From 587745046ffb354fb75007deecc5fcbd6f55d447 Mon Sep 17 00:00:00 2001 From: Ralph Khreish <35776126+Crunchyman-ralph@users.noreply.github.com> Date: Mon, 8 Sep 2025 18:30:40 -0700 Subject: [PATCH] chore: fix format --- .../profiles/roo-files-inclusion.test.js | 8 +++-- .../profiles/rules-files-inclusion.test.js | 8 +++-- tests/unit/prompt-manager.test.js | 35 ++++++++++--------- 3 files changed, 31 insertions(+), 20 deletions(-) diff --git a/tests/integration/profiles/roo-files-inclusion.test.js b/tests/integration/profiles/roo-files-inclusion.test.js index b795479e..77451241 100644 --- a/tests/integration/profiles/roo-files-inclusion.test.js +++ b/tests/integration/profiles/roo-files-inclusion.test.js @@ -103,10 +103,14 @@ describe('Roo Files Inclusion in Package', () => { test('source Roo files exist in public/assets directory', () => { // Verify that the source files for Roo integration exist expect( - fs.existsSync(path.join(process.cwd(), 'public', 'assets', 'roocode', '.roo')) + fs.existsSync( + path.join(process.cwd(), 'public', 'assets', 'roocode', '.roo') + ) ).toBe(true); expect( - fs.existsSync(path.join(process.cwd(), 'public', 'assets', 'roocode', '.roomodes')) + fs.existsSync( + path.join(process.cwd(), 'public', 'assets', 'roocode', '.roomodes') + ) ).toBe(true); }); }); diff --git a/tests/integration/profiles/rules-files-inclusion.test.js b/tests/integration/profiles/rules-files-inclusion.test.js index 00b65bed..c8cdbe03 100644 --- a/tests/integration/profiles/rules-files-inclusion.test.js +++ b/tests/integration/profiles/rules-files-inclusion.test.js @@ -89,10 +89,14 @@ describe('Rules Files Inclusion in Package', () => { test('source Roo files exist in public/assets directory', () => { // Verify that the source files for Roo integration exist expect( - fs.existsSync(path.join(process.cwd(), 'public', 'assets', 'roocode', '.roo')) + fs.existsSync( + path.join(process.cwd(), 'public', 'assets', 'roocode', '.roo') + ) ).toBe(true); expect( - fs.existsSync(path.join(process.cwd(), 'public', 'assets', 'roocode', '.roomodes')) + fs.existsSync( + path.join(process.cwd(), 'public', 'assets', 'roocode', '.roomodes') + ) ).toBe(true); }); }); diff --git a/tests/unit/prompt-manager.test.js b/tests/unit/prompt-manager.test.js index 8abe1c41..f135f7d0 100644 --- a/tests/unit/prompt-manager.test.js +++ b/tests/unit/prompt-manager.test.js @@ -62,11 +62,11 @@ describe('PromptManager', () => { describe('loadPrompt', () => { it('should load and render a prompt from actual files', () => { // Test with an actual prompt that exists - const result = promptManager.loadPrompt('research', { + const result = promptManager.loadPrompt('research', { query: 'test query', projectContext: 'test context' }); - + expect(result.systemPrompt).toBeDefined(); expect(result.userPrompt).toBeDefined(); expect(result.userPrompt).toContain('test query'); @@ -87,7 +87,7 @@ describe('PromptManager', () => { }); const result = promptManager.loadPrompt('test-prompt', { name: 'John' }); - + expect(result.userPrompt).toBe('Hello John, your age is '); }); @@ -100,13 +100,13 @@ describe('PromptManager', () => { it('should use cache for repeated calls', () => { // First call with a real prompt const result1 = promptManager.loadPrompt('research', { query: 'test' }); - + // Mark the result to verify cache is used result1._cached = true; - + // Second call with same parameters should return cached result const result2 = promptManager.loadPrompt('research', { query: 'test' }); - + expect(result2._cached).toBe(true); expect(result1).toBe(result2); // Same object reference }); @@ -127,7 +127,7 @@ describe('PromptManager', () => { const result = promptManager.loadPrompt('array-prompt', { items: ['one', 'two', 'three'] }); - + // The actual implementation doesn't handle {{this}} properly, check what it does produce expect(result.userPrompt).toContain('Item:'); }); @@ -145,10 +145,14 @@ describe('PromptManager', () => { } }); - const withData = promptManager.loadPrompt('conditional-prompt', { hasData: true }); + const withData = promptManager.loadPrompt('conditional-prompt', { + hasData: true + }); expect(withData.userPrompt).toBe('Data exists'); - const withoutData = promptManager.loadPrompt('conditional-prompt', { hasData: false }); + const withoutData = promptManager.loadPrompt('conditional-prompt', { + hasData: false + }); expect(withoutData.userPrompt).toBe('No data'); }); }); @@ -162,7 +166,7 @@ describe('PromptManager', () => { age: 30 } }; - + const result = promptManager.renderTemplate(template, variables); expect(result).toBe('User: John, Age: 30'); }); @@ -172,7 +176,7 @@ describe('PromptManager', () => { const variables = { special: '<>&"\'' }; - + const result = promptManager.renderTemplate(template, variables); expect(result).toBe('Special: <>&"\''); }); @@ -183,8 +187,8 @@ describe('PromptManager', () => { const prompts = promptManager.listPrompts(); expect(prompts).toBeInstanceOf(Array); expect(prompts.length).toBeGreaterThan(0); - - const ids = prompts.map(p => p.id); + + const ids = prompts.map((p) => p.id); expect(ids).toContain('analyze-complexity'); expect(ids).toContain('expand-task'); expect(ids).toContain('add-task'); @@ -192,7 +196,6 @@ describe('PromptManager', () => { }); }); - describe('validateTemplate', () => { it('should validate a correct template', () => { const result = promptManager.validateTemplate('research'); @@ -202,7 +205,7 @@ describe('PromptManager', () => { it('should reject invalid template', () => { const result = promptManager.validateTemplate('non-existent'); expect(result.valid).toBe(false); - expect(result.error).toContain("not found"); + expect(result.error).toContain('not found'); }); }); -}); \ No newline at end of file +});