Skip to content

Commit 5d567f3

Browse files
e2e test: includes-excludes use fail instead of log warning (#6742)
### Intent Now that #6651 is done, we can make these tests fail instead of just warn when the hidden interpreter is not installed ### QA Notes @:interpreter All tests should pass. We won't see the release tests pass until tonight's test run.
1 parent dfca2fd commit 5d567f3

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/e2e/tests/interpreters/includes-excludes.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ test.describe('Interpreter Includes/Excludes', {
2424
await userSettings.set([['python.interpreters.include', '["/home/runner/scratch/python-env"]']], true);
2525
await app.workbench.interpreter.selectInterpreter(InterpreterType.Python, hiddenPython, true);
2626
} else {
27-
logger.log('Hidden Python version not set'); // use this for now so release test can essentially skip this case
27+
fail('Hidden Python version not set');
2828
}
2929
});
3030

@@ -36,7 +36,7 @@ test.describe('Interpreter Includes/Excludes', {
3636
await userSettings.set([['positron.r.customRootFolders', '["/home/runner/scratch"]']], true);
3737
await app.workbench.interpreter.selectInterpreter(InterpreterType.R, hiddenR, true);
3838
} else {
39-
logger.log('Hidden R version not set'); // use this for now so release test can essentially skip this case
39+
fail('Hidden R version not set');
4040
}
4141
});
4242

0 commit comments

Comments
 (0)