Fix spacing

Emyrk · Emyrk · commit 05ca60ec21c2 · 2023-10-04T10:12:17.000-05:00
diff --git a/coderd/database/dbfake/dbfake.go b/coderd/database/dbfake/dbfake.go
@@ -6648,7 +6648,8 @@ func (q *FakeQuerier) GetAuthorizedWorkspaces(ctx context.Context, arg database.
 				statusMatch = job.JobStatus == database.ProvisionerJobStatusSucceeded &&
 					build.Transition == database.WorkspaceTransitionStop
 			case database.WorkspaceStatusRunning:
-				statusMatch = job.JobStatus == database.ProvisionerJobStatusSucceeded
+				statusMatch = job.JobStatus == database.ProvisionerJobStatusSucceeded &&
+					build.Transition == database.WorkspaceTransitionStart
 			default:
 				statusMatch = job.JobStatus == database.ProvisionerJobStatus(arg.Status)
 				if !statusMatch {
diff --git a/coderd/database/queries/workspaces.sql b/coderd/database/queries/workspaces.sql
@@ -154,10 +154,13 @@ WHERE
 					latest_build.job_status = 'succeeded'::provisioner_job_status AND
 					latest_build.transition = 'start'::workspace_transition
 
-			    -- Special case. A workspace status of "running" is a job status
-			    -- of "succeeded". This is annoying...
+			    -- Special case where the provisioner status and workspace status
+			    -- differ. A workspace is "running" if the job is "succeeded" and
+			    -- the transition is "start". This is because a workspace starts
+			    -- running when a job is complete.
 			    WHEN @status = 'running' THEN
-			    	latest_build.job_status = 'succeeded'::provisioner_job_status
+					latest_build.job_status = 'succeeded'::provisioner_job_status AND
+					latest_build.transition = 'start'::workspace_transition
 
 				WHEN @status != '' THEN
 				    -- By default just match the job status exactly
