tests: Use a fancy generator provider in ReleaseNotesTest
authorTimo Tijhof <krinklemail@gmail.com>
Tue, 2 Jul 2019 18:48:20 +0000 (19:48 +0100)
committerTimo Tijhof <krinklemail@gmail.com>
Tue, 2 Jul 2019 18:49:43 +0000 (19:49 +0100)
Follows-up ffc23ee333fffb.

Change-Id: I99bf0d7115f6d2a84eb499d104d9cd227331c20f

tests/phpunit/documentation/ReleaseNotesTest.php

index 3ac69ae..acbb04a 100644 (file)
@@ -46,11 +46,9 @@ class ReleaseNotesTest extends MediaWikiTestCase {
                        "SECURITY",
                ];
 
-               $testCases = [];
                foreach ( $rootFiles as $rootFile ) {
-                       $testCases["$rootFile file"] = [ "$IP/$rootFile" ];
+                       yield "$rootFile file" => [ "$IP/$rootFile" ];
                }
-               return $testCases;
        }
 
        /**
@@ -82,9 +80,10 @@ class ReleaseNotesTest extends MediaWikiTestCase {
                        }
                        $errors[] = "line $num: length $length > $max_length:\n$line";
                }
-               # Using assertSame() to show the full line
+               # Use assertSame() instead of assertEqual(), to show the full line in the diff
                $this->assertSame(
-                       [], $errors,
+                       [],
+                       $errors,
                        "$type file '$fileName' lines " .
                        "have at most $max_length characters"
                );