Fix a couple of tests.

* Null-containing sets don't need to be tested as they can no longer be constructed. * Use vendered guava.
apache · Jul 8, 2024 · 6f15be1 · 6f15be1
1 parent e43e691
commit 6f15be1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/...-java/src/test/java/org/apache/beam/runners/core/metrics/MonitoringInfoEncodingsTest.java b/...-java/src/test/java/org/apache/beam/runners/core/metrics/MonitoringInfoEncodingsTest.java
@@ -93,11 +93,6 @@ public void testStringSetEncoding() {
     data = StringSetData.create(ImmutableSet.of("ab", "", "ef"));
     payload = encodeStringSet(data);
     assertEquals(data, decodeStringSet(payload));
-
-    // test null string encoding
-    data = StringSetData.create(new HashSet<>(Arrays.asList("ab", null, "cd")));
-    StringSetData finalData = data;
-    Assert.assertThrows(RuntimeException.class, () -> encodeStringSet(finalData));
   }
 
   @Test

diff --git a/sdks/java/core/src/test/java/org/apache/beam/sdk/metrics/StringSetResultTest.java b/sdks/java/core/src/test/java/org/apache/beam/sdk/metrics/StringSetResultTest.java
@@ -21,11 +21,11 @@
 import static org.junit.Assert.assertThrows;
 import static org.junit.Assert.assertTrue;
 
-import com.google.common.collect.Sets;
-import com.google.common.collect.Sets.SetView;
 import java.util.Arrays;
 import java.util.HashSet;
 import java.util.Set;
+import org.apache.beam.vendor.guava.v32_1_2_jre.com.google.common.collect.Sets;
+import org.apache.beam.vendor.guava.v32_1_2_jre.com.google.common.collect.Sets.SetView;
 import org.junit.Test;
 
 public class StringSetResultTest {