support boolean eq

Author: meir

src/json_path.rs

@@ -304,24 +304,50 @@ impl<'i, 'j, S:SelectValue> TermEvaluationResult<'i, 'j, S> {
     }
 
     fn eq(&self, s: &Self) -> bool {
-        if let (TermEvaluationResult::Value(v1), TermEvaluationResult::Value(v2)) = (self, s) {
-            if v1 == v2 {
-                true
-            } else {
-                false
+        match (self, s){
+            (TermEvaluationResult::Bool(b1), TermEvaluationResult::Bool(b2)) => {
+                if *b1 == *b2 {
+                    true
+                } else {
+                    false
+                }
+            }
+            (TermEvaluationResult::Value(v1), TermEvaluationResult::Bool(b2)) => {
+                if v1.get_type() == SelectValueType::Bool {
+                    let b1 = v1.get_bool();
+                    if b1 == *b2 {
+                        true
+                    } else {
+                        false
+                    }
+                } else {
+                    false
+                }
+            }
+            (TermEvaluationResult::Bool(_), TermEvaluationResult::Value(_)) => {
+                s.eq(self)
+            }
+            (TermEvaluationResult::Value(v1), TermEvaluationResult::Value(v2)) => {
+                if v1 == v2 {
+                    true
+                } else {
+                    false
+                }
+            }
+            (_, _) => {
+                match self.cmp(s) {
+                    CmpResult::Ord(o) => o.is_eq(),
+                    CmpResult::NotCmparable => false,
+               }
             }
-        } else {
-            match self.cmp(s) {
-                CmpResult::Ord(o) => o.is_eq(),
-                CmpResult::NotCmparable => false,
-           }
         }
     }
 
     fn ne(&self, s: &Self) -> bool {
-        match self.cmp(s) {
-            CmpResult::Ord(o) => o.is_ne(),
-            CmpResult::NotCmparable => false,
+        if self.eq(s) {
+            false
+        } else {
+            true
         }
     }
 

src/lib.rs

@@ -372,6 +372,12 @@ mod json_path_tests {
         verify_json!(path:"$*[?(@.code==\"2\")].code", json:[{"code":"1"},{"code":"2"}], results:["2"]);
     }
 
+    #[test]
+    fn test_filter_bool() {
+        verify_json!(path:"$.*[?(@==true)]", json:{"a":true, "b":false}, results:[true]);
+        verify_json!(path:"$.*[?(@==false)]", json:{"a":true, "b":false}, results:[false]);
+    }
+
     #[test]
     fn test_complex_filter_from_root() {
         verify_json!(path:"$.bar.*[?@ == $.foo]",

tests/op.rs

@@ -315,7 +315,7 @@ fn op_ne_for_complex_value() {
     select_and_then_compare(
         r#"$.[?("1" != @.a)]"#,
         json!({ "a": { "b": 1 } }),
-        json!([]),
+        json!([{ "a": { "b": 1 } }]),
     );
 }