mirror of
https://github.com/QwenLM/qwen-code.git
synced 2026-04-28 19:52:02 +00:00
fix: increase wait times in AskUserQuestionDialog tests for CI stability
The tests used fixed wait times (200ms) that were too close to the auto-advance timeout (150ms). In CI environments (especially Windows with Node 20), timing can be less predictable due to event loop scheduling differences, causing race conditions. Changes: - Increased wait(200) to wait(300) for auto-advance tests - Increased wait() to wait(100) for navigation tests in custom input state test Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
This commit is contained in:
parent
64201e8848
commit
c074960dd7
1 changed files with 8 additions and 8 deletions
|
|
@ -402,7 +402,7 @@ describe('<AskUserQuestionDialog />', () => {
|
|||
|
||||
// Select first option in Q1
|
||||
stdin.write('\r');
|
||||
await wait(200); // Wait for auto-advance timeout (150ms)
|
||||
await wait(300); // Wait for auto-advance timeout (150ms) + buffer for CI
|
||||
|
||||
// Should now show Q2
|
||||
expect(lastFrame()).toContain('Second question?');
|
||||
|
|
@ -495,11 +495,11 @@ describe('<AskUserQuestionDialog />', () => {
|
|||
|
||||
// Answer Q1
|
||||
stdin.write('\r'); // Select Red
|
||||
await wait(200);
|
||||
await wait(300); // Wait for auto-advance (150ms) + buffer for CI
|
||||
|
||||
// Answer Q2
|
||||
stdin.write('\r'); // Select Red
|
||||
await wait(200);
|
||||
await wait(300); // Wait for auto-advance (150ms) + buffer for CI
|
||||
|
||||
// Now on Submit tab, press Enter to submit
|
||||
stdin.write('\r');
|
||||
|
|
@ -649,7 +649,7 @@ describe('<AskUserQuestionDialog />', () => {
|
|||
|
||||
// Answer Q1
|
||||
stdin.write('\r'); // Select Red
|
||||
await wait(200);
|
||||
await wait(300); // Wait for auto-advance (150ms) + buffer for CI
|
||||
|
||||
// Q2 is now active; check that Q1 shows ✓
|
||||
expect(lastFrame()).toContain('Q1');
|
||||
|
|
@ -776,23 +776,23 @@ describe('<AskUserQuestionDialog />', () => {
|
|||
// Navigate to custom input (index 3)
|
||||
for (let i = 0; i < 3; i++) {
|
||||
stdin.write('\u001B[B');
|
||||
await wait();
|
||||
await wait(100); // Increased wait for reliable navigation
|
||||
}
|
||||
|
||||
// Type something - should auto-check
|
||||
stdin.write('Custom');
|
||||
await wait();
|
||||
await wait(100); // Wait for state update
|
||||
|
||||
// Should already be checked (auto-checked on type)
|
||||
expect(lastFrame()).toContain('[✓]');
|
||||
|
||||
// Navigate away
|
||||
stdin.write('\u001B[A'); // Up
|
||||
await wait();
|
||||
await wait(100);
|
||||
|
||||
// Navigate back to custom input
|
||||
stdin.write('\u001B[B'); // Down
|
||||
await wait();
|
||||
await wait(100);
|
||||
|
||||
// Should still be checked
|
||||
const output = lastFrame()!;
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue