Skip to content

Commit f482149

Browse files
Smoke test: do not wait for a verification on open when an earlier test verified the fixture
The server only verifies on the first file opened, so after the state error test the smoke test waited for diagnostics that never came whenever that test's notifications had all arrived before it ended. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 parent 950ac38 commit f482149

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

‎client/src/test/smoke.test.ts‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,9 @@ suite('Bundled LiquidJava extension', () => {
4141
const initialDiagnostics = nextFixtureDiagnostics();
4242
const document = await vscode.workspace.openTextDocument(uri);
4343
await vscode.window.showTextDocument(document);
44-
await initialDiagnostics;
44+
// the server only verifies on the first file opened, so an earlier test may have verified this one
45+
const verified = api.getState().diagnostics.some(d => vscode.Uri.file(path.resolve(d.file)).fsPath === uri.fsPath);
46+
if (!verified) await initialDiagnostics;
4547

4648
const manualDiagnostics = nextFixtureDiagnostics();
4749
await vscode.commands.executeCommand('liquidjava.verify');

0 commit comments

Comments
 (0)