feat(pr): remove diff comment (#103)

cyberspace7 · Jan 26, 2024 · 0e4a16a · 0e4a16a
1 parent 611452b
commit 0e4a16a
Show file tree

Hide file tree

Showing 10 changed files with 20 additions and 1,770 deletions.
diff --git a/__tests__/libs/release-notes.test.ts b/__tests__/libs/release-notes.test.ts
@@ -1,7 +1,4 @@
-import {
-  createReleasePullRequestBody,
-  getDiffMarkdown,
-} from "../../src/libs/release-notes";
+import { createReleasePullRequestBody } from "../../src/libs/release-notes";
 
 describe("createReleasePullRequestBody()", () => {
   it("should return notes", () => {
@@ -47,41 +44,3 @@ describe("createReleasePullRequestBody()", () => {
     expect(result).toEqual("Line 1\n## Line 2\n");
   });
 });
-
-describe("getDiffMarkdown()", () => {
-  it("should return diff when adds", () => {
-    const oldContent = ["Line 1", "Line 2"].join("\n");
-    const newContent = ["Line 1", "Line 2", "Line 3"].join("\n");
-
-    const result = getDiffMarkdown(oldContent, newContent);
-
-    expect(result).toEqual("```diff\n+  Line 3\n```");
-  });
-
-  it("should return diff when changes", () => {
-    const oldContent = ["Line 1", "Line 2", "", "Line 3"].join("\n");
-    const newContent = ["Line 1", "Line 1a", "", "Line 3"].join("\n");
-
-    const result = getDiffMarkdown(oldContent, newContent);
-
-    expect(result).toEqual("```diff\n-  Line 2\n+  Line 1a\n```");
-  });
-
-  it("should return `null` when no changes", () => {
-    const oldContent = ["Line 1", "Line 2", "", "Line 3"].join("\n");
-    const newContent = ["Line 1", "Line 2", "", "Line 3"].join("\n");
-
-    const result = getDiffMarkdown(oldContent, newContent);
-
-    expect(result).toEqual(null);
-  });
-
-  it("should return `null` when empty", () => {
-    const oldContent = "";
-    const newContent = " ";
-
-    const result = getDiffMarkdown(oldContent, newContent);
-
-    expect(result).toEqual(null);
-  });
-});
diff --git a/__tests__/prepare.test.ts b/__tests__/prepare.test.ts
@@ -606,13 +606,7 @@ describe("main()", () => {
         pull_number: 1,
         body: "Release notes\n\n- title (#4)\n- title (#5)\n- title (#6)\n- title (#7)\n\nThe end.",
       });
-      expect(octokit.rest.issues.createComment).toHaveBeenCalledTimes(1);
-      expect(octokit.rest.issues.createComment).toHaveBeenCalledWith({
-        owner: "owner",
-        repo: "repository",
-        issue_number: 1,
-        body: "Content has been updated:\n\n```diff\n+  - title (#6)\n+  - title (#7)\n```",
-      });
+      expect(octokit.rest.issues.createComment).toHaveBeenCalledTimes(0);
     });
 
     it("should prepare when changes (new version)", async () => {
@@ -691,13 +685,7 @@ describe("main()", () => {
         pull_number: 1,
         body: "Release notes\n\n- title (#4)\n- title (#5)\n- title (#6)\n- title (#7)\n\nThe end.",
       });
-      expect(octokit.rest.issues.createComment).toHaveBeenCalledTimes(1);
-      expect(octokit.rest.issues.createComment).toHaveBeenCalledWith({
-        owner: "owner",
-        repo: "repository",
-        issue_number: 1,
-        body: "Content has been updated:\n\n```diff\n+  - title (#6)\n+  - title (#7)\n```",
-      });
+      expect(octokit.rest.issues.createComment).toHaveBeenCalledTimes(0);
     });
 
     it("should prepare when only change is pre-release", async () => {
@@ -762,7 +750,7 @@ describe("main()", () => {
         owner: "owner",
         repo: "repository",
         issue_number: 1,
-        body: "Version `1.3.0-beta.0` has been manually requested by @actor.\n\nContent has been updated:\n\n```diff\n+  - title (#6)\n+  - title (#7)\n```",
+        body: "Version `1.3.0-beta.0` has been manually requested by @actor.",
       });
     });
   });