fix unit test

Author: shijiesheng

.buildkite/pipeline.yml

@@ -52,6 +52,7 @@ steps:
 
   - label: ":java: Unit test with test services"
     artifact_paths:
+      - "build/reports/tests/test/**/*"
       - "build/reports/jacoco/test/jacocoTestReport.xml"
       - "build/reports/metadata.txt"
     timeout_in_minutes: 30

src/test/java/com/uber/cadence/internal/compatibility/MapperTestUtil.java

@@ -47,24 +47,18 @@ void assertNoMissingFields(M message, Class<E> fields) {
   }
 
   public static void assertNoMissingFields(Object message) {
-    List<String> nullFields = getMissingFields(message.toString());
+    Set<String> nullFields = getMissingFields(message.toString());
 
-    Assert.assertArrayEquals(
-        "All fields expected to be set in the text",
-        new String[0],
-        nullFields.toArray(new String[0]));
+    Assert.assertEquals("All fields expected to be set in the text", new HashSet<>(), nullFields);
   }
 
   public static void assertMissingFields(Object message, Set<String> values) {
-    List<String> nullFields = getMissingFields(message.toString());
-    Assert.assertArrayEquals(
-        "Expected missing fields but get different",
-        values.toArray(new String[0]),
-        nullFields.toArray(new String[0]));
+    Set<String> nullFields = getMissingFields(message.toString());
+    Assert.assertEquals("Expected missing fields but get different", values, nullFields);
   }
 
-  private static List<String> getMissingFields(String text) {
-    List<String> nullFields = new ArrayList<>();
+  private static Set<String> getMissingFields(String text) {
+    Set<String> nullFields = new HashSet<>();
     // Regex to find fieldName=null
     Pattern pattern = Pattern.compile("(\\w+)=null");
     Matcher matcher = pattern.matcher(text);

src/test/java/com/uber/cadence/internal/compatibility/proto/mappers/RequestMapperTest.java

@@ -247,7 +247,11 @@ public static Iterable<Object[]> cases() {
             ClientObjects.UPDATE_DOMAIN_REQUEST,
             // Data and replicationConfiguration are copied incorrectly due to a bug :(
             ProtoObjects.UPDATE_DOMAIN_REQUEST,
-            RequestMapper::updateDomainRequest),
+            RequestMapper::updateDomainRequest,
+            // TODO new fields that are not yet supported
+            "queueConfig",
+            "predefinedQueueName",
+            "queueType"),
         testCase(
             ClientObjects.LIST_CLOSED_WORKFLOW_EXECUTIONS_REQUEST,
             ProtoObjects.LIST_CLOSED_WORKFLOW_EXECUTIONS_REQUEST,