Closed (won't fix)
Project:
Drupal core
Version:
8.3.x-dev
Component:
system.module
Priority:
Major
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
4 May 2014 at 20:33 UTC
Updated:
20 Sep 2016 at 11:11 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
sunComment #3
sun1: test.iu_.1.patch queued for re-testing.
Comment #4
sunJust noticed that some additional/duplicated test coverage for "required" modules exists in
Drupal\system\Tests\Module\RequiredTestalready.Comment #5
sun1: test.iu_.1.patch queued for re-testing.
Comment #6
jthorson commented1: test.iu_.1.patch queued for re-testing.
Comment #8
jthorson commented... Make him fly!
Comment #9
sunJust because today is today.
Comment #10
xanoNot yet. ModuleHandler needs some more test coverage.
Comment #11
xanoOh, there is web test coverage. It's not pretty, but it does the job. Sorry for delaying this.
Comment #12
catchI think alexpott mentioned that this was the only test that caught some relatively recent regressions, assigning so he can comment.
Comment #13
sun@alexpott raised concerns about a different issue, #2258247: Remove useless Drupal\config\Tests\ConfigImportAllTest
Comment #14
alexpottWe should prove that we have covered all the regression listed in the issue summary.
The trickiest will be
Comment #15
jhedstromSetting at needs work based on #14.
Comment #16
webchickIn talking to the DrupalCI folks, they say this is a blocker to testbot testing itself, since this test runs for a completely silly amount of time.
If anyone is looking for something fun to do at the sprint tomorrow, here's a good one. :)
Comment #17
mparker17Unassigning so other people can work on it as per #16 and https://twitter.com/webchick/status/588448754358882304
Comment #19
fgmBumping to 8.2.x now that 8.1.x has been released.
Comment #21
alexpottThis test again proves it is not obsolete... without this test we would not have caught #2485385-137: Move highwater field support to the source plugin, and do not expose its internals on MigrationInterface and worked on a fix in #2776235: Cached autoloader misses cause failures when missed class becomes available.
It is time to close this issue with won't fix as the premise is wrong.