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.

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

Rewrite reportportal test results reporting #3223

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Rewrite reportportal test results reporting #3223

wants to merge 3 commits into from

Conversation

kkaarreell
Copy link
Collaborator

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
  • write the documentation
  • extend the test coverage
  • update the specification
  • adjust plugin docstring
  • modify the json schema
  • mention the version
  • 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
happz and others added 2 commits September 23, 2024 09:28
@happz @kkaarreell
Provide mapping between results and tests for easier reporting
3efb2ba
@kkaarreell
Test results reporting rewrite
c52e735 
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.
@kkaarreell
Copy link
Collaborator Author

Semanticdiff link
https://app.semanticdiff.com/gh/teemtee/tmt/pull/3223/changes#tmt/steps/execute/__init__.py?ignore_comments=false

@kkaarreell
Fix error when guest's primary_address is not set
4ba756e 
Fixes the following error:
    'ResultGuestData' object has no attribute 'primary_address'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@psss psss Awaiting requested review from psss psss is a code owner

@lukaszachy lukaszachy Awaiting requested review from lukaszachy lukaszachy is a code owner

@happz happz Awaiting requested review from happz happz is a code owner

@thrix thrix Awaiting requested review from thrix thrix is a code owner

@janhavlin janhavlin Awaiting requested review from janhavlin janhavlin is a code owner

@4N0body5 4N0body5 Awaiting requested review from 4N0body5

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
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
Milestone
1.37
Development

Successfully merging this pull request may close these issues.
3 participants
@kkaarreell @psss @happz