MDL-68473 core_h5p: moved fixture to anonymous class
authorVíctor Déniz Falcón <victor@moodle.com>
Fri, 1 May 2020 13:19:59 +0000 (14:19 +0100)
committerVíctor Déniz Falcón <victor@moodle.com>
Fri, 1 May 2020 13:25:11 +0000 (14:25 +0100)
h5p/tests/editor_test.php

index d8d69b6..fb007bf 100644 (file)
@@ -27,15 +27,11 @@ namespace core_h5p;
 
 defined('MOODLE_INTERNAL') || die();
 
-global $CFG;
-
 use advanced_testcase;
 use core_h5p\local\library\autoloader;
 use MoodleQuickForm;
 use page_requirements_manager;
 
-require_once($CFG->libdir . '/formslib.php');
-
 /**
  *
  * Test class covering the editor class.
@@ -48,6 +44,34 @@ require_once($CFG->libdir . '/formslib.php');
  */
 class editor_testcase extends advanced_testcase {
 
+    /**
+     * Form object to be used in test case.
+     */
+    protected function get_test_form() {
+        global $CFG;
+
+        require_once($CFG->libdir . '/formslib.php');
+
+        return new class extends \moodleform {
+            /**
+             * Form definition.
+             */
+            public function definition(): void {
+                // No definition required.
+            }
+
+            /**
+             * Returns form reference.
+             *
+             * @return MoodleQuickForm
+             */
+            public function getform() {
+                $mform = $this->_form;
+                return $mform;
+            }
+        };
+    }
+
     /**
      * Test that existing content is properly set.
      */
@@ -159,7 +183,7 @@ class editor_testcase extends advanced_testcase {
         global $PAGE, $CFG;
 
         // Get form data.
-        $form = new temp_form();
+        $form = $this->get_test_form();
         $mform = $form->getform();
 
         // Call method.
@@ -255,27 +279,3 @@ class editor_testcase extends advanced_testcase {
         $this->assertNotEmpty($out);
     }
 }
-
-/**
- * Form object to be used in test case.
- */
-class temp_form extends \moodleform {
-    /**
-     * Form definition.
-     */
-    public function definition(): void {
-        // No definition required.
-    }
-
-    /**
-     * Returns form reference.
-     *
-     * @return MoodleQuickForm
-     */
-    public function getform() {
-        $mform = $this->_form;
-        // Set submitted flag, to simulate submission.
-        $mform->_flagSubmitted = true;
-        return $mform;
-    }
-}