MDL-52060 phpunit: Check file only if needed
authorRajesh Taneja <rajesh@moodle.com>
Wed, 11 Nov 2015 07:57:10 +0000 (15:57 +0800)
committerRajesh Taneja <rajesh@moodle.com>
Tue, 22 Dec 2015 05:19:06 +0000 (13:19 +0800)
lib/testing/classes/util.php

index 0864d3b..1b08d6f 100644 (file)
@@ -269,17 +269,15 @@ abstract class testing_util {
      * @return array  $table=>$records
      */
     protected static function get_tabledata() {
-        global $CFG;
-
-        $framework = self::get_framework();
+        if (!isset(self::$tabledata)) {
+            $framework = self::get_framework();
 
-        $datafile = self::get_dataroot() . '/' . $framework . '/tabledata.ser';
-        if (!file_exists($datafile)) {
-            // Not initialised yet.
-            return array();
-        }
+            $datafile = self::get_dataroot() . '/' . $framework . '/tabledata.ser';
+            if (!file_exists($datafile)) {
+                // Not initialised yet.
+                return array();
+            }
 
-        if (!isset(self::$tabledata)) {
             $data = file_get_contents($datafile);
             self::$tabledata = unserialize($data);
         }
@@ -297,17 +295,15 @@ abstract class testing_util {
      * @return array $table=>$records
      */
     public static function get_tablestructure() {
-        global $CFG;
-
-        $framework = self::get_framework();
+        if (!isset(self::$tablestructure)) {
+            $framework = self::get_framework();
 
-        $structurefile = self::get_dataroot() . '/' . $framework . '/tablestructure.ser';
-        if (!file_exists($structurefile)) {
-            // Not initialised yet.
-            return array();
-        }
+            $structurefile = self::get_dataroot() . '/' . $framework . '/tablestructure.ser';
+            if (!file_exists($structurefile)) {
+                // Not initialised yet.
+                return array();
+            }
 
-        if (!isset(self::$tablestructure)) {
             $data = file_get_contents($structurefile);
             self::$tablestructure = unserialize($data);
         }