Changes multimap state key() tests to not care about order. There is no guarantee on the order keys are returned. Also fixes a couple warnings from other FnApi tests. (#36178)

Author: acrites

sdks/java/harness/src/test/java/org/apache/beam/fn/harness/control/HarnessMonitoringInfosInstructionHandlerTest.java

@@ -30,7 +30,10 @@
 import org.apache.beam.sdk.metrics.Counter;
 import org.apache.beam.sdk.metrics.MetricsEnvironment;
 import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.JUnit4;
 
+@RunWith(JUnit4.class)
 public class HarnessMonitoringInfosInstructionHandlerTest {
 
   @Test

sdks/java/harness/src/test/java/org/apache/beam/fn/harness/control/ProcessBundleHandlerTest.java

@@ -236,6 +236,7 @@ public void finishBundle(FinishBundleContext context) {
     }
   }
 
+  @SuppressWarnings("ExtendsAutoValue")
   private static class TestBundleProcessor extends BundleProcessor {
     static int resetCnt = 0;
 

sdks/java/harness/src/test/java/org/apache/beam/fn/harness/state/MultimapUserStateTest.java

@@ -21,6 +21,7 @@
 import static java.util.Collections.singletonList;
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.hamcrest.Matchers.emptyIterable;
+import static org.hamcrest.collection.ArrayMatching.arrayContainingInAnyOrder;
 import static org.hamcrest.core.Is.is;
 import static org.junit.Assert.assertArrayEquals;
 import static org.junit.Assert.assertEquals;
@@ -167,7 +168,9 @@ public void testKeys() throws Exception {
     userState.put(A3, "V1");
     userState.put(A1, "V3");
     assertArrayEquals(new byte[][] {A1, A2}, Iterables.toArray(initKeys, byte[].class));
-    assertArrayEquals(new byte[][] {A1, A2, A3}, Iterables.toArray(userState.keys(), byte[].class));
+    assertThat(
+        Iterables.toArray(userState.keys(), byte[].class),
+        is(arrayContainingInAnyOrder(A1, A2, A3)));
 
     userState.clear();
     assertArrayEquals(new byte[][] {A1, A2}, Iterables.toArray(initKeys, byte[].class));
@@ -822,8 +825,9 @@ public void testKeysCached() throws Exception {
 
       userState.put(A2, "V1");
       userState.put(A3, "V1");
-      assertArrayEquals(
-          new byte[][] {A1, A2, A3}, Iterables.toArray(userState.keys(), byte[].class));
+      assertThat(
+          Iterables.toArray(userState.keys(), byte[].class),
+          is(arrayContainingInAnyOrder(A1, A2, A3)));
       userState.asyncClose();
     }
 
@@ -841,8 +845,9 @@ public void testKeysCached() throws Exception {
               ByteArrayCoder.of(),
               StringUtf8Coder.of());
 
-      assertArrayEquals(
-          new byte[][] {A1, A2, A3}, Iterables.toArray(userState.keys(), byte[].class));
+      assertThat(
+          Iterables.toArray(userState.keys(), byte[].class),
+          is(arrayContainingInAnyOrder(A1, A2, A3)));
       userState.asyncClose();
     }
   }