Closed Bug 1570278 Opened 6 years ago Closed 6 years ago

Some async test262 tests for dynamic import fail

Categories

(Core :: JavaScript Engine, task, P2)

task

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: anba, Assigned: anba)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

When enabling dynamic import() tests when running test262 in the browser (bug 1570011), the following tests failed:

  • test262/language/expressions/dynamic-import/catch/nested-arrow-import-catch-instn-iee-err-ambiguous-import.js
  • test262/language/expressions/dynamic-import/catch/nested-async-function-await-instn-iee-err-ambiguous-import.js
  • test262/language/expressions/dynamic-import/catch/nested-block-labeled-instn-iee-err-ambiguous-import.js
  • test262/language/expressions/dynamic-import/catch/nested-do-while-instn-iee-err-ambiguous-import.js
  • test262/language/expressions/dynamic-import/catch/nested-else-import-catch-instn-iee-err-ambiguous-import.js
  • test262/language/expressions/dynamic-import/catch/nested-if-import-catch-instn-iee-err-ambiguous-import.js
  • test262/language/expressions/dynamic-import/catch/nested-while-import-catch-instn-iee-err-ambiguous-import.js

It looks like jsTestDriverEnd is called before $DONE, so maybe it's necessary to modify gDelayTestDriverEnd and manually call jsTestDriverEnd for async tests.

Different try-runs show failures in different tests, so it seems best to keep all dynamic import() tests disabled in the browser until this bug is fixed.

Priority: -- → P2

Depends on D40087

Assignee: nobody → andrebargull
Status: NEW → ASSIGNED
Keywords: checkin-needed

Pushed by apavel@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fa8554671087
Part 1: Add 'async' flag for asynchronous test262 tests. r=jonco
https://hg.mozilla.org/integration/autoland/rev/c5ee864490dc
Part 2: Reimport test262 files. r=jonco

Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: