jettypod 4.1.2 → 4.1.4

package/features/work-commands/validation.test.js

@@ -1,88 +0,0 @@
-// Unit tests for work commands validation logic
-// These test the error handling and validation without full database integration
-
-const { VALID_STATUSES } = require('../../lib/constants');
-
-describe('Work Commands - Validation', () => {
-  describe('startWork validation', () => {
-    test('should validate work item ID is numeric', () => {
-      const invalidIds = [null, undefined, 'abc', '', {}, []];
-      invalidIds.forEach(id => {
-        const isValid = !!(id && !isNaN(id) && id >= 1);
-        expect(isValid).toBe(false);
-      });
-    });
-
-    test('should validate work item ID is positive', () => {
-      expect(!!(0 && !isNaN(0) && 0 >= 1)).toBe(false);
-      expect(!!(-1 && !isNaN(-1) && -1 >= 1)).toBe(false);
-    });
-
-    test('should accept valid work item IDs', () => {
-      const validIds = [1, 2, 100, 999];
-      validIds.forEach(id => {
-        const isValid = id && !isNaN(id) && id >= 1;
-        expect(isValid).toBe(true);
-      });
-    });
-  });
-
-  describe('stopWork validation', () => {
-    test('should validate status is in VALID_STATUSES', () => {
-      const invalidStatuses = ['invalid', 'foo', '', null, undefined];
-      invalidStatuses.forEach(status => {
-        const isValid = !!(status && VALID_STATUSES.includes(status));
-        expect(isValid).toBe(false);
-      });
-    });
-
-    test('should accept valid statuses', () => {
-      VALID_STATUSES.forEach(status => {
-        const isValid = VALID_STATUSES.includes(status);
-        expect(isValid).toBe(true);
-      });
-    });
-
-    test('should accept null status (means no status change)', () => {
-      const status = null;
-      // Null is valid - means we're just stopping without changing status
-      expect(status === null).toBe(true);
-    });
-  });
-
-  describe('Path validation', () => {
-    test('should check jettypod directory exists', () => {
-      const fs = require('fs');
-      const path = require('path');
-
-      // In a non-initialized directory
-      const nonExistentPath = '/nonexistent/path/.jettypod';
-      expect(fs.existsSync(nonExistentPath)).toBe(false);
-    });
-
-    test('should check database file exists', () => {
-      const fs = require('fs');
-      const path = require('path');
-
-      // In a non-initialized directory
-      const nonExistentDb = '/nonexistent/path/.jettypod/work.db';
-      expect(fs.existsSync(nonExistentDb)).toBe(false);
-    });
-  });
-
-  describe('Status transition logic', () => {
-    test('should change todo to in_progress', () => {
-      const currentStatus = 'todo';
-      const finalStatus = currentStatus === 'todo' ? 'in_progress' : currentStatus;
-      expect(finalStatus).toBe('in_progress');
-    });
-
-    test('should preserve non-todo statuses', () => {
-      const statuses = ['backlog', 'in_progress', 'blocked', 'done', 'cancelled'];
-      statuses.forEach(status => {
-        const finalStatus = status === 'todo' ? 'in_progress' : status;
-        expect(finalStatus).toBe(status);
-      });
-    });
-  });
-});

package/features/work-commands/work-commands.feature

@@ -1,13 +0,0 @@
-Feature: Work Item Commands
-  As a developer
-  I want to start and stop work on items
-  So that my current work is tracked automatically
-
-  Scenario: Integration - work commands with existing work tracking
-    Given I have initialized jettypod
-    And I create an epic with id 1
-    And I create a feature with id 2 under epic 1
-    When I start work on item 2
-    Then the current work file exists
-    And the current work contains item 2
-    And item 2 status is in_progress or backlog

package/features/work-tracking/discovery-validation.test.js

@@ -1,228 +0,0 @@
-// Tests for work-tracking discovery validation
-// Ensures epics and features cannot be created during project discovery
-
-const { createTestEnvironment } = require('../../lib/test-helpers');
-const workTracking = require('./index');
-const config = require('../../lib/config');
-
-describe('Work Tracking - Discovery Validation', () => {
-  let testEnv;
-
-  beforeEach(() => {
-    testEnv = createTestEnvironment();
-    process.chdir(testEnv.testDir);
-  });
-
-  afterEach(() => {
-    testEnv.cleanup();
-  });
-
-  describe('epic creation during discovery', () => {
-    test('should block epic creation when project discovery in_progress', async () => {
-      // Start project discovery
-      config.update({
-        project_discovery: {
-          status: 'in_progress',
-          prototypes: [],
-          winner: null,
-          rationale: null,
-          started_date: new Date().toISOString(),
-          completed_date: null
-        }
-      });
-
-      await expect(workTracking.create('epic', 'Test Epic', 'Description'))
-        .rejects.toThrow('Cannot create epics while project discovery is in progress');
-    });
-
-    test('should allow epic creation when discovery is not_started', async () => {
-      const newId = await workTracking.create('epic', 'Test Epic', 'Description');
-      expect(newId).toBeDefined();
-      expect(typeof newId).toBe('number');
-      expect(newId).toBeGreaterThan(0);
-    });
-
-    test('should allow epic creation when discovery is completed', async () => {
-      // Complete discovery
-      config.update({
-        project_discovery: {
-          status: 'completed',
-          prototypes: ['proto1.js'],
-          winner: 'proto1.js',
-          rationale: 'Best approach',
-          started_date: new Date().toISOString(),
-          completed_date: new Date().toISOString()
-        }
-      });
-
-      const newId = await workTracking.create('epic', 'Test Epic', 'Description');
-      expect(newId).toBeDefined();
-      expect(typeof newId).toBe('number');
-    });
-  });
-
-  describe('feature creation during discovery', () => {
-    test('should block feature creation when project discovery in_progress', async () => {
-      // Start project discovery
-      config.update({
-        project_discovery: {
-          status: 'in_progress',
-          prototypes: [],
-          winner: null,
-          rationale: null,
-          started_date: new Date().toISOString(),
-          completed_date: null
-        }
-      });
-
-      await expect(workTracking.create('feature', 'Test Feature', 'Description'))
-        .rejects.toThrow('Cannot create features while project discovery is in progress');
-    });
-
-    test('should allow feature creation when discovery is not_started', async () => {
-      const newId = await workTracking.create('feature', 'Test Feature', 'Description');
-      expect(newId).toBeDefined();
-      expect(typeof newId).toBe('number');
-    });
-
-    test('should allow feature creation when discovery is completed', async () => {
-      // Complete discovery
-      config.update({
-        project_discovery: {
-          status: 'completed',
-          prototypes: ['proto1.js'],
-          winner: 'proto1.js',
-          rationale: 'Best approach',
-          started_date: new Date().toISOString(),
-          completed_date: new Date().toISOString()
-        }
-      });
-
-      const newId = await workTracking.create('feature', 'Test Feature', 'Description');
-      expect(newId).toBeDefined();
-      expect(typeof newId).toBe('number');
-    });
-  });
-
-  describe('chore creation during discovery', () => {
-    test('should allow chore creation when project discovery in_progress', async () => {
-      // Start project discovery
-      config.update({
-        project_discovery: {
-          status: 'in_progress',
-          prototypes: [],
-          winner: null,
-          rationale: null,
-          started_date: new Date().toISOString(),
-          completed_date: null
-        }
-      });
-
-      const newId = await workTracking.create('chore', 'Test Chore', 'Description');
-      expect(newId).toBeDefined();
-      expect(typeof newId).toBe('number');
-    });
-
-    test('should allow chore creation in all discovery states', async () => {
-      // not_started (default)
-      let newId = await workTracking.create('chore', 'Chore 1', 'Description');
-      expect(newId).toBeDefined();
-      expect(typeof newId).toBe('number');
-
-      // in_progress
-      config.update({
-        project_discovery: {
-          status: 'in_progress',
-          prototypes: [],
-          winner: null,
-          rationale: null,
-          started_date: new Date().toISOString(),
-          completed_date: null
-        }
-      });
-      newId = await workTracking.create('chore', 'Chore 2', 'Description');
-      expect(newId).toBeDefined();
-      expect(typeof newId).toBe('number');
-
-      // completed
-      config.update({
-        project_discovery: {
-          status: 'completed',
-          prototypes: ['proto1.js'],
-          winner: 'proto1.js',
-          rationale: 'Best approach',
-          started_date: new Date().toISOString(),
-          completed_date: new Date().toISOString()
-        }
-      });
-      newId = await workTracking.create('chore', 'Chore 3', 'Description');
-      expect(newId).toBeDefined();
-      expect(typeof newId).toBe('number');
-    });
-  });
-
-  describe('bug creation during discovery', () => {
-    test('should allow bug creation when project discovery in_progress', async () => {
-      // Start project discovery
-      config.update({
-        project_discovery: {
-          status: 'in_progress',
-          prototypes: [],
-          winner: null,
-          rationale: null,
-          started_date: new Date().toISOString(),
-          completed_date: null
-        }
-      });
-
-      const newId = await workTracking.create('bug', 'Test Bug', 'Description');
-      expect(newId).toBeDefined();
-      expect(typeof newId).toBe('number');
-    });
-  });
-
-  describe('error messages', () => {
-    test('should provide helpful error message for epic creation', async () => {
-      config.update({
-        project_discovery: {
-          status: 'in_progress',
-          prototypes: [],
-          winner: null,
-          rationale: null,
-          started_date: new Date().toISOString(),
-          completed_date: null
-        }
-      });
-
-      try {
-        await workTracking.create('epic', 'Test Epic', 'Description');
-      } catch (error) {
-        expect(error.message).toContain('Cannot create epics');
-        expect(error.message).toContain('discovery is in progress');
-        expect(error.message).toContain('jettypod project discover complete');
-        expect(error.message).toContain('Claude Code');
-      }
-    });
-
-    test('should provide helpful error message for feature creation', async () => {
-      config.update({
-        project_discovery: {
-          status: 'in_progress',
-          prototypes: [],
-          winner: null,
-          rationale: null,
-          started_date: new Date().toISOString(),
-          completed_date: null
-        }
-      });
-
-      try {
-        await workTracking.create('feature', 'Test Feature', 'Description');
-      } catch (error) {
-        expect(error.message).toContain('Cannot create features');
-        expect(error.message).toContain('discovery is in progress');
-        expect(error.message).toContain('jettypod project discover complete');
-      }
-    });
-  });
-});