every: non collections should evaluate to false (#253)

See https://github.com/open-policy-agent/opa/pull/6763

Signed-off-by: Anand Krishnamoorthi <anakrish@microsoft.com>

src/interpreter.rs

@@ -791,10 +791,9 @@ impl Interpreter {
                     }
                 }
             }
-            Value::Undefined | Value::Null => r = false,
-            // Other types cause every to evaluate to true even though
-            // it is supposed to happen only for empty domain.
-            _ => (),
+            _ => {
+                r = false;
+            }
         };
         self.contexts.pop();
         self.scopes.pop();

tests/interpreter/cases/every/tests.yaml

@@ -114,7 +114,6 @@ cases:
       x1: 100
       x2: 100
       x3: 100
-      x4: 100
 
   - note: negative
     data: {}