Remove duplicate test policies and use the ones from cel-policy as SoT

PiperOrigin-RevId: 918667081

Author: l46kok

policy/src/test/java/dev/cel/policy/BUILD.bazel

@@ -9,6 +9,7 @@ java_library(
     srcs = glob(["*.java"]),
     resources = [
         "//testing:policy_test_resources",
+        "@cel_policy//conformance:testdata",
     ],
     deps = [
         "//:java_truth",

policy/src/test/java/dev/cel/policy/PolicyTestHelper.java

@@ -74,11 +74,11 @@ enum TestYamlPolicy {
         "required_labels",
         true,
         "cel.@block([spec.labels.filter(@it:0:0, !(@it:0:0 in resource.labels)), spec.labels,"
-            + " resource.labels, @index2.filter(@it:0:0, @it:0:0 in @index1 && @index1[@it:0:0] !="
-            + " @index2[@it:0:0])], (@index0.size() > 0) ? optional.of(\"missing one or more"
-            + " required labels: [\"\" + @index0.join(\",\") + \"\"]\") : ((@index3.size() > 0) ?"
-            + " optional.of(\"invalid values provided on one or more labels: [\"\" +"
-            + " @index3.join(\",\") + \"\"]\") : optional.none()))"),
+            + " resource.labels.transformList(@it:0:1, @it2:0:1, @it:0:1 in @index1 && @it2:0:1 !="
+            + " @index1[@it:0:1], @it:0:1)], (@index0.size() > 0) ? optional.of(\"missing one or"
+            + " more required labels: [\"\" + @index0.join(\"\", \"\") + \"\"]\") :"
+            + " ((@index2.size() > 0) ? optional.of(\"invalid values provided on one or more"
+            + " labels: [\"\" + @index2.join(\"\", \"\") + \"\"]\") : optional.none()))"),
     RESTRICTED_DESTINATIONS(
         "restricted_destinations",
         false,
@@ -102,9 +102,10 @@ enum TestYamlPolicy {
         "cel.@block([spec.single_int32], (@index0 > 10) ? optional.of(\"invalid spec, got"
             + " single_int32=\" + string(@index0) + \", wanted <= 10\") : ((spec.standalone_enum =="
             + " cel.expr.conformance.proto3.TestAllTypes.NestedEnum.BAR ||"
-            + " dev.cel.testing.testdata.proto3.StandaloneGlobalEnum.SGAR =="
-            + " dev.cel.testing.testdata.proto3.StandaloneGlobalEnum.SGOO) ? optional.of(\"invalid"
-            + " spec, neither nested nor imported enums may refer to BAR\") : optional.none()))"),
+            + " cel.expr.conformance.proto3.TestAllTypes.NestedEnum.BAZ in"
+            + " spec.repeated_nested_enum || cel.expr.conformance.proto3.GlobalEnum.GAR =="
+            + " cel.expr.conformance.proto3.GlobalEnum.GOO) ? optional.of(\"invalid spec, neither"
+            + " nested nor repeated enums may refer to BAR or BAZ\") : optional.none()))"),
     LIMITS(
         "limits",
         true,
@@ -136,16 +137,20 @@ String getUnparsed() {
     }
 
     String readPolicyYamlContent() throws IOException {
-      return readFromYaml(String.format("policy/%s/policy.yaml", name));
+      return readFromYaml(
+          String.format("third_party/cel/policy/conformance/testdata/%s/policy.yaml", name));
     }
 
     String readConfigYamlContent() throws IOException {
-      return readFromYaml(String.format("policy/%s/config.yaml", name));
+      return readFromYaml(
+          String.format("third_party/cel/policy/conformance/testdata/%s/config.yaml", name));
     }
 
     PolicyTestSuite readTestYamlContent() throws IOException {
       Yaml yaml = new Yaml(new Constructor(PolicyTestSuite.class, new LoaderOptions()));
-      String testContent = readFile(String.format("policy/%s/tests.yaml", name));
+      String testContent =
+          readFile(
+              String.format("third_party/cel/policy/conformance/testdata/%s/tests.yaml", name));
 
       return yaml.load(testContent);
     }
@@ -163,9 +168,18 @@ static String readFromYaml(String yamlPath) throws IOException {
    */
   @VisibleForTesting
   public static final class PolicyTestSuite {
+    private String name;
     private String description;
     private List<PolicyTestSection> section;
 
+    public void setName(String name) {
+      this.name = name;
+    }
+
+    public String getName() {
+      return name;
+    }
+
     public void setDescription(String description) {
       this.description = description;
     }

repositories.bzl

@@ -34,8 +34,8 @@ def bazel_common_dependency():
     )
 
 def cel_policy_dependency():
-    cel_policy_tag = "569292f1c4eaa41894c1e37ee94eb146e284bcfa"
-    cel_policy_sha = "5a68318d906f6ce18492ad6f82b5f8bb083fd9d694cf567d399216c11da03157"
+    cel_policy_tag = "e4c38defbbf34dfff2dc448dc58e93a9733ae8b1"
+    cel_policy_sha = "46378e0d17a16465899f9fefc94c3d44e1f40aedd8a31c9c0b2b6198048eabd6"
     http_archive(
         name = "cel_policy",
         sha256 = cel_policy_sha,

testing/src/test/java/dev/cel/testing/testrunner/BUILD.bazel

@@ -157,14 +157,13 @@ java_test(
 
 cel_java_test(
     name = "test_runner_sample_yaml",
-    cel_expr = "nested_rule/policy.yaml",
+    cel_expr = "@cel_policy//conformance:testdata/nested_rule/policy.yaml",
     proto_deps = [
         "@cel_spec//proto/cel/expr/conformance/proto2:test_all_types_proto",
         "@cel_spec//proto/cel/expr/conformance/proto3:test_all_types_proto",
     ],
-    test_data_path = "//testing/src/test/resources/policy",
     test_src = ":user_test",
-    test_suite = "nested_rule/testrunner_tests.yaml",
+    test_suite = "@cel_policy//conformance:testdata/nested_rule/tests.yaml",
     deps = [
         "@cel_spec//proto/cel/expr/conformance/proto2:test_all_types_java_proto",
         "@cel_spec//proto/cel/expr/conformance/proto3:test_all_types_java_proto",
@@ -173,7 +172,7 @@ cel_java_test(
 
 cel_java_test(
     name = "unknown_set_yaml",
-    cel_expr = "nested_rule/policy.yaml",
+    cel_expr = "@cel_policy//conformance:testdata/nested_rule/policy.yaml",
     proto_deps = [
         "@cel_spec//proto/cel/expr/conformance/proto2:test_all_types_proto",
         "@cel_spec//proto/cel/expr/conformance/proto3:test_all_types_proto",
@@ -211,15 +210,14 @@ cel_java_test(
 
 cel_java_test(
     name = "context_pb_user_test_runner_sample",
-    cel_expr = "context_pb/policy.yaml",
-    config = "context_pb/config.yaml",
+    cel_expr = "@cel_policy//conformance:testdata/context_pb/policy.yaml",
+    config = "@cel_policy//conformance:testdata/context_pb/config.yaml",
     proto_deps = [
         "@cel_spec//proto/cel/expr/conformance/proto2:test_all_types_proto",
         "@cel_spec//proto/cel/expr/conformance/proto3:test_all_types_proto",
     ],
-    test_data_path = "//testing/src/test/resources/policy",
     test_src = ":context_pb_user_test",
-    test_suite = "context_pb/tests.yaml",
+    test_suite = "@cel_policy//conformance:testdata/context_pb/tests.yaml",
     deps = [
         "@cel_spec//proto/cel/expr/conformance/proto2:test_all_types_java_proto",
         "@cel_spec//proto/cel/expr/conformance/proto3:test_all_types_java_proto",
@@ -228,15 +226,14 @@ cel_java_test(
 
 cel_java_test(
     name = "additional_config_test_runner_sample",
-    cel_expr = "nested_rule/policy.yaml",
-    config = "nested_rule/config.yaml",
+    cel_expr = "@cel_policy//conformance:testdata/nested_rule/policy.yaml",
+    config = "@cel_policy//conformance:testdata/nested_rule/config.yaml",
     proto_deps = [
         "@cel_spec//proto/cel/expr/conformance/proto2:test_all_types_proto",
         "@cel_spec//proto/cel/expr/conformance/proto3:test_all_types_proto",
     ],
-    test_data_path = "//testing/src/test/resources/policy",
     test_src = ":env_config_user_test",
-    test_suite = "nested_rule/testrunner_tests.textproto",
+    test_suite = "@cel_policy//conformance:testdata/nested_rule/tests.textproto",
     deps = [
         "@cel_spec//proto/cel/expr/conformance/proto2:test_all_types_java_proto",
         "@cel_spec//proto/cel/expr/conformance/proto3:test_all_types_java_proto",
@@ -245,14 +242,13 @@ cel_java_test(
 
 cel_java_test(
     name = "test_runner_sample",
-    cel_expr = "nested_rule/policy.yaml",
+    cel_expr = "@cel_policy//conformance:testdata/nested_rule/policy.yaml",
     proto_deps = [
         "@cel_spec//proto/cel/expr/conformance/proto2:test_all_types_proto",
         "@cel_spec//proto/cel/expr/conformance/proto3:test_all_types_proto",
     ],
-    test_data_path = "//testing/src/test/resources/policy",
     test_src = ":user_test",
-    test_suite = "nested_rule/testrunner_tests.textproto",
+    test_suite = "@cel_policy//conformance:testdata/nested_rule/tests.textproto",
     deps = [
         "@cel_spec//proto/cel/expr/conformance/proto2:test_all_types_java_proto",
         "@cel_spec//proto/cel/expr/conformance/proto3:test_all_types_java_proto",
@@ -286,8 +282,8 @@ cel_java_test(
 
 cel_java_test(
     name = "context_message_user_test_runner_textproto_sample",
-    cel_expr = "context_pb/policy.yaml",
-    config = "context_pb/config.yaml",
+    cel_expr = "@cel_policy//conformance:testdata/context_pb/policy.yaml",
+    config = "@cel_policy//conformance:testdata/context_pb/config.yaml",
     proto_deps = [
         "@cel_spec//proto/cel/expr/conformance/proto2:test_all_types_proto",
         "@cel_spec//proto/cel/expr/conformance/proto3:test_all_types_proto",
@@ -303,15 +299,14 @@ cel_java_test(
 
 cel_java_test(
     name = "context_pb_user_test_runner_textproto_sample",
-    cel_expr = "context_pb/policy.yaml",
-    config = "context_pb/config.yaml",
+    cel_expr = "@cel_policy//conformance:testdata/context_pb/policy.yaml",
+    config = "@cel_policy//conformance:testdata/context_pb/config.yaml",
     proto_deps = [
         "@cel_spec//proto/cel/expr/conformance/proto2:test_all_types_proto",
         "@cel_spec//proto/cel/expr/conformance/proto3:test_all_types_proto",
     ],
-    test_data_path = "//testing/src/test/resources/policy",
     test_src = ":context_pb_user_test",
-    test_suite = "context_pb/tests.textproto",
+    test_suite = "@cel_policy//conformance:testdata/context_pb/tests.textproto",
     deps = [
         "@cel_spec//proto/cel/expr/conformance/proto2:test_all_types_java_proto",
         "@cel_spec//proto/cel/expr/conformance/proto3:test_all_types_java_proto",