Skip to content
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

Rewrite reportportal test results reporting #3223

Merged
merged 3 commits into from
Oct 1, 2024
Merged

Rewrite reportportal test results reporting #3223

merged 3 commits into from
Oct 1, 2024

Conversation

kkaarreell
Copy link
Collaborator

@kkaarreell kkaarreell commented Sep 18, 2024

Modifies test results reporting in the following way:

  • All results from execute.results() are reported.
  • All tests from discover.tests() that have not been executed are reported as 'skipped' at the end.

Pull Request Checklist

  • implement the feature
  • include a release note

@kkaarreell kkaarreell force-pushed the ks_rp_rewrite branch 3 times, most recently from b666d13 to d7d6ac6 Compare September 18, 2024 09:59
@kkaarreell kkaarreell changed the title Test results reporting rewrite reportportal test results reporting rewrite Sep 18, 2024
@psss psss added priority | should medium priority, should be included in the next release step | report Stuff related to the report step plugin | reportportal The reportportal report plugin labels Sep 19, 2024
@psss psss added this to the 1.37 milestone Sep 19, 2024
@psss psss changed the title reportportal test results reporting rewrite Rewrite reportportal test results reporting Sep 19, 2024
@kkaarreell
Copy link
Collaborator Author

@psss psss added the ci | full test Pull request is ready for the full test execution label Oct 1, 2024
Copy link
Collaborator

@psss psss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for solving this, @kkaarreell. Looks good to me. Added a short release note in 469c1bb.

@psss psss self-assigned this Oct 1, 2024
happz and others added 3 commits October 1, 2024 22:12
Modifies test results reporting in the following way:
 - All results from execute.results() are reported.
 - All tests from discover.tests() that have not been
   executed are reported as 'skipped' at the end.
Plus a minor style adjustment.
@psss
Copy link
Collaborator

psss commented Oct 1, 2024

Full test passed, failures seem to be irrelevant, merging.

@psss psss merged commit dab0f36 into main Oct 1, 2024
17 of 23 checks passed
@psss psss deleted the ks_rp_rewrite branch October 1, 2024 22:10
The-Mule pushed a commit to The-Mule/tmt that referenced this pull request Oct 14, 2024
Modifies test results reporting in the following way:
 - All results from execute.results() are reported.
 - All tests from discover.tests() that have not been
   executed are reported as 'skipped' at the end.

Co-authored-by: Miloš Prchlík <mprchlik@redhat.com>
Co-authored-by: Petr Šplíchal <psplicha@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci | full test Pull request is ready for the full test execution plugin | reportportal The reportportal report plugin priority | should medium priority, should be included in the next release step | report Stuff related to the report step
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants