address some comments

Author: Tejas-Kochar

databricks-sdk-java/src/main/java/com/databricks/sdk/core/ConfigAttributeAccessor.java

@@ -69,7 +69,7 @@ public void setValueOnConfig(DatabricksConfig cfg, String value) throws IllegalA
         field.set(cfg, ProxyConfig.ProxyAuthType.valueOf(value));
       } else if (List.class.isAssignableFrom(field.getType())) {
         // Handle List<String> fields (e.g., scopes)
-        // Parse comma and/or whitespace separated values from environment variable or config file
+        // Parse comma and/or whitespace separated values from environment variable or config file.
         if (field.getGenericType() instanceof ParameterizedType) {
           ParameterizedType paramType = (ParameterizedType) field.getGenericType();
           if (paramType.getActualTypeArguments().length > 0
@@ -111,7 +111,10 @@ public String toString() {
 
   public String getAsString(Object value) {
     if (value instanceof List) {
-      return String.join(" ", (List<String>) value);
+      List<?> list = (List<?>) value;
+      return list.stream()
+          .map(Object::toString)
+          .collect(Collectors.joining(", ", "[", "]"));
     }
     return value.toString();
   }

databricks-sdk-java/src/main/java/com/databricks/sdk/core/DatabricksConfig.java

@@ -213,7 +213,7 @@ private synchronized DatabricksConfig innerResolve() {
     }
   }
 
-  /** Sort scopes in-place for better de-duplication in the refresh token cache. */
+  // Sort scopes in-place for better de-duplication in the refresh token cache.
   private void sortScopes() {
     if (scopes != null && !scopes.isEmpty()) {
       java.util.Collections.sort(scopes);

databricks-sdk-java/src/test/java/com/databricks/sdk/core/DatabricksConfigTest.java

@@ -329,7 +329,6 @@ public void testDisableOauthRefreshTokenEnvironmentVariable() {
   }
 
   // Config File Scope Parsing Tests
-
   private static Stream<Arguments> provideConfigFileScopesTestCases() {
     return Stream.of(
         Arguments.of("Empty scopes defaults to all-apis", "scope-empty", Arrays.asList("all-apis")),
@@ -357,9 +356,6 @@ public void testConfigFileScopes(String testName, String profile, List<String> e
     config.resolve(new Environment(env, new ArrayList<>(), System.getProperty("os.name")));
 
     List<String> scopes = config.getScopes();
-    assertEquals(expectedScopes.size(), scopes.size());
-    for (int i = 0; i < expectedScopes.size(); i++) {
-      assertEquals(expectedScopes.get(i), scopes.get(i));
-    }
+    assertIterableEquals(expectedScopes, scopes);
   }
 }