Skip to content

Commit 7326e6a

Browse files
authored
test: ensure nightly scheduled tests run all samples unconditionally (#1505)
* test: ensure nightly scheduled tests run all samples unconditionally * Remove FORCE_JAVASCRIPT_ACTIONS_TO_NODE24 variable Removed FORCE_JAVASCRIPT_ACTIONS_TO_NODE24 environment variable. The 3P actions plugins in use here all depend on Node 20, so this is not needed. In general the internal runner node can use a different version of Node (even though Node 20 is slated for deprecation in September of 2026).
1 parent 9f59152 commit 7326e6a

2 files changed

Lines changed: 8 additions & 1 deletion

File tree

.github/workflows/tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ on:
3434
- cron: "0 12 * * *"
3535

3636
env:
37-
FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: true
3837
SKIP_DIST: ${{ github.event_name == 'pull_request' || github.event_name == 'schedule' }}
3938

4039
concurrency:
@@ -216,6 +215,7 @@ jobs:
216215
run: npx playwright test e2e/samples.spec.ts
217216
env:
218217
CI: true
218+
TEST_ALL_SAMPLES: "true"
219219

220220
- name: Upload Test Report Artifact
221221
uses: actions/upload-artifact@v4

e2e/samples.spec.ts

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,13 @@ const getChangedSampleFolders = (): string[] => {
4242
'find-changes.sh'
4343
);
4444

45+
if (process.env.TEST_ALL_SAMPLES === 'true') {
46+
console.log(
47+
'TEST_ALL_SAMPLES is set. Running tests for all samples.'
48+
);
49+
return getAllSampleFolders();
50+
}
51+
4552
if (!fs.existsSync(scriptPath)) {
4653
console.warn(
4754
`Warning: find-changes.sh not found at ${scriptPath}. Running tests for all samples.`

0 commit comments

Comments
 (0)