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

@@ -7,8 +7,9 @@ java_library(
     name = "tests",
     testonly = True,
     srcs = glob(["*.java"]),
-    resources = [
+    data = [
         "//testing:policy_test_resources",
+        "@cel_policy//conformance:testdata",
     ],
     deps = [
         "//:java_truth",
@@ -39,6 +40,7 @@ java_library(
         "//runtime:function_binding",
         "//testing:cel_runtime_flavor",
         "//testing/protos:single_file_java_proto",
+        "@bazel_tools//tools/java/runfiles",
         "@cel_spec//proto/cel/expr/conformance/proto3:test_all_types_java_proto",
         "@maven//:com_google_guava_guava",
         "@maven//:com_google_testparameterinjector_test_parameter_injector",

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

@@ -522,15 +522,24 @@ private String getPolicyFilePath() {
     }
 
     private String readPolicyYamlContent() throws IOException {
-      return readFromYaml(String.format("policy/%s/policy.yaml", name));
+      return readFromYaml(
+          String.format(
+              "google3/third_party/java/cel/testing/src/test/resources/policy/%s/policy.yaml",
+              name));
     }
 
     private String readConfigYamlContent() throws IOException {
-      return readFromYaml(String.format("policy/%s/config.yaml", name));
+      return readFromYaml(
+          String.format(
+              "google3/third_party/java/cel/testing/src/test/resources/policy/%s/config.yaml",
+              name));
     }
 
     private String readExpectedErrorsBaseline() throws IOException {
-      return readFromYaml(String.format("policy/%s/expected_errors.baseline", name));
+      return readFromYaml(
+          String.format(
+              "google3/third_party/java/cel/testing/src/test/resources/policy/%s/expected_errors.baseline",
+              name));
     }
 
     TestErrorYamlPolicy(String name) {

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

@@ -18,9 +18,10 @@
 
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.base.Ascii;
-import com.google.common.io.Resources;
+import com.google.common.io.Files;
+import com.google.devtools.build.runfiles.Runfiles;
+import java.io.File;
 import java.io.IOException;
-import java.net.URL;
 import java.util.List;
 import java.util.Map;
 import org.yaml.snakeyaml.LoaderOptions;
@@ -30,6 +31,8 @@
 /** Package-private class to assist with policy testing. */
 final class PolicyTestHelper {
 
+  private static final Runfiles runfiles = createRunfiles();
+
   enum TestYamlPolicy {
     NESTED_RULE(
         "nested_rule",
@@ -74,11 +77,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 +105,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 +140,23 @@ String getUnparsed() {
     }
 
     String readPolicyYamlContent() throws IOException {
-      return readFromYaml(String.format("policy/%s/policy.yaml", name));
+      return readFromYaml(
+          String.format(
+              "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(
+              "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(
+                  "cel_policy/conformance/testdata/%s/tests.yaml", name));
 
       return yaml.load(testContent);
     }
@@ -163,9 +174,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;
     }
@@ -258,12 +278,21 @@ public void setExpr(String expr) {
     }
   }
 
-  private static URL getResource(String path) {
-    return Resources.getResource(Ascii.toLowerCase(path));
+  private static String readFile(String rlocationPath) throws IOException {
+    String resolvedPath = runfiles.rlocation(Ascii.toLowerCase(rlocationPath));
+    if (resolvedPath == null || !new File(resolvedPath).exists()) {
+      throw new IOException("Runfile not found: " + rlocationPath);
+    }
+    File file = new File(resolvedPath);
+    return Files.asCharSource(file, UTF_8).read();
   }
 
-  private static String readFile(String path) throws IOException {
-    return Resources.toString(getResource(path), UTF_8);
+  private static Runfiles createRunfiles() {
+    try {
+      return Runfiles.create();
+    } catch (IOException e) {
+      throw new RuntimeException("Failed to initialize Runfiles", e);
+    }
   }
 
   private PolicyTestHelper() {}

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",