Add acceptance tests for auth logout

mihaimitrea-db · mihaimitrea-db · commit 66871b3b82df · 2026-03-03T12:30:38.000Z
Cover four scenarios: profile ordering and comments are preserved after deletion, deleting the last non-default profile leaves an empty DEFAULT section, deleting the DEFAULT profile itself clears its keys and restores the default comment, and error paths for non-existent profiles and missing --profile in non-interactive mode.
diff --git a/acceptance/cmd/auth/logout/default-profile/out.databrickscfg b/acceptance/cmd/auth/logout/default-profile/out.databrickscfg
@@ -0,0 +1,7 @@
+; Dev workspace
+[dev]
+host  = https://dev.cloud.databricks.com
+token = dev-token
+
+; The profile defined in the DEFAULT section is to be used as a fallback when no profile is explicitly specified.
+[DEFAULT]
diff --git a/acceptance/cmd/auth/logout/default-profile/out.test.toml b/acceptance/cmd/auth/logout/default-profile/out.test.toml
diff --git a/acceptance/cmd/auth/logout/default-profile/output.txt b/acceptance/cmd/auth/logout/default-profile/output.txt
@@ -0,0 +1,27 @@
+
+=== Initial config
+; The profile defined in the DEFAULT section is to be used as a fallback when no profile is explicitly specified.
+[DEFAULT]
+host = https://default.cloud.databricks.com
+token = default-token
+
+; Dev workspace
+[dev]
+host = https://dev.cloud.databricks.com
+token = dev-token
+
+=== Delete the DEFAULT profile
+>>> [CLI] auth logout --profile DEFAULT --force
+Successfully logged out of profile "DEFAULT".
+
+=== Backup file should exist
+OK: Backup file exists
+
+=== Config after logout — empty DEFAULT with comment should remain at top
+; Dev workspace
+[dev]
+host  = https://dev.cloud.databricks.com
+token = dev-token
+
+; The profile defined in the DEFAULT section is to be used as a fallback when no profile is explicitly specified.
+[DEFAULT]
diff --git a/acceptance/cmd/auth/logout/default-profile/script b/acceptance/cmd/auth/logout/default-profile/script
@@ -0,0 +1,27 @@
+sethome "./home"
+
+cat > "./home/.databrickscfg" <<'EOF'
+; The profile defined in the DEFAULT section is to be used as a fallback when no profile is explicitly specified.
+[DEFAULT]
+host = https://default.cloud.databricks.com
+token = default-token
+
+; Dev workspace
+[dev]
+host = https://dev.cloud.databricks.com
+token = dev-token
+EOF
+
+title "Initial config\n"
+cat "./home/.databrickscfg"
+
+title "Delete the DEFAULT profile"
+trace $CLI auth logout --profile DEFAULT --force
+
+title "Backup file should exist\n"
+assert_backup_exists
+
+title "Config after logout — empty DEFAULT with comment should remain at top\n"
+cat "./home/.databrickscfg"
+
+cp "./home/.databrickscfg" "./out.databrickscfg"
diff --git a/acceptance/cmd/auth/logout/default-profile/test.toml b/acceptance/cmd/auth/logout/default-profile/test.toml
@@ -0,0 +1 @@
+
diff --git a/acceptance/cmd/auth/logout/error-cases/out.test.toml b/acceptance/cmd/auth/logout/error-cases/out.test.toml
diff --git a/acceptance/cmd/auth/logout/error-cases/output.txt b/acceptance/cmd/auth/logout/error-cases/output.txt
@@ -0,0 +1,8 @@
+
+=== Logout of non-existent profileError: profile "nonexistent" not found. Available profiles: [dev]
+
+Exit code: 1
+
+=== Logout without --profile in non-interactive modeError: the command is being run in a non-interactive environment, please specify a profile to log out of using --profile
+
+Exit code: 1
diff --git a/acceptance/cmd/auth/logout/error-cases/script b/acceptance/cmd/auth/logout/error-cases/script
@@ -0,0 +1,16 @@
+sethome "./home"
+
+cat > "./home/.databrickscfg" <<'EOF'
+; The profile defined in the DEFAULT section is to be used as a fallback when no profile is explicitly specified.
+[DEFAULT]
+
+[dev]
+host = https://dev.cloud.databricks.com
+token = dev-token
+EOF
+
+title "Logout of non-existent profile"
+errcode $CLI auth logout --profile nonexistent --force
+
+title "Logout without --profile in non-interactive mode"
+errcode $CLI auth logout --force
diff --git a/acceptance/cmd/auth/logout/error-cases/test.toml b/acceptance/cmd/auth/logout/error-cases/test.toml
@@ -0,0 +1 @@
+
diff --git a/acceptance/cmd/auth/logout/last-non-default/out.databrickscfg b/acceptance/cmd/auth/logout/last-non-default/out.databrickscfg
@@ -0,0 +1,2 @@
+; The profile defined in the DEFAULT section is to be used as a fallback when no profile is explicitly specified.
+[DEFAULT]
diff --git a/acceptance/cmd/auth/logout/last-non-default/out.test.toml b/acceptance/cmd/auth/logout/last-non-default/out.test.toml
diff --git a/acceptance/cmd/auth/logout/last-non-default/output.txt b/acceptance/cmd/auth/logout/last-non-default/output.txt
@@ -0,0 +1,20 @@
+
+=== Initial config
+; The profile defined in the DEFAULT section is to be used as a fallback when no profile is explicitly specified.
+[DEFAULT]
+
+; The only non-default profile
+[only-profile]
+host = https://only.cloud.databricks.com
+token = only-token
+
+=== Delete the only non-default profile
+>>> [CLI] auth logout --profile only-profile --force
+Successfully logged out of profile "only-profile".
+
+=== Backup file should exist
+OK: Backup file exists
+
+=== Config after logout — DEFAULT section with comment should remain
+; The profile defined in the DEFAULT section is to be used as a fallback when no profile is explicitly specified.
+[DEFAULT]
diff --git a/acceptance/cmd/auth/logout/last-non-default/script b/acceptance/cmd/auth/logout/last-non-default/script
@@ -0,0 +1,25 @@
+sethome "./home"
+
+cat > "./home/.databrickscfg" <<'EOF'
+; The profile defined in the DEFAULT section is to be used as a fallback when no profile is explicitly specified.
+[DEFAULT]
+
+; The only non-default profile
+[only-profile]
+host = https://only.cloud.databricks.com
+token = only-token
+EOF
+
+title "Initial config\n"
+cat "./home/.databrickscfg"
+
+title "Delete the only non-default profile"
+trace $CLI auth logout --profile only-profile --force
+
+title "Backup file should exist\n"
+assert_backup_exists
+
+title "Config after logout — DEFAULT section with comment should remain\n"
+cat "./home/.databrickscfg"
+
+cp "./home/.databrickscfg" "./out.databrickscfg"
diff --git a/acceptance/cmd/auth/logout/last-non-default/test.toml b/acceptance/cmd/auth/logout/last-non-default/test.toml
@@ -0,0 +1 @@
+
diff --git a/acceptance/cmd/auth/logout/ordering-preserved/out.databrickscfg b/acceptance/cmd/auth/logout/ordering-preserved/out.databrickscfg
@@ -0,0 +1,8 @@
+; The profile defined in the DEFAULT section is to be used as a fallback when no profile is explicitly specified.
+[DEFAULT]
+host = https://default.cloud.databricks.com
+
+; Second workspace — beta
+[beta]
+host  = https://beta.cloud.databricks.com
+token = beta-token
diff --git a/acceptance/cmd/auth/logout/ordering-preserved/out.test.toml b/acceptance/cmd/auth/logout/ordering-preserved/out.test.toml
diff --git a/acceptance/cmd/auth/logout/ordering-preserved/output.txt b/acceptance/cmd/auth/logout/ordering-preserved/output.txt
@@ -0,0 +1,59 @@
+
+=== Initial config
+; The profile defined in the DEFAULT section is to be used as a fallback when no profile is explicitly specified.
+[DEFAULT]
+host = https://default.cloud.databricks.com
+
+; First workspace — alpha
+[alpha]
+host = https://alpha.cloud.databricks.com
+token = alpha-token
+cluster_id = alpha-cluster
+
+; Second workspace — beta
+[beta]
+host = https://beta.cloud.databricks.com
+token = beta-token
+
+; Third workspace — gamma
+[gamma]
+host = https://gamma.cloud.databricks.com
+token = gamma-token
+warehouse_id = gamma-warehouse
+
+=== Delete first non-default profile (alpha)
+>>> [CLI] auth logout --profile alpha --force
+Successfully logged out of profile "alpha".
+
+=== Backup file should exist
+OK: Backup file exists
+
+=== Config after deleting alpha
+; The profile defined in the DEFAULT section is to be used as a fallback when no profile is explicitly specified.
+[DEFAULT]
+host = https://default.cloud.databricks.com
+
+; Second workspace — beta
+[beta]
+host  = https://beta.cloud.databricks.com
+token = beta-token
+
+; Third workspace — gamma
+[gamma]
+host         = https://gamma.cloud.databricks.com
+token        = gamma-token
+warehouse_id = gamma-warehouse
+
+=== Delete last profile (gamma)
+>>> [CLI] auth logout --profile gamma --force
+Successfully logged out of profile "gamma".
+
+=== Config after deleting gamma
+; The profile defined in the DEFAULT section is to be used as a fallback when no profile is explicitly specified.
+[DEFAULT]
+host = https://default.cloud.databricks.com
+
+; Second workspace — beta
+[beta]
+host  = https://beta.cloud.databricks.com
+token = beta-token
diff --git a/acceptance/cmd/auth/logout/ordering-preserved/script b/acceptance/cmd/auth/logout/ordering-preserved/script
@@ -0,0 +1,44 @@
+sethome "./home"
+
+cat > "./home/.databrickscfg" <<'EOF'
+; The profile defined in the DEFAULT section is to be used as a fallback when no profile is explicitly specified.
+[DEFAULT]
+host = https://default.cloud.databricks.com
+
+; First workspace — alpha
+[alpha]
+host = https://alpha.cloud.databricks.com
+token = alpha-token
+cluster_id = alpha-cluster
+
+; Second workspace — beta
+[beta]
+host = https://beta.cloud.databricks.com
+token = beta-token
+
+; Third workspace — gamma
+[gamma]
+host = https://gamma.cloud.databricks.com
+token = gamma-token
+warehouse_id = gamma-warehouse
+EOF
+
+title "Initial config\n"
+cat "./home/.databrickscfg"
+
+title "Delete first non-default profile (alpha)"
+trace $CLI auth logout --profile alpha --force
+
+title "Backup file should exist\n"
+assert_backup_exists
+
+title "Config after deleting alpha\n"
+cat "./home/.databrickscfg"
+
+title "Delete last profile (gamma)"
+trace $CLI auth logout --profile gamma --force
+
+title "Config after deleting gamma\n"
+cat "./home/.databrickscfg"
+
+cp "./home/.databrickscfg" "./out.databrickscfg"
diff --git a/acceptance/cmd/auth/logout/ordering-preserved/test.toml b/acceptance/cmd/auth/logout/ordering-preserved/test.toml
@@ -0,0 +1 @@
+
diff --git a/acceptance/cmd/auth/logout/script.prepare b/acceptance/cmd/auth/logout/script.prepare
@@ -0,0 +1,7 @@
+assert_backup_exists() {
+    if [ -f "./home/.databrickscfg.bak" ]; then
+        echo "OK: Backup file exists"
+    else
+        echo "ERROR: Backup file does not exist"
+    fi
+}
diff --git a/acceptance/cmd/auth/logout/test.toml b/acceptance/cmd/auth/logout/test.toml
@@ -0,0 +1,3 @@
+Ignore = [
+    "home"
+]

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+; The profile defined in the DEFAULT section is to be used as a fallback when no profile is explicitly specified.`
	`2`	`+[DEFAULT]`