MDL-82148 phpunit: Remove all cases of multiple @coversDefaultClass

Only one by class is allowed (to be used as shortcut exclusively).
stronk7 · Jun 10, 2024 · d81c63d · d81c63d
1 parent 2d41ac4
commit d81c63d
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 19 deletions.
diff --git a/lib/tests/http_client_test.php b/lib/tests/http_client_test.php
@@ -29,12 +29,8 @@
  * @category   test
  * @copyright  2022 Safat Shahin <safat.shahin@moodle.com>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ *
  * @coversDefaultClass \core\http_client
- * @coversDefaultClass \core\local\guzzle\redirect_middleware
- * @coversDefaultClass \core\local\guzzle\check_request
- * @coversDefaultClass \core\local\guzzle\cache_item
- * @coversDefaultClass \core\local\guzzle\cache_handler
- * @coversDefaultClass \core\local\guzzle\cache_storage
  */
 class http_client_test extends \advanced_testcase {
 

diff --git a/mod/quiz/tests/quiz_question_restore_test.php b/mod/quiz/tests/quiz_question_restore_test.php
@@ -31,9 +31,6 @@
  * @copyright  2021 Catalyst IT Australia Pty Ltd
  * @author     Safat Shahin <safatshahin@catalyst-au.net>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- * @coversDefaultClass \mod_quiz\question\bank\qbank_helper
- * @coversDefaultClass \backup_quiz_activity_structure_step
- * @coversDefaultClass \restore_quiz_activity_structure_step
  */
 class quiz_question_restore_test extends \advanced_testcase {
     use \quiz_question_helper_test_trait;
@@ -58,7 +55,7 @@ public function setUp(): void {
     /**
      * Test a quiz backup and restore in a different course without attempts for course question bank.
      *
-     * @covers ::get_question_structure
+     * @covers \mod_quiz\question\bank\qbank_helper::get_question_structure
      */
     public function test_quiz_restore_in_a_different_course_using_course_question_bank() {
         $this->resetAfterTest();
@@ -97,7 +94,7 @@ public function test_quiz_restore_in_a_different_course_using_course_question_ba
     /**
      * Test a quiz backup and restore in a different course without attempts for quiz question bank.
      *
-     * @covers ::get_question_structure
+     * @covers \mod_quiz\question\bank\qbank_helper::get_question_structure
      */
     public function test_quiz_restore_in_a_different_course_using_quiz_question_bank() {
         $this->resetAfterTest();
@@ -197,7 +194,7 @@ public function test_quiz_duplicate_for_quiz_question_bank_questions() {
     /**
      * Test quiz restore with attempts.
      *
-     * @covers ::get_question_structure
+     * @covers \mod_quiz\question\bank\qbank_helper::get_question_structure
      */
     public function test_quiz_restore_with_attempts() {
         $this->resetAfterTest();
@@ -236,7 +233,7 @@ public function test_quiz_restore_with_attempts() {
     /**
      * Test pre 4.0 quiz restore for regular questions.
      *
-     * @covers ::process_quiz_question_legacy_instance
+     * @covers \restore_quiz_activity_structure_step::process_quiz_question_legacy_instance
      */
     public function test_pre_4_quiz_restore_for_regular_questions() {
         global $USER, $DB;
@@ -278,7 +275,7 @@ public function test_pre_4_quiz_restore_for_regular_questions() {
     /**
      * Test pre 4.0 quiz restore for random questions.
      *
-     * @covers ::process_quiz_question_legacy_instance
+     * @covers \restore_quiz_activity_structure_step::process_quiz_question_legacy_instance
      */
     public function test_pre_4_quiz_restore_for_random_questions() {
         global $USER, $DB;
@@ -326,7 +323,7 @@ public function test_pre_4_quiz_restore_for_random_questions() {
     /**
      * Test pre 4.0 quiz restore for random question tags.
      *
-     * @covers ::process_quiz_question_legacy_instance
+     * @covers \restore_quiz_activity_structure_step::process_quiz_question_legacy_instance
      */
     public function test_pre_4_quiz_restore_for_random_question_tags() {
         global $USER, $DB;
@@ -383,7 +380,7 @@ public function test_pre_4_quiz_restore_for_random_question_tags() {
     /**
      * Test pre 4.0 quiz restore for random question used on multiple quizzes.
      *
-     * @covers ::process_quiz_question_legacy_instance
+     * @covers \restore_quiz_activity_structure_step::process_quiz_question_legacy_instance
      */
     public function test_pre_4_quiz_restore_shared_random_question() {
         global $USER, $DB;

diff --git a/privacy/tests/legacy_polyfill_test.php b/privacy/tests/legacy_polyfill_test.php
@@ -21,7 +21,6 @@
  * @category    test
  * @copyright   2018 Andrew Nicols <andrew@nicols.co.uk>
  * @license     http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- * @coversDefaultClass \core_privacy\local\legacy_polyfill
  */
 
 namespace core_privacy;

diff --git a/question/bank/editquestion/tests/external/update_question_version_status_test.php b/question/bank/editquestion/tests/external/update_question_version_status_test.php
@@ -26,9 +26,9 @@
  * @copyright  2021 Catalyst IT Australia Pty Ltd
  * @author     Safat Shahin <safatshahin@catalyst-au.net>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- * @coversDefaultClass \core_question\local\bank\question_version_status
+ *
+ * @covers \core_question\local\bank\question_version_status
  * @coversDefaultClass \qbank_editquestion\external\update_question_version_status
- * @coversDefaultClass \qbank_editquestion\editquestion_helper
  */
 class update_question_version_status_test extends \advanced_testcase {
 
@@ -53,7 +53,7 @@ public function setUp(): void {
      * Test if the submit status webservice changes the status of the question.
      *
      * @covers ::execute
-     * @covers ::get_question_status_string
+     * @covers \qbank_editquestion\editquestion_helper::get_question_status_string
      */
     public function test_submit_status_updates_the_question_status() {
         global $DB;