Move function a bit to minimize Git diff

Author: RReverser

test/test_other.py

@@ -9246,76 +9246,6 @@ def build(args):
     # adding --metrics should not affect code size
     self.assertEqual(base_size, os.path.getsize('a.out.wasm'))
 
-  def check_output_sizes(self, *outputs: str, **metadata):
-    test_name = self.id().split('.')[-1]
-    results_file = test_file('code_size', test_name + '.json')
-
-    expected_results: dict = {}
-    try:
-      expected_results = json.loads(read_file(results_file))
-    except Exception:
-      if not common.EMTEST_REBASELINE:
-        raise
-
-    obtained_results = {}
-
-    total_output_size = 0
-    total_expected_size = 0
-    total_output_size_gz = 0
-    total_expected_size_gz = 0
-    for f in outputs:
-      f_gz = f + '.gz'
-      expected_size = expected_results.get(f, inf)
-      expected_size_gz = expected_results.get(f_gz, inf)
-      contents = read_binary(f)
-      size = len(contents)
-      size_gz = len(gzip.compress(contents))
-
-      obtained_results[f] = size
-      obtained_results[f_gz] = size_gz
-
-      if not common.EMTEST_REBASELINE and size != expected_size and (f.endswith(('.js', '.html'))):
-        print('Contents of ' + f + ': ')
-        print(contents.decode('utf-8', errors='replace'))
-
-      def print_diff(title, actual, expected):
-        diff = actual - expected
-        s = f'{title}={actual}, expected {expected}'
-        if diff > 0:
-          s += f', delta={diff} ({diff * 100.0 / expected:+.2f}%)'
-        print(s)
-
-      print_diff(f'size of {f}', size, expected_size)
-      print_diff(f'size of {f_gz}', size_gz, expected_size_gz)
-
-      # N.B. even though the test code above prints out gzip compressed sizes, regression testing is done against uncompressed sizes
-      # this is because optimizing for compressed sizes can be unpredictable and sometimes counterproductive
-      total_output_size += size
-      total_expected_size += expected_size
-
-      total_output_size_gz += size_gz
-      total_expected_size_gz += expected_size_gz
-
-    if len(outputs) > 1:
-      obtained_results['total'] = total_output_size
-      obtained_results['total_gz'] = total_output_size_gz
-
-    print_diff('Total output size', total_output_size, total_expected_size)
-    print_diff('Total output size gzipped', total_output_size_gz, total_expected_size_gz)
-
-    obtained_results.update(metadata)
-
-    if common.EMTEST_REBASELINE:
-      create_file(results_file, json.dumps(obtained_results, indent=2) + '\n', absolute=True)
-    else:
-      if total_output_size > total_expected_size:
-        print(f'Oops, overall generated code size regressed by {total_output_size - total_expected_size} bytes!')
-        print('If this is expected, rerun the test with --rebaseline to update the expected sizes')
-      if total_output_size < total_expected_size:
-        print(f'Hey amazing, overall generated code size was improved by {total_expected_size - total_output_size} bytes!')
-        print('If this is expected, rerun the test with --rebaseline to update the expected sizes')
-      self.assertDictEqual(obtained_results, expected_results)
-
   @crossplatform
   def test_unoptimized_code_size(self):
     # We don't care too about unoptimized code size but we would like to keep it
@@ -11962,6 +11892,76 @@ def test_minimal_runtime_code_size(self, test_name, wasm2js, compare_js_output=F
 
     self.check_output_sizes(*outputs)
 
+  def check_output_sizes(self, *outputs: str, **metadata):
+    test_name = self.id().split('.')[-1]
+    results_file = test_file('code_size', test_name + '.json')
+
+    expected_results: dict = {}
+    try:
+      expected_results = json.loads(read_file(results_file))
+    except Exception:
+      if not common.EMTEST_REBASELINE:
+        raise
+
+    obtained_results = {}
+
+    total_output_size = 0
+    total_expected_size = 0
+    total_output_size_gz = 0
+    total_expected_size_gz = 0
+    for f in outputs:
+      f_gz = f + '.gz'
+      expected_size = expected_results.get(f, inf)
+      expected_size_gz = expected_results.get(f_gz, inf)
+      contents = read_binary(f)
+      size = len(contents)
+      size_gz = len(gzip.compress(contents))
+
+      obtained_results[f] = size
+      obtained_results[f_gz] = size_gz
+
+      if not common.EMTEST_REBASELINE and size != expected_size and (f.endswith(('.js', '.html'))):
+        print('Contents of ' + f + ': ')
+        print(contents.decode('utf-8', errors='replace'))
+
+      def print_diff(title, actual, expected):
+        diff = actual - expected
+        s = f'{title}={actual}, expected {expected}'
+        if diff > 0:
+          s += f', delta={diff} ({diff * 100.0 / expected:+.2f}%)'
+        print(s)
+
+      print_diff(f'size of {f}', size, expected_size)
+      print_diff(f'size of {f_gz}', size_gz, expected_size_gz)
+
+      # N.B. even though the test code above prints out gzip compressed sizes, regression testing is done against uncompressed sizes
+      # this is because optimizing for compressed sizes can be unpredictable and sometimes counterproductive
+      total_output_size += size
+      total_expected_size += expected_size
+
+      total_output_size_gz += size_gz
+      total_expected_size_gz += expected_size_gz
+
+    if len(outputs) > 1:
+      obtained_results['total'] = total_output_size
+      obtained_results['total_gz'] = total_output_size_gz
+
+    print_diff('Total output size', total_output_size, total_expected_size)
+    print_diff('Total output size gzipped', total_output_size_gz, total_expected_size_gz)
+
+    obtained_results.update(metadata)
+
+    if common.EMTEST_REBASELINE:
+      create_file(results_file, json.dumps(obtained_results, indent=2) + '\n', absolute=True)
+    else:
+      if total_output_size > total_expected_size:
+        print(f'Oops, overall generated code size regressed by {total_output_size - total_expected_size} bytes!')
+        print('If this is expected, rerun the test with --rebaseline to update the expected sizes')
+      if total_output_size < total_expected_size:
+        print(f'Hey amazing, overall generated code size was improved by {total_expected_size - total_output_size} bytes!')
+        print('If this is expected, rerun the test with --rebaseline to update the expected sizes')
+      self.assertDictEqual(obtained_results, expected_results)
+
   # Tests the library_c_preprocessor.js functionality.
   @crossplatform
   def test_c_preprocessor(self):