Support quoted table names (#2616)

## Why make this change?
- Fixes #2385 

## What is this change?
- In [SqlMetadataProvider.PopulateColumnDefinitionsWithReadOnlyFlag](https://github.com/Azure/data-api-builder/blob/c9e836b551eb6bc729e88e71b3aa4d1c1b235345/src/Core/Services/MetadataProviders/SqlMetadataProvider.cs#L1355) the QuoteIdentifier-method of the SqlQueryBuilder was not used to add quotes to the table name

## How was this tested?

- [ ] Integration Tests -> I'm happy to extend the integration tests, if this fix seems like a good idea
- [ ] Unit Tests
- [x] manually


## Sample Request(s)
Sample configuraiton: 
```
"MyTable": {
  "source": {
    "object": "schema.\"MyTable\"",
    "type": "table"
  }
}
```
---------

Co-authored-by: Thomas Fröhle <[email protected]>
Co-authored-by: Aniruddh Munde <[email protected]>
Co-authored-by: aaronburtle <[email protected]>
Co-authored-by: aaron burtle <[email protected]>

Author: 4 people

src/Core/Resolvers/DWSqlQueryBuilder.cs

@@ -432,5 +432,11 @@ private string BuildAggregationColumns(GroupByMetadata metadata)
         {
             return string.Join(", ", metadata.Aggregations.Select(aggregation => Build(aggregation.Column, useAlias: true)));
         }
+
+        public string QuoteTableNameAsDBConnectionParam(string param)
+        {
+            // Table names in DWSql should not be quoted when used as DB Connection Params.
+            return param;
+        }
     }
 }

src/Core/Resolvers/IQueryBuilder.cs

@@ -84,5 +84,11 @@ public interface IQueryBuilder
         /// Adds database specific quotes to string identifier
         /// </summary>
         public string QuoteIdentifier(string identifier);
+
+        /// <summary>
+        /// Adds database specific quotes to the table name when used as part of a
+        /// DB Connection Param.
+        /// </summary>
+        public string QuoteTableNameAsDBConnectionParam(string param);
     }
 }

src/Core/Resolvers/MsSqlQueryBuilder.cs

@@ -594,5 +594,11 @@ public string BuildFetchEnabledTriggersQuery()
 
             return query;
         }
+
+        public string QuoteTableNameAsDBConnectionParam(string param)
+        {
+            // Table names in MSSQL should not be quoted when used as DB Connection Params.
+            return param;
+        }
     }
 }

src/Core/Resolvers/MySqlQueryBuilder.cs

@@ -367,5 +367,11 @@ public string BuildStoredProcedureResultDetailsQuery(string databaseObjectName)
         {
             throw new NotImplementedException();
         }
+
+        public string QuoteTableNameAsDBConnectionParam(string param)
+        {
+            // Table names in MySQL should not be quoted when used as DB Connection Params.
+            return param;
+        }
     }
 }

src/Core/Resolvers/PostgresQueryBuilder.cs

@@ -241,5 +241,12 @@ public string BuildQueryToGetReadOnlyColumns(string schemaParamName, string tabl
                 $"WHERE attrelid = ({schemaParamName} || '.' || {tableParamName})::regclass AND attgenerated = 's';";
             return query;
         }
+
+        public string QuoteTableNameAsDBConnectionParam(string param)
+        {
+            // PostreSQL uses same quoting for table name as DB Connection Param
+            // as when used directly in SQL text.
+            return QuoteIdentifier(param);
+        }
     }
 }

src/Core/Services/MetadataProviders/SqlMetadataProvider.cs

@@ -1355,12 +1355,13 @@ private async Task PopulateSourceDefinitionAsync(
         private async Task PopulateColumnDefinitionsWithReadOnlyFlag(string tableName, string schemaOrDatabaseName, SourceDefinition sourceDefinition)
         {
             string schemaOrDatabaseParamName = $"{BaseQueryStructure.PARAM_NAME_PREFIX}param0";
+            string quotedTableName = SqlQueryBuilder.QuoteTableNameAsDBConnectionParam(tableName);
             string tableParamName = $"{BaseQueryStructure.PARAM_NAME_PREFIX}param1";
             string queryToGetReadOnlyColumns = SqlQueryBuilder.BuildQueryToGetReadOnlyColumns(schemaOrDatabaseParamName, tableParamName);
             Dictionary<string, DbConnectionParam> parameters = new()
             {
                 { schemaOrDatabaseParamName, new(schemaOrDatabaseName, DbType.String) },
-                { tableParamName, new(tableName, DbType.String) }
+                { tableParamName, new(quotedTableName, DbType.String) }
             };
 
             List<string>? readOnlyFields = await QueryExecutor.ExecuteQueryAsync(