From f53a7aced35d5411d061e043196657baf80bd980 Mon Sep 17 00:00:00 2001 From: Sam Clegg Date: Mon, 16 Dec 2024 15:35:03 -0800 Subject: [PATCH] [test] Remove also_with_wasmfs_js. NFC (#23185) The only different here with `also_with_wasmfs` was that this decorator adds `FORCE_FILESYSTEM`, but the only test to use this decorator was already adding this flag. --- test/test_core.py | 19 +------------------ 1 file changed, 1 insertion(+), 18 deletions(-) diff --git a/test/test_core.py b/test/test_core.py index 6685b5a89918a..930bbda052810 100644 --- a/test/test_core.py +++ b/test/test_core.py @@ -157,23 +157,6 @@ def decorated(f): return decorated -# Similar to also_with_wasmfs, but also enables the full JS API -def also_with_wasmfs_js(func): - assert callable(func) - - @wraps(func) - def decorated(self): - func(self) - print('wasmfs') - if self.get_setting('STANDALONE_WASM'): - self.skipTest("test currently cannot run both with WASMFS and STANDALONE_WASM") - self.set_setting('WASMFS') - self.set_setting('FORCE_FILESYSTEM') - self.emcc_args = self.emcc_args.copy() + ['-DWASMFS'] - func(self) - return decorated - - def with_asyncify_and_jspi(f): assert callable(f) @@ -5757,7 +5740,7 @@ def test_fs_trackingdelegate(self): self.do_run_in_out_file_test('fs/test_trackingdelegate.c') @also_with_noderawfs - @also_with_wasmfs_js + @also_with_wasmfs def test_fs_writeFile(self): if self.get_setting('WASMFS'): self.set_setting("FORCE_FILESYSTEM")