Skip to content

Commit 7431da5

Browse files
committed
skip tmux test in gh actions
1 parent cda474d commit 7431da5

File tree

2 files changed

+12
-7
lines changed

2 files changed

+12
-7
lines changed

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

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -82,10 +82,7 @@ function tmux(args: string[]): Promise<string> {
8282
* Send input to the CLI using bracketed paste mode.
8383
* Standard send-keys doesn't work with OpenTUI - see tmux.knowledge.md
8484
*/
85-
async function sendCliInput(
86-
sessionName: string,
87-
text: string,
88-
): Promise<void> {
85+
async function sendCliInput(sessionName: string, text: string): Promise<void> {
8986
await tmux([
9087
'send-keys',
9188
'-t',
@@ -115,7 +112,7 @@ describe.skipIf(!tmuxAvailable || !sdkBuilt)(
115112
// Propagate environment into tmux server
116113
await Promise.all(
117114
entries.map(([key, value]) =>
118-
tmux(['set-environment', '-g', key, value]).catch(() => {}),
115+
tmux(['set-environment', '-g', key, value]),
119116
),
120117
)
121118
// Enable performance testing
@@ -162,10 +159,10 @@ describe.skipIf(!tmuxAvailable || !sdkBuilt)(
162159
// Log analysis for debugging
163160
console.log('\n📊 Re-render Analysis:')
164161
console.log(` Total re-renders: ${analysis.totalRerenders}`)
162+
console.log(` Max per message: ${analysis.maxRerenderPerMessage}`)
165163
console.log(
166-
` Max per message: ${analysis.maxRerenderPerMessage}`,
164+
` Messages tracked: ${analysis.rerendersByMessage.size}`,
167165
)
168-
console.log(` Messages tracked: ${analysis.rerendersByMessage.size}`)
169166
if (analysis.propChangeFrequency.size > 0) {
170167
console.log(' Prop change frequency:')
171168
for (const [prop, count] of analysis.propChangeFrequency) {

cli/src/__tests__/test-utils.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,16 @@ import path from 'path'
66
* Check if tmux is available and usable on the system.
77
* This checks both that tmux is installed AND that it can actually run
88
* (e.g., the tmux server socket directory exists and is accessible).
9+
*
10+
* Note: Always returns false on CI since tmux integration tests require
11+
* a real interactive terminal environment.
912
*/
1013
export function isTmuxAvailable(): boolean {
14+
// Skip on CI - tmux integration tests need a real terminal environment
15+
if (process.env.CI === 'true' || process.env.CI === '1') {
16+
return false
17+
}
18+
1119
try {
1220
// First check if tmux is installed
1321
execSync('which tmux', { stdio: 'pipe' })

0 commit comments

Comments
 (0)