More

internal/github/reporters_test.go

@@ -246,7 +246,6 @@ func TestCommitCommentReporter_report(t *testing.T) {
 					t.Fatalf("unexpected error: %v", err)
 				}
 				if diff := cmp.Diff(expected, comment); diff != "" {
-					//if !reflect.DeepEqual(expected, comment) {
 					t.Fatalf("expected cmt: (-have +want)\n%s", diff)
 				}
 			default:
@@ -268,3 +267,47 @@ func TestCommitCommentReporter_report(t *testing.T) {
 		}
 	}
 }
+
+func TestInlineCommitCommentReporter_report(t *testing.T) {
+	var (
+		expectedOwner   = "owner"
+		expectedRepo    = "repo"
+		expectedCmtBody = "body"
+		expectedCmtPath = "path"
+		expectedCmtPos  = 4
+		expectedSHA     = "abc123"
+	)
+
+	ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		decoder := json.NewDecoder(r.Body)
+		switch r.RequestURI {
+		case fmt.Sprintf("/repos/%v/%v/commits/%v/comments", expectedOwner, expectedRepo, expectedSHA):
+			expected := github.RepositoryComment{
+				Body:     github.String(expectedCmtBody),
+				Path:     github.String(expectedCmtPath),
+				Position: github.Int(expectedCmtPos),
+			}
+			var comment github.RepositoryComment
+			err := decoder.Decode(&comment)
+			if err != nil {
+				t.Fatalf("unexpected error: %v", err)
+			}
+			if diff := cmp.Diff(expected, comment); diff != "" {
+				t.Fatalf("expected cmt: (-have +want)\n%s", diff)
+			}
+		default:
+			t.Logf(r.RequestURI)
+		}
+	}))
+	defer ts.Close()
+
+	r := NewInlineCommitCommentReporter(github.NewClient(nil), expectedOwner, expectedRepo, expectedSHA)
+	r.client.BaseURL, _ = url.Parse(ts.URL)
+
+	var issues = []db.Issue{{Path: expectedCmtPath, HunkPos: expectedCmtPos, Issue: expectedCmtBody}}
+
+	err := r.Report(context.Background(), issues)
+	if err != nil {
+		t.Fatalf("unexpected error: %v", err)
+	}
+}