Skip to content

Commit 7917d00

Browse files
committed
Temporarily enable PTY tests in CI to capture debug logs from plugin
1 parent 149a4a7 commit 7917d00

2 files changed

Lines changed: 0 additions & 6 deletions

File tree

test/pty-echo.test.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ describe('PTY Echo Behavior', () => {
2020
})
2121

2222
it('should echo input characters in interactive bash session', async () => {
23-
if (process.env.CI) return // Skip PTY tests in CI due to manager issues
2423
const receivedOutputs: string[] = []
2524

2625
// Subscribe to raw output events

test/web-server.test.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,6 @@ describe('Web Server', () => {
129129
})
130130

131131
it('should return individual session', async () => {
132-
if (process.env.CI) return // Skip PTY tests in CI due to manager issues
133132
// Create a test session first
134133
const session = manager.spawn({
135134
command: 'cat',
@@ -157,7 +156,6 @@ describe('Web Server', () => {
157156
})
158157

159158
it('should return 404 for non-existent session', async () => {
160-
if (process.env.CI) return // Skip PTY tests in CI due to manager issues
161159
const nonexistentId = `nonexistent-${Math.random().toString(36).substr(2, 9)}`
162160
console.log('Testing non-existent session ID:', nonexistentId)
163161
const response = await fetch(`${serverUrl}/api/sessions/${nonexistentId}`)
@@ -166,7 +164,6 @@ describe('Web Server', () => {
166164
})
167165

168166
it('should handle input to session', async () => {
169-
if (process.env.CI) return // Skip PTY tests in CI due to manager issues
170167
// Create a session to test input
171168
const session = manager.spawn({
172169
command: 'cat',
@@ -198,7 +195,6 @@ describe('Web Server', () => {
198195
})
199196

200197
it('should handle kill session', async () => {
201-
if (process.env.CI) return // Skip PTY tests in CI due to manager issues
202198
const session = manager.spawn({
203199
command: 'echo',
204200
args: ['test output'],
@@ -223,7 +219,6 @@ describe('Web Server', () => {
223219
})
224220

225221
it('should return session output', async () => {
226-
if (process.env.CI) return // Skip PTY tests in CI due to manager issues
227222
// Create a session that produces output
228223
const session = manager.spawn({
229224
command: 'echo',

0 commit comments

Comments
 (0)