Apply suggestions from code review

DariaBod · labkey-tchad · web-flow · commit 3978aab0effc · 2026-02-17T16:07:13.000-08:00
Co-authored-by: Trey Chadick &lt;tchad@labkey.com&gt;
diff --git a/src/org/labkey/test/components/ui/grids/ResponsiveGrid.java b/src/org/labkey/test/components/ui/grids/ResponsiveGrid.java
@@ -241,10 +241,11 @@ public String filterColumnExpectingError(CharSequence columnIdentifier, Filter.O
         return errorMsg;
     }
 
+private static final List<Filter.Operator> ARRAY_OPERATORS = List.of(ARRAY_CONTAINS_ALL, ARRAY_CONTAINS_ANY, ARRAY_CONTAINS_EXACT, ARRAY_CONTAINS_NONE,
+                ARRAY_CONTAINS_NOT_EXACT, ARRAY_ISEMPTY, ARRAY_ISNOTEMPTY);
+                
     private GridFilterModal initFilterColumn(CharSequence columnIdentifier, Filter.Operator operator, Object value)
     {
-        List<Filter.Operator> listOperators = List.of(IN, ARRAY_CONTAINS_ALL, ARRAY_CONTAINS_ANY, ARRAY_CONTAINS_EXACT, ARRAY_CONTAINS_NONE,
-                ARRAY_CONTAINS_NOT_EXACT, ARRAY_ISEMPTY, ARRAY_ISNOTEMPTY);
         clickColumnMenuItem(columnIdentifier, "Filter...", false);
         GridFilterModal filterModal = new GridFilterModal(getDriver(), this);
         if (operator != null)

Original file line number	Diff line number	Diff line change
`@@ -241,10 +241,11 @@ public String filterColumnExpectingError(CharSequence columnIdentifier, Filter.O`
`241`	`241`	`return errorMsg;`
`242`	`242`	`}`
`243`	`243`
	`244`	`+private static final List<Filter.Operator> ARRAY_OPERATORS = List.of(ARRAY_CONTAINS_ALL, ARRAY_CONTAINS_ANY, ARRAY_CONTAINS_EXACT, ARRAY_CONTAINS_NONE,`
	`245`	`+ ARRAY_CONTAINS_NOT_EXACT, ARRAY_ISEMPTY, ARRAY_ISNOTEMPTY);`
	`246`	`+`
`244`	`247`	`private GridFilterModal initFilterColumn(CharSequence columnIdentifier, Filter.Operator operator, Object value)`
`245`	`248`	`{`
`246`		`- List<Filter.Operator> listOperators = List.of(IN, ARRAY_CONTAINS_ALL, ARRAY_CONTAINS_ANY, ARRAY_CONTAINS_EXACT, ARRAY_CONTAINS_NONE,`
`247`		`- ARRAY_CONTAINS_NOT_EXACT, ARRAY_ISEMPTY, ARRAY_ISNOTEMPTY);`
`248`	`249`	`clickColumnMenuItem(columnIdentifier, "Filter...", false);`
`249`	`250`	`GridFilterModal filterModal = new GridFilterModal(getDriver(), this);`
`250`	`251`	`if (operator != null)`