Issue 53197: randomFieldName() to include WIDE_PLACEHOLDER

Author: cnathe

src/org/labkey/test/tests/component/GridPanelTest.java

@@ -64,11 +64,11 @@ public class GridPanelTest extends GridPanelBaseTest
     // Column names.
     private static final String FILTER_NAME_COL = "Name";
     private static final String FILTER_EXPDATE_COL = "Expiration Date";
-    private static final String FILTER_STRING_COL = "Str";
-    private static final String FILTER_INT_COL = "Int";
-    private static final String FILTER_EXTEND_CHAR_COL = "\u0106\u00D8\u0139";
-    private static final String FILTER_BOOL_COL = "Bool";
-    private static final String FILTER_DATE_COL = "Date";
+    private static final String FILTER_STRING_COL = TestDataGenerator.randomFieldName("Str", 0, 5);
+    private static final String FILTER_INT_COL = TestDataGenerator.randomFieldName("Int", 0, 5);
+    private static final String FILTER_EXTEND_CHAR_COL = TestDataGenerator.randomFieldName("\u0106\u00D8\u0139", 0, 5);
+    private static final String FILTER_BOOL_COL = TestDataGenerator.randomFieldName("Bool", 0, 5);
+    private static final String FILTER_DATE_COL = TestDataGenerator.randomFieldName("Date", 0, 5);
     private static final String FILTER_STORED_AMOUNT_COL = "Amount";
 
     // Views and columns used in the views. The views are only applied to the small sample type (Small_SampleType).

src/org/labkey/test/util/TestDataGenerator.java

@@ -576,12 +576,11 @@ public static String randomFieldName(String part, int numStartChars, int numEndC
     public static String randomFieldName(@NotNull String part, int numStartChars, int numEndChars, @Nullable String exclusion)
     {
         // use the characters that we know are encoded in fieldKeys plus characters that we know clients are using
-        // Issue 53197: Field name with double byte character can cause client side exception in Firefox when trying to customize grid view.
-        String chars = ALL_ILLEGAL_QUERY_KEY_CHARACTERS + " %()=+-[]_|*`'\":;<>?!@#^" + NON_LATIN_STRING;// + WIDE_PLACEHOLDER ;
+        String chars = ALL_ILLEGAL_QUERY_KEY_CHARACTERS + " %()=+-[]_|*`'\":;<>?!@#^" + NON_LATIN_STRING + WIDE_PLACEHOLDER ;
 
         String randomFieldName = randomName(part, numStartChars, numEndChars, chars, exclusion);
         TestLogger.log("Generated random field name: " + randomFieldName);
-        return randomFieldName;
+        return randomFieldName + WIDE_PLACEHOLDER; // TODO remove this hardcoded WIDE_PLACEHOLDER after a few TC runs
     }
 
     public static String randomChoice(List<String> choices)