[test] Cleanup make_reftest part 2. NFC

Avoid including/inlining browser_reporting.js in the middle of reftest.js.
emscripten-core · Sep 25, 2024 · f657328 · f657328
1 parent a0fc53c
commit f657328
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 19 deletions.
diff --git a/test/common.py b/test/common.py
@@ -2191,7 +2191,6 @@ def run_browser(self, html_file, expected=None, message=None, timeout=None, extr
   def make_reftest(self, expected, manually_trigger=False):
     # make sure the pngs used here have no color correction, using e.g.
     #   pngcrush -rem gAMA -rem cHRM -rem iCCP -rem sRGB infile outfile
-    reporting = read_file(test_file('browser_reporting.js'))
     shutil.copyfile(expected, 'expected.png')
     create_file('reftest.js', '''
       function doReftest() {
@@ -2239,30 +2238,16 @@ def make_reftest(self, expected, manually_trigger=False):
             }
             // floor, to allow some margin of error for antialiasing
             var wrong = Math.floor(total / (img.width*img.height*3));
-
-            function reportResult(result) {
-              // If the main JS file is in a worker, or modularize, then we need to supply our own
-              // reporting logic.
-              if (typeof reportResultToServer === 'undefined') {
-                (() => {
-                  %s
-                  reportResultToServer(result);
-                })();
-              } else {
-                reportResultToServer(result);
-              }
-            }
-
             var rebaseline = %s;
             if (wrong || rebaseline) {
               // Generate a png of the actual rendered image and send it back
               // to the server.
               Module.canvas.toBlob((blob) => {
                 sendFileToServer('actual.png', blob);
-                reportResult(wrong);
+                reportResultToServer(wrong);
               })
             } else {
-              reportResult(wrong);
+              reportResultToServer(wrong);
             }
           };
           actualImage.src = actualUrl;
@@ -2311,7 +2296,7 @@ def make_reftest(self, expected, manually_trigger=False):
       }
 
       setupRefTest();
-''' % (reporting, EMTEST_REBASELINE, int(manually_trigger)))
+''' % (EMTEST_REBASELINE, int(manually_trigger)))
 
   def compile_btest(self, filename, args, reporting=Reporting.FULL):
     # Inject support code for reporting results. This adds an include a header so testcases can

diff --git a/test/test_browser.py b/test/test_browser.py
@@ -852,9 +852,11 @@ def test_sdl_canvas(self, args):
 
   def post_manual_reftest(self):
     assert os.path.exists('reftest.js')
+    shutil.copyfile(test_file('browser_reporting.js'), 'browser_reporting.js')
     html = read_file('test.html')
     html = html.replace('</body>', '''
-<script src="reftest.js"/>
+<script src="browser_reporting.js"></script>
+<script src="reftest.js"></script>
 <script>
 var windowClose = window.close;
 window.close = () => {