test: Rework Pipeline.run() tests to ease declaration with dataclasses

[silvanocerza]

Related Issues

• Part of Write better tests for Pipeline.run() #7611

Proposed Changes:

Add a new PipelineRunData dataclass that holds all the information necessary to run a Pipeline and verify its correct execution. This dataclass will be returned by the functions marked with @given together with the Pipeline instance to run.

I also added a _PipelineResult dataclass used in the boilerplate code that runs and verifies the Pipeline.run() execution.

How did you test it?

I ran tests locally.

Notes for the reviewer

I also changes slightly the tests for the "bad" Pipeline runs as they were unnecesssarily returning the expected Exception class too. This made it simpler to structure the PipelineRunData dataclass.

Checklist

• I have read the contributors guidelines and the code of conduct
• I have updated the related issue with new insights and changes
• I added unit tests and updated the docstrings
• I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.
• I documented my code
• I ran pre-commit hooks and fixed any issue

[coveralls]

Pull Request Test Coverage Report for Build 9350724425

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 89.75%

---

Totals
Change from base Build 9348644629:	0.0%
Covered Lines:	6690
Relevant Lines:	7454

---

💛 - Coveralls