Skip to content

test: enable bazel e2e windows ci job - #23853

Closed
jbedard wants to merge 5 commits into
angular:mainfrom
jbedard:legacy-test-windows
Closed

test: enable bazel e2e windows ci job#23853
jbedard wants to merge 5 commits into
angular:mainfrom
jbedard:legacy-test-windows

Conversation

@jbedard

@jbedard jbedard commented Sep 2, 2022

Copy link
Copy Markdown
Contributor

Extension of #23074 to get running on windows while keeping that one green.

@jbedard
jbedard force-pushed the legacy-test-windows branch 24 times, most recently from 389adf8 to cd06f7d Compare September 9, 2022 22:50
@jbedard
jbedard force-pushed the legacy-test-windows branch 6 times, most recently from 2581e4c to 367b2e9 Compare September 12, 2022 20:47
@jbedard
jbedard force-pushed the legacy-test-windows branch 23 times, most recently from cadea32 to ece191e Compare September 20, 2022 19:25
The NPM packages being tested must be pre-compiled and the tar packages specified via --package. This way the real packages such as snapshots, release artifacts or cached packages can be tested. Previously the e2e tests compiled and packaged during test execution.
Using execPath ensures the child processes are the same node binary without depending on PATH.
@jbedard

jbedard commented Sep 20, 2022

Copy link
Copy Markdown
Contributor Author

Moving to #23074 now that it's passing

@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant