-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[wasm] Add Wasm.Build tests, for testing wasm app builds #47683
Commits on Mar 3, 2021
-
[wasm] Add Wasm.Build tests, for testing wasm app builds
These tests will build wasm test projects, as part of each test method, and run them. Other library tests are run with `xharness`, and the test assembly is run under wasm. But here we want to run them with xunit, outside wasm. So, this has different requirements for the helix payload, eg, the sdk, xunit console runner etc. To make it work, a new Scenario - `BuildWasmApps` is added, which emits it's archives in a `buildwasmapps/` folder, which makes it easy to pick up for the helix test run. The tests are added under `src/tests/BuildWasmApps/Wasm.Build.Tests`, but they use `Directory.Build*` from `src/libraries`, similar to how FunctionalTests do it. Another use case of this kinda scenario are the wasm debugger tests, in which the individual test methods launch wasm apps, and then debug them. (TBD) Tests: - The initial set of tests are just proof-of-concept, and more will be added once this is merged. Note: The individual tests build test projects, and then run them with `xharness`, under `v8`, and Chrome.
Configuration menu - View commit details
-
Copy full SHA for 2739be7 - Browse repository at this point
Copy the full SHA 2739be7View commit details -
[wasm] Disable il stripping completely
This uses `mono-cil-strip` from a mono installation. And in it's current form it can cause issues, so disabling it for now.
Configuration menu - View commit details
-
Copy full SHA for 25c2340 - Browse repository at this point
Copy the full SHA 25c2340View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3af97d1 - Browse repository at this point
Copy the full SHA 3af97d1View commit details -
[wasm] Cleanup builing RunScriptCommand
.. this allows supporting other properties from xunit*targets, eg. to run a particular test `$(XUnitMethodName)`, which adds `-method foobar` to the command line.
Configuration menu - View commit details
-
Copy full SHA for 8ef0923 - Browse repository at this point
Copy the full SHA 8ef0923View commit details -
Configuration menu - View commit details
-
Copy full SHA for 03e8730 - Browse repository at this point
Copy the full SHA 03e8730View commit details -
Configuration menu - View commit details
-
Copy full SHA for 9aee29e - Browse repository at this point
Copy the full SHA 9aee29eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 33f6995 - Browse repository at this point
Copy the full SHA 33f6995View commit details -
Configuration menu - View commit details
-
Copy full SHA for ef0e104 - Browse repository at this point
Copy the full SHA ef0e104View commit details
Commits on Mar 4, 2021
-
Configuration menu - View commit details
-
Copy full SHA for b074297 - Browse repository at this point
Copy the full SHA b074297View commit details -
Configuration menu - View commit details
-
Copy full SHA for e0c5a7c - Browse repository at this point
Copy the full SHA e0c5a7cView commit details -
[wasm] Enable verbose output for xunit
Instead of writing all the output to stdout also, use `-verbose` which gives output like: ``` Wasm.Build.Tests.WasmBuildAppTest.InvariantGlobalization(config: "Debug", aot: False, invariantGlobalization: null) [STARTING] ============== wasm test ============= ============== wasm test-browser ============= Wasm.Build.Tests.WasmBuildAppTest.InvariantGlobalization(config: "Debug", aot: False, invariantGlobalization: null) [FINISHED] Time: 8.6357275s ``` We log the detailed output to files anyway.
Configuration menu - View commit details
-
Copy full SHA for 9600a13 - Browse repository at this point
Copy the full SHA 9600a13View commit details -
Configuration menu - View commit details
-
Copy full SHA for d10510e - Browse repository at this point
Copy the full SHA d10510eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 8aeec63 - Browse repository at this point
Copy the full SHA 8aeec63View commit details -
Configuration menu - View commit details
-
Copy full SHA for b94865f - Browse repository at this point
Copy the full SHA b94865fView commit details
Commits on Mar 5, 2021
-
Configuration menu - View commit details
-
Copy full SHA for 33efd20 - Browse repository at this point
Copy the full SHA 33efd20View commit details -
Configuration menu - View commit details
-
Copy full SHA for a31092d - Browse repository at this point
Copy the full SHA a31092dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 204b3ee - Browse repository at this point
Copy the full SHA 204b3eeView commit details -
Configuration menu - View commit details
-
Copy full SHA for ac147c7 - Browse repository at this point
Copy the full SHA ac147c7View commit details
Commits on Mar 8, 2021
-
Co-authored-by: Mitchell Hwang <mitchhwang1418@gmail.com>
Configuration menu - View commit details
-
Copy full SHA for 2bf3845 - Browse repository at this point
Copy the full SHA 2bf3845View commit details -
Update eng/testing/tests.mobile.targets
Co-authored-by: Mitchell Hwang <mitchhwang1418@gmail.com>
Configuration menu - View commit details
-
Copy full SHA for 31d5d2f - Browse repository at this point
Copy the full SHA 31d5d2fView commit details -
Instead, this is moved to a different AOT PR. Revert "[wasm] Disable il stripping completely" This reverts commit 25c2340.
Configuration menu - View commit details
-
Copy full SHA for ead13ee - Browse repository at this point
Copy the full SHA ead13eeView commit details -
Configuration menu - View commit details
-
Copy full SHA for 17ce3d8 - Browse repository at this point
Copy the full SHA 17ce3d8View commit details
Commits on Mar 9, 2021
-
Revert "Remove unrelated commit"
This is needed because `mono-cil-strip` isn't available on helix. And we want to disable cil stripping anyway. This reverts commit ead13ee.
Configuration menu - View commit details
-
Copy full SHA for 7be536d - Browse repository at this point
Copy the full SHA 7be536dView commit details