MediaMetz/gemini-cli
1
0
Fork 0
mirror of https://github.com/google-gemini/gemini-cli.git synced 2026-05-15 14:23:02 -07:00
Code Issues Packages Projects Releases Wiki Activity

ci: run suspect tests with threads in CI and improve AppRig fake timer support

Browse Source
This commit is contained in:
mkorwel
2026-04-16 22:46:39 +00:00
parent 6b4f7743f0
commit 74f7a72f79
2 changed files with 26 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.github/workflows/ci-bundling-trial.yml
+1 -1
View File
@@ -27,7 +27,7 @@ jobs:
mode: 'source'
- name: 'Run Suspect Tests'
run: 'npx vitest run --pool=forks packages/cli/src/test-utils/render.test.tsx packages/cli/src/ui/App.test.tsx packages/cli/src/test-utils/AppRig.test.tsx packages/cli/src/ui/components/SettingsDialog.test.tsx packages/cli/src/ui/components/messages/DiffRenderer.test.tsx packages/cli/src/ui/components/messages/ThinkingMessage.test.tsx packages/cli/src/ui/components/messages/ToolResultDisplay.test.tsx packages/cli/src/ui/components/messages/ToolResultDisplayOverflow.test.tsx packages/cli/src/ui/components/messages/UserMessage.test.tsx packages/cli/src/ui/components/shared/SearchableList.test.tsx packages/cli/src/ui/components/shared/TextInput.test.tsx packages/cli/src/ui/components/views/ExtensionRegistryView.test.tsx'
run: 'npx vitest run --pool=threads packages/cli/src/test-utils/render.test.tsx packages/cli/src/ui/App.test.tsx packages/cli/src/test-utils/AppRig.test.tsx packages/cli/src/ui/components/SettingsDialog.test.tsx packages/cli/src/ui/components/messages/DiffRenderer.test.tsx packages/cli/src/ui/components/messages/ThinkingMessage.test.tsx packages/cli/src/ui/components/messages/ToolResultDisplay.test.tsx packages/cli/src/ui/components/messages/ToolResultDisplayOverflow.test.tsx packages/cli/src/ui/components/messages/UserMessage.test.tsx packages/cli/src/ui/components/shared/SearchableList.test.tsx packages/cli/src/ui/components/shared/TextInput.test.tsx packages/cli/src/ui/components/views/ExtensionRegistryView.test.tsx'
shell: 'bash'
packages/cli/src/test-utils/AppRig.tsx
+25 -4
View File
@@ -314,6 +314,7 @@ export class AppRig {
authType: authMethod,
proxy: gcConfig.getProxy(),
apiKey: process.env['GEMINI_API_KEY'] || 'test-api-key',
fakeResponses: this.options.fakeResponsesPath,
};
gcConfig.contentGenerator = await createContentGenerator(
@@ -437,6 +438,7 @@ export class AppRig {
uiState: {
terminalHeight: this.options.terminalHeight ?? 40,
},
clearScreenOnRender: false,
},
);
});
@@ -500,7 +502,11 @@ export class AppRig {
}
await act(async () => {
await new Promise((resolve) => setTimeout(resolve, interval));
if (vi.isFakeTimers()) {
await vi.advanceTimersByTimeAsync(interval);
} else {
await new Promise((resolve) => setTimeout(resolve, interval));
}
});
}
}
@@ -562,7 +568,11 @@ export class AppRig {
await this.waitUntil(
async () => {
await act(async () => {
await new Promise((resolve) => setTimeout(resolve, 0));
if (vi.isFakeTimers()) {
await vi.advanceTimersByTimeAsync(0);
} else {
await new Promise((resolve) => setTimeout(resolve, 0));
}
});
confirmation = this.getPendingConfirmations()[0];
// Now that we have a code-powered signal, this should be perfectly deterministic.
@@ -677,7 +687,11 @@ export class AppRig {
this.renderResult!.stdin.write(text);
});
await act(async () => {
await new Promise((resolve) => setTimeout(resolve, 50));
if (vi.isFakeTimers()) {
await vi.advanceTimersByTimeAsync(50);
} else {
await new Promise((resolve) => setTimeout(resolve, 50));
}
});
}
@@ -691,7 +705,11 @@ export class AppRig {
this.renderResult!.stdin.write(key);
});
await act(async () => {
await new Promise((resolve) => setTimeout(resolve, 50));
if (vi.isFakeTimers()) {
await vi.advanceTimersByTimeAsync(50);
} else {
await new Promise((resolve) => setTimeout(resolve, 50));
}
});
}
@@ -728,6 +746,9 @@ export class AppRig {
this.awaitingResponse = true;
await this.type(text);
await this.pressEnter();
if (vi.isFakeTimers()) {
await vi.advanceTimersByTimeAsync(1000);
}
}
async unmount() {