Cherrypick test improvements

mkruskal-google · mkruskal-google · commit d365736f4f6e · 2025-09-09T13:52:09.000-07:00
diff --git a/java/core/src/test/java/com/google/protobuf/ImportOptionTest.java b/java/core/src/test/java/com/google/protobuf/ImportOptionTest.java
@@ -5,12 +5,12 @@
 import com.google.protobuf.Descriptors.Descriptor;
 import com.google.protobuf.Descriptors.FieldDescriptor;
 import com.google.protobuf.Descriptors.FileDescriptor;
-import proto2_unittest.UnittestCustomOptions;
-import proto2_unittest_import_option.TestMessage;
-import proto2_unittest_import_option.UnittestImportOptionProto;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.junit.runners.JUnit4;
+import proto2_unittest.UnittestCustomOptions;
+import proto2_unittest_import_option.TestMessage;
+import proto2_unittest_import_option.UnittestImportOptionProto;
 
 /** Unit test for import option. */
 @RunWith(JUnit4.class)
@@ -36,5 +36,8 @@ public void testImportOption() throws Exception {
     // TODO: Since `option_deps` are treated as `deps` in Bazel 7, the unknown
     // fields are not empty.  Once we drop Bazel 7 support we can test that these are filled with
     // unknown fields.
+    assertThat(unknownFieldsFile.asMap().size()).isAtMost(1);
+    assertThat(unknownFieldsMessage.asMap().size()).isAtMost(1);
+    assertThat(unknownFieldsField.asMap().size()).isAtMost(1);
   }
 }
