Add interpreter test for protolite messages

PiperOrigin-RevId: 746242496

Author: l46kok

common/src/main/java/dev/cel/common/internal/ProtoJavaQualifiedNames.java

@@ -28,7 +28,7 @@
 import java.util.ArrayDeque;
 
 /**
- * Helper class for constructing a fully qualified Java class name from a protobuf descriptor. * *
+ * Helper class for constructing a fully qualified Java class name from a protobuf descriptor.
  *
  * <p>CEL Library Internals. Do Not Use.
  */

common/src/main/java/dev/cel/common/values/ProtoLiteCelValueConverter.java

@@ -154,9 +154,9 @@ CelValue getDefaultCelValue(String protoTypeName, String fieldName) {
     if (defaultValue instanceof MessageLite) {
       return fromProtoMessageToCelValue(
           fieldDescriptor.getFieldProtoTypeName(), (MessageLite) defaultValue);
-    } else {
-      return fromJavaObjectToCelValue(getDefaultValue(fieldDescriptor));
     }
+
+    return fromJavaObjectToCelValue(defaultValue);
   }
 
   private Object getDefaultValue(FieldLiteDescriptor fieldDescriptor) {
@@ -198,7 +198,7 @@ private Object getScalarDefaultValue(FieldLiteDescriptor fieldDescriptor) {
         return ByteString.EMPTY;
       case MESSAGE:
         if (WellKnownProto.isWrapperType(fieldDescriptor.getFieldProtoTypeName())) {
-          return NullValue.NULL_VALUE;
+          return com.google.protobuf.NullValue.NULL_VALUE;
         }
 
         return getDefaultMessageBuilder(fieldDescriptor.getFieldProtoTypeName()).build();

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

@@ -184,6 +184,7 @@ java_library(
         ["*.java"],
         # keep sorted
         exclude = [
+            "CelLiteInterpreterTest.java",
             "CelValueInterpreterTest.java",
             "InterpreterTest.java",
         ] + ANDROID_TESTS,
@@ -316,6 +317,23 @@ cel_android_local_test(
     ],
 )
 
+java_library(
+    name = "cel_lite_interpreter_test",
+    testonly = 1,
+    srcs = [
+        "CelLiteInterpreterTest.java",
+    ],
+    deps = [
+        "//common/values:proto_message_lite_value_provider",
+        "//extensions:optional_library",
+        "//runtime",
+        "//testing:base_interpreter_test",
+        "//testing:test_all_types_cel_java_proto3",
+        "@maven//:com_google_testparameterinjector_test_parameter_injector",
+        "@maven//:junit_junit",
+    ],
+)
+
 junit4_test_suites(
     name = "test_suites",
     shard_count = 4,
@@ -325,6 +343,7 @@ junit4_test_suites(
     ],
     src_dir = "src/test/java",
     deps = [
+        ":cel_lite_interpreter_test",
         ":cel_value_interpreter_test",
         ":interpreter_test",
         ":tests",

runtime/src/test/java/dev/cel/runtime/CelLiteInterpreterTest.java

@@ -0,0 +1,102 @@
+// Copyright 2025 Google LLC
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//      https://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package dev.cel.runtime;
+
+import com.google.testing.junit.testparameterinjector.TestParameter;
+import com.google.testing.junit.testparameterinjector.TestParameterInjector;
+import dev.cel.common.values.ProtoMessageLiteValueProvider;
+import dev.cel.expr.conformance.proto3.TestAllTypesProto3CelDescriptor;
+import dev.cel.extensions.CelOptionalLibrary;
+import dev.cel.testing.BaseInterpreterTest;
+import org.junit.runner.RunWith;
+
+/**
+ * Exercises a suite of interpreter tests defined in {@link BaseInterpreterTest} using {@link
+ * ProtoMessageLiteValueProvider} and <b>full version of protobuf messages</b>.
+ */
+@RunWith(TestParameterInjector.class)
+public class CelLiteInterpreterTest extends BaseInterpreterTest {
+  public CelLiteInterpreterTest(@TestParameter InterpreterTestOption testOption) {
+    super(
+        testOption.celOptions.toBuilder().enableCelValue(true).build(),
+        testOption.useNativeCelType,
+        CelRuntimeFactory.standardCelRuntimeBuilder()
+            .setValueProvider(
+                ProtoMessageLiteValueProvider.newInstance(
+                    TestAllTypesProto3CelDescriptor.getDescriptor()))
+            .addLibraries(CelOptionalLibrary.INSTANCE)
+            .setOptions(testOption.celOptions.toBuilder().enableCelValue(true).build())
+            .build());
+  }
+
+  @Override
+  public void dynamicMessage_adapted() throws Exception {
+    // Dynamic message is not supported in Protolite
+    skipBaselineVerification();
+  }
+
+  @Override
+  public void dynamicMessage_dynamicDescriptor() throws Exception {
+    // Dynamic message is not supported in Protolite
+    skipBaselineVerification();
+  }
+
+  // All the tests below rely on message creation with fields populated. They are excluded for time
+  // being until this support is added.
+  @Override
+  public void wrappers() throws Exception {
+    skipBaselineVerification();
+  }
+
+  @Override
+  public void jsonConversions() {
+    skipBaselineVerification();
+  }
+
+  @Override
+  public void nestedEnums() {
+    skipBaselineVerification();
+  }
+
+  @Override
+  public void messages() throws Exception {
+    skipBaselineVerification();
+  }
+
+  @Override
+  public void packUnpackAny() {
+    skipBaselineVerification();
+  }
+
+  @Override
+  public void lists() throws Exception {
+    skipBaselineVerification();
+  }
+
+  @Override
+  public void maps() throws Exception {
+    skipBaselineVerification();
+  }
+
+  @Override
+  public void jsonValueTypes() {
+    skipBaselineVerification();
+  }
+
+  @Override
+  public void messages_error() {
+    skipBaselineVerification();
+  }
+}

testing/src/main/java/dev/cel/testing/BaseInterpreterTest.java

@@ -124,14 +124,21 @@ protected enum InterpreterTestOption {
   private CelRuntime celRuntime;
 
   public BaseInterpreterTest(CelOptions celOptions, boolean useNativeCelType) {
-    super(useNativeCelType);
-    this.celOptions = celOptions;
-    this.celRuntime =
+    this(
+        celOptions,
+        useNativeCelType,
         CelRuntimeFactory.standardCelRuntimeBuilder()
             .addLibraries(CelOptionalLibrary.INSTANCE)
             .addFileTypes(TEST_FILE_DESCRIPTORS)
             .setOptions(celOptions)
-            .build();
+            .build());
+  }
+
+  public BaseInterpreterTest(
+      CelOptions celOptions, boolean useNativeCelType, CelRuntime celRuntime) {
+    super(useNativeCelType);
+    this.celOptions = celOptions;
+    this.celRuntime = celRuntime;
   }
 
   @Override