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..ca32293c1cb7 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 @@ -38,6 +38,7 @@ import java.math.BigInteger; import java.nio.ByteBuffer; import java.util.Collections; +import java.util.HashSet; import java.util.List; import java.util.Set; @@ -820,7 +821,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, new HashSet<>(identifiers), deets.getRequiredBindings()); Assert.assertEquals(expression, scalars, deets.getScalarVariables()); Assert.assertEquals(expression, arrays, deets.getArrayVariables()); @@ -828,7 +829,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, new HashSet<>(identifiers), roundTripDeets.getRequiredBindings()); Assert.assertEquals(expression, scalars, roundTripDeets.getScalarVariables()); Assert.assertEquals(expression, arrays, roundTripDeets.getArrayVariables()); }