-
Notifications
You must be signed in to change notification settings - Fork 5.1k
[wasm] Testing with NodeJs on CI #62779
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label. |
Tagging subscribers to 'arch-wasm': @lewing Issue DetailsOnly in
|
/azp run runtime-manual |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-manual |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-manual |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-manual |
Azure Pipelines successfully started running 1 pipeline(s). |
359fe47
to
bcedca9
Compare
/azp run runtime-manual |
Azure Pipelines successfully started running 1 pipeline(s). |
bcedca9
to
50a4298
Compare
/azp run runtime-manual |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-manual |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-manual |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-manual |
Azure Pipelines successfully started running 1 pipeline(s). |
79a562f
to
69b8d73
Compare
/azp run runtime-manual |
/azp run runtime-manual |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-manual |
Azure Pipelines successfully started running 1 pipeline(s). |
src/tests/BuildWasmApps/Wasm.Build.Tests/data/RunScriptTemplate.cmd
Outdated
Show resolved
Hide resolved
/azp run runtime-manual |
Azure Pipelines successfully started running 1 pipeline(s). |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. It would be nice to have activeissue set on the nodejs failing tests.
/azp run runtime-manual |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-manual |
Azure Pipelines successfully started running 1 pipeline(s). |
Set BROWSER_PATH variable only when HELIX_CORRELATION_PAYLOAD variable is set. Fixes change in the dotnet#62779.
Set BROWSER_PATH variable only when HELIX_CORRELATION_PAYLOAD variable is set. Fixes change in the #62779.
Add WASM test on NodeJS on CI
wasmtestonnodejs
inruntime-manual
pipelinewebassembly-20210707133424-12f133e
with NodeJs upgradeRunScript improvements
SetScriptCommands
early in the script templatesSCENARIO
andJS_ENGINE
env variablesSendToHelixHelp
WasmTestOnNodeJs
scenario triggersNeedsEMSDKNode
NeedsEMSDKNode
would take emscripten NodeJs binaries and push it as HelixCorrelationPayloadFixes #47336
Fixes #41861
Fixes #62901