Fix start.test.ts to match updated signal handling

[mcollina]

Summary

Updated test/start.test.ts to work with the new signal handling implementation in lib/start.js that uses process.once('SIGINT') instead of close-with-grace.

Changes

• Removed close-with-grace mocking: The library is no longer used in the implementation
• Added client.close() mocking: Properly mocks and verifies client.close() is called before server.close()
• Updated signal handling tests: Changed from testing close-with-grace to testing process.once('SIGINT')
• Fixed timeout handling: Added mocks for setTimeout and clearTimeout to prevent the 10-minute recording timeout from hanging tests
• Combined test cases: Merged two separate tests into one to avoid module caching issues with Node.js experimental module mocking
• Added setupSignals verification: Ensures create is called with setupSignals: false

Test Results

All CLI tests now pass:

# tests 6
# suites 3
# pass 6
# fail 0

Test Coverage

The updated test verifies:

• Server starts with the correct runtime (test/start.test.ts:145-160)
• Record mode functionality with proper request handling (test/start.test.ts:197-204)
• SIGINT signal handling and cleanup sequence (test/start.test.ts:210-234)
• Client and server are properly closed in the correct order (test/start.test.ts:229-234)

🤖 Generated with Claude Code