Skip to content

build: define node_script_test targets for unit and E2E tests - #3347

Open
Lightning00Blade wants to merge 1 commit into
gn-migration/07-ts-library-tests-and-drop-tscfrom
gn-migration/08-node-script-tests
Open

Lightning00Blade wants to merge 1 commit into
gn-migration/07-ts-library-tests-and-drop-tscfrom
gn-migration/08-node-script-tests

Conversation

@Lightning00Blade

@Lightning00Blade Lightning00Blade commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator
  • Define run_chrome_devtools_mcp_unit_tests and run_chrome_devtools_mcp_e2e_tests via node_script_test in tests/BUILD.gn and wire them into the root default group in BUILD.gn.
  • Update scripts/test.js to accept --build-dir and --suite=unit|e2e|all and run Node using process.execPath from the project root.
  • Resolve compiled entrypoints and snapshot paths relative to import.meta.dirname and spawn child Node processes via process.execPath so wrapper scripts run from any build directory.

Stack created with GitHub Stacks CLI • Give Feedback 💬

@Lightning00Blade
Lightning00Blade added this pull request to stack #3333 October 26, 2026 13:36
@Lightning00Blade
Lightning00Blade marked this pull request as draft October 26, 2026 13:38
@Lightning00Blade
Lightning00Blade marked this pull request as ready for review October 26, 2026 14:33
@Lightning00Blade
Lightning00Blade force-pushed the gn-migration/08-node-script-tests branch from a326a05 to 8df7de1 Compare October 26, 2026 14:33
@Lightning00Blade Lightning00Blade changed the title build: define node_script_test targets and implement Test Executable Protocol build: define node_script_test targets for unit and E2E tests Oct 26, 2026
@Lightning00Blade
Lightning00Blade force-pushed the gn-migration/08-node-script-tests branch from 8df7de1 to 0776334 Compare October 26, 2026 14:46
@Lightning00Blade
Lightning00Blade force-pushed the gn-migration/08-node-script-tests branch from 0776334 to 097c1f9 Compare October 26, 2026 15:03
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.

1 participant