resourceloader: Improve coverage of SkinModule
authorTimo Tijhof <krinklemail@gmail.com>
Fri, 4 May 2018 16:30:33 +0000 (17:30 +0100)
committerTimo Tijhof <krinklemail@gmail.com>
Fri, 4 May 2018 16:31:04 +0000 (17:31 +0100)
commitb6a2e22acdedaa9e6fa827a7217945390c94ac1f
tree1ad78e092b246340e504ada9dfc951908dfc78f6
parent42f201ebe78d1aba18176f8b3cef5cf733c018a8
resourceloader: Improve coverage of SkinModule

* Ignore getLogoData() which is a one-line proxy to ::getLogo,
  that can't really be tested because static methods can't be stubbed.
  It exists so that this method can be stubbed instead. The actual
  method is already covered.

* Simplify @covers for getStyles() to allow indirect coverage within
  the class because it's intended as a higher-level integration tests.
  The other tests in the suite still cover a specific method only.

Change-Id: I1445a016c1f12a6d8ceaaf745023a28cf20e5371
includes/resourceloader/ResourceLoaderSkinModule.php
tests/phpunit/includes/resourceloader/ResourceLoaderSkinModuleTest.php