pull latest

acceptance/telemetry/upload-timeout/script

@@ -1,8 +1,8 @@
 export DATABRICKS_CLI_TELEMETRY_PID_FILE=./telemetry.pid
 export DATABRICKS_CLI_TELEMETRY_UPLOAD_LOGS_FILE=./out.upload_process.txt
 
-# Configure a timeout of 0 seconds. This ensures that the timeout is respected and the
-# telemetry process does not try to upload logs.
+# Configure a timeout of 0 seconds. This test validates that the timeout is respected
+# and the telemetry process does not try to upload logs.
 export DATABRICKS_CLI_TELEMETRY_UPLOAD_TIMEOUT="0s"
 
 trace $CLI selftest send-telemetry

cmd/root/root.go

@@ -176,7 +176,6 @@ func inheritEnvVars() []string {
 	return out
 }
 
-// TODO: Add tests validating the auth resolution in the telemetry worker.
 func uploadTelemetry(ctx context.Context, cmdStr string, start, end time.Time, exitCode int) {
 	// Nothing to upload.
 	if !telemetry.HasLogs(ctx) {

cmd/selftest/selftest.go

@@ -11,7 +11,6 @@ func New() *cobra.Command {
 		Hidden: true,
 	}
 
-	// TODO: Run the acceptance tests as integration tests?
 	cmd.AddCommand(newSendTelemetry())
 	return cmd
 }