fix: build reason

mtojek · mtojek · commit e6a5aef55cbd · 2024-07-15T16:05:51.000+02:00
diff --git a/coderd/provisionerdserver/provisionerdserver.go b/coderd/provisionerdserver/provisionerdserver.go
@@ -1100,7 +1100,7 @@ func (s *server) notifyWorkspaceBuildFailed(ctx context.Context, workspace datab
 	if build.Reason.Valid() && build.Reason == database.BuildReasonInitiator {
 		return // failed workspace build initiated by a user should not notify
 	}
-	reason = "initiated by autobuild"
+	reason = string(build.Reason)
 	initiator := "autobuild"
 
 	if _, err := s.NotificationEnqueuer.Enqueue(ctx, workspace.OwnerID, notifications.WorkspaceAutobuildFailed,
diff --git a/coderd/provisionerdserver/provisionerdserver_test.go b/coderd/provisionerdserver/provisionerdserver_test.go
@@ -1702,9 +1702,8 @@ func TestNotifications(t *testing.T) {
 		tests := []struct {
 			name string
 
-			buildReason           database.BuildReason
-			shouldNotify          bool
-			shouldDeleteWorkspace bool
+			buildReason  database.BuildReason
+			shouldNotify bool
 		}{
 			{
 				name:         "initiated by owner",
@@ -1793,7 +1792,6 @@ func TestNotifications(t *testing.T) {
 
 				workspace, err = db.GetWorkspaceByID(ctx, workspace.ID)
 				require.NoError(t, err)
-				require.Equal(t, tc.shouldDeleteWorkspace, workspace.Deleted)
 
 				if tc.shouldNotify {
 					// Validate that the notification was sent and contained the expected values.

Original file line number	Diff line number	Diff line change
`@@ -1100,7 +1100,7 @@ func (s *server) notifyWorkspaceBuildFailed(ctx context.Context, workspace datab`
`1100`	`1100`	`if build.Reason.Valid() && build.Reason == database.BuildReasonInitiator {`
`1101`	`1101`	`return // failed workspace build initiated by a user should not notify`
`1102`	`1102`	`}`
`1103`		`- reason = "initiated by autobuild"`
	`1103`	`+ reason = string(build.Reason)`
`1104`	`1104`	`initiator := "autobuild"`
`1105`	`1105`
`1106`	`1106`	`if _, err := s.NotificationEnqueuer.Enqueue(ctx, workspace.OwnerID, notifications.WorkspaceAutobuildFailed,`