- Issue created by @mondrake
- 🇮🇹Italy mondrake 🇮🇹
To review this, you need to look at the raw output of a test of a core CI job.
For the sake of demonstrating the outcome, I broke a test so that its execution log can be dumped to the raw log.
See https://git.drupalcode.org/issue/drupal-3483931/-/jobs/3175261/viewer and search for 'PHPUnit 10' to see the result of the change.
- 🇮🇹Italy mondrake 🇮🇹
Rebased.
Added before and after screenshots - it's not the same test but they give pretty well the sense of the change IMHO.
Before
After
- 🇮🇹Italy mondrake 🇮🇹
Reverted the change made on purpose in #3 to demo the outcome.
- 🇺🇸United States smustgrave
Thanks for providing the before/after screenshots. I can't think of any reason not to use it.
-
longwave →
committed cf0ae4ae on 11.0.x
Issue #3483931 by mondrake: [CI] Use testdox and colors in tests spawned...
-
longwave →
committed cf0ae4ae on 11.0.x
- 🇬🇧United Kingdom longwave UK
This is really nice. Committed and pushed 6e60f0b5009 to 11.x and c22e5e6be8d to 11.1.x and cf0ae4aefa0 to 11.0.x. Thanks!
If there isn't one already can we have a followup to clean up the passes as well?
---- Drupal\Tests\Core\Session\CalculatedPermissionsItemTest ---- Status Group Filename Line Function -------------------------------------------------------------------------------- Pass Other CalculatedPermiss 27 Drupal\Tests\Core\Session\Calculate Pass Other CalculatedPermiss 47 Drupal\Tests\Core\Session\Calculate Pass Other CalculatedPermiss 59 Drupal\Tests\Core\Session\Calculate ---- Drupal\Tests\Core\Session\UserSessionTest ---- Status Group Filename Line Function -------------------------------------------------------------------------------- Pass Other UserSessionTest.p 39 Drupal\Tests\Core\Session\UserSessi Pass Other UserSessionTest.p 60 Drupal\Tests\Core\Session\UserSessi Pass Other UserSessionTest.p 71 Drupal\Tests\Core\Session\UserSessi
This could be so much more readable!
-
longwave →
committed c22e5e6b on 11.1.x
Issue #3483931 by mondrake: [CI] Use testdox and colors in tests spawned...
-
longwave →
committed c22e5e6b on 11.1.x
-
longwave →
committed 6e60f0b5 on 11.x
Issue #3483931 by mondrake: [CI] Use testdox and colors in tests spawned...
-
longwave →
committed 6e60f0b5 on 11.x
- 🇮🇹Italy mondrake 🇮🇹
@longwave re #10 I really believe 🐛 Allow run-tests.sh to report skipped/risky/incomplete PHPUnit-based tests Needs work would be a step change for run-tests.sh output.
I will rebase it now and add before/after screenshot there as well.
Automatically closed - issue fixed for 2 weeks with no activity.