refactor: Isolate logic that distributes Components output after run

[silvanocerza]

Related Issues

• Part of Optimize Pipeline.run() #7614

Proposed Changes:

Isolate logic that distributes Components output after run.

This also slightly changes the _run_component() method to resetting variadics as it was done in the wrong place.

How did you test it?

I added tests and ran them locally.

Notes for the reviewer

N/A

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 9468686654

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 43 unchanged lines in 1 file lost coverage.
• Overall coverage increased (+0.1%) to 89.927%

---

Files with Coverage Reduction	New Missed Lines	%
core/pipeline/pipeline.py	43	72.13%

Totals
Change from base Build 9467243080:	0.1%
Covered Lines:	6874
Relevant Lines:	7644

---

💛 - Coveralls

[masci]

Can we move those methods to the base class?

[coveralls]

Pull Request Test Coverage Report for Build 9485718095

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 38 unchanged lines in 2 files lost coverage.
• Overall coverage increased (+0.08%) to 89.892%

---

Files with Coverage Reduction	New Missed Lines	%
core/pipeline/base.py	6	92.76%
core/pipeline/pipeline.py	32	68.31%

Totals
Change from base Build 9467243080:	0.08%
Covered Lines:	6928
Relevant Lines:	7707

---

💛 - Coveralls

[masci]

LGTM

[coveralls]

Pull Request Test Coverage Report for Build 9501066047

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 39 unchanged lines in 2 files lost coverage.
• Overall coverage decreased (-0.08%) to 89.738%

---

Files with Coverage Reduction	New Missed Lines	%
core/pipeline/base.py	7	92.5%
core/pipeline/pipeline.py	32	68.31%

Totals
Change from base Build 9467243080:	-0.08%
Covered Lines:	6917
Relevant Lines:	7708

---

💛 - Coveralls

[coveralls]

Pull Request Test Coverage Report for Build 9514060310

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 38 unchanged lines in 2 files lost coverage.
• Overall coverage decreased (-0.07%) to 89.75%

---

Files with Coverage Reduction	New Missed Lines	%
core/pipeline/base.py	6	92.76%
core/pipeline/pipeline.py	32	68.31%

Totals
Change from base Build 9467243080:	-0.07%
Covered Lines:	6917
Relevant Lines:	7707

---

💛 - Coveralls

[coveralls]

Pull Request Test Coverage Report for Build 9514365962

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 39 unchanged lines in 2 files lost coverage.
• Overall coverage decreased (-0.08%) to 89.738%

---

Files with Coverage Reduction	New Missed Lines	%
core/pipeline/base.py	7	92.5%
core/pipeline/pipeline.py	32	68.31%

Totals
Change from base Build 9467243080:	-0.08%
Covered Lines:	6917
Relevant Lines:	7708

---

💛 - Coveralls