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

Fix gherkin terminal report colouring #372

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Fix gherkin terminal report colouring #372

wants to merge 7 commits into from

Commits on Jun 18, 2023

  1. Add skipped parameter to steps 

    - the skipped parameter can be used to track which steps have been skipped and
adjust the behaviour appropriatelly (ex. change the output colour to yellow)
    @dsax7 @hristiy4n
    dsax7 authored and hristiy4n committed Jun 18, 2023
    Configuration menu
    Copy the full SHA
    8897a41 View commit details
    Browse the repository at this point in the history

  2. Add hook and plugin to modify steps when skipped 

    - the hook and the pluging work in a similar way to the fail hook and plugin
where they set a flag wether the step is skipped or not
    @dsax7 @hristiy4n
    dsax7 authored and hristiy4n committed Jun 18, 2023
    Configuration menu
    Copy the full SHA
    a2e48b1 View commit details
    Browse the repository at this point in the history

  3. Use the skip hook and plugin if the step is skipped 

    - trigger the skip hook if a step is skipped
    @dsax7 @hristiy4n
    dsax7 authored and hristiy4n committed Jun 18, 2023
    Configuration menu
    Copy the full SHA
    0a8a4e3 View commit details
    Browse the repository at this point in the history

  4. Color the steps independently of the scenario 

    - use the output of the individual steps to decide the output colour
    @dsax7 @hristiy4n
    dsax7 authored and hristiy4n committed Jun 18, 2023
    Configuration menu
    Copy the full SHA
    bf2057a View commit details
    Browse the repository at this point in the history

  5. Modify report test 

    - the PR proposes to expand the step with an additional skipped bool, to track weather the
step has been skipped or not, therefor the expected behaviour of the steps in the tests should
be modified
    @dsax7 @hristiy4n
    dsax7 authored and hristiy4n committed Jun 18, 2023
    Configuration menu
    Copy the full SHA
    9529e46 View commit details
    Browse the repository at this point in the history

  6. Extend test_report with a skipping scenario 

    - when a step is skipped the step should have skipped set to True
    @dsax7 @hristiy4n
    dsax7 authored and hristiy4n committed Jun 18, 2023
    Configuration menu
    Copy the full SHA
    6eca0a3 View commit details
    Browse the repository at this point in the history

  7. [pre-commit.ci] auto fixes from pre-commit.com hooks 

    for more information, see https://pre-commit.ci
    @pre-commit-ci
    pre-commit-ci[bot] committed Jun 18, 2023
    Configuration menu
    Copy the full SHA
    4533677 View commit details
    Browse the repository at this point in the history