test

Author: pranavcodeant

tests/reviewApiHelper.test.js

@@ -345,18 +345,22 @@ describe('ReviewApiHelper integration tests', () => {
       expect(result.file_contents).not.toHaveProperty('docs/guide.md');
     });
 
-    it('limits to MAX_REVIEW_FILES (15) unique files', async () => {
-      for (let i = 1; i <= 18; i++) {
-        writeFile(`src/file${i}.js`, `const f${i} = ${i};\n`);
+    it.each([18, 40, 43])('applies the 40-file limit to %i staged files', async (totalFiles) => {
+      for (let i = 1; i <= totalFiles; i++) {
+        writeFile(`src/cap/file${i}.js`, `const f${i} = ${i};\n`);
       }
-      git('add -A');
+      git('add src/cap');
 
       const helper = await createHelper();
       const result = await helper.buildReviewApiRequest('staged-only');
 
       const fileCount = Object.keys(result.file_contents).length;
-      expect(fileCount).toBe(15);
-      expect(result._meta.max_files).toBe(15);
+      expect(fileCount).toBe(Math.min(totalFiles, 40));
+      expect(result._meta.reviewed_files).toHaveLength(fileCount);
+      expect(result.diff_content.match(/^diff --git /gm)).toHaveLength(fileCount);
+      expect(result._meta.max_files).toBe(40);
+      expect(result._meta.capped).toBe(totalFiles > 40);
+      expect(result._meta.skipped).toHaveLength(Math.max(totalFiles - 40, 0));
     });
 
     it('skips files exceeding MAX_FILE_LINES (5000)', async () => {