-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Open
Labels
Description
testament silently ignores top-level tests, eg tests/trunner.nim was silently ignored (which I'm working around in #14377 by moving it under tests/misc/trunner.nim)
this should be considered a bug, as echoed by @alaviss here #14377 (comment)
Sounds like a bug to me. We should fix this given that testament is now a part of the official Nim distribution.
Example
tests/trunner.nim
Current Output
silently ignored
Expected Output
- (not preferred but better) give an error
- (preferred) run the test, under a special category called "toplevel" (so that
testament pcat toplevelworks); and give an error if a toplevel dir also exists to avoid clashes
Additional Information
- 40ac195 1.3.5
test_nimscript.nimsis already being tested separately in koch.nim; IIRC onlynimfiles are being tested in testament, so nothing would change for this file; note that we can't make testament test nims files because of config files liketests/newconfig/tfoo.nimswhich is attached totests/newconfig/tfoo.niminstead of being a standalone test