Merge pull request #321 from martiner/ne

make $ne operator type handling consistent with $eq

Author: twillouer

src/main/java/com/github/fakemongo/impl/ExpressionParser.java

@@ -182,7 +182,8 @@ private boolean isEqual(Object obj1, Object obj2) {
                 return false;
               }
 
-              return obj1.equals(obj2);
+              Integer result = compareObjects(obj1, obj2, true);
+              return result != null && result.intValue() == 0;
             }
           };
         }

src/test/java/com/github/fakemongo/impl/ExpressionParserTest.java

@@ -109,6 +109,9 @@ public void testBasicOperators() {
     assertQuery(new BasicDBObject("a", new BasicDBObject("$eq", 1)), Arrays.<DBObject>asList(
         new BasicDBObject("n", "neil").append("a", 1)
     ));
+    assertQuery(new BasicDBObject("a", new BasicDBObject("$eq", 1L)), Arrays.<DBObject>asList(
+        new BasicDBObject("n", "neil").append("a", 1)
+    ));
   }
 
   @Test
@@ -152,6 +155,20 @@ public void testNeOperator() {
     ), results);
   }
 
+  @Test
+  public void testNeOperatorDifferentTypes() {
+    DBObject query = new BasicDBObjectBuilder().push("a").add("$ne", 1L).pop().get();
+
+    List<DBObject> results = doFilter(
+            query,
+            new BasicDBObject("a", 1),
+            new BasicDBObject("a", 2)
+    );
+    assertEquals(Collections.<DBObject>singletonList(
+            new BasicDBObject("a", 2)
+    ), results);
+  }
+
   @Test
   public void testNeOperatorWithNullValue() {
 	  DBObject query = new BasicDBObjectBuilder().push("a").add("$ne", null).pop().get();