The ResourceLoaderModule::getConfig method falls back to the global service locator. This code was unreachable until recently when two separate issues were allowed to be merge because of this fallback preventing it from failing.
- getConfig() is called in the SkinModule constructor for reading wgUseNewMediaStructure. It does so before the Config object is set by the ResourceLoader service. This issue was observed during development, but addressed by working around it in the unit tests.
- Also in SkinModule, the value for wgResourceBasePath is read from global state instead of from Config. This has also made the test significantly more complicated than one would like for a test, to the point that the test is as complex if not more so than the code it is testing.