review: trim redundant comments on mergeLabels and its regression test

Author: james00012

modules/gcp/compute.go

@@ -442,8 +442,7 @@ func (i *Instance) SetLabelsWithClient(ctx context.Context, service *compute.Ser
 	return nil
 }
 
-// mergeLabels merges new key-value pairs into existing labels, preserving any keys not in the new set.
-// Keys present in newLabels overwrite the existing values.
+// mergeLabels returns a new map with existing labels overlaid by newLabels.
 func mergeLabels(existing, newLabels map[string]string) map[string]string {
 	merged := make(map[string]string, len(existing)+len(newLabels))
 	for k, v := range existing {

modules/gcp/compute_test.go

@@ -195,9 +195,6 @@ func TestSetLabelsWithClient(t *testing.T) {
 	require.NoError(t, inst.SetLabelsWithClient(context.Background(), svc, map[string]string{"env": "unit"}))
 }
 
-// TestSetLabelsWithClientMergesExisting — regression test for the SetLabels-clobbers-existing bug.
-// The instance already carries a label; SetLabels should merge the new label in, not drop the
-// existing one.
 func TestSetLabelsWithClientMergesExisting(t *testing.T) {
 	t.Parallel()