fixing issue with index null checking, and equality check for numerics (#23)

* fixing issue with index null checking, and equality check for numerics

Author: slorello89

src/Redis.OM/Common/ExpressionTranslator.cs

@@ -260,7 +260,7 @@ internal static RedisQuery BuildQueryFromExpression(Expression expression, Type
         /// <returns>The index field type.</returns>
         internal static SearchFieldType DetermineIndexFieldsType(MemberInfo member)
         {
-            if (TypeDeterminationUtilities.IsNumeric(member.DeclaringType!))
+            if (member is PropertyInfo info && TypeDeterminationUtilities.IsNumeric(info.PropertyType))
             {
                 return SearchFieldType.NUMERIC;
             }

src/Redis.OM/Redis.OM.csproj

@@ -6,8 +6,8 @@
     <RootNamespace>Redis.OM</RootNamespace>
     <Nullable>enable</Nullable>
     <AutoGenerateBindingRedirects>true</AutoGenerateBindingRedirects>
-    <PackageVersion>0.1.1</PackageVersion>
-    <Version>0.1.1</Version>
+    <PackageVersion>0.1.2</PackageVersion>
+    <Version>0.1.2</Version>
     <Description>Object Mapping and More for Redis</Description>
     <Title>Redis OM</Title>
     <Authors>Steve Lorello</Authors>

src/Redis.OM/Searching/RedisQueryProvider.cs

@@ -111,7 +111,7 @@ public SearchResponse<T> ExecuteQuery<T>(Expression expression)
                 attr = type.GetCustomAttribute<DocumentAttribute>();
             }
 
-            if (attr == null || string.IsNullOrEmpty(attr.IndexName))
+            if (attr == null)
             {
                 throw new InvalidOperationException("Searches can only be performed on objects decorated with a RedisObjectDefinitionAttribute that specifies a particular index");
             }

test/Redis.OM.Unit.Tests/RediSearchTests/PersonNoName.cs

@@ -0,0 +1,11 @@
+using Redis.OM.Modeling;
+
+namespace Redis.OM.Unit.Tests.RediSearchTests
+{
+    [Document]
+    public class PersonNoName
+    {
+        [Indexed]
+        public int Age { get; set; }
+    }
+}

test/Redis.OM.Unit.Tests/RediSearchTests/SearchTests.cs

@@ -85,6 +85,57 @@ public void TestBasicQueryWithVariable()
                 "100"));
         }
 
+        [Fact]
+        public void TestBasicQueryWithExactNumericMatch()
+        {
+            _mock.Setup(x => x.Execute(It.IsAny<string>(), It.IsAny<string[]>()))
+                .Returns(_mockReply);            
+            var y = 33;
+            var collection = new RedisCollection<Person>(_mock.Object);
+            var res = collection.Where(x => x.Age == y).ToList();
+            _mock.Verify(x => x.Execute(
+                "FT.SEARCH",
+                "person-idx",
+                "(@Age:[33 33])",
+                "LIMIT",
+                "0",
+                "100"));
+        }
+
+        [Fact]
+        public void TestBasicFirstOrDefaultQuery()
+        {
+            _mock.Setup(x => x.Execute(It.IsAny<string>(), It.IsAny<string[]>()))
+                .Returns(_mockReply);            
+            var y = 33;
+            var collection = new RedisCollection<Person>(_mock.Object);
+            var res = collection.FirstOrDefault(x => x.Age == y);
+            _mock.Verify(x => x.Execute(
+                "FT.SEARCH",
+                "person-idx",
+                "(@Age:[33 33])",
+                "LIMIT",
+                "0",
+                "1"));
+        }
+        
+        [Fact]
+        public void TestBasicQueryNoNameIndex()
+        {
+            _mock.Setup(x => x.Execute(It.IsAny<string>(), It.IsAny<string[]>()))
+                .Returns(_mockReply);            
+            var y = 33;
+            var collection = new RedisCollection<PersonNoName>(_mock.Object);
+            var res = collection.FirstOrDefault(x => x.Age == y);
+            _mock.Verify(x => x.Execute(
+                "FT.SEARCH",
+                "personnoname-idx",
+                "(@Age:[33 33])",
+                "LIMIT",
+                "0",
+                "1"));
+        }
+
         [Fact]
         public void TestBasicOrQuery()
         {