add test

node-src/git/git.test.ts

@@ -1,12 +1,16 @@
 import { execaCommand } from 'execa';
-import { describe, expect, it, vi } from 'vitest';
+import { afterEach, describe, expect, it, vi } from 'vitest';
 
-import { getCommit, getSlug, hasPreviousCommit, mergeQueueBranchMatch } from './git';
+import { NULL_BYTE, findFilesFromRepositoryRoot, getCommit, getSlug, hasPreviousCommit, mergeQueueBranchMatch } from './git';
 
 vi.mock('execa');
 
 const command = vi.mocked(execaCommand);
 
+afterEach(() => {
+  vi.clearAllMocks();
+})
+
 describe('getCommit', () => {
   it('parses log output', async () => {
     command.mockImplementation(
@@ -106,3 +110,33 @@ describe('mergeQueueBranchMatch', () => {
     );
   });
 });
+
+describe('findFilesFromRepositoryRoot', () => {
+  it('finds files relative to the repository root', async () => {
+    const filesFound = [
+      'package.json',
+      'another/package/package.json',
+    ];
+
+    // first call from getRepositoryRoot()
+    command.mockImplementationOnce(
+      () =>
+        Promise.resolve({
+          all: '/root',
+        }) as any
+    );
+
+    command.mockImplementationOnce(
+      () =>
+        Promise.resolve({
+          all: filesFound.join(NULL_BYTE),
+        }) as any
+    );
+
+    const results = await findFilesFromRepositoryRoot('package.json', '**/package.json');
+
+    expect(command).toBeCalledTimes(2);
+    expect(command).toHaveBeenNthCalledWith(2, "git ls-files --full-name -z '/root/package.json' '/root/**/package.json'", expect.any(Object));
+    expect(results).toEqual(filesFound);
+  });
+});

node-src/git/git.ts

@@ -11,6 +11,7 @@ import gitNotInstalled from '../ui/messages/errors/gitNotInstalled';
 import path from 'node:path';
 
 const newline = /\r\n|\r|\n/; // Git may return \n even on Windows, so we can't use EOL
+export const NULL_BYTE = '\0'; // Separator used when running `git ls-files` with `-z`
 
 export async function execGitCommand(command: string) {
   try {
@@ -295,7 +296,7 @@ export async function findFilesFromRepositoryRoot(...patterns: string[]) {
   // not the directory in which this is executed from
   const gitCommand = `git ls-files --full-name -z ${patternsFromRoot.map((p) => `'${p}'`).join(' ')}`;
   const files = await execGitCommand(gitCommand);
-  return files.split('\0').filter(Boolean);
+  return files.split(NULL_BYTE).filter(Boolean);
 }
 
 export async function mergeQueueBranchMatch(branch) {