test: WPT for Wasm jsapi including new ESM Integration tests#59034
test: WPT for Wasm jsapi including new ESM Integration tests#59034guybedford wants to merge 8 commits intonodejs:mainfrom
Conversation
|
Review requested:
|
41d1b4b to
aacf993
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #59034 +/- ##
==========================================
+ Coverage 89.67% 89.68% +0.01%
==========================================
Files 676 676
Lines 206501 206738 +237
Branches 39540 39600 +60
==========================================
+ Hits 185177 185412 +235
- Misses 13457 13463 +6
+ Partials 7867 7863 -4 🚀 New features to boost your workflow:
|
7cb3437 to
7e36e0d
Compare
|
|
@miladfarca FYI it looks like some of these tests are failing on Linux s390x. |
|
So, it seems it this is for round-tripping v128: The value returned is consistent with big endian handling of the SIMD path, which formally speaking is a V8 bug since Wasm is supposed to have little endian behaviours. There is a SIMD test for this in V8, but V8 only runs x390x simulator assuming I've pushed a temporary commit which replicates the internal simd test suite case in question, to see if that also fails on the hardware. If so, then this is a V8 bug on s390x separate to the WPT work here. |
|
I can confirm the same failure, so this is a v8 bug for Wasm SIMD on s390x - https://ci.nodejs.org/job/node-test-commit-aix/61569/nodes=aix72-power9/testReport/junit/(root)/parallel/test_wasm_simd_global/. I've added a specific skip for this environment to resolve the issue, given this is not a Node.js or WPT issue. |
FWIW AIX is not Linux s390x, but both are big endian platforms. |
This adds the jsapi test suite for WebAssembly including the new ESM Integration jsapi tests from WebAssembly/esm-integration#110 with upstream PR in web-platform-tests/wpt#53718.
Tests for string builtins and namespaceInstance are pending #59020 and #59024 respectively.