-
Notifications
You must be signed in to change notification settings - Fork 11
Adjust frontend to display skipped tests #597
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.
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
Conversation
82730b2 to
a1494c0
Compare
|
There are things that could be improved if this is necessary:
|
f4bec53 to
5d87292
Compare
yes, we need them to be displayed according to the order of their execution |
javihernandez
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aside that minor comment, LGTM
8b10608 to
a02670f
Compare
anfimovdm
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
during the testing I found a few issues that should be fixed:
- test logs from builds (that built before your changes) can't be opened
- errors during compilation

Fixes: AlmaLinux/build-system#485
AlmaLinux/alts#163
AlmaLinux/albs-web-server#1171
If we want to skip a test - we skip the execution of a test function in
alts. If this happens - artifacts are not created onaltsside. In order to address this, we pass names of skipped tests fromalts, and manually display skipped test (name, status, log), along with executed tests that already contain such data