Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
simpletest.module
Priority:
Major
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
6 Oct 2015 at 11:46 UTC
Updated:
20 Oct 2015 at 21:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
dawehnerThere we go.
Comment #3
lewisnymanI assume this includes frontend regression testing? +1
Comment #4
webflo commentedSubmodule? Delete the .git folder in vendor/hmlb/phpunit-vw and create a new patch.
Comment #5
jelle_s#2: I assume the patch will be green, so we can go straight to RTBC!
Comment #6
dawehnerAh that explained the odd behaviour!
Comment #7
wim leersSo much win. We would have shipped years ago, with 40% better performance, if we had had this sooner.
Ship it.
Comment #8
yched commentedMh - that means bringing https://www.drupal.org/project/bad_judgement in core, right ?
Comment #9
yched commented"Any similarities with a current event concerning (but not limited to) a multinational automobile manufacturer are purely coincidental"
If that brings us closer to releasing the drupalmobile, I say do it.
Comment #10
gábor hojtsyThis should lead to lower emissions, because all those developers don't need to use that power to keep their laptops on to fix the bugs. Less coal to burn, better planet!
Comment #11
dawehnerThis also ensures that on production we run with full fuels and just on testing ensure we waste less energy.
Comment #12
legolasboAs a Bad Judgement co-maintainer I hereby pledge to add this testing feature to the Bad Judgement module if the core maintainers make the good judgement to not add this to core.
Comment #13
MixologicThis would save us some serious $$ in testbot costs. +1
Comment #14
catchCommitted/pushed to 8.0.x, thanks!
Comment #15
catchMoving to 7.x for backport.
This is already fixed in 6.x because it doesn't have a test suite.
Comment #16
almaudoh commentedDo we need a change record? ;)
Comment #17
skwashd commented@catch this only works for phpunit so no backport for D7 is needed. I can't see the commit in the repo, did you forget to push it?
Comment #18
gábor hojtsy@skwashd the push only worked in the testing environment, not on prod, that is by design. :)
@almaudoh I don't think we need a change record, maybe in a decade or so... if they figure it out! ssssh!