chore: Fix tests.

Closes #17
joberstein · Jul 30, 2023 · 188e106 · 188e106
1 parent 5ea835f
commit 188e106
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 10 deletions.
diff --git a/dist/events/gitEvent.test.js b/dist/events/gitEvent.test.js
diff --git a/dist/events/gitEvent.test.js.map b/dist/events/gitEvent.test.js.map
diff --git a/dist/events/push.js b/dist/events/push.js
diff --git a/dist/events/push.js.map b/dist/events/push.js.map
diff --git a/src/events/gitEvent.test.ts b/src/events/gitEvent.test.ts
@@ -48,7 +48,6 @@ describe("src/events/gitEvent", () => {
 
     it("Validates a git event with no target ref", async () => {
         const event = new GitEvent({ target: '' }, options);
-        console.log(execSync('git log', options).toString());
 
         await expect(event.validateCommits()).resolves.not.toThrow();
 

diff --git a/src/events/push.ts b/src/events/push.ts
@@ -43,14 +43,18 @@ export default class Push extends GitEvent {
         const { ref } = this.#args;
         const { options } = this;
 
-        const refsToExclude = execSync(`git for-each-ref --format="%(refname)" refs/heads`, options)
+        const refs = execSync(`git for-each-ref --format="%(refname)" refs/heads`, options)
             .toString()
             .trim()
-            .split('\n')
+            .split('\n');
+
+        console.log(refs);
+
+        const refsToExclude = refs
             .filter(seenRef => ref !== seenRef)
             .join(' ');
 
-        const [commit, ] = execSync(`git rev-list --no-merges --not '${refsToExclude}' '${ref}'`, options)
+        const [commit, ] = execSync(`git rev-list --no-merges '${ref}' --not ${refsToExclude}`, options)
             .toString()
             .trim()
             .split('\n')