StableLibraryOverrideTest does not respect experimental modules

A test with a hidden experimental module perhaps?

Although then we're testing a test… :P

Add an experimental module with a library. getAllLibraries() grabs all libraries, correct? So it shouldn't appear in that group. Does \Drupal::moduleHandler()->getModuleList(); only return installed modules? Is that sufficient to check, or do assets actually need to be checked?

As as I can tell there is also nothing checking the other module types, so should there be individual checks for all of them? A dummy experimental module, test module, contrib module, hidden module.

Tim pointed out to me that the existing condition is already checking for hidden modules, and is excluding those. So, we can't write a test with a hidden module, because that'd already work without any changes.

Therefore the only way to prove this works IMO is to write a FAIL test here, but commit #2.

Finally, I want to stress again that writing tests for this would mean writing test coverage for a test. So, removing the Needs tests issue tag.

#7 Looks good to me. I tested all three patches and got the expected results.

Let's get this in!

+1

Makes sense. Committed and pushed bbfb417 to 8.2.x and 9365344 to 8.1.x. Thanks!