Squash actions#113

jjerphan · Nov 15, 2022 · 1ef5c18 · 1ef5c18
2 parents 5c75392 + 3118216
commit 1ef5c18
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 3 deletions.
diff --git a/__tests__/main.test.ts b/__tests__/main.test.ts
@@ -51,7 +51,7 @@ describe("run", () => {
     let mockInput = {
       "repo-token": "foo",
       "configuration-path": "bar",
-      "sync-labels": true,
+      "sync-labels": "true",
     };
 
     jest
@@ -82,7 +82,31 @@ describe("run", () => {
     let mockInput = {
       "repo-token": "foo",
       "configuration-path": "bar",
-      "sync-labels": false,
+      "sync-labels": "false",
+    };
+
+    jest
+      .spyOn(core, "getInput")
+      .mockImplementation((name: string, ...opts) => mockInput[name]);
+
+    usingLabelerConfigYaml("only_pdfs.yml");
+    mockGitHubResponseChangedFiles("foo.txt");
+    getPullMock.mockResolvedValue(<any>{
+      data: {
+        labels: [{ name: "touched-a-pdf-file" }],
+      },
+    });
+
+    await run();
+
+    expect(addLabelsMock).toHaveBeenCalledTimes(0);
+    expect(removeLabelMock).toHaveBeenCalledTimes(0);
+  });
+
+  it("(with sync-labels not specified) it behaves like sync-labels: false", async () => {
+    let mockInput = {
+      "repo-token": "foo",
+      "configuration-path": "bar",
     };
 
     jest

diff --git a/src/labeler.ts b/src/labeler.ts
@@ -15,7 +15,8 @@ export async function run() {
   try {
     const token = core.getInput("repo-token", { required: true });
     const configPath = core.getInput("configuration-path", { required: true });
-    const syncLabels = !!core.getInput("sync-labels", { required: false });
+    const syncLabels =
+      core.getInput("sync-labels", { required: false }) === "true";
 
     const prNumber = getPrNumber();
     if (!prNumber) {