cleanup test

Author: lkwinta

test/test_other.py

@@ -3959,8 +3959,9 @@ def test_file_packager_standalone_modularize(self):
 
     create_file('test.c', '''
     #include <stdio.h>
+    #include <emscripten.h>
 
-    int main() {
+    EMSCRIPTEN_KEEPALIVE int test_fun() {
       FILE* f = fopen("data.txt", "r");
       char buf[64] = {0};
       int rtn = fread(buf, 1, 64, f);
@@ -3976,9 +3977,10 @@ def test_file_packager_standalone_modularize(self):
     import loadDataFile from './dataFileLoader.js'
     import {default as loadModule} from './moduleFile.js'
 
-    const sleep = (delay) => new Promise((resolve) => setTimeout(resolve, delay))
-    loadModule().then(async (module) => {
-      loadDataFile(module).then(() => {
+    loadModule().then((module) => {
+      loadDataFile(module)
+        .catch((cause) => Promise.reject(cause))
+        .then(() => {
           module._test_fun();
         }
       );

tools/file_packager.py

@@ -709,7 +709,7 @@ def generate_js(data_target, data_files, metadata):
     create_data = '''// canOwn this data in the filesystem, it is a slide into the heap that will never change
           Module['FS_createDataFile'](this.name, null, byteArray, true, true, true);
           Module['removeRunDependency'](`fp ${that.name}`);'''
-    ready_promise = '''         
+    ready_promise = '''
           readyPromiseResolve();'''
 
     if not options.lz4:
@@ -986,7 +986,7 @@ def generate_js(data_target, data_files, metadata):
           return;
         }'''.strip()
 
-    reject_promise = '''         
+    reject_promise = '''
           readyPromiseReject();'''
 
     ret += '''