Skip to content

Commit a132cda

Browse files
committed
fixes
1 parent 571b07c commit a132cda

1 file changed

Lines changed: 13 additions & 6 deletions

File tree

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

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -647,7 +647,9 @@ public void testEnsureResolvedResolvesHostMetadataWhenUnifiedHost() throws IOExc
647647
try (FixtureServer server =
648648
new FixtureServer().with("GET", "/.well-known/databricks-config", response, 200)) {
649649
DatabricksConfig config =
650-
new DatabricksConfig().setHost(server.getUrl()).setExperimentalIsUnifiedHost(true);
650+
new DatabricksConfig()
651+
.setHost(server.getUrl())
652+
.setExperimentalIsUnifiedHost(true);
651653
config.resolve(emptyEnv());
652654
assertEquals(DUMMY_ACCOUNT_ID, config.getAccountId());
653655
assertEquals(DUMMY_WORKSPACE_ID, config.getWorkspaceId());
@@ -672,10 +674,11 @@ public void testEnsureResolvedSkipsHostMetadataWhenNotUnified() throws IOExcepti
672674
public void testEnsureResolvedHostMetadataFailureNonFatal() throws IOException {
673675
try (FixtureServer server =
674676
new FixtureServer()
675-
.with(
676-
"GET", "/.well-known/databricks-config", "{\"error\": \"internal error\"}", 500)) {
677+
.with("GET", "/.well-known/databricks-config", "{\"error\": \"internal error\"}", 500)) {
677678
DatabricksConfig config =
678-
new DatabricksConfig().setHost(server.getUrl()).setExperimentalIsUnifiedHost(true);
679+
new DatabricksConfig()
680+
.setHost(server.getUrl())
681+
.setExperimentalIsUnifiedHost(true);
679682
// Should not throw — metadata failure is non-fatal
680683
config.resolve(emptyEnv());
681684
assertNull(config.getAccountId());
@@ -689,7 +692,9 @@ public void testEnsureResolvedHostMetadataNoOidcEndpointNonFatal() throws IOExce
689692
try (FixtureServer server =
690693
new FixtureServer().with("GET", "/.well-known/databricks-config", response, 200)) {
691694
DatabricksConfig config =
692-
new DatabricksConfig().setHost(server.getUrl()).setExperimentalIsUnifiedHost(true);
695+
new DatabricksConfig()
696+
.setHost(server.getUrl())
697+
.setExperimentalIsUnifiedHost(true);
693698
config.resolve(emptyEnv());
694699
assertEquals(DUMMY_ACCOUNT_ID, config.getAccountId());
695700
assertNull(config.getDiscoveryUrl());
@@ -704,7 +709,9 @@ public void testEnsureResolvedHostMetadataMissingAccountIdWithPlaceholderNonFata
704709
try (FixtureServer server =
705710
new FixtureServer().with("GET", "/.well-known/databricks-config", response, 200)) {
706711
DatabricksConfig config =
707-
new DatabricksConfig().setHost(server.getUrl()).setExperimentalIsUnifiedHost(true);
712+
new DatabricksConfig()
713+
.setHost(server.getUrl())
714+
.setExperimentalIsUnifiedHost(true);
708715
config.resolve(emptyEnv());
709716
// DiscoveryURL should not be set because account_id is empty and placeholder can't be
710717
// substituted

0 commit comments

Comments
 (0)