-
-
Notifications
You must be signed in to change notification settings - Fork 688
Open
Description
As a follow-up to #32973 and #32981, there are a few other ways we could improve the --warn-long flag to keep our test suite speedy:
- Add a flag like
-Werrorto the test runner, so that warnings are turned into errors that are reflected in the exit code. This will allow the CI to catch them. Another idea would be to have an--error-longflag that triggers errors instead of warnings at an ostensibly higher limit. - Summarize the
--warn-longwarnings at the end of the test run, so that they're all visible to the user without having to dig through a log file. - Add a separate option for warning about
# long timetests. Currently when running with--long, you have to use a very long warning time, which then essentially hides the warnings for non-long tests that take too long.
CC: @tornaria
Component: doctest framework
Issue created by migration from https://trac.sagemath.org/ticket/32995