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

Enhancement: Extract TestIdentifier #356

Merged
merged 1 commit into from
Nov 2, 2023
Merged

Conversation

localheinz
Copy link
Member

This pull request

  • extracts a TestIdentifier

Related to #291.

Copy link

codecov bot commented Nov 2, 2023

Codecov Report

Merging #356 (c80f5ac) into main (010eb3b) will increase coverage by 1.15%.
Report is 4 commits behind head on main.
The diff coverage is 100.00%.

❗ Current head c80f5ac differs from pull request most recent head c920cb8. Consider uploading reports for the commit c920cb8 to get more accurate results

@@             Coverage Diff              @@
##               main     #356      +/-   ##
============================================
+ Coverage     95.74%   96.89%   +1.15%     
- Complexity       85       90       +5     
============================================
  Files            19       21       +2     
  Lines           376      387      +11     
============================================
+ Hits            360      375      +15     
+ Misses           16       12       -4     
Files Coverage Δ
src/Collector/DefaultCollector.php 54.54% <100.00%> (ø)
src/Exception/InvalidTestIdentifier.php 100.00% <100.00%> (ø)
src/Reporter/DefaultReporter.php 95.83% <100.00%> (ø)
src/SlowTest.php 100.00% <100.00%> (ø)
src/Subscriber/TestPassedSubscriber.php 97.56% <100.00%> (+0.06%) ⬆️
src/Subscriber/TestPreparedSubscriber.php 100.00% <100.00%> (ø)
src/TestIdentifier.php 100.00% <100.00%> (ø)
src/TimeKeeper.php 100.00% <100.00%> (+30.76%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@localheinz localheinz force-pushed the feature/test-identifier branch 2 times, most recently from 7087271 to c80f5ac Compare November 2, 2023 12:34
@localheinz localheinz merged commit ce1633e into main Nov 2, 2023
16 checks passed
@localheinz localheinz deleted the feature/test-identifier branch November 2, 2023 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants