Skip to content

Commit 3f0188c

Browse files
committed
Cleanup test_modularize_init_error. NFC
- Use browser_reporting.js directly. - Delete redundant unhandledrejection handler. browser_reporting handles this already.
1 parent c08204e commit 3f0188c

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

test/test_browser.py

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3343,8 +3343,10 @@ def test_modularize(self, opts):
33433343

33443344
def test_modularize_network_error(self):
33453345
browser_reporting_js_path = test_file('browser_reporting.js')
3346-
self.compile_btest('browser_test_hello_world.c', ['-sMODULARIZE', '-sEXPORT_NAME="createModule"', '--extern-pre-js', browser_reporting_js_path], reporting=Reporting.NONE)
3346+
self.compile_btest('browser_test_hello_world.c', ['-sMODULARIZE', '-sEXPORT_NAME=createModule'], reporting=Reporting.NONE)
3347+
shutil.copy(test_file('browser_reporting.js'), '.')
33473348
create_file('a.html', '''
3349+
<script src="browser_reporting.js"></script>
33483350
<script src="a.out.js"></script>
33493351
<script>
33503352
createModule()
@@ -3361,16 +3363,12 @@ def test_modularize_network_error(self):
33613363
self.run_browser('a.html', '/report_result?Aborted(both async and sync fetching of the wasm failed)')
33623364

33633365
def test_modularize_init_error(self):
3364-
browser_reporting_js_path = test_file('browser_reporting.js')
3365-
self.compile_btest('browser/test_modularize_init_error.cpp', ['-sMODULARIZE', '-sEXPORT_NAME="createModule"', '--extern-pre-js', browser_reporting_js_path], reporting=Reporting.NONE)
3366+
self.compile_btest('browser/test_modularize_init_error.cpp', ['-sMODULARIZE', '-sEXPORT_NAME=createModule'], reporting=Reporting.NONE)
3367+
shutil.copy(test_file('browser_reporting.js'), '.')
33663368
create_file('a.html', '''
3369+
<script src="browser_reporting.js"></script>
33673370
<script src="a.out.js"></script>
33683371
<script>
3369-
if (typeof window === 'object') {
3370-
window.addEventListener('unhandledrejection', function(event) {
3371-
reportResultToServer("Unhandled promise rejection: " + event.reason.message);
3372-
});
3373-
}
33743372
createModule()
33753373
.then(() => {
33763374
reportResultToServer("Module creation succeeded when it should have failed");

0 commit comments

Comments
 (0)