From 785ee5d59a60df89413d8f0dd1b7916279f6e786 Mon Sep 17 00:00:00 2001 From: Miguel Solorio Date: Thu, 7 Aug 2025 16:11:35 -0700 Subject: Use semantic colors in themes (#5796) Co-authored-by: Jacob Richman --- packages/cli/src/ui/themes/theme.test.ts | 50 -------------------------------- 1 file changed, 50 deletions(-) (limited to 'packages/cli/src/ui/themes/theme.test.ts') diff --git a/packages/cli/src/ui/themes/theme.test.ts b/packages/cli/src/ui/themes/theme.test.ts index c1e4dc00..6359a922 100644 --- a/packages/cli/src/ui/themes/theme.test.ts +++ b/packages/cli/src/ui/themes/theme.test.ts @@ -36,25 +36,6 @@ describe('validateCustomTheme', () => { expect(result.error).toBeUndefined(); }); - it('should return isValid: false for a theme with a missing required field', () => { - const invalidTheme = { - ...validTheme, - name: undefined as unknown as string, - }; - const result = validateCustomTheme(invalidTheme); - expect(result.isValid).toBe(false); - expect(result.error).toBe('Missing required field: name'); - }); - - it('should return isValid: false for a theme with an invalid color format', () => { - const invalidTheme = { ...validTheme, Background: 'not-a-color' }; - const result = validateCustomTheme(invalidTheme); - expect(result.isValid).toBe(false); - expect(result.error).toBe( - 'Invalid color format for Background: not-a-color', - ); - }); - it('should return isValid: false for a theme with an invalid name', () => { const invalidTheme = { ...validTheme, name: ' ' }; const result = validateCustomTheme(invalidTheme); @@ -71,37 +52,6 @@ describe('validateCustomTheme', () => { expect(result.error).toBeUndefined(); }); - it('should return a warning if DiffAdded and DiffRemoved are missing', () => { - const legacyTheme: Partial = { ...validTheme }; - delete legacyTheme.DiffAdded; - delete legacyTheme.DiffRemoved; - const result = validateCustomTheme(legacyTheme); - expect(result.isValid).toBe(true); - expect(result.warning).toBe('Missing field(s) DiffAdded, DiffRemoved'); - }); - - it('should return a warning if only DiffRemoved is missing', () => { - const legacyTheme: Partial = { ...validTheme }; - delete legacyTheme.DiffRemoved; - const result = validateCustomTheme(legacyTheme); - expect(result.isValid).toBe(true); - expect(result.warning).toBe('Missing field(s) DiffRemoved'); - }); - - it('should return isValid: false for a theme with an invalid DiffAdded color', () => { - const invalidTheme = { ...validTheme, DiffAdded: 'invalid' }; - const result = validateCustomTheme(invalidTheme); - expect(result.isValid).toBe(false); - expect(result.error).toBe('Invalid color format for DiffAdded: invalid'); - }); - - it('should return isValid: false for a theme with an invalid DiffRemoved color', () => { - const invalidTheme = { ...validTheme, DiffRemoved: 'invalid' }; - const result = validateCustomTheme(invalidTheme); - expect(result.isValid).toBe(false); - expect(result.error).toBe('Invalid color format for DiffRemoved: invalid'); - }); - it('should return isValid: false for a theme with a very long name', () => { const invalidTheme = { ...validTheme, name: 'a'.repeat(51) }; const result = validateCustomTheme(invalidTheme); -- cgit v1.2.3