Review comments

dannykopping · dannykopping · commit ac85d98cee15 · 2024-07-10T11:11:34.000Z
Signed-off-by: Danny Kopping &lt;danny@coder.com&gt;
diff --git a/coderd/notifications/notifications_test.go b/coderd/notifications/notifications_test.go
@@ -217,7 +217,7 @@ func TestBackpressure(t *testing.T) {
 
 	// SETUP
 	if !dbtestutil.WillUsePostgres() {
-		t.Skip("This test requires postgres; it relies on business-logic implemented in the database")
+		t.Skip("This test requires postgres; it relies on business-logic only implemented in the database")
 	}
 
 	ctx, logger, db := setup(t)
@@ -305,7 +305,7 @@ func TestRetries(t *testing.T) {
 
 	// SETUP
 	if !dbtestutil.WillUsePostgres() {
-		t.Skip("This test requires postgres; it relies on business-logic implemented in the database")
+		t.Skip("This test requires postgres; it relies on business-logic only implemented in the database")
 	}
 
 	const maxAttempts = 3
@@ -397,7 +397,7 @@ func TestExpiredLeaseIsRequeued(t *testing.T) {
 
 	// SETUP
 	if !dbtestutil.WillUsePostgres() {
-		t.Skip("This test requires postgres; it relies on business-logic implemented in the database")
+		t.Skip("This test requires postgres; it relies on business-logic only implemented in the database")
 	}
 
 	ctx, logger, db := setup(t)

Original file line number	Diff line number	Diff line change
`@@ -217,7 +217,7 @@ func TestBackpressure(t *testing.T) {`
`217`	`217`
`218`	`218`	`// SETUP`
`219`	`219`	`if !dbtestutil.WillUsePostgres() {`
`220`		`- t.Skip("This test requires postgres; it relies on business-logic implemented in the database")`
	`220`	`+ t.Skip("This test requires postgres; it relies on business-logic only implemented in the database")`
`221`	`221`	`}`
`222`	`222`
`223`	`223`	`ctx, logger, db := setup(t)`
`@@ -305,7 +305,7 @@ func TestRetries(t *testing.T) {`
`305`	`305`
`306`	`306`	`// SETUP`
`307`	`307`	`if !dbtestutil.WillUsePostgres() {`
`308`		`- t.Skip("This test requires postgres; it relies on business-logic implemented in the database")`
	`308`	`+ t.Skip("This test requires postgres; it relies on business-logic only implemented in the database")`
`309`	`309`	`}`
`310`	`310`
`311`	`311`	`const maxAttempts = 3`
`@@ -397,7 +397,7 @@ func TestExpiredLeaseIsRequeued(t *testing.T) {`
`397`	`397`
`398`	`398`	`// SETUP`
`399`	`399`	`if !dbtestutil.WillUsePostgres() {`
`400`		`- t.Skip("This test requires postgres; it relies on business-logic implemented in the database")`
	`400`	`+ t.Skip("This test requires postgres; it relies on business-logic only implemented in the database")`
`401`	`401`	`}`
`402`	`402`
`403`	`403`	`ctx, logger, db := setup(t)`