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

feat: junit report named like config file #264

Merged
merged 1 commit into from
Apr 6, 2023
Merged

Conversation

Pog3k
Copy link
Contributor

@Pog3k Pog3k commented Mar 29, 2023

junit report includes now config name. Improved traceability

@codecov
Copy link

codecov bot commented Mar 29, 2023

Codecov Report

Merging #264 (b6ffe04) into master (440923d) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #264      +/-   ##
==========================================
- Coverage   96.91%   96.91%   -0.01%     
==========================================
  Files          77       77              
  Lines        6193     6189       -4     
==========================================
- Hits         6002     5998       -4     
  Misses        191      191              
Impacted Files Coverage Δ
src/pykiso/cli.py 100.00% <100.00%> (ø)
src/pykiso/test_coordinator/test_execution.py 97.64% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@@ -394,11 +397,21 @@ def test_config_registry_and_test_execution_with_junit_reporting(tmp_test, capsy
Validation criteria:
- run is executed without error
"""

class AnyPathWith(str):
"""Check if part string is in path object"""
Copy link
Contributor

Choose a reason for hiding this comment

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

I do not understand the description.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

maybe clearer now ?

@Pog3k Pog3k force-pushed the feature/name_junit_report branch 2 times, most recently from d3d92b6 to d22cd1c Compare April 3, 2023 15:04
@sebclrsn
Copy link
Contributor

sebclrsn commented Apr 6, 2023

Please extend the what's new and we're good to go

@Pog3k Pog3k force-pushed the feature/name_junit_report branch from d22cd1c to b6ffe04 Compare April 6, 2023 11:26
@Pog3k Pog3k merged commit 81c8dfb into master Apr 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants