Skip to content

Commit 960b0ea

Browse files
authored
tests: remove support for legacy test suite names. NFC (#17498)
Continuing to magically support them only seems to prevent my fingers from learning the new names.
1 parent fbc5327 commit 960b0ea

File tree

1 file changed

+4
-9
lines changed

1 file changed

+4
-9
lines changed

tests/runner.py

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -238,21 +238,16 @@ def show():
238238
atexit.register(show)
239239

240240

241-
def replace_legacy_suite_names(args):
242-
newargs = []
243-
241+
def error_on_legacy_suite_names(args):
244242
for a in args:
245243
if a.startswith('wasm') and not any(a.startswith(p) for p in ('wasm2js', 'wasmfs', 'wasm64')):
246-
print('warning: test suites in test_core.py have been renamed from `wasm` to `core`. Please use the new names')
247-
a = a.replace('wasm', 'core', 1)
248-
newargs.append(a)
249-
250-
return newargs
244+
new = a.replace('wasm', 'core', 1)
245+
utils.exit_with_error('`%s` test suite has been replaced with `%s`', a, new)
251246

252247

253248
def load_test_suites(args, modules):
254249
loader = unittest.TestLoader()
255-
args = replace_legacy_suite_names(args)
250+
error_on_legacy_suite_names(args)
256251
unmatched_test_names = set(args)
257252
suites = []
258253
for m in modules:

0 commit comments

Comments
 (0)