pytest-bdd icon indicating copy to clipboard operation
pytest-bdd copied to clipboard

Modify gherkin terminal reporter to print tags

Open guillego opened this issue 3 years ago • 3 comments
trafficstars

As mentioned in #476, this PR implements a simple parser for printing the tags in the verbose gherkin terminal reporter.

guillego avatar Jan 04 '22 23:01 guillego

Codecov Report

Merging #477 (ef7681b) into master (2fb5a11) will increase coverage by 0.44%. The diff coverage is 100.00%.

:exclamation: Current head ef7681b differs from pull request most recent head 463bf19. Consider uploading reports for the commit 463bf19 to get more accurate results Impacted file tree graph

@@            Coverage Diff             @@
##           master     #477      +/-   ##
==========================================
+ Coverage   95.48%   95.93%   +0.44%     
==========================================
  Files          48       49       +1     
  Lines        1573     1648      +75     
  Branches      171      179       +8     
==========================================
+ Hits         1502     1581      +79     
+ Misses         44       41       -3     
+ Partials       27       26       -1     
Impacted Files Coverage Δ
pytest_bdd/gherkin_terminal_reporter.py 82.85% <100.00%> (+2.21%) :arrow_up:
pytest_bdd/types.py 100.00% <0.00%> (ø)
tests/feature/test_report.py 75.67% <0.00%> (ø)
tests/feature/test_outline.py 100.00% <0.00%> (ø)
tests/feature/test_outline_empty_values.py 100.00% <0.00%> (ø)
tests/args/test_arg_fixture_mix.py 100.00% <0.00%> (ø)
pytest_bdd/scenario.py 91.66% <0.00%> (+0.59%) :arrow_up:
pytest_bdd/parser.py 99.54% <0.00%> (+1.56%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 2fb5a11...463bf19. Read the comment docs.

codecov[bot] avatar Jan 04 '22 23:01 codecov[bot]

@olegpidsadnyi This is my first time contributing, can you guide me through the PR process? Is there anything else I can do?

guillego avatar Jan 17 '22 14:01 guillego

Hi @guillego, this looks like something we would like to merge, but it would needs some tests to be added to the PR.

youtux avatar Jul 24 '22 18:07 youtux