chore: remove skipped placeholder tests

Author: jonathanpeterwu

src/__tests__/integration/cli-integration.test.ts

@@ -54,28 +54,14 @@ describe('CLI Integration', () => {
       });
 
       expect(result).toContain('StackMemory initialized');
-      
+
       // Check that .stackmemory directory was created
       const stackmemoryDir = path.join(testDir, '.stackmemory');
       expect(fs.existsSync(stackmemoryDir)).toBe(true);
     });
   });
 
   describe('Status Command', () => {
-    it.skip('should show status after init - needs database schema', () => {
-      // Initialize first
-      execSync(`node ${cliPath} init`, { cwd: testDir });
-
-      // Check status
-      const result = execSync(`node ${cliPath} status`, {
-        cwd: testDir,
-        encoding: 'utf8',
-      });
-
-      expect(result).toContain('StackMemory Status');
-      expect(result).toContain('Initialized:');
-    });
-
     it('should handle status when not initialized', () => {
       try {
         execSync(`node ${cliPath} status`, {
@@ -100,19 +86,6 @@ describe('CLI Integration', () => {
 
       expect(result).toContain('Context Usage');
     });
-
-    it.skip('should check if clear is recommended - needs git repo', () => {
-      // Initialize first
-      execSync(`node ${cliPath} init`, { cwd: testDir });
-
-      const result = execSync(`node ${cliPath} clear --check`, {
-        cwd: testDir,
-        encoding: 'utf8',
-      });
-
-      // Should provide some recommendation
-      expect(result.length).toBeGreaterThan(0);
-    });
   });
 
   describe('Workflow Command', () => {
@@ -141,4 +114,4 @@ describe('CLI Integration', () => {
       expect(result).toBeDefined();
     });
   });
-});
+});

src/__tests__/integration/e2e/full-session.test.ts

@@ -4,8 +4,15 @@
  */
 
 import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { TestEnvironment, TestSession, ActivityRecord } from '../helpers/test-environment.js';
-import { generateTestFrames, generateSearchQueries } from '../fixtures/test-data-generator.js';
+import {
+  TestEnvironment,
+  TestSession,
+  ActivityRecord,
+} from '../helpers/test-environment.js';
+import {
+  generateTestFrames,
+  generateSearchQueries,
+} from '../fixtures/test-data-generator.js';
 
 describe('Full Session Lifecycle', () => {
   let env: TestEnvironment;
@@ -24,13 +31,13 @@ describe('Full Session Lifecycle', () => {
     const project = await env.createProject('test-app');
     expect(project.id).toContain('e2e-test-project');
     expect(project.path).toContain('test-app');
-    
+
     // Step 2: Start development session
     const session = await env.startSession('dev-session-1');
     expect(session).toBeDefined();
     expect(session.frameManager).toBeDefined();
     expect(session.sharedContext).toBeDefined();
-    
+
     // Step 3: Record development activities
     const activities: ActivityRecord[] = [
       { type: 'file_edit', file: 'src/app.ts' },
@@ -39,28 +46,28 @@ describe('Full Session Lifecycle', () => {
       { type: 'test_run', status: 'pass' },
       { type: 'commit', message: 'Fix test failures' },
     ];
-    
+
     const frameIds = await session.recordActivity(activities);
     expect(frameIds).toHaveLength(5);
-    
+
     // Step 4: Save context before clear
     const savedContext = await session.saveContext();
     expect(savedContext.frames).toHaveLength(5);
     expect(savedContext.sessionId).toBe('dev-session-1');
-    
+
     // Step 5: Simulate Claude clear
     await env.simulateClear();
-    
+
     // Step 6: Restore context after clear
     const restoredContext = await env.restoreContext('dev-session-1');
     expect(restoredContext.frames).toHaveLength(5);
     expect(restoredContext.sessionId).toBe('dev-session-1');
-    
+
     // Verify frame content preserved
     const firstFrame = restoredContext.frames[0];
     expect(firstFrame.name).toContain('Edit');
     expect(firstFrame.type).toBe('operation');
-    
+
     // Step 7: Generate handoff document
     const handoff = await session.generateHandoff();
     expect(handoff).toContain('# Session Handoff');
@@ -76,36 +83,36 @@ describe('Full Session Lifecycle', () => {
       { type: 'file_edit', file: 'api.ts' },
       { type: 'command', command: 'npm test' },
     ]);
-    
+
     const contextA = await sessionA.saveContext();
     expect(contextA.frames).toHaveLength(2);
-    
+
     // Session B reads and extends context
     const sessionB = await env.startSession('session-b');
-    
+
     // Session B should be able to query Session A's frames
     const retriever = sessionB.retriever;
     const searchResult = await retriever.retrieveContext({
       text: 'api.ts',
       maxResults: 5,
     });
-    
+
     expect(searchResult.contexts.length).toBeGreaterThan(0);
     expect(searchResult.contexts[0].frame.name).toContain('api.ts');
-    
+
     // Session B adds more work
     await sessionB.recordActivity([
       { type: 'file_edit', file: 'api.test.ts' },
       { type: 'test_run', status: 'pass' },
     ]);
-    
+
     const contextB = await sessionB.saveContext();
     expect(contextB.frames.length).toBeGreaterThanOrEqual(2);
   });
 
   it('should handle error recovery workflow', async () => {
     const session = await env.startSession('error-recovery');
-    
+
     // Simulate error scenario
     const activities: ActivityRecord[] = [
       { type: 'command', command: 'npm run build' },
@@ -114,24 +121,24 @@ describe('Full Session Lifecycle', () => {
       { type: 'command', command: 'npm run build' },
       { type: 'test_run', status: 'pass' },
     ];
-    
+
     const frameIds = await session.recordActivity(activities);
-    
+
     // Verify error was recorded
     const db = await env.getDatabase();
     const errorFrame = await db.getFrame(frameIds[1]);
     expect(errorFrame?.type).toBe('error');
     expect(errorFrame?.state).toBe('error');
-    
+
     // Search for error patterns
     const errorSearch = await session.retriever.retrieveContext({
       text: 'build failed',
       maxResults: 10,
     });
-    
+
     expect(errorSearch.contexts.length).toBeGreaterThan(0);
     expect(errorSearch.contexts[0].frame.digest_text).toContain('Build failed');
-    
+
     // Generate handoff should include error context
     const handoff = await session.generateHandoff();
     expect(handoff).toContain('[error]');
@@ -141,7 +148,7 @@ describe('Full Session Lifecycle', () => {
   it('should handle large session with many frames', async () => {
     const session = await env.startSession('large-session');
     const db = await env.getDatabase();
-    
+
     // Generate many activities
     const activities: ActivityRecord[] = [];
     for (let i = 0; i < 100; i++) {
@@ -151,10 +158,10 @@ describe('Full Session Lifecycle', () => {
         status: i % 3 === 0 ? 'fail' : 'pass',
       });
     }
-    
+
     const frameIds = await session.recordActivity(activities);
     expect(frameIds).toHaveLength(100);
-    
+
     // Test search performance
     const startTime = Date.now();
     const searchResult = await db.search({
@@ -163,25 +170,25 @@ describe('Full Session Lifecycle', () => {
       limit: 20,
     });
     const searchTime = Date.now() - startTime;
-    
+
     expect(searchResult.length).toBeLessThanOrEqual(20);
     expect(searchTime).toBeLessThan(100); // Should be fast
-    
+
     // Test context retrieval with large dataset
     const queries = generateSearchQueries();
     for (const query of queries.slice(0, 5)) {
       const result = await session.retriever.retrieveContext({
         text: query,
         maxResults: 10,
       });
-      
+
       expect(result.retrievalTimeMs).toBeLessThan(200);
     }
   });
 
   it('should handle workflow transitions', async () => {
     const session = await env.startSession('workflow-test');
-    
+
     // Start TDD workflow
     await session.recordActivity([
       { type: 'command', command: 'stackmemory workflow start tdd' },
@@ -191,39 +198,19 @@ describe('Full Session Lifecycle', () => {
       { type: 'test_run', status: 'pass' },
       { type: 'file_edit', file: 'src/feature.ts' }, // Refactor
     ]);
-    
+
     // Save workflow state
     const context = await session.saveContext();
     expect(context.frames).toHaveLength(6);
-    
+
     // Verify workflow pattern detected
     const frames = context.frames;
-    const hasTestFirst = frames.some(f => f.name.includes('test') && f.state === 'completed');
-    const hasImplementation = frames.some(f => f.name.includes('feature.ts'));
-    
+    const hasTestFirst = frames.some(
+      (f) => f.name.includes('test') && f.state === 'completed'
+    );
+    const hasImplementation = frames.some((f) => f.name.includes('feature.ts'));
+
     expect(hasTestFirst).toBe(true);
     expect(hasImplementation).toBe(true);
   });
-
-  it.skip('should handle CLI commands integration - needs separate setup', async () => {
-    // Test CLI commands in the environment
-    
-    // Initialize via CLI
-    const initOutput = await env.runCLICommand('init');
-    expect(initOutput).toContain('StackMemory initialized');
-    
-    // Check status
-    const statusOutput = await env.runCLICommand('status');
-    expect(statusOutput).toContain('StackMemory Status');
-    
-    // Test workflow commands
-    const workflowList = await env.runCLICommand('workflow --list');
-    expect(workflowList).toContain('Available Workflows');
-    expect(workflowList).toContain('tdd');
-    expect(workflowList).toContain('feature');
-    
-    // Test clear status
-    const clearStatus = await env.runCLICommand('clear --status');
-    expect(clearStatus).toContain('Context Usage');
-  });
-});
+});

src/cli/commands/__tests__/integration.test.ts

@@ -47,32 +47,6 @@ describe('CLI Integration Tests', () => {
       expect(result).toContain('Context Usage Status');
       expect(result).toContain('Usage:');
     });
-
-    it.skip('should save continuity ledger - NEEDS IMPLEMENTATION', () => {
-      const result = execSync(cli('clear --save'), {
-        cwd: testDir,
-        encoding: 'utf8',
-      });
-
-      expect(result).toContain('Continuity ledger saved');
-
-      // Check that ledger file was created
-      const ledgerPath = path.join(testDir, '.stackmemory', 'continuity.json');
-      expect(fs.existsSync(ledgerPath)).toBe(true);
-    });
-
-    it.skip('should restore from ledger - NEEDS IMPLEMENTATION', () => {
-      // First save a ledger
-      execSync(cli('clear --save'), { cwd: testDir });
-
-      // Then restore
-      const result = execSync(cli('clear --restore'), {
-        cwd: testDir,
-        encoding: 'utf8',
-      });
-
-      expect(result).toContain('restored');
-    });
   });
 
   describe('Workflow Commands', () => {
@@ -166,33 +140,4 @@ describe('CLI Integration Tests', () => {
       expect(result).toBeDefined();
     });
   });
-
-  describe('Feature Integration', () => {
-    it.skip('should handle full workflow - NEEDS IMPLEMENTATION', () => {
-      // Start a workflow
-      let result = execSync(cli('workflow --start tdd'), {
-        cwd: testDir,
-        encoding: 'utf8',
-      });
-
-      expect(result).toContain('workflow');
-
-      // Generate handoff
-      result = execSync(cli('handoff capture'), {
-        cwd: testDir,
-        encoding: 'utf8',
-      });
-
-      // Just verify it ran
-      expect(result).toBeDefined();
-
-      // Save and clear
-      result = execSync(cli('clear --save'), {
-        cwd: testDir,
-        encoding: 'utf8',
-      });
-
-      expect(result).toContain('Continuity ledger saved');
-    });
-  });
 });