From b7789cfe773249748bf5d4f4c4b419859623ffbb Mon Sep 17 00:00:00 2001
From: BrunoQuaresma <bruno_nonato_quaresma@hotmail.com>
Date: Tue, 15 Oct 2024 12:23:26 +0000
Subject: [PATCH] test(coderd): fix flake on TestWorkspaceBuildTimings

---
 coderd/workspacebuilds_test.go | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/coderd/workspacebuilds_test.go b/coderd/workspacebuilds_test.go
index 580b01fdec1a9..87c0dfe6b43ee 100644
--- a/coderd/workspacebuilds_test.go
+++ b/coderd/workspacebuilds_test.go
@@ -1214,16 +1214,15 @@ func TestWorkspaceBuildTimings(t *testing.T) {
 		ActiveVersionID: version.ID,
 		CreatedBy:       owner.UserID,
 	})
-	ws := dbgen.Workspace(t, db, database.Workspace{
-		OwnerID:        owner.UserID,
-		OrganizationID: owner.OrganizationID,
-		TemplateID:     template.ID,
-	})
 
-	// Create a build to attach timings
-	var buildNumber int32
+	// Tests will run in parallel. To avoid conflicts and race conditions on the
+	// build number, each test will have its own workspace and build.
 	makeBuild := func() database.WorkspaceBuild {
-		buildNumber++
+		ws := dbgen.Workspace(t, db, database.Workspace{
+			OwnerID:        owner.UserID,
+			OrganizationID: owner.OrganizationID,
+			TemplateID:     template.ID,
+		})
 		jobID := uuid.New()
 		job := dbgen.ProvisionerJob(t, db, pubsub, database.ProvisionerJob{
 			ID:             jobID,
@@ -1235,7 +1234,7 @@ func TestWorkspaceBuildTimings(t *testing.T) {
 			TemplateVersionID: version.ID,
 			InitiatorID:       owner.UserID,
 			JobID:             job.ID,
-			BuildNumber:       buildNumber,
+			BuildNumber:       1,
 		})
 	}