Skip to content

Commit

Permalink
MDL-75880 phpunit: fix all @coversDefaultClass missing @Covers
Browse files Browse the repository at this point in the history
Because @coversDefaultClass doesn't indicate any coverage (it's
just an alias to avoid having to write the class name in @Covers),
this commit fixes all the files that were using that annotation
and missing any @Covers.

Basically, replacing one by the other and done.
  • Loading branch information
stronk7 committed Nov 4, 2022
1 parent fee1b8c commit d590b08
Show file tree
Hide file tree
Showing 17 changed files with 45 additions and 89 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
* @category test
* @copyright 2021 Sara Arjona (sara@moodle.com)
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @coversDefaultClass \tool_admin_presets\event\preset_deleted
* @covers \tool_admin_presets\event\preset_deleted
*/
class preset_deleted_test extends \advanced_testcase {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
* @category test
* @copyright 2021 Sara Arjona (sara@moodle.com)
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @coversDefaultClass \tool_admin_presets\event\preset_downloaded
* @covers \tool_admin_presets\event\preset_downloaded
*/
class preset_downloaded_test extends \advanced_testcase {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
* @category test
* @copyright 2021 Sara Arjona (sara@moodle.com)
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @coversDefaultClass \tool_admin_presets\event\preset_exported
* @covers \tool_admin_presets\event\preset_exported
*/
class preset_exported_test extends \advanced_testcase {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
* @category test
* @copyright 2021 Sara Arjona (sara@moodle.com)
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @coversDefaultClass \tool_admin_presets\event\preset_imported
* @covers \tool_admin_presets\event\preset_imported
*/
class preset_imported_test extends \advanced_testcase {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
* @category test
* @copyright 2021 Sara Arjona (sara@moodle.com)
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @coversDefaultClass \tool_admin_presets\event\preset_loaded
* @covers \tool_admin_presets\event\preset_loaded
*/
class preset_loaded_test extends \advanced_testcase {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
* @category test
* @copyright 2021 Sara Arjona (sara@moodle.com)
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @coversDefaultClass \tool_admin_presets\event\preset_previewed
* @covers \tool_admin_presets\event\preset_previewed
*/
class preset_previewed_test extends \advanced_testcase {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
* @category test
* @copyright 2021 Sara Arjona (sara@moodle.com)
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @coversDefaultClass \tool_admin_presets\event\preset_reverted
* @covers \tool_admin_presets\event\preset_reverted
*/
class preset_reverted_test extends \advanced_testcase {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
* @category test
* @copyright 2021 Sara Arjona (sara@moodle.com)
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @coversDefaultClass \tool_admin_presets\event\presets_listed
* @covers \tool_admin_presets\event\presets_listed
*/
class presets_listed_test extends \advanced_testcase {

Expand Down
2 changes: 1 addition & 1 deletion course/format/tests/output/local/state/state_test.php
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ public static function setupBeforeClass(): void {
* Test the behaviour of state::export_for_template().
*
* @dataProvider state_provider
* @coversDefaultClass \core_courseformat\output\local\state
* @covers \core_courseformat\output\local\state
*
* @param string $format The course format of the course where the method will be executed.
*/
Expand Down
15 changes: 5 additions & 10 deletions files/tests/archive_writer_test.php
Original file line number Diff line number Diff line change
Expand Up @@ -14,15 +14,6 @@
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.

/**
* Unit tests for core_files\local\archive_writer/zip_archive.
*
* @package core_files
* @category test
* @copyright 2020 Mark Nelson <mdjnelson@gmail.com>
* @license http://www.gnu.org/copyleft/gpl.html GNU Public License
*/

namespace core_files;

use advanced_testcase;
Expand All @@ -31,7 +22,11 @@
/**
* Unit tests for \core_files\archive_writer.
*
* @coversDefaultClass \core_files\archive_writer
* @package core_files
* @category test
* @copyright 2020 Mark Nelson <mdjnelson@gmail.com>
* @license http://www.gnu.org/copyleft/gpl.html GNU Public License
* @covers \core_files\archive_writer
*/
class archive_writer_test extends advanced_testcase {

Expand Down
15 changes: 5 additions & 10 deletions files/tests/local/archive_writer/zip_writer_test.php
Original file line number Diff line number Diff line change
Expand Up @@ -14,15 +14,6 @@
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.

/**
* Unit tests for \core_files\local\archive_writer\zip_writer.
*
* @package core_files
* @category test
* @copyright 2020 Mark Nelson <mdjnelson@gmail.com>
* @license http://www.gnu.org/copyleft/gpl.html GNU Public License
*/

namespace core_files\local\archive_writer;

use advanced_testcase;
Expand All @@ -33,7 +24,11 @@
/**
* Unit tests for \core_files\local\archive_writer\zip_writer.
*
* @coversDefaultClass \core_files\local\archive_writer\zip_writer
* @package core_files
* @category test
* @copyright 2020 Mark Nelson <mdjnelson@gmail.com>
* @license http://www.gnu.org/copyleft/gpl.html GNU Public License
* @covers \core_files\local\archive_writer\zip_writer
*/
class zip_writer_test extends advanced_testcase {

Expand Down
11 changes: 1 addition & 10 deletions lib/table/tests/local/filter/filter_test.php
Original file line number Diff line number Diff line change
Expand Up @@ -14,15 +14,6 @@
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.

/**
* Unit tests for core_table\local\filter\filter.
*
* @package core_table
* @category test
* @copyright 2020 Andrew Nicols <andrew@nicols.co.uk>
* @license http://www.gnu.org/copyleft/gpl.html GNU Public License
*/

declare(strict_types=1);

namespace core_table\local\filter;
Expand All @@ -33,11 +24,11 @@
/**
* Unit tests for core_table\local\filter\filter.
*
* @coversDefaultClass \core_table\local\filter\filter
* @package core_table
* @category test
* @copyright 2020 Andrew Nicols <andrew@nicols.co.uk>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @covers \core_table\local\filter\filter
*/
class filter_test extends advanced_testcase {
/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,15 +14,6 @@
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.

/**
* Unit tests for core\content\exportable_items\exportable_filearea.
*
* @package core
* @category test
* @copyright 2020 Andrew Nicols <andrew@nicols.co.uk>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/

declare(strict_types=1);

namespace core\content\export\exportable_items;
Expand All @@ -38,7 +29,11 @@
/**
* Unit tests for the `exportable_filearea` export item class.
*
* @coversDefaultClass \core\content\exportable_items\exportable_filearea
* @package core
* @category test
* @copyright 2020 Andrew Nicols <andrew@nicols.co.uk>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @covers \core\content\exportable_items\exportable_filearea
*/
class exportable_filearea_test extends advanced_testcase {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,15 +14,6 @@
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.

/**
* Unit tests for core\content\exportable_items\exportable_stored_file.
*
* @package core
* @category test
* @copyright 2020 Andrew Nicols <andrew@nicols.co.uk>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/

declare(strict_types=1);

namespace core\content\export\exportable_items;
Expand All @@ -37,7 +28,11 @@
/**
* Unit tests for the `exportable_stored_file` export item class.
*
* @coversDefaultClass \core\content\exportable_items\exportable_stored_file
* @package core
* @category test
* @copyright 2020 Andrew Nicols <andrew@nicols.co.uk>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @covers \core\content\exportable_items\exportable_stored_file
*/
class exportable_stored_file_test extends advanced_testcase {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,15 +14,6 @@
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.

/**
* Unit tests for core\content\exportable_items\exportable_textarea.
*
* @package core
* @category test
* @copyright 2020 Andrew Nicols <andrew@nicols.co.uk>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/

declare(strict_types=1);

namespace core\content\export\exportable_items;
Expand All @@ -38,7 +29,11 @@
/**
* Unit tests for the `exportable_textarea` export item class.
*
* @coversDefaultClass \core\content\exportable_items\exportable_textarea
* @package core
* @category test
* @copyright 2020 Andrew Nicols <andrew@nicols.co.uk>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @covers \core\content\exportable_items\exportable_textarea
*/
class exportable_textarea_test extends advanced_testcase {

Expand Down
15 changes: 5 additions & 10 deletions lib/tests/content/export/exporters/course_exporter_test.php
Original file line number Diff line number Diff line change
Expand Up @@ -14,15 +14,6 @@
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.

/**
* Unit tests for core\content\export\exporters\course_exporter.
*
* @package core
* @category test
* @copyright 2020 Simey Lameze <simey@moodle.com>
* @license http://www.gnu.org/copyleft/gpl.html GNU Public License
*/

declare(strict_types=1);

namespace core\content\export\exporters;
Expand All @@ -36,7 +27,11 @@
/**
* Unit tests for activity exporter.
*
* @coversDefaultClass \core\content\export\exporters\course_exporter
* @package core
* @category test
* @copyright 2020 Simey Lameze <simey@moodle.com>
* @license http://www.gnu.org/copyleft/gpl.html GNU Public License
* @covers \core\content\export\exporters\course_exporter
*/
class course_exporter_test extends advanced_testcase {

Expand Down
15 changes: 5 additions & 10 deletions lib/tests/content/export/zipwriter_test.php
Original file line number Diff line number Diff line change
Expand Up @@ -14,15 +14,6 @@
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.

/**
* Unit tests for core\content\zipwriter.
*
* @package core
* @category test
* @copyright 2020 Simey Lameze <simey@moodle.com>
* @license http://www.gnu.org/copyleft/gpl.html GNU Public License
*/

declare(strict_types=1);

namespace core\content\export;
Expand All @@ -35,7 +26,11 @@
/**
* Unit tests for core\content\zipwriter.
*
* @coversDefaultClass \core\content\export\zipwriter
* @package core
* @category test
* @copyright 2020 Simey Lameze <simey@moodle.com>
* @license http://www.gnu.org/copyleft/gpl.html GNU Public License
* @covers \core\content\export\zipwriter
*/
class zipwriter_test extends advanced_testcase {

Expand Down

0 comments on commit d590b08

Please sign in to comment.