fix: Casting to IQueryable no longer drops chunk size (#455)

* fix: Casting to IQueryable no longer drops chunk size * including saveState * adding contributor to readme --------- Co-authored-by: Lennon <[email protected]> Co-authored-by: slorello89 <[email protected]>
redis · Jul 11, 2024 · eff0dc9 · eff0dc9
1 parent 66c2f0c
commit eff0dc9
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -472,6 +472,7 @@ We'd love your contributions! If you want to contribute please read our [Contrib
 * [@axnetg](https://github.com/axnetg)
 * [@abbottdev](https://github.com/abbottdev)
 * [@PrudiusVladislav](https://github.com/PrudiusVladislav)
+* [@CormacLennon](https://github.com/CormacLennon)
 
 <!-- Logo -->
 [Logo]: images/logo.svg

diff --git a/src/Redis.OM/Searching/RedisQueryProvider.cs b/src/Redis.OM/Searching/RedisQueryProvider.cs
@@ -84,7 +84,11 @@ public IQueryable CreateQuery(Expression expression)
                    (IQueryable)Activator.CreateInstance(
                        typeof(RedisCollection<>).MakeGenericType(elementType),
                        this,
-                       expression);
+                       expression,
+                       StateManager,
+                       null,
+                       _saveState,
+                       _chunkSize);
             }
             catch (TargetInvocationException e)
             {
@@ -102,7 +106,7 @@ public IQueryable<TElement> CreateQuery<TElement>(Expression expression)
             where TElement : notnull
         {
             var booleanExpression = expression as Expression<Func<TElement, bool>>;
-            return new RedisCollection<TElement>(this, expression, StateManager, booleanExpression, true);
+            return new RedisCollection<TElement>(this, expression, StateManager, booleanExpression, _saveState, _chunkSize);
         }
 
         /// <inheritdoc/>

diff --git a/test/Redis.OM.Unit.Tests/RediSearchTests/SearchTests.cs b/test/Redis.OM.Unit.Tests/RediSearchTests/SearchTests.cs
@@ -3822,5 +3822,22 @@ public async Task EnumerateAllButAllExpired()
 
             Assert.Empty(people);
         }
+
+        [Fact]
+        public void TestBasicQueryCastToIQueryable()
+        {
+            _substitute.ClearSubstitute();
+            _substitute.Execute(Arg.Any<string>(), Arg.Any<object[]>()).Returns(_mockReply);
+            var y = 33;
+            var collection = new RedisCollection<Person>(_substitute, 1234) as IQueryable<Person>;
+            _ = collection.Where(x => x.Age < y).ToList();
+            _substitute.Received().Execute(
+                "FT.SEARCH",
+                "person-idx",
+                "(@Age:[-inf (33])",
+                "LIMIT",
+                "0",
+                "1234");
+        }
     }
 }