MDL-65204 phpunit: make testcases match their expected signature
authorEloy Lafuente (stronk7) <stronk7@moodle.org>
Sun, 24 Mar 2019 22:21:48 +0000 (23:21 +0100)
committerJun Pataleta <jun@moodle.com>
Wed, 3 Apr 2019 02:39:19 +0000 (10:39 +0800)
lib/phpunit/classes/advanced_testcase.php
lib/phpunit/classes/autoloader.php
lib/phpunit/classes/basic_testcase.php
lib/phpunit/classes/database_driver_testcase.php
lib/phpunit/classes/hint_resultprinter.php
lib/tests/completionlib_test.php

index e18fddf..b65db0c 100644 (file)
@@ -63,7 +63,7 @@ abstract class advanced_testcase extends base_testcase {
      * Runs the bare test sequence.
      * @return void
      */
-    final public function runBare() {
+    final public function runBare(): void {
         global $DB;
 
         if (phpunit_util::$lastdbwrites != $DB->perf_get_writes()) {
index c39ec4a..50052db 100644 (file)
@@ -43,7 +43,7 @@
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 class phpunit_autoloader implements \PHPUnit\Runner\TestSuiteLoader {
-    public function load($suiteClassName, $suiteClassFile = '') {
+    public function load(string $suiteClassName, string $suiteClassFile = ''): ReflectionClass {
         global $CFG;
 
         // Let's guess what user entered on the commandline...
@@ -197,7 +197,7 @@ class phpunit_autoloader implements \PHPUnit\Runner\TestSuiteLoader {
         return new ReflectionClass($classname);
     }
 
-    public function reload(ReflectionClass $aClass) {
+    public function reload(ReflectionClass $aClass): ReflectionClass {
         return $aClass;
     }
 }
index ba7d000..497f260 100644 (file)
@@ -57,7 +57,7 @@ abstract class basic_testcase extends base_testcase {
      * Runs the bare test sequence and log any changes in global state or database.
      * @return void
      */
-    final public function runBare() {
+    final public function runBare(): void {
         global $DB;
 
         try {
index 5236a6e..63a99d4 100644 (file)
@@ -138,7 +138,7 @@ abstract class database_driver_testcase extends base_testcase {
      * Runs the bare test sequence.
      * @return void
      */
-    public function runBare() {
+    public function runBare(): void {
         try {
             parent::runBare();
 
index fb10e5a..228db3e 100644 (file)
@@ -48,7 +48,7 @@ class Hint_ResultPrinter extends PHPUnit\TextUI\ResultPrinter {
         parent::__construct(null, false, self::COLOR_DEFAULT, false);
     }
 
-    protected function printDefectTrace(PHPUnit\Framework\TestFailure $defect) {
+    protected function printDefectTrace(PHPUnit\Framework\TestFailure $defect): void {
         global $CFG;
 
         parent::printDefectTrace($defect);
index cc2dfbb..1b4f163 100644 (file)
@@ -76,7 +76,7 @@ class core_completionlib_testcase extends advanced_testcase {
      * @param  boolean $canonicalize
      * @param  boolean $ignoreCase
      */
-    public static function assertEquals($expected, $actual, $message = '', $delta = 0, $maxDepth = 10, $canonicalize = FALSE, $ignoreCase = FALSE) {
+    public static function assertEquals($expected, $actual, string $message = '', float $delta = 0, int $maxDepth = 10, bool $canonicalize = false, bool $ignoreCase = false): void {
         // Nasty cheating hack: prevent random failures on timemodified field.
         if (is_object($expected) and is_object($actual)) {
             if (property_exists($expected, 'timemodified') and property_exists($actual, 'timemodified')) {