File size: 8,325 Bytes
1dbc34b
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
import { EventEmitter } from 'events';
import path from 'path';
import os from 'os';
import fs from 'fs/promises';
import { spawn, execSync } from 'child_process';

// Mock child_process
vi.mock('child_process', () => ({
  spawn: vi.fn(),
  execSync: vi.fn(),
  execFile: vi.fn(),
}));

// Mock secure-fs
vi.mock('@/lib/secure-fs.js', () => ({
  access: vi.fn(),
}));

// Mock net
vi.mock('net', () => ({
  default: {
    createServer: vi.fn(),
  },
  createServer: vi.fn(),
}));

import * as secureFs from '@/lib/secure-fs.js';
import net from 'net';

describe('DevServerService Persistence & Sync', () => {
  let testDataDir: string;
  let worktreeDir: string;
  let mockEmitter: EventEmitter;

  beforeEach(async () => {
    vi.clearAllMocks();
    vi.resetModules();

    testDataDir = path.join(os.tmpdir(), `dev-server-persistence-test-${Date.now()}`);
    worktreeDir = path.join(os.tmpdir(), `dev-server-worktree-test-${Date.now()}`);
    await fs.mkdir(testDataDir, { recursive: true });
    await fs.mkdir(worktreeDir, { recursive: true });

    mockEmitter = new EventEmitter();

    // Default mock for secureFs.access - return resolved (file exists)
    vi.mocked(secureFs.access).mockResolvedValue(undefined);

    // Default mock for net.createServer - port available
    const mockServer = new EventEmitter() as any;
    mockServer.listen = vi.fn().mockImplementation((port: number, host: string) => {
      process.nextTick(() => mockServer.emit('listening'));
    });
    mockServer.close = vi.fn();
    vi.mocked(net.createServer).mockReturnValue(mockServer);

    // Default mock for execSync - no process on port
    vi.mocked(execSync).mockImplementation(() => {
      throw new Error('No process found');
    });
  });

  afterEach(async () => {
    try {
      await fs.rm(testDataDir, { recursive: true, force: true });
      await fs.rm(worktreeDir, { recursive: true, force: true });
    } catch {
      // Ignore cleanup errors
    }
  });

  it('should emit dev-server:starting when startDevServer is called', async () => {
    const { getDevServerService } = await import('@/services/dev-server-service.js');
    const service = getDevServerService();
    await service.initialize(testDataDir, mockEmitter as any);

    const mockProcess = createMockProcess();
    vi.mocked(spawn).mockReturnValue(mockProcess as any);

    const events: any[] = [];
    mockEmitter.on('dev-server:starting', (payload) => events.push(payload));

    await service.startDevServer(worktreeDir, worktreeDir);

    expect(events.length).toBe(1);
    expect(events[0].worktreePath).toBe(worktreeDir);
  });

  it('should prevent concurrent starts for the same worktree', async () => {
    const { getDevServerService } = await import('@/services/dev-server-service.js');
    const service = getDevServerService();
    await service.initialize(testDataDir, mockEmitter as any);

    // Delay spawn to simulate long starting time
    vi.mocked(spawn).mockImplementation(() => {
      const p = createMockProcess();
      // Don't return immediately, simulate some work
      return p as any;
    });

    // Start first one (don't await yet if we want to test concurrency)
    const promise1 = service.startDevServer(worktreeDir, worktreeDir);

    // Try to start second one immediately
    const result2 = await service.startDevServer(worktreeDir, worktreeDir);

    expect(result2.success).toBe(false);
    expect(result2.error).toContain('already starting');

    await promise1;
  });

  it('should persist state to dev-servers.json when started', async () => {
    const { getDevServerService } = await import('@/services/dev-server-service.js');
    const service = getDevServerService();
    await service.initialize(testDataDir, mockEmitter as any);

    const mockProcess = createMockProcess();
    vi.mocked(spawn).mockReturnValue(mockProcess as any);

    await service.startDevServer(worktreeDir, worktreeDir);

    const statePath = path.join(testDataDir, 'dev-servers.json');
    const exists = await fs
      .access(statePath)
      .then(() => true)
      .catch(() => false);
    expect(exists).toBe(true);

    const content = await fs.readFile(statePath, 'utf-8');
    const state = JSON.parse(content);
    expect(state.length).toBe(1);
    expect(state[0].worktreePath).toBe(worktreeDir);
  });

  it('should load state from dev-servers.json on initialize', async () => {
    // 1. Create a fake state file
    const persistedInfo = [
      {
        worktreePath: worktreeDir,
        allocatedPort: 3005,
        port: 3005,
        url: 'http://localhost:3005',
        startedAt: new Date().toISOString(),
        urlDetected: true,
        customCommand: 'npm run dev',
      },
    ];
    await fs.writeFile(path.join(testDataDir, 'dev-servers.json'), JSON.stringify(persistedInfo));

    // 2. Mock port as IN USE (so it re-attaches)
    const mockServer = new EventEmitter() as any;
    mockServer.listen = vi.fn().mockImplementation((port: number, host: string) => {
      // Fail to listen = port in use
      process.nextTick(() => mockServer.emit('error', new Error('EADDRINUSE')));
    });
    vi.mocked(net.createServer).mockReturnValue(mockServer);

    const { getDevServerService } = await import('@/services/dev-server-service.js');
    const service = getDevServerService();
    await service.initialize(testDataDir, mockEmitter as any);

    expect(service.isRunning(worktreeDir)).toBe(true);
    const info = service.getServerInfo(worktreeDir);
    expect(info?.port).toBe(3005);
  });

  it('should prune stale servers from state on initialize if port is available', async () => {
    // 1. Create a fake state file
    const persistedInfo = [
      {
        worktreePath: worktreeDir,
        allocatedPort: 3005,
        port: 3005,
        url: 'http://localhost:3005',
        startedAt: new Date().toISOString(),
        urlDetected: true,
      },
    ];
    await fs.writeFile(path.join(testDataDir, 'dev-servers.json'), JSON.stringify(persistedInfo));

    // 2. Mock port as AVAILABLE (so it prunes)
    const mockServer = new EventEmitter() as any;
    mockServer.listen = vi.fn().mockImplementation((port: number, host: string) => {
      process.nextTick(() => mockServer.emit('listening'));
    });
    mockServer.close = vi.fn();
    vi.mocked(net.createServer).mockReturnValue(mockServer);

    const { getDevServerService } = await import('@/services/dev-server-service.js');
    const service = getDevServerService();
    await service.initialize(testDataDir, mockEmitter as any);

    expect(service.isRunning(worktreeDir)).toBe(false);

    // Give it a moment to complete the pruning saveState
    await new Promise((resolve) => setTimeout(resolve, 100));

    // Check if file was updated
    const content = await fs.readFile(path.join(testDataDir, 'dev-servers.json'), 'utf-8');
    const state = JSON.parse(content);
    expect(state.length).toBe(0);
  });

  it('should update persisted state when URL is detected', async () => {
    const { getDevServerService } = await import('@/services/dev-server-service.js');
    const service = getDevServerService();
    await service.initialize(testDataDir, mockEmitter as any);

    const mockProcess = createMockProcess();
    vi.mocked(spawn).mockReturnValue(mockProcess as any);

    await service.startDevServer(worktreeDir, worktreeDir);

    // Simulate output with URL
    mockProcess.stdout.emit('data', Buffer.from('Local: http://localhost:5555/\n'));

    // Give it a moment to process and save (needs to wait for saveQueue)
    await new Promise((resolve) => setTimeout(resolve, 300));

    const content = await fs.readFile(path.join(testDataDir, 'dev-servers.json'), 'utf-8');
    const state = JSON.parse(content);
    expect(state[0].url).toBe('http://localhost:5555/');
    expect(state[0].port).toBe(5555);
    expect(state[0].urlDetected).toBe(true);
  });
});

// Helper to create a mock child process
function createMockProcess() {
  const mockProcess = new EventEmitter() as any;
  mockProcess.stdout = new EventEmitter();
  mockProcess.stderr = new EventEmitter();
  mockProcess.kill = vi.fn();
  mockProcess.killed = false;
  mockProcess.pid = 12345;
  mockProcess.unref = vi.fn();
  return mockProcess;
}