From 5c99623c00c276761abc72c4dba31d69a3490680 Mon Sep 17 00:00:00 2001 From: Andreas Arvidsson Date: Sat, 26 Apr 2025 16:40:23 +0200 Subject: [PATCH] Fail scope tests with no scopes --- .../src/suite/scopes.vscode.test.ts | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/packages/cursorless-vscode-e2e/src/suite/scopes.vscode.test.ts b/packages/cursorless-vscode-e2e/src/suite/scopes.vscode.test.ts index e4453990b2..e05213868d 100644 --- a/packages/cursorless-vscode-e2e/src/suite/scopes.vscode.test.ts +++ b/packages/cursorless-vscode-e2e/src/suite/scopes.vscode.test.ts @@ -123,7 +123,7 @@ async function runTest(file: string, languageId: string, facetId: string) { const editor = ide.activeTextEditor!; - const outputFixture = ((): string => { + const [outputFixture, numScopes] = ((): [string, number] => { const config = { visibleOnly: false, scopeType, @@ -137,17 +137,21 @@ async function runTest(file: string, languageId: string, facetId: string) { includeNestedTargets: false, }, ); - return serializeIterationScopeFixture(code, iterationScopes); + return [ + serializeIterationScopeFixture(code, iterationScopes), + iterationScopes.length, + ]; } const scopes = scopeProvider.provideScopeRanges(editor, config); - return serializeScopeFixture(facetId, code, scopes); + return [serializeScopeFixture(facetId, code, scopes), scopes.length]; })(); if (shouldUpdateFixtures()) { await fsp.writeFile(file, outputFixture); } else { + assert.isAbove(numScopes, 0, "No scopes found"); assert.equal(outputFixture, fixture); } }