Add random name suffix to SQL Server test tables

Multiple test functions create tables with names from
testTableNameTestData, which can cause flaky failures when running them
concurrently. Adding these random suffixes should make sure the table
names never collide.

Author: LBPeraza

plugin/trino-sqlserver/src/test/java/io/trino/plugin/sqlserver/TestSqlServerConnectorTest.java

@@ -178,6 +178,7 @@ private void testInsertWriteBulkinessWithTimestamps(String timestampType)
     public void testCreateAndDropTableWithSpecialCharacterName()
     {
         for (String tableName : testTableNameTestData()) {
+            tableName = addNameSuffix(tableName);
             String tableNameInSql = "\"" + tableName.replace("\"", "\"\"") + "\"";
             // Until https://github.com/trinodb/trino/issues/17 the table name is effectively lowercase
             tableName = tableName.toLowerCase(ENGLISH);
@@ -215,6 +216,7 @@ public void testRenameColumnNameAdditionalTests()
     public void testRenameFromToTableWithSpecialCharacterName()
     {
         for (String tableName : testTableNameTestData()) {
+            tableName = addNameSuffix(tableName);
             String tableNameInSql = "\"" + tableName.replace("\"", "\"\"") + "\"";
             String sourceTableName = "test_rename_source_" + randomNameSuffix();
             assertUpdate("CREATE TABLE " + sourceTableName + " AS SELECT 123 x", 1);
@@ -276,4 +278,14 @@ private List<String> testTableNameTestData()
                 .add("close]bracket")
                 .build();
     }
+
+    /**
+     * Returns a new name with a random suffix which maintains the shape of the name.
+     *
+     * <p>In particular, the suffix is added before any trailing spaces.
+     */
+    private static String addNameSuffix(String name)
+    {
+        return name.replaceFirst("(\\s*)$", randomNameSuffix() + "$1");
+    }
 }