tidy

Signed-off-by: John Kastner <[email protected]>

Author: john-h-kastner-aws

cedar-policy-core/src/extensions/ipaddr.rs

@@ -1033,7 +1033,7 @@ mod tests {
         let entities = basic_entities();
         let eval = Evaluator::new(request, &entities, &exts);
 
-        // Requires at least one argument.
+        // Requires exactly two arguments
         assert_ipaddr_wrong_num_args_err(
             eval.interpret_inline_policy(&Expr::call_extension_fn(
                 Name::parse_unqualified_name("isInRange").expect("should be a valid identifier"),
@@ -1046,6 +1046,15 @@ mod tests {
             )),
             "isInRange",
         );
+        assert_ipaddr_wrong_num_args_err(
+            eval.interpret_inline_policy(&Expr::call_extension_fn(
+                Name::parse_unqualified_name("isInRange").expect("should be a valid identifier"),
+                vec![
+                    ip("192.168.1.100"),
+                ],
+            )),
+            "isInRange",
+        );
     }
 
     #[test]

cedar-policy-core/src/validator/typecheck/test/extensions.rs

@@ -98,8 +98,8 @@ fn ip_extension_typecheck_fails() {
     );
 }
 
-#[cfg(not(feature = "variadic-is-in-range"))]
 #[test]
+#[cfg(not(feature = "variadic-is-in-range"))]
 fn ip_extension_not_variadic() {
     let src = "ip(\"127.0.0.1\").isInRange(ip(\"192.168.0.1/24\"), ip(\"192.167.0.1/24\"), ip(\"192.167.0.3/24\"))";
     let expr = Expr::from_str(src).expect("parsing should succeed");
@@ -111,8 +111,8 @@ fn ip_extension_not_variadic() {
     );
 }
 
-#[cfg(feature = "variadic-is-in-range")]
 #[test]
+#[cfg(feature = "variadic-is-in-range")]
 fn ip_extension_variadic() {
     use crate::ast::Name;
     let ipaddr_name = Name::parse_unqualified_name("ipaddr").expect("should be a valid identifier");