diff --git a/extensions-core/protobuf-extensions/src/test/java/org/apache/druid/data/input/protobuf/ProtobufInputFormatTest.java b/extensions-core/protobuf-extensions/src/test/java/org/apache/druid/data/input/protobuf/ProtobufInputFormatTest.java index 56b774f15eee..bfdc473cae0b 100644 --- a/extensions-core/protobuf-extensions/src/test/java/org/apache/druid/data/input/protobuf/ProtobufInputFormatTest.java +++ b/extensions-core/protobuf-extensions/src/test/java/org/apache/druid/data/input/protobuf/ProtobufInputFormatTest.java @@ -24,6 +24,7 @@ import com.fasterxml.jackson.databind.ObjectMapper; import com.google.common.collect.ImmutableList; import com.google.common.collect.ImmutableMap; +import com.google.common.collect.ImmutableSet; import com.google.common.collect.Lists; import com.google.common.io.Files; import org.apache.druid.common.config.NullHandling; @@ -368,7 +369,7 @@ public void testParseNestedDataSchemaless() throws Exception InputRow row = transformingReader.read().next(); Assert.assertEquals( - ImmutableList.of( + ImmutableSet.of( "someOtherId", "someIntColumn", "isValid", @@ -381,7 +382,7 @@ public void testParseNestedDataSchemaless() throws Exception "id", "someBytesColumn" ), - row.getDimensions() + ImmutableSet.copyOf(row.getDimensions()) ); Assert.assertEquals(ImmutableMap.of("bar", "baz"), row.getRaw("foo")); diff --git a/processing/src/test/java/org/apache/druid/math/expr/ParserTest.java b/processing/src/test/java/org/apache/druid/math/expr/ParserTest.java index acb6345a2e9a..2b28fd6db1a3 100644 --- a/processing/src/test/java/org/apache/druid/math/expr/ParserTest.java +++ b/processing/src/test/java/org/apache/druid/math/expr/ParserTest.java @@ -820,7 +820,7 @@ private void validateParser( } final Expr.BindingAnalysis deets = parsed.analyzeInputs(); Assert.assertEquals(expression, expected, parsed.toString()); - Assert.assertEquals(expression, identifiers, deets.getRequiredBindingsList()); + Assert.assertEquals(expression, ImmutableSet.copyOf(identifiers), ImmutableSet.copyOf(deets.getRequiredBindingsList())); Assert.assertEquals(expression, scalars, deets.getScalarVariables()); Assert.assertEquals(expression, arrays, deets.getArrayVariables()); @@ -828,7 +828,7 @@ private void validateParser( final Expr roundTrip = Parser.parse(parsedNoFlatten.stringify(), ExprMacroTable.nil()); Assert.assertEquals(parsed.stringify(), roundTrip.stringify()); final Expr.BindingAnalysis roundTripDeets = roundTrip.analyzeInputs(); - Assert.assertEquals(expression, identifiers, roundTripDeets.getRequiredBindingsList()); + Assert.assertEquals(expression, ImmutableSet.copyOf(identifiers), ImmutableSet.copyOf(roundTripDeets.getRequiredBindingsList())); Assert.assertEquals(expression, scalars, roundTripDeets.getScalarVariables()); Assert.assertEquals(expression, arrays, roundTripDeets.getArrayVariables()); }