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

Add --exclude-testsuite option #4657

Closed
wants to merge 3 commits into from

Conversation

alecsmrekar
Copy link
Contributor

Covers issues #4588
I add the logic for a new parameter, "--exclude-testsuite"

@codecov
Copy link

codecov bot commented Apr 22, 2021

Codecov Report

Merging #4657 (cd907ff) into master (1b36757) will decrease coverage by 0.04%.
The diff coverage is 63.63%.

❗ Current head cd907ff differs from pull request most recent head e982dc0. Consider uploading reports for the commit e982dc0 to get more accurate results
Impacted file tree graph

@@             Coverage Diff              @@
##             master    #4657      +/-   ##
============================================
- Coverage     83.34%   83.30%   -0.05%     
- Complexity     4527     4535       +8     
============================================
  Files           292      292              
  Lines         11930    11947      +17     
============================================
+ Hits           9943     9952       +9     
- Misses         1987     1995       +8     
Impacted Files Coverage Δ Complexity Δ
src/TextUI/CliArguments/Builder.php 79.44% <50.00%> (-0.28%) 121.00 <0.00> (+1.00) ⬇️
src/TextUI/CliArguments/Configuration.php 67.35% <50.00%> (-0.28%) 272.00 <4.00> (+3.00) ⬇️
src/TextUI/CliArguments/Mapper.php 77.97% <50.00%> (-0.34%) 83.00 <0.00> (+1.00) ⬇️
src/TextUI/TestSuiteMapper.php 80.00% <83.33%> (-1.09%) 19.00 <19.00> (+3.00) ⬇️
src/TextUI/Command.php 75.21% <100.00%> (+0.07%) 123.00 <0.00> (ø)

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 1b36757...e982dc0. Read the comment docs.

@sebastianbergmann sebastianbergmann added feature/test-runner CLI test runner type/enhancement A new idea that should be implemented labels Apr 22, 2021
@sebastianbergmann sebastianbergmann added this to the PHPUnit 10.0 milestone Apr 22, 2021
@alecsmrekar
Copy link
Contributor Author

I'm a bit of out my depth about why this specific PHP 8.1 test is failing, any clues?

@sebastianbergmann sebastianbergmann changed the title Issue #4588: Adding logic for skipping a certain testsuite Add --exclude-testsuite option May 19, 2021
@sebastianbergmann
Copy link
Owner

Merged manually, thanks.

sebastianbergmann added a commit that referenced this pull request Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/test-runner CLI test runner type/enhancement A new idea that should be implemented
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants