From d8cf850032e78b0252d6b5801e8d0754b3411594 Mon Sep 17 00:00:00 2001 From: Timo Tijhof Date: Wed, 22 Nov 2017 11:01:54 -0800 Subject: [PATCH] InstallDocFormatter: Add missing @covers scope The format() method is a one-line wrapper around execute(), which is the real method being tested here. Given the class doesn't contain any other methods, increase the scope to the whole class. Change-Id: I8b2c7736c6708c0915a9ce15fef0dc85c63621ea --- includes/installer/InstallDocFormatter.php | 2 +- .../phpunit/includes/installer/InstallDocFormatterTest.php | 6 +----- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/includes/installer/InstallDocFormatter.php b/includes/installer/InstallDocFormatter.php index 4163e2f958..08cfd8689a 100644 --- a/includes/installer/InstallDocFormatter.php +++ b/includes/installer/InstallDocFormatter.php @@ -21,7 +21,7 @@ */ class InstallDocFormatter { - static function format( $text ) { + public static function format( $text ) { $obj = new self( $text ); return $obj->execute(); diff --git a/tests/phpunit/includes/installer/InstallDocFormatterTest.php b/tests/phpunit/includes/installer/InstallDocFormatterTest.php index 36b6d64f79..9584d4b8c4 100644 --- a/tests/phpunit/includes/installer/InstallDocFormatterTest.php +++ b/tests/phpunit/includes/installer/InstallDocFormatterTest.php @@ -1,12 +1,8 @@