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

test: Add utility then step in feature testing to draw pipeline to file #8209

Merged
merged 1 commit into from
Aug 13, 2024

Conversation

silvanocerza
Copy link
Contributor

Proposed Changes:

Small utility that ease visualization of a Pipeline graph.

To use it we just need to add a new Then step in a scenario like so:

    Scenario Outline: Running a correct Pipeline
        Given a pipeline <kind>
        When I run the Pipeline
        Then draw it to file
        Then it should return the expected result
        And components ran in the expected order

The images will be saved in a test_pipeline_graphs folder in the project's root.
This just an utility step meant to be used locally to ease some debugging.

How did you test it?

I added a Then step to a scenario and run it.

Notes for the reviewer

N/A

Checklist

@silvanocerza silvanocerza added the ignore-for-release-notes PRs with this flag won't be included in the release notes. label Aug 13, 2024
@silvanocerza silvanocerza self-assigned this Aug 13, 2024
@silvanocerza silvanocerza requested a review from a team as a code owner August 13, 2024 09:22
@silvanocerza silvanocerza requested review from julian-risch and removed request for a team August 13, 2024 09:22
@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 10367114382

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 90.113%

Totals Coverage Status
Change from base Build 10351241196: 0.0%
Covered Lines: 6954
Relevant Lines: 7717

💛 - Coveralls

@shadeMe
Copy link
Collaborator

shadeMe commented Aug 13, 2024

Test failure looks unrelated

@silvanocerza
Copy link
Contributor Author

Merging, tests are failing for unrelated reasons.

@silvanocerza silvanocerza merged commit ab7eb25 into main Aug 13, 2024
14 of 15 checks passed
@silvanocerza silvanocerza deleted the draw-step branch August 13, 2024 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ignore-for-release-notes PRs with this flag won't be included in the release notes. topic:tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants