pytest-dev / pytest-bdd

BDD library for the pytest runner
https://pytest-bdd.readthedocs.io/en/latest/
MIT License
1.32k stars 221 forks source link

Modify gherkin terminal reporter to print tags #477

Open guillego opened 2 years ago

guillego commented 2 years ago

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

codecov[bot] commented 2 years ago

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.

guillego commented 2 years ago

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

youtux commented 2 years ago

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