New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make testplan timeout obvious for users in report #1184

Draft

rnemes wants to merge 2 commits into morganstanley:main from rnemes:timeout

+79 −12

Contributor

rnemes commented Feb 20, 2025 •

edited

Loading

Bug / Requirement Description

When testplan level timeout happens, the error message is only visible if we click the 'home' button on UI, which is very well hidden from the users.

Solution description

Add a test level synthesize entry named "Testplan Timeout"
Let's see if test type information is available, and add the test type icon
Fix the '-> Passed' issue in stdout, make it Incomplete

Checklist:

Test
Example (both test_plan.py and .rst)
Documentation (API)
News fragment present for release notes
MS info leakage check
For new driver: driver index page
For new assertion: ui/pdf/std renderers, documentation
For new cmdline arg: documentation

rnemes added 2 commits

February 20, 2025 08:43


          Several major changes around timeout handling

b2ce9a6


          Collecting data immediately

0d86152

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet