Remove obvious comments from test code

Cleaned up redundant inline comments while keeping essential test-level documentation

Co-authored-by: silverwind <115237+silverwind@users.noreply.github.com>

Author: Copilot

services/issue/commit_test.go

@@ -307,55 +307,50 @@ func TestUpdateIssuesCommit_SelfReference(t *testing.T) {
 	// PR #2 (issue_id=2) has merged_commit_id: 1a8823cd1a9549fde083f992f6b9b87a7ab74fb3
 	pushCommits := []*repository.PushCommit{
 		{
-			Sha1:           "1a8823cd1a9549fde083f992f6b9b87a7ab74fb3", // This is the merge commit for PR #2
+			Sha1:           "1a8823cd1a9549fde083f992f6b9b87a7ab74fb3",
 			CommitterEmail: "user2@example.com",
 			CommitterName:  "User Two",
 			AuthorEmail:    "user2@example.com",
 			AuthorName:     "User Two",
-			Message:        "Merge pull request 'issue2' (#2) from branch1 into master", // References its own PR
+			Message:        "Merge pull request 'issue2' (#2) from branch1 into master",
 		},
 	}
 
 	repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
 
-	// This comment should NOT be created (self-reference)
 	selfRefCommentBean := &issues_model.Comment{
 		Type:      issues_model.CommentTypeCommitRef,
 		CommitSHA: "1a8823cd1a9549fde083f992f6b9b87a7ab74fb3",
 		PosterID:  user.ID,
-		IssueID:   2, // PR #2 references itself
+		IssueID:   2,
 	}
 
 	unittest.AssertNotExistsBean(t, selfRefCommentBean)
 	assert.NoError(t, UpdateIssuesCommit(t.Context(), user, repo, pushCommits, repo.DefaultBranch))
-	// The self-reference comment should still not exist
 	unittest.AssertNotExistsBean(t, selfRefCommentBean)
 	unittest.CheckConsistencyFor(t, &activities_model.Action{})
 
-	// Test that normal (non-self-referencing) commit comments are still created
-	// Use a different commit that is NOT a merge commit for any PR
+	// Test that normal commit references are still created
 	pushCommits2 := []*repository.PushCommit{
 		{
-			Sha1:           "abcdef9876543210", // Different commit, not a merge commit
+			Sha1:           "abcdef9876543210",
 			CommitterEmail: "user2@example.com",
 			CommitterName:  "User Two",
 			AuthorEmail:    "user2@example.com",
 			AuthorName:     "User Two",
-			Message:        "Fix bug, refs #1", // References issue #1
+			Message:        "Fix bug, refs #1",
 		},
 	}
 
-	// This comment SHOULD be created (reference to a different issue from a non-merge commit)
 	otherRefCommentBean := &issues_model.Comment{
 		Type:      issues_model.CommentTypeCommitRef,
 		CommitSHA: "abcdef9876543210",
 		PosterID:  user.ID,
-		IssueID:   1, // References issue #1
+		IssueID:   1,
 	}
 
 	unittest.AssertNotExistsBean(t, otherRefCommentBean)
 	assert.NoError(t, UpdateIssuesCommit(t.Context(), user, repo, pushCommits2, repo.DefaultBranch))
-	// The reference to issue #1 should exist
 	unittest.AssertExistsAndLoadBean(t, otherRefCommentBean)
 	unittest.CheckConsistencyFor(t, &activities_model.Action{})
 }