MDL-73278 phpunit: Rename already namespaced classes to match file name

These are the only cases 100% safe to apply the renaming of the testcase class names to match the file names. All other cases are not safe, because they are missing namespace and may enter into name conflicts. Adding namespaces is not as simple as imagined because it implies to, also, add a good number of modifications to core. See the issue for more details.
abgreeve · Dec 7, 2021 · deb0e76 · deb0e76
1 parent 5ee0b62
commit deb0e76
Show file tree

Hide file tree

Showing 138 changed files with 138 additions and 138 deletions.
diff --git a/admin/tool/brickfield/tests/filters_test.php b/admin/tool/brickfield/tests/filters_test.php
@@ -26,7 +26,7 @@
  * @author     Jay Churchward ([email protected])
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class filter_test extends \advanced_testcase {
+class filters_test extends \advanced_testcase {
     public function test_constructor() {
         $this->resetAfterTest();
 

diff --git a/admin/tool/customlang/tests/local/mlang/langstring_test.php b/admin/tool/customlang/tests/local/mlang/langstring_test.php
@@ -36,7 +36,7 @@
  * @copyright  2020 Ferran Recio <[email protected]>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class langstring_testcase extends advanced_testcase {
+class langstring_test extends advanced_testcase {
 
     /**
      * Sanity 1.x string

diff --git a/admin/tool/customlang/tests/local/mlang/phpparser_test.php b/admin/tool/customlang/tests/local/mlang/phpparser_test.php
@@ -34,7 +34,7 @@
  * @copyright  2015 Damyon Wiese
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class phpparser_testcase extends advanced_testcase {
+class phpparser_test extends advanced_testcase {
 
 
     /**

diff --git a/admin/tool/moodlenet/tests/import_handler_info_test.php b/admin/tool/moodlenet/tests/import_handler_info_test.php
@@ -33,7 +33,7 @@
 /**
  * Class tool_moodlenet_import_handler_info_testcase, providing test cases for the import_handler_info class.
  */
-class tool_moodlenet_import_handler_info_testcase extends \advanced_testcase {
+class import_handler_info_test extends \advanced_testcase {
 
     /**
      * Test init and the getters.

diff --git a/admin/tool/moodlenet/tests/import_handler_registry_test.php b/admin/tool/moodlenet/tests/import_handler_registry_test.php
@@ -36,7 +36,7 @@
 /**
  * Class tool_moodlenet_import_handler_registry_testcase, providing test cases for the import_handler_registry class.
  */
-class tool_moodlenet_import_handler_registry_testcase extends \advanced_testcase {
+class import_handler_registry_test extends \advanced_testcase {
 
     /**
      * Test confirming the behaviour of get_resource_handlers_for_strategy with different params.

diff --git a/admin/tool/moodlenet/tests/import_info_test.php b/admin/tool/moodlenet/tests/import_info_test.php
@@ -33,7 +33,7 @@
 /**
  * Class tool_moodlenet_import_info_testcase, providing test cases for the import_info class.
  */
-class tool_moodlenet_import_info_testcase extends \advanced_testcase {
+class import_info_test extends \advanced_testcase {
 
     /**
      * Create some test objects.

diff --git a/admin/tool/moodlenet/tests/import_processor_test.php b/admin/tool/moodlenet/tests/import_processor_test.php
@@ -36,7 +36,7 @@
 /**
  * Class tool_moodlenet_import_processor_testcase, providing test cases for the import_processor class.
  */
-class tool_moodlenet_import_processor_testcase extends \advanced_testcase {
+class import_processor_test extends \advanced_testcase {
 
     /**
      * An integration test, this confirms the ability to construct an import processor and run the import for the current user.

diff --git a/admin/tool/moodlenet/tests/remote_resource_test.php b/admin/tool/moodlenet/tests/remote_resource_test.php
@@ -32,7 +32,7 @@
 /**
  * Class tool_moodlenet_remote_resource_testcase, providing test cases for the remote_resource class.
  */
-class tool_moodlenet_remote_resource_testcase extends \advanced_testcase {
+class remote_resource_test extends \advanced_testcase {
 
     /**
      * Test getters.

diff --git a/admin/tool/moodlenet/tests/url_test.php b/admin/tool/moodlenet/tests/url_test.php
@@ -31,7 +31,7 @@
 /**
  * Class tool_moodlenet_url_testcase, providing test cases for the url class.
  */
-class tool_moodlenet_url_testcase extends \advanced_testcase {
+class url_test extends \advanced_testcase {
 
     /**
      * Test the parsing to host + path components.

diff --git a/blocks/html/tests/search_content_test.php b/blocks/html/tests/search_content_test.php
@@ -33,7 +33,7 @@
  * @copyright 2017 The Open University
  * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class search_content_testcase extends \advanced_testcase {
+class search_content_test extends \advanced_testcase {
 
     /**
      * Creates an HTML block on a course.

diff --git a/completion/tests/completion_criteria_test.php b/completion/tests/completion_criteria_test.php
@@ -24,7 +24,7 @@
  * @copyright 2021 Mikhail Golenkov <[email protected]>
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class core_completion_criteria_testcase extends \advanced_testcase {
+class completion_criteria_test extends \advanced_testcase {
 
     /**
      * Test setup.

diff --git a/contentbank/tests/content_test.php b/contentbank/tests/content_test.php
@@ -39,7 +39,7 @@
  * @coversDefaultClass \core_contentbank\content
  *
  */
-class core_contenttype_content_testcase extends \advanced_testcase {
+class content_test extends \advanced_testcase {
 
     /**
      * Setup to ensure that fixtures are loaded.

diff --git a/contentbank/tests/contenttype_test.php b/contentbank/tests/contenttype_test.php
@@ -40,7 +40,7 @@
  * @coversDefaultClass \core_contentbank\contenttype
  *
  */
-class core_contenttype_contenttype_testcase extends \advanced_testcase {
+class contenttype_test extends \advanced_testcase {
 
     /** @var int Identifier for the manager role. */
     protected $managerroleid;

diff --git a/contentbank/tests/external/delete_content_test.php b/contentbank/tests/external/delete_content_test.php
@@ -43,7 +43,7 @@
  * @copyright  2020 Sara Arjona <[email protected]>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class delete_content_testcase extends externallib_advanced_testcase {
+class delete_content_test extends externallib_advanced_testcase {
 
     /**
      * Test the behaviour of delete_content().

diff --git a/contentbank/tests/external/rename_content_test.php b/contentbank/tests/external/rename_content_test.php
@@ -43,7 +43,7 @@
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  * @coversDefaultClass \core_contentbank\external
  */
-class rename_content_testcase extends \externallib_advanced_testcase {
+class rename_content_test extends \externallib_advanced_testcase {
 
     /**
      * Data provider for test_rename_content.

diff --git a/contentbank/tests/privacy_test.php b/contentbank/tests/privacy_test.php
@@ -43,7 +43,7 @@
  * @copyright  2020 Carlos Escobedo <[email protected]>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class core_contentbank_privacy_testcase extends provider_testcase {
+class privacy_test extends provider_testcase {
 
     /**
      * Setup to ensure that fixtures are loaded.

diff --git a/course/tests/caching_content_item_readonly_repository_test.php b/course/tests/caching_content_item_readonly_repository_test.php
@@ -35,7 +35,7 @@
  * @copyright  2020 Jake Dallimore <[email protected]>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class caching_content_item_readonly_repository_testcase extends \advanced_testcase {
+class caching_content_item_readonly_repository_test extends \advanced_testcase {
     /**
      * Test verifying that content items are cached and returned from the cache in subsequent same-request calls.
      */

diff --git a/course/tests/category_hooks_test.php b/course/tests/category_hooks_test.php
@@ -35,7 +35,7 @@
 /**
  * Functional test for class core_course_category methods invoking hooks.
  */
-class core_course_category_hooks_testcase extends \advanced_testcase {
+class category_hooks_test extends \advanced_testcase {
 
     protected function setUp(): void {
         $this->resetAfterTest();

diff --git a/course/tests/content_item_readonly_repository_test.php b/course/tests/content_item_readonly_repository_test.php
@@ -35,7 +35,7 @@
  * @copyright  2020 Jake Dallimore <[email protected]>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class content_item_readonly_repository_testcase extends \advanced_testcase {
+class content_item_readonly_repository_test extends \advanced_testcase {
     /**
      * Test the repository method, find_all_for_course().
      */

diff --git a/course/tests/content_item_test.php b/course/tests/content_item_test.php
@@ -38,7 +38,7 @@
  * @copyright  2020 Jake Dallimore <[email protected]>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class content_item_testcase extends \advanced_testcase {
+class content_item_test extends \advanced_testcase {
 
     /**
      * Test the content_item class.

diff --git a/course/tests/course_image_cache_test.php b/course/tests/course_image_cache_test.php
@@ -31,7 +31,7 @@
  * @copyright  2021 Catalyst IT
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class course_image_cache_testcase extends \advanced_testcase {
+class course_image_cache_test extends \advanced_testcase {
 
     /**
      * Initial setup.

diff --git a/course/tests/course_summary_exporter_test.php b/course/tests/course_summary_exporter_test.php
@@ -29,7 +29,7 @@
  * @copyright  2021 Catalyst IT
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class course_summary_exporter_testcase extends \advanced_testcase {
+class course_summary_exporter_test extends \advanced_testcase {
 
     /**
      * Test that if no course overview images uploaded get_course_image returns false.

diff --git a/course/tests/exporters_content_item_test.php b/course/tests/exporters_content_item_test.php
@@ -35,7 +35,7 @@
  * @copyright  2020 Jake Dallimore <[email protected]>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class exporters_course_content_item_testcase extends \advanced_testcase {
+class exporters_content_item_test extends \advanced_testcase {
 
     /**
      * Test confirming a content_item can be exported for a course.

diff --git a/course/tests/exporters_content_items_test.php b/course/tests/exporters_content_items_test.php
@@ -35,7 +35,7 @@
  * @copyright  2020 Jake Dallimore <[email protected]>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class exporters_course_content_items_testcase extends \advanced_testcase {
+class exporters_content_items_test extends \advanced_testcase {
 
     /**
      * Test confirming the collection of content_items can be exported for a course.

diff --git a/course/tests/services_content_item_service_test.php b/course/tests/services_content_item_service_test.php
@@ -35,7 +35,7 @@
  * @copyright  2020 Jake Dallimore <[email protected]>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class services_content_item_service_testcase extends \advanced_testcase {
+class services_content_item_service_test extends \advanced_testcase {
 
     /**
      * Test confirming that content items are returned by the service.

diff --git a/dataformat/pdf/tests/writer_test.php b/dataformat/pdf/tests/writer_test.php
@@ -36,7 +36,7 @@
  * @copyright  2020 Paul Holden <[email protected]>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class writer_testcase extends \advanced_testcase {
+class writer_test extends \advanced_testcase {
 
     /**
      * Test writing data whose content contains an image with pluginfile.php source

diff --git a/enrol/fee/tests/payment/service_provider_test.php b/enrol/fee/tests/payment/service_provider_test.php
@@ -30,7 +30,7 @@
  *
  * @coversDefaultClass service_provider
  */
-class service_provider_testcase extends \advanced_testcase {
+class service_provider_test extends \advanced_testcase {
 
     /**
      * Test for service_provider::get_payable().

diff --git a/enrol/meta/tests/external_add_instances_test.php b/enrol/meta/tests/external_add_instances_test.php
@@ -30,7 +30,7 @@
  * @copyright  2021 WKS KV Bildung
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class add_instances_testcase extends \externallib_advanced_testcase {
+class external_add_instances_test extends \externallib_advanced_testcase {
 
     /**
      * Test setup

diff --git a/enrol/meta/tests/external_delete_instances_test.php b/enrol/meta/tests/external_delete_instances_test.php
@@ -30,7 +30,7 @@
  * @copyright  2021 WKS KV Bildung
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class delete_instances_testcase extends \externallib_advanced_testcase {
+class external_delete_instances_test extends \externallib_advanced_testcase {
 
     /**
      * Test setup

diff --git a/files/tests/archive_writer_test.php b/files/tests/archive_writer_test.php
@@ -33,7 +33,7 @@
  *
  * @coversDefaultClass \core_files\archive_writer
  */
-class archive_writer_testcase extends advanced_testcase {
+class archive_writer_test extends advanced_testcase {
 
     /**
      * Test get_file_writer().

diff --git a/files/tests/local/archive_writer/zip_writer_test.php b/files/tests/local/archive_writer/zip_writer_test.php
@@ -35,7 +35,7 @@
  *
  * @coversDefaultClass \core_files\local\archive_writer\zip_writer
  */
-class zip_writer_testcase extends advanced_testcase {
+class zip_writer_test extends advanced_testcase {
 
     /**
      * Test add_file_from_filepath().

diff --git a/grade/grading/form/guide/tests/generator_test.php b/grade/grading/form/guide/tests/generator_test.php
@@ -37,7 +37,7 @@
  * @copyright  2019 Andrew Nicols <[email protected]>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class generator_testcase extends \advanced_testcase {
+class generator_test extends \advanced_testcase {
 
     /**
      * Test guide creation.

diff --git a/grade/grading/form/guide/tests/grades_grader_gradingpanel_guide_external_fetch_test.php b/grade/grading/form/guide/tests/grades_grader_gradingpanel_guide_external_fetch_test.php
@@ -42,7 +42,7 @@
  * @copyright 2019 Andrew Nicols <[email protected]>
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class fetch_test extends advanced_testcase {
+class grades_grader_gradingpanel_guide_external_fetch_test extends advanced_testcase {
 
     public static function setupBeforeClass(): void {
         global $CFG;

diff --git a/grade/grading/form/guide/tests/grades_grader_gradingpanel_guide_external_store_test.php b/grade/grading/form/guide/tests/grades_grader_gradingpanel_guide_external_store_test.php
@@ -42,7 +42,7 @@
  * @copyright 2019 Andrew Nicols <[email protected]>
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class store_test extends advanced_testcase {
+class grades_grader_gradingpanel_guide_external_store_test extends advanced_testcase {
 
     public static function setupBeforeClass(): void {
         global $CFG;

diff --git a/grade/grading/form/rubric/tests/generator_test.php b/grade/grading/form/rubric/tests/generator_test.php
@@ -38,7 +38,7 @@
  * @copyright  2019 Andrew Nicols <[email protected]>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class generator_testcase extends advanced_testcase {
+class generator_test extends advanced_testcase {
 
     /**
      * Test rubric creation.

diff --git a/grade/grading/form/rubric/tests/grades_grader_gradingpanel_rubric_external_fetch_test.php b/grade/grading/form/rubric/tests/grades_grader_gradingpanel_rubric_external_fetch_test.php
@@ -43,7 +43,7 @@
  * @copyright 2019 Mathew May <mathew.solutions>
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class fetch_test extends advanced_testcase {
+class grades_grader_gradingpanel_rubric_external_fetch_test extends advanced_testcase {
 
     public static function setupBeforeClass(): void {
         global $CFG;

diff --git a/grade/grading/form/rubric/tests/grades_grader_gradingpanel_rubric_external_store_test.php b/grade/grading/form/rubric/tests/grades_grader_gradingpanel_rubric_external_store_test.php
@@ -42,7 +42,7 @@
  * @copyright 2019 Mathew May <mathew.solutions>
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class store_test extends advanced_testcase {
+class grades_grader_gradingpanel_rubric_external_store_test extends advanced_testcase {
 
     public static function setupBeforeClass(): void {
         global $CFG;

diff --git a/grade/grading/form/rubric/tests/privacy_test.php b/grade/grading/form/rubric/tests/privacy_test.php
@@ -37,7 +37,7 @@
  * @copyright  2018 Adrian Greeve <adriangreeve.com>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class privacy_testcase extends provider_testcase {
+class privacy_test extends provider_testcase {
 
     /**
      * Test the export of rubric data.

diff --git a/grade/grading/tests/generator_test.php b/grade/grading/tests/generator_test.php
@@ -38,7 +38,7 @@
  * @copyright  2019 Andrew Nicols <[email protected]>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class generator_testcase extends advanced_testcase {
+class generator_test extends advanced_testcase {
 
     /**
      * Test gradingform controller creation.

diff --git a/grade/tests/external/create_gradecategories_test.php b/grade/tests/external/create_gradecategories_test.php
@@ -34,7 +34,7 @@
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  * @since      Moodle 3.11
  */
-class create_gradecategories_testcase extends \externallib_advanced_testcase {
+class create_gradecategories_test extends \externallib_advanced_testcase {
 
     /**
      * Test create_gradecategories.

diff --git a/grade/tests/grades_grader_gradingpanel_point_external_fetch_test.php b/grade/tests/grades_grader_gradingpanel_point_external_fetch_test.php
@@ -42,7 +42,7 @@
  * @copyright 2019 Andrew Nicols <[email protected]>
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class fetch_test extends advanced_testcase {
+class grades_grader_gradingpanel_point_external_fetch_test extends advanced_testcase {
 
     public static function setupBeforeClass(): void {
         global $CFG;

diff --git a/grade/tests/grades_grader_gradingpanel_point_external_store_test.php b/grade/tests/grades_grader_gradingpanel_point_external_store_test.php
@@ -44,7 +44,7 @@
  * @copyright 2019 Andrew Nicols <[email protected]>
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class store_test extends advanced_testcase {
+class grades_grader_gradingpanel_point_external_store_test extends advanced_testcase {
 
     public static function setupBeforeClass(): void {
         global $CFG;

diff --git a/grade/tests/grades_grader_gradingpanel_scale_external_fetch_test.php b/grade/tests/grades_grader_gradingpanel_scale_external_fetch_test.php
@@ -42,7 +42,7 @@
  * @copyright 2019 Andrew Nicols <[email protected]>
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class fetch_test extends advanced_testcase {
+class grades_grader_gradingpanel_scale_external_fetch_test extends advanced_testcase {
 
     public static function setupBeforeClass(): void {
         global $CFG;

diff --git a/grade/tests/grades_grader_gradingpanel_scale_external_store_test.php b/grade/tests/grades_grader_gradingpanel_scale_external_store_test.php
@@ -44,7 +44,7 @@
  * @copyright 2019 Andrew Nicols <[email protected]>
  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class store_test extends advanced_testcase {
+class grades_grader_gradingpanel_scale_external_store_test extends advanced_testcase {
 
     public static function setupBeforeClass(): void {
         global $CFG;