add a comment

Signed-off-by: Boyang Lyu <boyangly@gmail.com>
tikv · Aug 16, 2024 · 3c1af75 · 3c1af75
1 parent e3517c8
commit 3c1af75
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/tests/integrations/client/http_client_test.go b/tests/integrations/client/http_client_test.go
@@ -899,8 +899,8 @@ func (suite *httpClientTestSuite) TestGetGCSafePoint() {
 		re.Equal("Delete service GC safepoint successfully.", msg)
 	}
 
-	_, err4 := client.DeleteGCSafePoint(ctx, "gc_worker")
-	re.Error(err4)
+	_, err = client.DeleteGCSafePoint(ctx, "gc_worker")
+	re.Error(err)
 
 	// check that the safepoitns are indeed deleted
 	l, err = client.GetGCSafePoint(ctx)

diff --git a/tools/pd-ctl/tests/safepoint/safepoint_test.go b/tools/pd-ctl/tests/safepoint/safepoint_test.go
@@ -125,6 +125,6 @@ func TestSafepoint(t *testing.T) {
 	output, err = tests.ExecuteCommand(cmd, args...)
 	re.NoError(err)
 
-	// try again
+	// output should be an error message
 	re.Equal("Failed to delete service GC safepoint: request pd http api failed with status: '500 Internal Server Error', body: '\"cannot remove service safe point of gc_worker\"'\n", string(output))
 }