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

Show paths custom #7

Merged
merged 5 commits into from
Feb 27, 2024
Merged

Conversation

mytestopia
Copy link
Contributor

  • -r gitlab --gitlab-show-paths -> print paths for all scenarios (skipped / passed)
➜  docker-compose exec  e2e /venv/bin/python3 bootstrap.py -r gitlab --gitlab-show-paths
Scenarios
* get options
 ✗ try to get options as guest (0.27s)
   |> scenarios/get_options/try_to_get_options_as_guest.py
   ✔ given_guest (0.00s)
   ...
   ✗ and_it_should_return_failure_body (0.00s)
...
AssertionError: assert False

 ✔ try to get options with empty feature (0.18s)
   |> scenarios/get_options/try_to_get_options_with_empty_feature.py
  • -r gitlab --gitlab-show-paths failed -> print paths for specified scenarios (failed in example)
➜  docker-compose exec  e2e /venv/bin/python3 bootstrap.py -r gitlab --gitlab-show-paths failed
Scenarios
* get options
 ✗ try to get options as guest (0.29s)
   |> scenarios/get_options/try_to_get_options_as_guest.py
   ✔ given_guest (0.00s)
    ...
   ✗ and_it_should_return_failure_body (0.00s)
...
AssertionError: assert False

 ✔ try to get options with empty feature (0.21s)
  • -r gitlab -> print no paths
➜  docker-compose exec  e2e /venv/bin/python3 bootstrap.py -r gitlab 
Scenarios
* get options
 ✗ try to get options as guest (0.25s)
   ✔ given_guest (0.00s)
   ...
   ✗ and_it_should_return_failure_body (0.00s)
AssertionError: assert False

 ✔ try to get options with empty feature (0.14s)

setup.cfg Show resolved Hide resolved
vedro_gitlab_reporter/_gitlab_reporter.py Outdated Show resolved Hide resolved
vedro_gitlab_reporter/_gitlab_reporter.py Show resolved Hide resolved
@tsv1
Copy link
Member

tsv1 commented Feb 12, 2024

It appears that the linter has encountered issues within the pipeline. Could we take a closer look to identify and address the specific causes?

@mytestopia mytestopia requested a review from tsv1 February 27, 2024 08:11
Copy link

codecov bot commented Feb 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.00%. Comparing base (04a4696) to head (69a14cc).

Additional details and impacted files
@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
+ Coverage   94.35%   95.00%   +0.64%     
==========================================
  Files           3        3              
  Lines         177      180       +3     
  Branches       23       26       +3     
==========================================
+ Hits          167      171       +4     
  Misses          4        4              
+ Partials        6        5       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@tsv1 tsv1 left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution!

@tsv1 tsv1 merged commit 647db7a into vedro-universe:master Feb 27, 2024
7 checks passed
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.

2 participants