Skip to content

[test] Remove EMTEST_ALL_ENGINES#27396

Open
sbc100 wants to merge 1 commit into
emscripten-core:mainfrom
sbc100:remove_all_engines
Open

[test] Remove EMTEST_ALL_ENGINES#27396
sbc100 wants to merge 1 commit into
emscripten-core:mainfrom
sbc100:remove_all_engines

Conversation

@sbc100

@sbc100 sbc100 commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

This simplifies the test runner by removing the ability to run a single test under multiple engines.

The way that out testing works today is that we run all tests (or a subset of tests such as --crossplatform-only) with a given engine configured. We don't have any CI jobs, for example, that use the --all-engines / EMTEST_ALL_ENGINES feature and I can't remember the last time I used it myself.

This change is part of an effort to simplify emscripten config file.

@sbc100
sbc100 requested a review from kripken July 21, 2026 20:24
@sbc100
sbc100 force-pushed the remove_all_engines branch 3 times, most recently from 1c9e6a4 to 2a7b911 Compare July 21, 2026 20:37
This simplifies the test runner by removing the ability to run a single
test under multiple engines.

The way that out testing works today is that we run all tests (or a
subset of tests such as `--crossplatform-only`) with a given engine
configured.  We don't have any CI jobs, for example, that use the
`--all-engines` / `EMTEST_ALL_ENGINES` feature and I can't remember the
last time I used it myself.

This change is part of an effort to simplify emscripten config file.
@sbc100
sbc100 force-pushed the remove_all_engines branch from 2a7b911 to 3983f28 Compare July 21, 2026 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants