[test] Parameterize test_function_exports_are_small. NFC

Author: sbc100

test/test_other.py

@@ -10703,10 +10703,19 @@ def test_emscripten_license(self, expect_license, args):
   # unminified exported name exactly once.  (need to contain the export name once for unminified
   # access from calling code, and should not have the unminified name exist more than once, that
   # would be wasteful for size)
-  def test_function_exports_are_small(self):
+  @parameterized({
+    '': ([],),
+    'closure': (['--closure=1'],),
+  })
+  @parameterized({
+    'O2': (['-O2'],),
+    'O2': (['-O2'],),
+    'Os': (['-Os'],),
+  })
+  def test_function_exports_are_small(self, opt, closure):
     def test(args, closure, opt):
+      print(args)
       extra_args = args + opt + closure
-      print(extra_args)
       args = [EMCC, test_file('long_function_name_in_export.c'), '-o', 'a.html', '-sENVIRONMENT=web', '-sDECLARE_ASM_MODULE_EXPORTS=0', '-Werror'] + extra_args
       self.run_process(args)
 
@@ -10720,10 +10729,8 @@ def test(args, closure, opt):
         num_times_export_is_referenced = output.count('thisIsAFunctionExportedFromAsmJsOrWasmWithVeryLongFunction')
         self.assertEqual(num_times_export_is_referenced, 1)
 
-    for closure in [[], ['--closure=1']]:
-      for opt in [['-O2'], ['-O3'], ['-Os']]:
-        test(['-sWASM=0', '-Wno-closure'], closure, opt)
-        test(['-sWASM_ASYNC_COMPILATION=0'], closure, opt)
+    test(['-sWASM=0', '-Wno-closure'], closure, opt)
+    test(['-sWASM_ASYNC_COMPILATION=0'], closure, opt)
 
   @parameterized({
     'hello_world_wasm': ('hello_world', False, True),