ts-repo-utils 7.8.1 → 7.8.2

package/src/functions/diff.test.mts

@@ -82,7 +82,7 @@ describe('diff', () => {
     };
   };
 
-  describe('getUntrackedFiles', () => {
+  describe(getUntrackedFiles, () => {
     test('should return empty array when no files are changed', async () => {
       const { repoPath, cleanup } = await createTempRepo();
       const repoFunctions = createRepoFunctions(repoPath);
@@ -91,6 +91,7 @@ describe('diff', () => {
         const result = await repoFunctions.getUntrackedFiles({ silent: true });
 
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           expect(Array.isArray(result.value)).toBe(true);
         }
@@ -113,8 +114,10 @@ describe('diff', () => {
         const result = await repoFunctions.getUntrackedFiles({ silent: true });
 
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           const files = result.value;
+
           expect(files).toContain(testFileName);
         }
       } finally {
@@ -142,8 +145,10 @@ describe('diff', () => {
         const result = await repoFunctions.getUntrackedFiles({ silent: true });
 
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           const files = result.value;
+
           expect(files).not.toContain(testFileName);
         }
       } finally {
@@ -176,8 +181,10 @@ describe('diff', () => {
         const result = await repoFunctions.getUntrackedFiles({ silent: true });
 
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           const files = result.value;
+
           expect(files).toContain(newFile);
           expect(files).not.toContain(modifyFile);
         }
@@ -194,11 +201,13 @@ describe('diff', () => {
         const result = await repoFunctions.getUntrackedFiles({ silent: true });
 
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           const files = result.value;
           // Verify no deleted files are included (status 'D')
           for (const file of files) {
-            expect(typeof file).toBe('string');
+            expectTypeOf(file).toBeString();
+
             expect(file.length).toBeGreaterThan(0);
           }
         }
@@ -229,12 +238,14 @@ describe('diff', () => {
         const result = await repoFunctions.getUntrackedFiles({ silent: true });
 
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           const files = result.value;
 
           // Each file should be a non-empty string
           for (const file of files) {
-            expect(typeof file).toBe('string');
+            expectTypeOf(file).toBeString();
+
             expect(file.trim()).toBe(file); // No leading/trailing whitespace
             expect(file.length).toBeGreaterThan(0);
           }
@@ -252,6 +263,7 @@ describe('diff', () => {
         const result = await repoFunctions.getUntrackedFiles({ silent: true });
 
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           expect(Array.isArray(result.value)).toBe(true);
         }
@@ -261,14 +273,16 @@ describe('diff', () => {
     });
   });
 
-  describe('getStagedFiles', () => {
+  describe(getStagedFiles, () => {
     test('should return empty array when no files are staged', async () => {
       const { repoPath, cleanup } = await createTempRepo();
       const repoFunctions = createRepoFunctions(repoPath);
 
       try {
         const result = await repoFunctions.getStagedFiles({ silent: true });
+
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           expect(Array.isArray(result.value)).toBe(true);
         }
@@ -290,9 +304,12 @@ describe('diff', () => {
         await execInRepo(`git add ${testFileName}`, { silent: true });
 
         const result = await repoFunctions.getStagedFiles({ silent: true });
+
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           const files = result.value;
+
           expect(files).toContain(testFileName);
         }
       } finally {
@@ -318,9 +335,12 @@ describe('diff', () => {
         await execInRepo(`git add ${file1} ${file2}`, { silent: true });
 
         const result = await repoFunctions.getStagedFiles({ silent: true });
+
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           const files = result.value;
+
           expect(files).toContain(file1);
           expect(files).toContain(file2);
         }
@@ -355,9 +375,12 @@ describe('diff', () => {
         const resultExclude = await repoFunctions.getStagedFiles({
           silent: true,
         });
+
         expect(Result.isOk(resultExclude)).toBe(true);
+
         if (Result.isOk(resultExclude)) {
           const files = resultExclude.value;
+
           expect(files).not.toContain(testFileName);
         }
 
@@ -375,9 +398,12 @@ describe('diff', () => {
             excludeDeleted: false,
             silent: true,
           });
+
           expect(Result.isOk(resultInclude)).toBe(true);
+
           if (Result.isOk(resultInclude)) {
             const files = resultInclude.value;
+
             expect(files).toContain(testFileName);
           }
         } else {
@@ -397,12 +423,15 @@ describe('diff', () => {
 
       try {
         const result = await repoFunctions.getStagedFiles({ silent: true });
+
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           const files = result.value;
           // Each file should be a non-empty string
           for (const file of files) {
-            expect(typeof file).toBe('string');
+            expectTypeOf(file).toBeString();
+
             expect(file.trim()).toBe(file); // No leading/trailing whitespace
           }
         }
@@ -417,7 +446,9 @@ describe('diff', () => {
 
       try {
         const result = await repoFunctions.getStagedFiles({ silent: true });
+
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           expect(Array.isArray(result.value)).toBe(true);
         }
@@ -432,6 +463,7 @@ describe('diff', () => {
 
       try {
         const result = await repoFunctions.getStagedFiles({ silent: true });
+
         // Should always return a Result, either Ok or Err
         expect(Result.isOk(result) || Result.isErr(result)).toBe(true);
       } finally {
@@ -440,14 +472,16 @@ describe('diff', () => {
     });
   });
 
-  describe('getModifiedFiles', () => {
+  describe(getModifiedFiles, () => {
     test('should return empty array when no files are modified', async () => {
       const { repoPath, cleanup } = await createTempRepo();
       const repoFunctions = createRepoFunctions(repoPath);
 
       try {
         const result = await repoFunctions.getModifiedFiles({ silent: true });
+
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           expect(Array.isArray(result.value)).toBe(true);
         }
@@ -476,9 +510,12 @@ describe('diff', () => {
         await fs.writeFile(testFilePath, 'modified content');
 
         const result = await repoFunctions.getModifiedFiles({ silent: true });
+
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           const files = result.value;
+
           expect(files).toContain(testFileName);
         }
       } finally {
@@ -511,9 +548,12 @@ describe('diff', () => {
         await fs.writeFile(filePath2, 'modified content 2');
 
         const result = await repoFunctions.getModifiedFiles({ silent: true });
+
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           const files = result.value;
+
           expect(files).toContain(file1);
           expect(files).toContain(file2);
         }
@@ -545,9 +585,12 @@ describe('diff', () => {
         const resultExclude = await repoFunctions.getModifiedFiles({
           silent: true,
         });
+
         expect(Result.isOk(resultExclude)).toBe(true);
+
         if (Result.isOk(resultExclude)) {
           const files = resultExclude.value;
+
           expect(files).not.toContain(testFileName);
         }
 
@@ -556,9 +599,12 @@ describe('diff', () => {
           excludeDeleted: false,
           silent: true,
         });
+
         expect(Result.isOk(resultInclude)).toBe(true);
+
         if (Result.isOk(resultInclude)) {
           const files = resultInclude.value;
+
           expect(files).toContain(testFileName);
         }
       } finally {
@@ -572,12 +618,15 @@ describe('diff', () => {
 
       try {
         const result = await repoFunctions.getModifiedFiles({ silent: true });
+
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           const files = result.value;
           // Each file should be a non-empty string
           for (const file of files) {
-            expect(typeof file).toBe('string');
+            expectTypeOf(file).toBeString();
+
             expect(file.trim()).toBe(file); // No leading/trailing whitespace
           }
         }
@@ -592,7 +641,9 @@ describe('diff', () => {
 
       try {
         const result = await repoFunctions.getModifiedFiles({ silent: true });
+
         expect(Result.isOk(result)).toBe(true);
+
         if (Result.isOk(result)) {
           expect(Array.isArray(result.value)).toBe(true);
         }
@@ -607,6 +658,7 @@ describe('diff', () => {
 
       try {
         const result = await repoFunctions.getModifiedFiles({ silent: true });
+
         // Should always return a Result, either Ok or Err
         expect(Result.isOk(result) || Result.isErr(result)).toBe(true);
       } finally {
@@ -615,7 +667,7 @@ describe('diff', () => {
     });
   });
 
-  describe('getDiffFrom', () => {
+  describe(getDiffFrom, () => {
     test('should work with silent option', async () => {
       const { repoPath, cleanup, execInRepo } = await createTempRepo();
       const repoFunctions = createRepoFunctions(repoPath);

package/src/functions/exec-async.test.mts

@@ -21,11 +21,13 @@ describe('exec-async', () => {
       (globalThis as any).echo = originalEcho;
     }
   };
+
   describe('basic command execution', () => {
     test('should execute simple command successfully', async () => {
       const result = await $('echo "hello world"', { silent: true });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(result.value.stdout.trim()).toBe('hello world');
         expect(result.value.stderr).toBe('');
@@ -36,6 +38,7 @@ describe('exec-async', () => {
       const result = await $('echo "line1\nline2\nline3"', { silent: true });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(result.value.stdout.trim()).toBe('line1\nline2\nline3');
         expect(result.value.stderr).toBe('');
@@ -46,6 +49,7 @@ describe('exec-async', () => {
       const result = await $('true', { silent: true });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(result.value.stdout).toBe('');
         expect(result.value.stderr).toBe('');
@@ -58,6 +62,7 @@ describe('exec-async', () => {
       const result = await $('nonexistent_command_xyz', { silent: true });
 
       expect(Result.isErr(result)).toBe(true);
+
       if (Result.isErr(result)) {
         expect(result.value).toBeDefined();
         expect(result.value.code).toBeDefined();
@@ -68,6 +73,7 @@ describe('exec-async', () => {
       const result = await $('exit 1', { silent: true });
 
       expect(Result.isErr(result)).toBe(true);
+
       if (Result.isErr(result)) {
         expect(result.value).toBeDefined();
         expect(result.value.code).toBe(1);
@@ -80,6 +86,7 @@ describe('exec-async', () => {
       });
 
       expect(Result.isErr(result)).toBe(true);
+
       if (Result.isErr(result)) {
         expect(result.value).toBeDefined();
       }
@@ -115,8 +122,9 @@ describe('exec-async', () => {
       const result = await $('echo "test"', { silent: true });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
-        expect(typeof result.value.stdout).toBe('string');
+        expectTypeOf(result.value.stdout).toBeString();
       }
     });
 
@@ -127,6 +135,7 @@ describe('exec-async', () => {
       });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(Buffer.isBuffer(result.value.stdout)).toBe(true);
         expect(Buffer.isBuffer(result.value.stderr)).toBe(true);
@@ -137,6 +146,7 @@ describe('exec-async', () => {
       const result = await $('echo "test"', { silent: true, encoding: null });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(Buffer.isBuffer(result.value.stdout)).toBe(true);
         expect(Buffer.isBuffer(result.value.stderr)).toBe(true);
@@ -150,8 +160,10 @@ describe('exec-async', () => {
       });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
-        expect(typeof result.value.stdout).toBe('string');
+        expectTypeOf(result.value.stdout).toBeString();
+
         expect(result.value.stdout.trim()).toBe('test 日本語');
       }
     });
@@ -164,6 +176,7 @@ describe('exec-async', () => {
       });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(result.value.stdout.trim()).toBe('hello world');
       }
@@ -173,6 +186,7 @@ describe('exec-async', () => {
       const result = await $('echo "first" && echo "second"', { silent: true });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(result.value.stdout.trim()).toBe('first\nsecond');
       }
@@ -182,6 +196,7 @@ describe('exec-async', () => {
       const result = await $('echo "first"; echo "second"', { silent: true });
 
       expect(Result.isOk(result)).toBe(true);
+
       if (Result.isOk(result)) {
         expect(result.value.stdout.trim()).toBe('first\nsecond');
       }