MDL-72701 unit tests: Generate default coverage

abgreeve · Dec 21, 2021 · 3a70983 · 3a70983
1 parent bb9741d
commit 3a70983
Show file tree

Hide file tree

Showing 16 changed files with 29 additions and 561 deletions.
diff --git a/admin/tool/dataprivacy/tests/coverage.php b/admin/tool/dataprivacy/tests/coverage.php
diff --git a/grade/grading/form/guide/tests/coverage.php b/grade/grading/form/guide/tests/coverage.php
diff --git a/grade/grading/form/rubric/tests/coverage.php b/grade/grading/form/rubric/tests/coverage.php
diff --git a/grade/grading/tests/coverage.php b/grade/grading/tests/coverage.php
diff --git a/grade/tests/coverage.php b/grade/tests/coverage.php
diff --git a/h5p/tests/coverage.php b/h5p/tests/coverage.php
diff --git a/lib/phpunit/classes/coverage_info.php b/lib/phpunit/classes/coverage_info.php
@@ -68,6 +68,19 @@ class phpunit_coverage_info {
     final public function get_includelists(string $plugindir) : array {
         $coverages = [];
 
+        $includelistfolders = array_merge([
+            'classes',
+            'tests/generator',
+        ], $this->includelistfolders);;
+
+        $includelistfiles = array_merge([
+            'externallib.php',
+            'lib.php',
+            'locallib.php',
+            'renderer.php',
+            'rsslib.php',
+        ], $this->includelistfiles);
+
         if (!empty($plugindir)) {
             $plugindir .= "/";
 
@@ -79,21 +92,16 @@ final public function get_includelists(string $plugindir) : array {
                     "Please, replace them with \$includelistfolders and " .
                     "\$includelistfiles in {$plugindir}tests/coverage.php\n";
 
-                foreach ($this->whitelistfolders as $folder) {
-                    $coverages[] = html_writer::tag('directory', "{$plugindir}{$folder}", ['suffix' => '.php']);
-                }
-
-                foreach ($this->whitelistfiles as $file) {
-                    $coverages[] = html_writer::tag('file', "{$plugindir}{$file}");
-                }
+                $includelistfolders = array_merge($includelistfolders, $this->whitelistfolders);
+                $includelistfiles = array_merge($includelistfiles, $this->whitelistfiles);
             }
         }
 
-        foreach ($this->includelistfolders as $folder) {
+        foreach ($includelistfolders as $folder) {
             $coverages[] = html_writer::tag('directory', "{$plugindir}{$folder}", ['suffix' => '.php']);
         }
 
-        foreach ($this->includelistfiles as $file) {
+        foreach ($includelistfiles as $file) {
             $coverages[] = html_writer::tag('file', "{$plugindir}{$file}");
         }
 

diff --git a/lib/phpunit/classes/util.php b/lib/phpunit/classes/util.php
@@ -973,7 +973,7 @@ protected static function get_coverage_config(array $includelists, array $exclud
      * @param   string  $fulldir The directory to find the coverage info file in.
      * @return  phpunit_coverage_info
      */
-    protected static function get_coverage_info(string $fulldir): ?phpunit_coverage_info {
+    protected static function get_coverage_info(string $fulldir): phpunit_coverage_info {
         $coverageconfig = "{$fulldir}/tests/coverage.php";
         if (file_exists($coverageconfig)) {
             $coverageinfo = require($coverageconfig);
@@ -984,7 +984,7 @@ protected static function get_coverage_info(string $fulldir): ?phpunit_coverage_
             return $coverageinfo;
         }
 
-        return null;
+        return new phpunit_coverage_info();;
     }
 
     /**

diff --git a/lib/table/tests/coverage.php b/lib/table/tests/coverage.php