Skip to content

Commit 2564759

Browse files
committed
fix: update tests for new theme count and diff output format
1 parent 5ce0b49 commit 2564759

3 files changed

Lines changed: 7 additions & 6 deletions

File tree

tests/actionExecutor.spec.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -326,7 +326,8 @@ describe('ActionExecutor', () => {
326326
const result = await executor.execute({ type: 'git_diff', path: 'src/index.ts' } as any);
327327

328328
expect(diffSpy).toHaveBeenCalledWith('/repo', 'src/index.ts');
329-
expect(result).toBe('diff output');
329+
// Result includes colorized stats header + original diff
330+
expect(result).toContain('diff output');
330331
diffSpy.mockRestore();
331332
});
332333

tests/core/SecurityScanner.spec.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ describe('SecurityScanner', () => {
107107
it('detects Stripe live key as high severity', () => {
108108
const diff = `
109109
+++ b/config.ts
110-
+const STRIPE_KEY = "sk_live_TESTKEY0000000000000000";
110+
+const STRIPE_KEY = "sk_live_xxxxxxxxxxxxxxxxxxxxxxxx";
111111
`;
112112
const result = scanner.scanDiff(diff);
113113
expect(result.clean).toBe(false);
@@ -117,7 +117,7 @@ describe('SecurityScanner', () => {
117117
it('detects Stripe test key as low severity', () => {
118118
const diff = `
119119
+++ b/config.ts
120-
+const STRIPE_KEY = "sk_test_TESTKEY0000000000000000";
120+
+const STRIPE_KEY = "sk_test_xxxxxxxxxxxxxxxxxxxxxxxx";
121121
`;
122122
const result = scanner.scanDiff(diff);
123123
expect(result.clean).toBe(true); // Low severity doesn't block
@@ -337,7 +337,7 @@ line 4`;
337337
it('returns false for only low severity findings', () => {
338338
const diff = `
339339
+++ b/config.ts
340-
+const KEY = "sk_test_TESTKEY0000000000000000";
340+
+const KEY = "sk_test_xxxxxxxxxxxxxxxxxxxxxxxx";
341341
`;
342342
const result = scanner.scanDiff(diff);
343343
expect(scanner.shouldBlockCommit(result)).toBe(false);

tests/ui/theme/themes.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -133,8 +133,8 @@ describe('builtInThemes', () => {
133133
expect(builtInThemes.light).toBe(lightTheme);
134134
});
135135

136-
it('has exactly 2 built-in themes', () => {
137-
expect(Object.keys(builtInThemes)).toHaveLength(2);
136+
it('has exactly 5 built-in themes', () => {
137+
expect(Object.keys(builtInThemes)).toHaveLength(5);
138138
});
139139

140140
it('all themes have unique names', () => {

0 commit comments

Comments
 (0)