Fix warnings in tsgen tests. NFC

Also don't report clearing of out/test between each test, which makes
the test runner less chatty by default.

Author: sbc100

test/common.py

@@ -894,14 +894,14 @@ def setUp(self):
         if EMTEST_SAVE_DIR == 2:
           print('Not clearing existing test directory')
         else:
-          print('Clearing existing test directory')
+          logger.debug('Clearing existing test directory: %s', self.working_dir)
           # Even when --save-dir is used we still try to start with an empty directory as many tests
           # expect this.  --no-clean can be used to keep the old contents for the new test
           # run. This can be useful when iterating on a given test with extra files you want to keep
           # around in the output directory.
           force_delete_contents(self.working_dir)
       else:
-        print('Creating new test output directory')
+        logger.debug('Creating new test output directory: %s', self.working_dir)
         ensure_dir(self.working_dir)
       utils.write_file(LAST_TEST, self.id() + '\n')
     os.chdir(self.working_dir)

test/test_other.py

@@ -3023,25 +3023,26 @@ def test_embind_tsgen_ignore(self):
     create_file('fail.js', 'assert(false);')
     # These extra arguments are not related to TS binding generation but we want to
     # verify that they do not interfere with it.
-    extra_args = ['-sALLOW_MEMORY_GROWTH=1',
-                  '-sMAXIMUM_MEMORY=4GB',
-                  '--pre-js', 'fail.js',
-                  '--post-js', 'fail.js',
-                  '--extern-pre-js', 'fail.js',
-                  '--extern-post-js', 'fail.js',
-                  '-sENVIRONMENT=worker',
-                  '--use-preload-cache',
-                  '--preload-file', 'fail.js',
-                  '-O3',
-                  '-msimd128',
-                  '-sUSE_PTHREADS',
-                  '-sPROXY_TO_PTHREAD',
-                  '-sPTHREAD_POOL_SIZE=1',
-                  '-sSINGLE_FILE',
-                  '-lembind', # Test duplicated link option.
-                  ]
+    self.emcc_args += ['-sALLOW_MEMORY_GROWTH=1',
+                       '-Wno-pthreads-mem-growth',
+                       '-sMAXIMUM_MEMORY=4GB',
+                       '--pre-js', 'fail.js',
+                       '--post-js', 'fail.js',
+                       '--extern-pre-js', 'fail.js',
+                       '--extern-post-js', 'fail.js',
+                       '-sENVIRONMENT=worker',
+                       '--use-preload-cache',
+                       '--preload-file', 'fail.js',
+                       '-O3',
+                       '-msimd128',
+                       '-pthread',
+                       '-sPROXY_TO_PTHREAD',
+                       '-sPTHREAD_POOL_SIZE=1',
+                       '-sSINGLE_FILE',
+                       '-lembind', # Test duplicated link option.
+                      ]
     self.run_process([EMCC, test_file('other/embind_tsgen.cpp'),
-                      '-lembind', '--embind-emit-tsd', 'embind_tsgen.d.ts'] + extra_args)
+                      '-lembind', '--embind-emit-tsd', 'embind_tsgen.d.ts'] + self.get_emcc_args())
     self.assertFileContents(test_file('other/embind_tsgen.d.ts'), read_file('embind_tsgen.d.ts'))
     # Test these args separately since they conflict with arguments in the first test.
     extra_args = ['-sMODULARIZE',
@@ -3055,7 +3056,11 @@ def test_embind_tsgen_ignore(self):
   def test_embind_tsgen_test_embind(self):
     self.run_process([EMCC, test_file('embind/embind_test.cpp'),
                       '-lembind', '--embind-emit-tsd', 'embind_tsgen_test_embind.d.ts',
-                      '-DSKIP_UNBOUND_TYPES']) # TypeScript generation requires all type to be bound.
+                      # This test explicitly creates std::string from unsigned char pointers
+                      # which is deprecated in upstream LLVM.
+                      '-Wno-deprecated-declarations',
+                      # TypeScript generation requires all type to be bound.
+                      '-DSKIP_UNBOUND_TYPES'] + self.get_emcc_args())
     self.assertExists('embind_tsgen_test_embind.d.ts')
 
   def test_embind_tsgen_val(self):
@@ -3076,7 +3081,8 @@ def test_embind_tsgen_bigint(self):
   def test_embind_tsgen_memory64(self):
     # Check that when memory64 is enabled longs & unsigned longs are mapped to bigint in the generated TS bindings
     self.run_process([EMCC, test_file('other/embind_tsgen_memory64.cpp'),
-                      '-lembind', '--embind-emit-tsd', 'embind_tsgen_memory64.d.ts', '-sMEMORY64'])
+                      '-lembind', '--embind-emit-tsd', 'embind_tsgen_memory64.d.ts', '-sMEMORY64', '-Wno-experimental'] +
+                     self.get_emcc_args())
     self.assertFileContents(test_file('other/embind_tsgen_memory64.d.ts'), read_file('embind_tsgen_memory64.d.ts'))
 
   def test_emconfig(self):