fix: fix formatting with gofmt (remove trailing whitespace)

Claude · Claude · commit ba268a78511d · 2025-03-13T15:37:47.000Z
diff --git a/cli/server.go b/cli/server.go
@@ -420,40 +420,40 @@ func (r *RootCmd) Server(newAPI func(context.Context, *coderd.Options) (*coderd.
 			config := r.createConfig()
 
 			builtinPostgres := false
-			
+
 			// Check if we have individual PostgreSQL connection parameters
-			hasIndividualParams := len(vals.PostgresHost.String()) > 0 && 
-				len(vals.PostgresUsername.String()) > 0 && 
-				len(vals.PostgresPassword.String()) > 0 && 
+			hasIndividualParams := len(vals.PostgresHost.String()) > 0 &&
+				len(vals.PostgresUsername.String()) > 0 &&
+				len(vals.PostgresPassword.String()) > 0 &&
 				len(vals.PostgresDatabase.String()) > 0
-			
+
 			// Build a connection URL from individual components if provided and no connection URL exists
 			if !vals.InMemoryDatabase.Value() && vals.PostgresURL == "" && hasIndividualParams {
 				port := vals.PostgresPort.String()
 				if port == "" {
 					port = "5432" // Default PostgreSQL port
 				}
-				
+
 				// Build the base connection string
-				connURL := fmt.Sprintf("postgres://%s:%s@%s:%s/%s", 
+				connURL := fmt.Sprintf("postgres://%s:%s@%s:%s/%s",
 					vals.PostgresUsername.String(),
 					vals.PostgresPassword.String(),
 					vals.PostgresHost.String(),
 					port,
 					vals.PostgresDatabase.String())
-				
+
 				// Add options if provided
 				if len(vals.PostgresOptions.String()) > 0 {
 					connURL = connURL + "?" + vals.PostgresOptions.String()
 				}
-				
+
 				logger.Debug(ctx, "using individual PostgreSQL connection parameters")
 				err = vals.PostgresURL.Set(connURL)
 				if err != nil {
 					return xerrors.Errorf("set postgres url from components: %w", err)
 				}
 			}
-			
+
 			// Only use built-in if PostgreSQL URL and individual parameters aren't specified!
 			if !vals.InMemoryDatabase.Value() && vals.PostgresURL == "" && !hasIndividualParams {
 				var closeFunc func() error
@@ -2770,7 +2770,7 @@ func getAndMigratePostgresDB(ctx context.Context, logger slog.Logger, postgresUR
 	// The postgresURL is constructed earlier by caller from either the
 	// CODER_PG_CONNECTION_URL or the individual components (host, port, etc.)
 	// So we just validate it here and don't need to reconstruct it
-	
+
 	dbURL, err := escapePostgresURLUserInfo(postgresURL)
 	if err != nil {
 		return nil, "", xerrors.Errorf("escaping postgres URL: %w", err)
diff --git a/cli/server_test.go b/cli/server_test.go
@@ -180,24 +180,24 @@ func TestServer(t *testing.T) {
 			return err == nil && rawURL != ""
 		}, superDuperLong, testutil.IntervalFast, "failed to get access URL")
 	})
-	
+
 	t.Run("PostgresConnectionParams", func(t *testing.T) {
 		if testing.Short() {
 			t.SkipNow()
 		}
 		t.Parallel()
-		
+
 		// Use URL of built-in test database
 		dbURL, err := dbtestutil.Open(t)
 		require.NoError(t, err)
-		
-		// Rather than testing individual parameters directly (which would require access to 
+
+		// Rather than testing individual parameters directly (which would require access to
 		// unexported struct fields), test the CLI flags with the actual server
-		
+
 		// Parse the URL to extract components
 		u, err := url.Parse(dbURL)
 		require.NoError(t, err, "Failed to parse database URL")
-		
+
 		// Extract components
 		username := u.User.Username()
 		password, _ := u.User.Password()
@@ -207,11 +207,11 @@ func TestServer(t *testing.T) {
 			port = "5432" // Use default PostgreSQL port
 		}
 		database := strings.TrimPrefix(u.Path, "/")
-		
+
 		// Test with CLI flags
 		t.Run("WithFlags", func(t *testing.T) {
 			t.Parallel()
-			
+
 			inv, cfg := clitest.New(t,
 				"server",
 				"--http-address", ":0",
@@ -223,40 +223,40 @@ func TestServer(t *testing.T) {
 				"--postgres-database", database,
 				"--postgres-options", "sslmode=disable",
 			)
-			
+
 			const longTimeout = testutil.WaitLong * 3
 			ctx := testutil.Context(t, longTimeout)
 			clitest.Start(t, inv.WithContext(ctx))
-			
+
 			//nolint:gocritic // Test server takes a while to start
 			require.Eventually(t, func() bool {
 				rawURL, err := cfg.URL().Read()
 				return err == nil && rawURL != ""
 			}, longTimeout, testutil.IntervalFast, "failed to get access URL with individual PostgreSQL parameters")
 		})
-		
+
 		// Test with environment variables
 		t.Run("WithEnvironmentVariables", func(t *testing.T) {
 			t.Parallel()
-			
+
 			inv, cfg := clitest.New(t,
 				"server",
 				"--http-address", ":0",
 				"--access-url", "http://example.com",
 			)
-			
+
 			// Set environment variables
 			inv.Environ.Set("CODER_PG_HOST", host)
 			inv.Environ.Set("CODER_PG_PORT", port)
 			inv.Environ.Set("CODER_PG_USERNAME", username)
 			inv.Environ.Set("CODER_PG_PASSWORD", password)
 			inv.Environ.Set("CODER_PG_DATABASE", database)
 			inv.Environ.Set("CODER_PG_OPTIONS", "sslmode=disable")
-			
+
 			const longTimeout = testutil.WaitLong * 3
 			ctx := testutil.Context(t, longTimeout)
 			clitest.Start(t, inv.WithContext(ctx))
-			
+
 			//nolint:gocritic // Test server takes a while to start
 			require.Eventually(t, func() bool {
 				rawURL, err := cfg.URL().Read()
