resourceloader: Add missing (string) cast to ClientHtml tests
authorTimo Tijhof <krinklemail@gmail.com>
Sat, 6 Apr 2019 23:02:28 +0000 (00:02 +0100)
committerTimo Tijhof <krinklemail@gmail.com>
Sat, 6 Apr 2019 23:14:22 +0000 (00:14 +0100)
commit9e1e44910e2b518073402d94b44eae2f5d67e6eb
treec0b375a37733c5a6e80b280d6f581a739183fea3
parentd9891f872f9d92f7109d7c6b9fcb6894e4c083bc
resourceloader: Add missing (string) cast to ClientHtml tests

One of the assertions already did this, but the others not yet.

The reason we do this is because the return value is a WrappedString
object. The assertEquals() method is able to satisfy the expected
value because of loose comparison casting it to a string for the
case where it matches.

But, when it fails during development, PHPUnit was still printing
the object as "Actual" and the string as "Expected" which makes
it difficult to understand what's going on, and also no diff.

Change-Id: Ice9fe5061fb76867e9bee2f34c88d3ddac3144f9
tests/phpunit/includes/resourceloader/ResourceLoaderClientHtmlTest.php