Fixing tests

dannykopping · dannykopping · commit 28fa2f7868dd · 2024-08-15T15:36:20.000+02:00
Signed-off-by: Danny Kopping &lt;danny@coder.com&gt;
diff --git a/coderd/database/dbauthz/dbauthz_test.go b/coderd/database/dbauthz/dbauthz_test.go
@@ -2463,6 +2463,13 @@ func (s *MethodTestSuite) TestSystemFunctions() {
 			JobID: j.ID,
 		}).Asserts( /*rbac.ResourceSystem, policy.ActionCreate*/ )
 	}))
+	s.Run("InsertProvisionerJobTimings", s.Subtest(func(db database.Store, check *expects) {
+		// TODO: we need to create a ProvisionerJob resource
+		j := dbgen.ProvisionerJob(s.T(), db, nil, database.ProvisionerJob{})
+		check.Args(database.InsertProvisionerJobTimingsParams{
+			JobID: j.ID,
+		}).Asserts( /*rbac.ResourceSystem, policy.ActionCreate*/ )
+	}))
 	s.Run("UpsertProvisionerDaemon", s.Subtest(func(db database.Store, check *expects) {
 		org := dbgen.Organization(s.T(), db, database.Organization{})
 		pd := rbac.ResourceProvisionerDaemon.InOrg(org.ID)
diff --git a/coderd/workspacebuilds.go b/coderd/workspacebuilds.go
@@ -1011,21 +1011,3 @@ func convertWorkspaceStatus(jobStatus codersdk.ProvisionerJobStatus, transition
 	// return error status since we should never get here
 	return codersdk.WorkspaceStatusFailed
 }
-
-func convertWorkspaceBuildStats(stats database.ProvisionerJobStat) codersdk.WorkspaceBuildStats {
-	return codersdk.WorkspaceBuildStats{
-		ID:             stats.JobID,
-		JobStatus:      codersdk.ProvisionerJobStatus(stats.JobStatus),
-		WorkerID:       stats.WorkspaceID,
-		WorkspaceID:    stats.WorkspaceID,
-		Error:          stats.Error.String,
-		ErrorCode:      stats.ErrorCode.String,
-		UpdatedAt:      stats.UpdatedAt,
-		QueuedSecs:     stats.QueuedSecs,
-		CompletionSecs: stats.CompletionSecs,
-		CanceledSecs:   stats.CanceledSecs,
-		InitSecs:       stats.InitSecs,
-		PlanSecs:       stats.PlanSecs,
-		ApplySecs:      stats.ApplySecs,
-	}
-}
diff --git a/provisioner/terraform/timings_test.go b/provisioner/terraform/timings_test.go
@@ -18,6 +18,8 @@ import (
 // This log content is then used to usher the provisioning process along as if terraform has run, and consequently
 // the timing data is extracted from the log content and validated against the expected values.
 func TestTimingsFromProvision(t *testing.T) {
+	t.Parallel()
+
 	cwd, err := os.Getwd()
 	require.NoError(t, err)
 
diff --git a/site/e2e/google/protobuf/timestampGenerated.ts b/site/e2e/google/protobuf/timestampGenerated.ts
