MDL-67673 phpunit: Fix the return type of template methods
authorEloy Lafuente (stronk7) <stronk7@moodle.org>
Sun, 2 Aug 2020 08:59:12 +0000 (10:59 +0200)
committerEloy Lafuente (stronk7) <stronk7@moodle.org>
Wed, 21 Oct 2020 10:45:59 +0000 (12:45 +0200)
All the setup/teardown/pre/post/conditions template methods
now are required to return void. This was warned with phpunit 7
and now is enforced.

At the same time, fix a few wrong function names,
provider data and param types, return statements...

421 files changed:
admin/tool/behat/tests/manager_util_test.php
admin/tool/capability/tests/events_test.php
admin/tool/cohortroles/tests/api_test.php
admin/tool/cohortroles/tests/privacy_test.php
admin/tool/dataprivacy/tests/expired_data_requests_test.php
admin/tool/dataprivacy/tests/task_test.php
admin/tool/langimport/tests/events_test.php
admin/tool/log/store/database/tests/privacy_test.php
admin/tool/log/store/legacy/tests/privacy_test.php
admin/tool/log/store/standard/tests/privacy_test.php
admin/tool/log/store/standard/tests/store_test.php
admin/tool/log/tests/privacy_test.php
admin/tool/lp/tests/externallib_test.php
admin/tool/lpmigrate/tests/processor_test.php
admin/tool/messageinbound/tests/manager_test.php
admin/tool/messageinbound/tests/privacy_test.php
admin/tool/mobile/tests/privacy_provider_test.php
admin/tool/monitor/tests/eventobservers_test.php
admin/tool/monitor/tests/events_test.php
admin/tool/monitor/tests/generator_test.php
admin/tool/monitor/tests/privacy_test.php
admin/tool/monitor/tests/rule_manager_test.php
admin/tool/monitor/tests/subscription_test.php
admin/tool/monitor/tests/task_check_subscriptions_test.php
admin/tool/monitor/tests/task_clean_events_test.php
admin/tool/policy/tests/externallib_test.php
admin/tool/policy/tests/privacy_provider_test.php
admin/tool/recyclebin/tests/category_bin_test.php
admin/tool/recyclebin/tests/course_bin_test.php
admin/tool/recyclebin/tests/events_test.php
admin/tool/usertours/tests/accessdate_filter_test.php
admin/tool/usertours/tests/manager_test.php
admin/tool/usertours/tests/role_filter_test.php
admin/tool/usertours/tests/step_test.php
admin/tool/usertours/tests/tour_test.php
analytics/tests/course_test.php
analytics/tests/dataset_manager_test.php
analytics/tests/model_test.php
analytics/tests/prediction_actions_test.php
analytics/tests/prediction_test.php
analytics/tests/privacy_test.php
analytics/tests/stats_test.php
auth/email/tests/external_test.php
auth/manual/tests/manual_test.php
auth/manual/tests/privacy_provider_test.php
auth/mnet/tests/privacy_provider_test.php
auth/oauth2/tests/privacy_provider_test.php
auth/tests/external_test.php
auth/tests/privacy_test.php
availability/condition/completion/tests/condition_test.php
availability/condition/date/tests/condition_test.php
availability/condition/group/tests/condition_test.php
availability/condition/grouping/tests/condition_test.php
availability/condition/profile/tests/condition_test.php
availability/tests/info_test.php
availability/tests/tree_test.php
backup/controller/tests/controller_test.php
backup/converter/moodle1/tests/moodle1_converter_test.php
backup/moodle2/tests/backup_encrypted_content_test.php
backup/moodle2/tests/backup_xml_transformer_test.php
backup/tests/automated_backup_test.php
backup/tests/course_copy_test.php
backup/tests/externallib_test.php
backup/util/checks/tests/checks_test.php
backup/util/dbops/tests/backup_dbops_test.php
backup/util/factories/tests/factories_test.php
backup/util/helper/tests/restore_structure_parser_processor_test.php
backup/util/plan/tests/plan_test.php
backup/util/plan/tests/step_test.php
backup/util/plan/tests/task_test.php
backup/util/structure/tests/structure_test.php
badges/tests/badgeslib_test.php
badges/tests/external_test.php
badges/tests/privacy_test.php
blocks/comments/tests/events_test.php
blocks/comments/tests/privacy_provider_test.php
blocks/online_users/tests/online_users_test.php
blocks/recentlyaccesseditems/tests/observer_test.php
blocks/rss_client/tests/privacy_test.php
blocks/tests/privacy_test.php
blog/tests/events_test.php
blog/tests/external_test.php
blog/tests/lib_test.php
blog/tests/privacy_test.php
cache/stores/apcu/tests/apcu_test.php
cache/stores/redis/tests/compressor_test.php
cache/stores/redis/tests/redis_test.php
cache/tests/administration_helper_test.php
cache/tests/cache_test.php
cache/tests/config_writer_test.php
cache/tests/fixtures/stores.php
calendar/tests/calendartype_test.php
calendar/tests/container_test.php
calendar/tests/events_related_objects_cache_test.php
calendar/tests/events_test.php
calendar/tests/externallib_test.php
calendar/tests/lib_test.php
calendar/tests/local_api_test.php
calendar/tests/privacy_test.php
calendar/tests/rrule_manager_test.php
calendar/tests/std_proxy_test.php
cohort/tests/privacy_test.php
comment/tests/externallib_test.php
comment/tests/privacy_test.php
competency/tests/external_test.php
competency/tests/privacy_test.php
completion/tests/api_test.php
completion/tests/progress_test.php
course/format/weeks/tests/observer_test.php
course/tests/category_hooks_test.php
course/tests/category_test.php
course/tests/customfield_test.php
course/tests/events_test.php
course/tests/externallib_test.php
course/tests/search_test.php
customfield/field/checkbox/tests/plugin_test.php
customfield/field/date/tests/plugin_test.php
customfield/field/select/tests/plugin_test.php
customfield/field/text/tests/plugin_test.php
customfield/field/textarea/tests/plugin_test.php
enrol/flatfile/tests/privacy_provider_test.php
enrol/imsenterprise/tests/imsenterprise_test.php
enrol/lti/tests/data_connector_test.php
enrol/lti/tests/helper_test.php
enrol/lti/tests/lib_test.php
enrol/lti/tests/privacy_provider_test.php
enrol/lti/tests/sync_members_test.php
enrol/lti/tests/tool_provider_test.php
enrol/paypal/tests/privacy_provider_test.php
enrol/tests/course_enrolment_manager_test.php
enrol/tests/role_external_test.php
favourites/tests/component_favourite_service_test.php
favourites/tests/privacy_test.php
favourites/tests/repository_test.php
favourites/tests/user_favourite_service_test.php
filter/algebra/tests/filter_test.php
filter/displayh5p/tests/filter_test.php
filter/multilang/tests/filter_test.php
filter/tex/tests/filter_test.php
grade/export/ods/tests/logging_test.php
grade/export/txt/tests/logging_test.php
grade/export/xls/tests/logging_test.php
grade/export/xml/tests/logging_test.php
grade/import/csv/tests/load_data_test.php
grade/report/grader/tests/privacy_test.php
grade/report/overview/tests/externallib_test.php
grade/report/user/tests/lib_test.php
grade/report/user/tests/privacy_test.php
grade/tests/events_test.php
grade/tests/grades_grader_gradingpanel_point_external_store_test.php
grade/tests/privacy_test.php
h5p/tests/editor_ajax_test.php
h5p/tests/editor_framework_test.php
h5p/tests/event_h5p_deleted_test.php
h5p/tests/event_h5p_viewed_test.php
h5p/tests/external_test.php
h5p/tests/framework_test.php
h5p/tests/generator_test.php
h5p/tests/h5p_core_test.php
h5p/tests/h5p_file_storage_test.php
iplookup/tests/geoip_test.php
iplookup/tests/geoplugin_test.php
lib/antivirus/clamav/tests/scanner_test.php
lib/ddl/tests/ddl_test.php
lib/dml/tests/dml_test.php
lib/dml/tests/pgsql_native_recordset_test.php
lib/dml/tests/recordset_walk_test.php
lib/dml/tests/sqlsrv_native_moodle_database_test.php
lib/filebrowser/tests/file_browser_test.php
lib/filestorage/tests/file_system_filedir_test.php
lib/filestorage/tests/file_system_test.php
lib/filestorage/tests/zip_packer_test.php
lib/form/tests/dateselector_test.php
lib/form/tests/datetimeselector_test.php
lib/grade/tests/fixtures/lib.php
lib/phpunit/classes/database_driver_testcase.php
lib/phpunit/classes/restore_date_testcase.php
lib/phpunit/tests/basic_test.php
lib/tests/ajaxlib_test.php
lib/tests/antivirus_test.php
lib/tests/blocklib_test.php
lib/tests/calendar_cron_task_test.php
lib/tests/collator_test.php
lib/tests/component_test.php
lib/tests/core_media_player_native.php
lib/tests/csvclass_test.php
lib/tests/customcontext_test.php
lib/tests/event/contentbank_content_created_test.php
lib/tests/event/contentbank_content_deleted_test.php
lib/tests/event/contentbank_content_updated_test.php
lib/tests/event/contentbank_content_uploaded_test.php
lib/tests/event/contentbank_content_viewed_test.php
lib/tests/event_profile_field_test.php
lib/tests/event_user_graded_test.php
lib/tests/events_test.php
lib/tests/exporter_test.php
lib/tests/externallib_test.php
lib/tests/gdlib_test.php
lib/tests/h5p_get_content_types_task_test.php
lib/tests/lock_test.php
lib/tests/medialib_test.php
lib/tests/moodle_page_test.php
lib/tests/myprofilelib_test.php
lib/tests/notification_test.php
lib/tests/persistent_test.php
lib/tests/plugin_manager_test.php
lib/tests/plugininfo/base_test.php
lib/tests/questionlib_test.php
lib/tests/rsslib_test.php
lib/tests/session_redis_test.php
lib/tests/statslib_test.php
lib/tests/task_database_logger_test.php
lib/tests/task_logging_test.php
lib/tests/time_splittings_test.php
lib/tests/user_test.php
lib/tests/useragent_test.php
lib/xapi/tests/external/post_statement_test.php
media/player/html5audio/tests/player_test.php
media/player/html5video/tests/player_test.php
media/player/swf/tests/player_test.php
media/player/videojs/tests/player_test.php
media/player/vimeo/tests/player_test.php
media/player/youtube/tests/player_test.php
message/output/airnotifier/tests/externallib_test.php
message/output/airnotifier/tests/privacy_test.php
message/output/email/tests/privacy_test.php
message/output/jabber/tests/privacy_test.php
message/output/popup/tests/api_test.php
message/output/popup/tests/externallib_test.php
message/tests/events_test.php
message/tests/externallib_test.php
message/tests/helper_test.php
message/tests/inbound_test.php
message/tests/messagelib_test.php
message/tests/migrate_message_data_task_test.php
message/tests/search_received_test.php
message/tests/search_sent_test.php
mnet/service/enrol/tests/privacy_test.php
mnet/tests/events_test.php
mod/assign/feedback/editpdf/tests/privacy_test.php
mod/assign/tests/base_test.php
mod/book/tests/events_test.php
mod/book/tests/lib_test.php
mod/book/tests/search_test.php
mod/book/tool/exportimscp/tests/events_test.php
mod/book/tool/importhtml/tests/locallib_test.php
mod/book/tool/print/tests/events_test.php
mod/chat/tests/lib_test.php
mod/chat/tests/privacy_test.php
mod/choice/tests/events_test.php
mod/choice/tests/privacy_provider_test.php
mod/data/tests/events_test.php
mod/data/tests/externallib_test.php
mod/data/tests/import_test.php
mod/data/tests/lib_test.php
mod/data/tests/privacy_provider_test.php
mod/data/tests/search_test.php
mod/feedback/tests/events_test.php
mod/feedback/tests/external_test.php
mod/feedback/tests/privacy_test.php
mod/folder/tests/events_test.php
mod/folder/tests/lib_test.php
mod/folder/tests/search_test.php
mod/forum/tests/builders_exported_posts_test.php
mod/forum/tests/events_test.php
mod/forum/tests/exporters_discussion_test.php
mod/forum/tests/externallib_test.php
mod/forum/tests/generator_test.php
mod/forum/tests/lib_test.php
mod/forum/tests/mail_group_test.php
mod/forum/tests/mail_test.php
mod/forum/tests/maildigest_test.php
mod/forum/tests/managers_capability_test.php
mod/forum/tests/privacy_provider_test.php
mod/forum/tests/private_replies_test.php
mod/forum/tests/qanda_test.php
mod/forum/tests/search_test.php
mod/forum/tests/subscriptions_test.php
mod/forum/tests/vaults_discussion_list_test.php
mod/forum/tests/vaults_discussion_test.php
mod/forum/tests/vaults_post_test.php
mod/glossary/tests/events_test.php
mod/glossary/tests/privacy_provider_test.php
mod/glossary/tests/search_test.php
mod/label/tests/lib_test.php
mod/lesson/tests/events_test.php
mod/lesson/tests/external_test.php
mod/lesson/tests/privacy_test.php
mod/lti/service/gradebookservices/tests/privacy_provider_test.php
mod/lti/service/gradebookservices/tests/task_cleanup_test.php
mod/lti/service/memberships/tests/privacy_provider_test.php
mod/lti/tests/externallib_test.php
mod/lti/tests/lib_test.php
mod/lti/tests/task_clean_access_tokens_test.php
mod/page/tests/lib_test.php
mod/quiz/accessrule/seb/tests/access_manager_test.php
mod/quiz/accessrule/seb/tests/backup_restore_test.php
mod/quiz/accessrule/seb/tests/event_test.php
mod/quiz/accessrule/seb/tests/link_generator_test.php
mod/quiz/accessrule/seb/tests/quiz_settings_test.php
mod/quiz/accessrule/seb/tests/rule_test.php
mod/quiz/accessrule/seb/tests/template_test.php
mod/quiz/tests/external_test.php
mod/quiz/tests/repaginate_test.php
mod/resource/tests/events_test.php
mod/resource/tests/lib_test.php
mod/resource/tests/search_test.php
mod/scorm/report/basic/tests/privacy_test.php
mod/scorm/report/interactions/tests/privacy_test.php
mod/scorm/report/objectives/tests/privacy_test.php
mod/scorm/tests/events_test.php
mod/scorm/tests/externallib_test.php
mod/scorm/tests/lib_test.php
mod/scorm/tests/locallib_test.php
mod/survey/tests/events_test.php
mod/survey/tests/externallib_test.php
mod/survey/tests/lib_test.php
mod/survey/tests/privacy_test.php
mod/url/tests/lib_test.php
mod/wiki/tests/events_test.php
mod/wiki/tests/externallib_test.php
mod/wiki/tests/privacy_test.php
mod/wiki/tests/search_test.php
mod/workshop/allocation/random/tests/allocator_test.php
mod/workshop/eval/best/tests/lib_test.php
mod/workshop/form/accumulative/tests/lib_test.php
mod/workshop/form/numerrors/tests/lib_test.php
mod/workshop/form/rubric/tests/lib_test.php
mod/workshop/tests/events_test.php
mod/workshop/tests/external_test.php
mod/workshop/tests/locallib_test.php
mod/workshop/tests/portfolio_caller_test.php
mod/workshop/tests/privacy_provider_test.php
my/tests/events_test.php
notes/tests/events_test.php
notes/tests/lib_test.php
privacy/classes/tests/provider_testcase.php
privacy/tests/manager_test.php
privacy/tests/writer_test.php
question/behaviour/adaptive/tests/behaviourtype_test.php
question/behaviour/adaptive/tests/mark_display_test.php
question/behaviour/deferredcbm/tests/behaviourtype_test.php
question/behaviour/deferredfeedback/tests/behaviourtype_test.php
question/behaviour/immediatecbm/tests/behaviourtype_test.php
question/behaviour/immediatefeedback/tests/behaviourtype_test.php
question/behaviour/informationitem/tests/behaviourtype_test.php
question/behaviour/interactive/tests/behaviourtype_test.php
question/behaviour/interactivecountback/tests/behaviourtype_test.php
question/behaviour/manualgraded/tests/behaviourtype_test.php
question/behaviour/missing/tests/behaviourtype_test.php
question/engine/tests/helpers.php
question/engine/tests/questionattempt_test.php
question/engine/tests/questionattempt_with_steps_test.php
question/engine/tests/questionattemptiterator_test.php
question/engine/tests/questionattemptstepiterator_test.php
question/engine/tests/questionusage_autosave_test.php
question/engine/tests/unitofwork_test.php
question/engine/upgrade/tests/helper.php
question/tests/category_class_test.php
question/tests/events_test.php
question/tests/externallib_test.php
question/type/calculated/tests/questiontype_test.php
question/type/calculatedsimple/tests/questiontype_test.php
question/type/ddimageortext/tests/questiontype_test.php
question/type/ddmarker/tests/questiontype_test.php
question/type/ddwtos/tests/questiontype_test.php
question/type/description/tests/questiontype_test.php
question/type/essay/tests/questiontype_test.php
question/type/gapselect/tests/questiontype_test.php
question/type/match/tests/questiontype_test.php
question/type/multianswer/tests/questiontype_test.php
question/type/multichoice/tests/questiontype_test.php
question/type/numerical/tests/questiontype_test.php
question/type/random/tests/questiontype_test.php
question/type/shortanswer/tests/questiontype_test.php
question/type/tests/question_first_matching_answer_grading_strategy_test.php
question/type/truefalse/tests/questiontype_test.php
rating/tests/externallib_test.php
rating/tests/rating_test.php
report/completion/tests/events_test.php
report/log/tests/events_test.php
report/log/tests/lib_test.php
report/loglive/tests/events_test.php
report/outline/tests/lib_test.php
report/questioninstances/tests/events_test.php
report/stats/tests/events_test.php
report/stats/tests/lib_test.php
report/usersessions/tests/lib_test.php
repository/flickr/tests/privacy_test.php
repository/nextcloud/tests/access_controlled_link_manager_test.php
repository/nextcloud/tests/lib_test.php
repository/nextcloud/tests/ocs_test.php
repository/onedrive/tests/privacy_test.php
repository/recent/tests/lib_test.php
repository/tests/privacy_test.php
rss/tests/privacy_test.php
search/engine/simpledb/tests/engine_test.php
search/engine/simpledb/tests/privacy_test.php
search/engine/solr/tests/engine_test.php
search/tests/base_activity_test.php
search/tests/base_test.php
search/tests/document_test.php
search/tests/engine_test.php
search/tests/external_test.php
search/tests/generator/lib.php
search/tests/manager_test.php
tag/tests/events_test.php
tag/tests/taglib_test.php
user/profile/field/checkbox/tests/privacy_test.php
user/profile/field/datetime/tests/privacy_test.php
user/profile/field/menu/tests/privacy_test.php
user/profile/field/text/tests/privacy_test.php
user/profile/field/textarea/tests/privacy_test.php
user/tests/search_test.php
webservice/tests/events_test.php
webservice/tests/externallib_test.php
webservice/tests/lib_test.php
webservice/tests/privacy_test.php
webservice/xmlrpc/tests/lib_test.php
webservice/xmlrpc/tests/locallib_test.php
webservice/xmlrpc/tests/xmlrpc_server_test.php

index 98ccd03..7162955 100644 (file)
@@ -87,7 +87,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
     /**
      * Setup test.
      */
-    public function setup() {
+    public function setUp(): void {
         global $CFG;
 
         $this->resetAfterTest();
index 30942ca..1e3e8d5 100644 (file)
@@ -36,7 +36,7 @@ class tool_capability_events_testcase extends advanced_testcase {
     /**
      * Setup testcase.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->setAdminUser();
         $this->resetAfterTest();
     }
index 2d99e5c..f023b71 100644 (file)
@@ -49,7 +49,7 @@ class tool_cohortroles_api_testcase extends advanced_testcase {
     /**
      * Setup function- we will create a course and add an assign instance to it.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest(true);
 
         // Create some users.
index 6399314..72446a2 100644 (file)
@@ -43,7 +43,7 @@ class tool_cohortroles_privacy_testcase extends \core_privacy\tests\provider_tes
     /**
      * Overriding setUp() function to always reset after tests.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index eec0bd5..32c4f56 100644 (file)
@@ -44,7 +44,7 @@ class tool_dataprivacy_expired_data_requests_testcase extends data_privacy_testc
     /**
      * Test tearDown.
      */
-    public function tearDown() {
+    public function tearDown(): void {
         \core_privacy\local\request\writer::reset();
     }
 
index 0dc614a..8ab8919 100644 (file)
@@ -39,7 +39,7 @@ class tool_dataprivacy_task_testcase extends data_privacy_testcase {
     /**
      * Test tearDown.
      */
-    public function tearDown() {
+    public function tearDown(): void {
         \core_privacy\local\request\writer::reset();
     }
 
index d2164cf..0633984 100644 (file)
@@ -36,7 +36,7 @@ class tool_langimport_events_testcase extends advanced_testcase {
     /**
      * Setup testcase.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->setAdminUser();
         $this->resetAfterTest();
     }
index 085ce1b..86b8b37 100644 (file)
@@ -50,7 +50,7 @@ require_once(__DIR__ . '/fixtures/event.php');
  */
 class logstore_database_privacy_testcase extends provider_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         global $CFG;
         $this->resetAfterTest();
         $this->preventResetByRollback(); // Logging waits till the transaction gets committed.
index d6d2a6c..ad0e992 100644 (file)
@@ -48,7 +48,7 @@ require_once(__DIR__ . '/fixtures/event.php');
  */
 class logstore_legacy_privacy_testcase extends provider_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index 1e23599..d324d79 100644 (file)
@@ -47,7 +47,7 @@ require_once(__DIR__ . '/fixtures/event.php');
  */
 class logstore_standard_privacy_testcase extends provider_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
         $this->preventResetByRollback(); // Logging waits till the transaction gets committed.
     }
index 1492718..aeb15c6 100644 (file)
@@ -540,7 +540,7 @@ class logstore_standard_store_testcase extends advanced_testcase {
     /**
      * Reset any garbage collector changes to the previous state at the end of the test.
      */
-    public function tearDown() {
+    public function tearDown(): void {
         if ($this->wedisabledgc) {
             gc_enable();
         }
index ae85b86..f697387 100644 (file)
@@ -51,7 +51,7 @@ require_once($CFG->dirroot . '/admin/tool/log/store/standard/tests/fixtures/even
  */
 class tool_log_privacy_testcase extends provider_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
         $this->preventResetByRollback(); // Logging waits till the transaction gets committed.
     }
index 8511c84..338bdeb 100644 (file)
@@ -75,7 +75,7 @@ class tool_lp_external_testcase extends externallib_advanced_testcase {
     /**
      * Setup function- we will create a course and add an assign instance to it.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB, $CFG;
 
         $this->resetAfterTest(true);
index c8db5c1..b133672 100644 (file)
@@ -46,7 +46,7 @@ class tool_lpmigrate_framework_processor_testcase extends advanced_testcase {
      * Then we create 2 courses, and in each 1 CM.
      * Then we attach some competencies from the first framework to courses and CM.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
         $dg = $this->getDataGenerator();
         $lpg = $dg->get_plugin_generator('core_competency');
index b88a742..0ba834c 100644 (file)
@@ -44,7 +44,7 @@ use tool_messageinbound\privacy\provider;
  */
 class tool_messageinbound_manager_testcase extends provider_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         global $CFG;
         $this->resetAfterTest();
 
index e98c027..65d6869 100644 (file)
@@ -46,7 +46,7 @@ use tool_messageinbound\privacy\provider;
  */
 class tool_messageinbound_privacy_testcase extends provider_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         global $CFG;
         $this->resetAfterTest();
 
index 6e1da9c..c1469f4 100644 (file)
@@ -40,7 +40,7 @@ class tool_mobile_privacy_testcase extends \core_privacy\tests\provider_testcase
     /**
      * Basic setup for these tests.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index f74c10b..35a90b4 100644 (file)
@@ -38,7 +38,7 @@ class tool_monitor_eventobservers_testcase extends advanced_testcase {
     /**
      * Set up method.
      */
-    public function setUp() {
+    public function setUp(): void {
         // Enable monitor.
         set_config('enablemonitor', 1, 'tool_monitor');
     }
index 4ed829d..109bc7f 100644 (file)
@@ -33,7 +33,7 @@ class tool_monitor_events_testcase extends advanced_testcase {
     /**
      * Tests set up.
      */
-    public function setUp() {
+    public function setUp(): void {
         set_config('enablemonitor', 1, 'tool_monitor');
         $this->resetAfterTest();
     }
index 7c4ae86..c2672a8 100644 (file)
@@ -39,7 +39,7 @@ class tool_monitor_generator_testcase extends advanced_testcase {
     /**
      * Set up method.
      */
-    public function setUp() {
+    public function setUp(): void {
         // Enable monitor.
         set_config('enablemonitor', 1, 'tool_monitor');
     }
@@ -138,4 +138,4 @@ class tool_monitor_generator_testcase extends advanced_testcase {
         $this->assertEquals(1, $historydata->userid);
         $this->assertEquals(1, $historydata->sid);
     }
-}
\ No newline at end of file
+}
index 3d419c7..871b11d 100644 (file)
@@ -43,7 +43,7 @@ class tool_monitor_privacy_testcase extends provider_testcase {
     /**
      * Set up method.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
         // Enable monitor.
         set_config('enablemonitor', 1, 'tool_monitor');
index a5173fa..fd334e6 100644 (file)
@@ -37,7 +37,7 @@ class tool_monitor_rule_manager_testcase extends advanced_testcase {
     /**
      * Set up method.
      */
-    public function setUp() {
+    public function setUp(): void {
         // Enable monitor.
         set_config('enablemonitor', 1, 'tool_monitor');
     }
@@ -192,4 +192,4 @@ class tool_monitor_rule_manager_testcase extends advanced_testcase {
         $this->assertEmpty(array_diff(array_keys($ruledata), $ruleids));
         $this->assertCount(10, $ruledata);
     }
-}
\ No newline at end of file
+}
index b3511b5..cb18815 100644 (file)
@@ -34,7 +34,7 @@ class tool_monitor_subscription_testcase extends advanced_testcase {
     /**
      * Test set up.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
 
         // Create the mock subscription.
index b526784..10a5e98 100644 (file)
@@ -36,7 +36,7 @@ class tool_monitor_task_check_subscriptions_testcase extends advanced_testcase {
     /**
      * Test set up.
      */
-    public function setUp() {
+    public function setUp(): void {
         global $DB;
         set_config('enablemonitor', 1, 'tool_monitor');
         $this->resetAfterTest(true);
index 9de3226..5f0a13c 100644 (file)
@@ -35,7 +35,7 @@ class tool_monitor_task_clean_events_testcase extends advanced_testcase {
     /**
      * Test set up.
      */
-    public function setUp() {
+    public function setUp(): void {
         set_config('enablemonitor', 1, 'tool_monitor');
         $this->resetAfterTest(true);
     }
index 8abff00..bec7faa 100644 (file)
@@ -46,7 +46,7 @@ class tool_policy_external_testcase extends externallib_advanced_testcase {
     /**
      * Setup function- we will create some policy docs.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
         $this->setAdminUser();
 
index c771cde..6fa4a0d 100644 (file)
@@ -51,7 +51,7 @@ class tool_policy_privacy_provider_testcase extends \core_privacy\tests\provider
     /**
      * Setup function. Will create a user.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest();
 
         $generator = $this->getDataGenerator();
index 0e24c31..199fca0 100644 (file)
@@ -46,7 +46,7 @@ class tool_recyclebin_category_bin_tests extends advanced_testcase {
     /**
      * Setup for each test.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest();
         $this->setAdminUser();
 
index 44f9db6..87c4e86 100644 (file)
@@ -46,7 +46,7 @@ class tool_recyclebin_course_bin_tests extends advanced_testcase {
     /**
      * Setup for each test.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest(true);
         $this->setAdminUser();
 
index c3ea0e6..c7be87b 100644 (file)
@@ -40,7 +40,7 @@ class tool_recyclebin_events_testcase extends advanced_testcase {
      *
      * This is executed before running any test in this file.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
 
         // We want the category and course bin to be enabled.
index 642ee0c..b60247f 100644 (file)
@@ -36,7 +36,7 @@ use tool_usertours\local\filter\accessdate;
  */
 class tool_usertours_accessdate_filter_test extends advanced_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index d5e4893..0deff0d 100644 (file)
@@ -47,7 +47,7 @@ class tool_usertours_manager_testcase extends advanced_testcase {
     /**
      * Setup to store the DB reference.
      */
-    public function setUp() {
+    public function setUp(): void {
         global $DB;
 
         $this->db = $DB;
@@ -56,7 +56,7 @@ class tool_usertours_manager_testcase extends advanced_testcase {
     /**
      * Tear down to restore the original DB reference.
      */
-    public function tearDown() {
+    public function tearDown(): void {
         global $DB;
 
         $DB = $this->db;
index 940e517..8fe9739 100644 (file)
@@ -58,7 +58,7 @@ class tool_usertours_role_filter_testcase extends advanced_testcase {
      */
     protected $roles;
 
-    public function setUp() {
+    public function setUp(): void {
         global $DB;
 
         $this->resetAfterTest(true);
index 005272e..fb6a3c1 100644 (file)
@@ -44,7 +44,7 @@ class step_testcase extends advanced_testcase {
     /**
      * Setup to store the DB reference.
      */
-    public function setUp() {
+    public function setUp(): void {
         global $DB;
 
         $this->db = $DB;
@@ -53,7 +53,7 @@ class step_testcase extends advanced_testcase {
     /**
      * Tear down to restore the original DB reference.
      */
-    public function tearDown() {
+    public function tearDown(): void {
         global $DB;
 
         $DB = $this->db;
index c5bf23f..d1014f7 100644 (file)
@@ -46,7 +46,7 @@ class tour_testcase extends advanced_testcase {
     /**
      * Setup to store the DB reference.
      */
-    public function setUp() {
+    public function setUp(): void {
         global $DB;
 
         $this->db = $DB;
@@ -55,7 +55,7 @@ class tour_testcase extends advanced_testcase {
     /**
      * Tear down to restore the original DB reference.
      */
-    public function tearDown() {
+    public function tearDown(): void {
         global $DB;
 
         $DB = $this->db;
index 00cfea3..13010f9 100644 (file)
@@ -33,7 +33,7 @@ defined('MOODLE_INTERNAL') || die();
  */
 class core_analytics_course_testcase extends advanced_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         global $DB;
 
         $this->course = $this->getDataGenerator()->create_course(['startdate' => 0]);
index f5c916c..fb600c8 100644 (file)
@@ -38,7 +38,7 @@ class dataset_manager_testcase extends advanced_testcase {
      *
      * @return null
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
 
         $this->sharedtoprows = array(
index 9d5bdd2..a1004fb 100644 (file)
@@ -41,7 +41,7 @@ require_once(__DIR__ . '/fixtures/test_analysis.php');
  */
 class analytics_model_testcase extends advanced_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
 
         $this->setAdminUser();
 
index f376878..f8f2866 100644 (file)
@@ -39,7 +39,7 @@ class analytics_prediction_actions_testcase extends advanced_testcase {
     /**
      * Common startup tasks
      */
-    public function setUp() {
+    public function setUp(): void {
         global $DB;
 
         $this->setAdminUser();
index 1fd9ac2..159bfd8 100644 (file)
@@ -62,7 +62,7 @@ class core_analytics_prediction_testcase extends advanced_testcase {
      *
      * @return null
      */
-    public function tearDown() {
+    public function tearDown(): void {
         $this->setAdminUser();
 
         $models = \core_analytics\manager::get_all_models();
index 21fdd28..d0fa280 100644 (file)
@@ -44,7 +44,7 @@ require_once(__DIR__ . '/fixtures/test_target_course_users.php');
  */
 class core_analytics_privacy_model_testcase extends \core_privacy\tests\provider_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
 
         $this->resetAfterTest(true);
         $this->setAdminUser();
index 5054a3f..5d3cb03 100644 (file)
@@ -39,7 +39,7 @@ class analytics_stats_testcase extends advanced_testcase {
     /**
      * Set up the test environment.
      */
-    public function setUp() {
+    public function setUp(): void {
 
         $this->setAdminUser();
     }
index f62f95c..ed1900a 100644 (file)
@@ -43,7 +43,7 @@ class auth_email_external_testcase extends externallib_advanced_testcase {
     /**
      * Set up for every test
      */
-    public function setUp() {
+    public function setUp(): void {
         global $CFG, $DB;
 
         $this->resetAfterTest(true);
index 156a997..7857e34 100644 (file)
@@ -44,7 +44,7 @@ class auth_manual_testcase extends advanced_testcase {
     /**
      * Setup test data.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest(true);
         $this->authplugin = new auth_plugin_manual();
         set_config('expiration', '1', 'auth_manual');
index 5572291..af741c3 100644 (file)
@@ -45,7 +45,7 @@ class auth_manual_privacy_testcase extends \core_privacy\tests\provider_testcase
     /**
      * Basic setup for these tests.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
         $this->authplugin = new auth_plugin_manual();
     }
index 7ecf9e6..b41db22 100644 (file)
@@ -43,7 +43,7 @@ class auth_mnet_privacy_testcase extends provider_testcase {
     /**
      * Set up method.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
         $this->setAdminUser();
     }
index d582d0a..1e190cf 100644 (file)
@@ -42,7 +42,7 @@ class auth_oauth2_privacy_testcase extends provider_testcase {
     /**
      * Set up method.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
         $this->setAdminUser();
     }
index 237852c..f3f53d5 100644 (file)
@@ -46,7 +46,7 @@ class core_auth_external_testcase extends externallib_advanced_testcase {
     /**
      * Set up for every test
      */
-    public function setUp() {
+    public function setUp(): void {
         global $CFG;
 
         $this->resetAfterTest(true);
@@ -60,7 +60,7 @@ class core_auth_external_testcase extends externallib_advanced_testcase {
     /**
      * Tear down to restore old logging..
      */
-    protected function tearDown() {
+    protected function tearDown(): void {
         ini_set('error_log', $this->oldlog);
         parent::tearDown();
     }
index 16701f2..d3586d3 100644 (file)
@@ -43,7 +43,7 @@ use core_auth\privacy\provider;
  */
 class core_auth_privacy_testcase extends provider_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index b1a0040..02d8476 100644 (file)
@@ -52,7 +52,7 @@ class availability_completion_condition_testcase extends advanced_testcase {
     /**
      * Load required classes.
      */
-    public function setUp() {
+    public function setUp(): void {
         availability_completion\condition::wipe_static_cache();
     }
 
index 7b62904..ae2b2a7 100644 (file)
@@ -38,7 +38,7 @@ class availability_date_condition_testcase extends advanced_testcase {
     /**
      * Load required classes.
      */
-    public function setUp() {
+    public function setUp(): void {
         // Load the mock info class so that it can be used.
         global $CFG;
         require_once($CFG->dirroot . '/availability/tests/fixtures/mock_info.php');
index 1c94323..2f7d618 100644 (file)
@@ -37,7 +37,7 @@ class availability_group_condition_testcase extends advanced_testcase {
     /**
      * Load required classes.
      */
-    public function setUp() {
+    public function setUp(): void {
         // Load the mock info class so that it can be used.
         global $CFG;
         require_once($CFG->dirroot . '/availability/tests/fixtures/mock_info.php');
index 94cd4ec..5f5f776 100644 (file)
@@ -37,7 +37,7 @@ class availability_grouping_condition_testcase extends advanced_testcase {
     /**
      * Load required classes.
      */
-    public function setUp() {
+    public function setUp(): void {
         // Load the mock info class so that it can be used.
         global $CFG;
         require_once($CFG->dirroot . '/availability/tests/fixtures/mock_info.php');
index e2250e2..c6dc98d 100644 (file)
@@ -45,7 +45,7 @@ class availability_profile_condition_testcase extends advanced_testcase {
     /** @var \core_availability\info Current info */
     private $info;
 
-    public function setUp() {
+    public function setUp(): void {
         global $DB, $CFG;
 
         $this->resetAfterTest();
index e8044df..befd3ff 100644 (file)
@@ -36,7 +36,7 @@ use core_availability\info_section;
  * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 class info_testcase extends advanced_testcase {
-    public function setUp() {
+    public function setUp(): void {
         // Load the mock condition so that it can be used.
         require_once(__DIR__ . '/fixtures/mock_condition.php');
     }
index 1b131a9..2760c58 100644 (file)
@@ -35,7 +35,7 @@ defined('MOODLE_INTERNAL') || die();
  * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 class tree_testcase extends \advanced_testcase {
-    public function setUp() {
+    public function setUp(): void {
         // Load the mock classes so they can be used.
         require_once(__DIR__ . '/fixtures/mock_condition.php');
         require_once(__DIR__ . '/fixtures/mock_info.php');
index f638be0..5c93fb6 100644 (file)
@@ -38,7 +38,7 @@ class core_backup_controller_testcase extends advanced_testcase {
     protected $courseid;  // course id used for testing
     protected $userid;    // user used if for testing
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB, $CFG;
 
         $this->resetAfterTest(true);
index 68589da..7065286 100644 (file)
@@ -41,7 +41,7 @@ class core_backup_moodle1_converter_testcase extends advanced_testcase {
     /** @var string saved hash of an icon file used during testing */
     protected $iconhash;
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $CFG;
 
         $this->tempdir = convert_helper::generate_id('unittest');
@@ -75,7 +75,7 @@ class core_backup_moodle1_converter_testcase extends advanced_testcase {
         );
     }
 
-    protected function tearDown() {
+    protected function tearDown(): void {
         global $CFG;
         if (empty($CFG->keeptempdirectoriesonbackup)) {
             fulldelete($this->tempdirpath);
index 5a609c8..d8397e0 100644 (file)
@@ -31,7 +31,7 @@ require_once($CFG->dirroot . '/backup/moodle2/backup_custom_fields.php');
 
 class core_backup_encrypted_content_testscase extends advanced_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         if (!function_exists('openssl_encrypt')) {
             $this->markTestSkipped('OpenSSL extension is not loaded.');
 
index 554aa0b..658f6c1 100644 (file)
@@ -42,7 +42,7 @@ class backup_xml_transformer_testcase extends advanced_testcase {
     /**
      * Initial set up.
      */
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
 
         $this->resetAfterTest(true);
index 9ac514f..c76d952 100644 (file)
@@ -46,7 +46,7 @@ class core_backup_automated_backup_testcase extends advanced_testcase {
      */
     protected $course;
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB, $CFG;
 
         $this->resetAfterTest(true);
index 906dbfe..40f2168 100644 (file)
@@ -66,7 +66,7 @@ class core_backup_course_copy_testcase extends advanced_testcase {
     /**
      * Set up tasks for all tests.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB, $CFG, $USER;
 
         $this->resetAfterTest(true);
@@ -633,4 +633,4 @@ class core_backup_course_copy_testcase extends advanced_testcase {
         $this->expectException(\moodle_exception::class);
         new \core_backup\copy\copy($formdata);
     }
-}
\ No newline at end of file
+}
index 8cb6e4d..df2b153 100644 (file)
@@ -44,7 +44,7 @@ class backup_external_testcase extends externallib_advanced_testcase {
     /**
      * Set up tasks for all tests.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $CFG;
 
         $this->resetAfterTest(true);
@@ -179,4 +179,4 @@ class backup_external_testcase extends externallib_advanced_testcase {
         $this->assertEquals(0, $restorerec->progress);
         $this->assertEquals('restore', $restorerec->operation);
     }
-}
\ No newline at end of file
+}
index 8e4ced3..3b89043 100644 (file)
@@ -39,7 +39,7 @@ class backup_check_testcase extends advanced_testcase {
     protected $courseid;  // course id used for testing
     protected $userid;    // user record id
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB, $CFG;
         parent::setUp();
 
index 219b530..7a18eb0 100644 (file)
@@ -37,7 +37,7 @@ class backup_dbops_testcase extends advanced_testcase {
     protected $courseid;  // course id used for testing
     protected $userid;      // user record used for testing
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB, $CFG;
         parent::setUp();
 
index 2a3fa1a..7cebfc5 100644 (file)
@@ -40,7 +40,7 @@ require_once($CFG->dirroot . '/backup/util/factories/backup_factory.class.php');
  */
 class backup_factories_testcase extends advanced_testcase {
 
-    function setUp() {
+    public function setUp(): void {
         global $CFG;
         parent::setUp();
 
@@ -57,7 +57,7 @@ class backup_factories_testcase extends advanced_testcase {
     /**
      * test get_logger_chain() method
      */
-    function test_backup_factory() {
+    public function test_backup_factory() {
         global $CFG;
 
         // Default instantiate, all levels = backup::LOG_NONE
index ea5ab2c..a5239a1 100644 (file)
@@ -42,7 +42,7 @@ class restore_structure_parser_processor_test extends advanced_testcase {
     /**
      * Initial set up.
      */
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
 
         $this->resetAfterTest(true);
index 3e9b2fe..a025736 100644 (file)
@@ -36,7 +36,7 @@ class backup_plan_testcase extends advanced_testcase {
     protected $courseid;  // course id used for testing
     protected $userid;      // user record used for testing
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB, $CFG;
         parent::setUp();
 
index 8b9f6e1..78d1a0c 100644 (file)
@@ -36,7 +36,7 @@ class backup_step_testcase extends advanced_testcase {
     protected $courseid;  // course id used for testing
     protected $userid;      // user record used for testing
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB, $CFG;
         parent::setUp();
 
index 40ee6ea..3cb0bef 100644 (file)
@@ -36,7 +36,7 @@ class backup_task_testcase extends advanced_testcase {
     protected $courseid;  // course id used for testing
     protected $userid;      // user record used for testing
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB, $CFG;
         parent::setUp();
 
index 93a51a7..3b9b807 100644 (file)
@@ -53,7 +53,7 @@ class backup_structure_testcase extends advanced_testcase {
     protected $contextid;
 
 
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
 
         $this->resetAfterTest(true);
index c4ec0e1..0dac8de 100644 (file)
@@ -43,7 +43,7 @@ class core_badges_badgeslib_testcase extends advanced_testcase {
     /** @var $assertion2 to define json format for Open badge version 2 */
     protected $assertion2;
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB, $CFG;
         $this->resetAfterTest(true);
         $CFG->enablecompletion = true;
index 0be1526..7b36cae 100644 (file)
@@ -45,7 +45,7 @@ class core_badges_external_testcase extends externallib_advanced_testcase {
     /**
      * Set up for every test
      */
-    public function setUp() {
+    public function setUp(): void {
         global $DB;
         $this->resetAfterTest();
         $this->setAdminUser();
index 8067b4e..12b8638 100644 (file)
@@ -48,7 +48,7 @@ require_once($CFG->libdir . '/badgeslib.php');
  */
 class core_badges_privacy_testcase extends provider_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index 1e7aae4..55db08c 100644 (file)
@@ -43,7 +43,7 @@ class block_comments_events_testcase extends advanced_testcase {
     /**
      * Setup test data.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
         $this->setAdminUser();
 
index 363245d..1962cc1 100644 (file)
@@ -51,7 +51,7 @@ class block_comments_privacy_provider_testcase extends \core_privacy\tests\provi
     /** @var stdClass A test course. */
     protected $course2;
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB;
 
         $this->resetAfterTest();
index da436ac..9f56f5f 100644 (file)
@@ -47,7 +47,7 @@ class block_online_users_testcase extends advanced_testcase {
      * Prepare the site with some courses, groups, users and
      * simulate various recent accesses.
      */
-    protected function setUp() {
+    protected function setUp(): void {
 
         // Generate (simulated) recently logged-in users.
         $generator = $this->getDataGenerator()->get_plugin_generator('block_online_users');
index fcc0eb2..6600709 100644 (file)
@@ -42,7 +42,7 @@ class block_recentlyaccesseditems_observer_testcase extends advanced_testcase {
     /**
      * Set up for every test
      */
-    public function setUp() {
+    public function setUp(): void {
         global $DB;
 
         $this->resetAfterTest();
@@ -157,4 +157,4 @@ class block_recentlyaccesseditems_observer_testcase extends advanced_testcase {
         $records = $DB->count_records($this->table, array('cmid' => $this->chat->cmid));
         $this->assertEquals(2, $records);
     }
-}
\ No newline at end of file
+}
index 68035ef..b3e9c1b 100644 (file)
@@ -38,7 +38,7 @@ class block_rss_client_testcase extends provider_testcase {
     /**
      * Basic setup for these tests.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index 20df9d9..4695a70 100644 (file)
@@ -44,7 +44,7 @@ use core_block\privacy\provider;
  */
 class core_block_privacy_testcase extends provider_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index 87f265d..e6105c8 100644 (file)
@@ -58,7 +58,7 @@ class core_blog_events_testcase extends advanced_testcase {
     /**
      * Setup the tests.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB;
         parent::setUp();
 
index 5e23d5f..7887a69 100644 (file)
@@ -44,7 +44,7 @@ class core_blog_external_testcase extends advanced_testcase {
     private $tagid;
     private $postid;
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB, $CFG;
         parent::setUp();
 
index a690e63..f1ad921 100644 (file)
@@ -41,7 +41,7 @@ class core_blog_lib_testcase extends advanced_testcase {
     private $tagid;
     private $postid;
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB;
         parent::setUp();
 
index 7002552..67de624 100644 (file)
@@ -47,7 +47,7 @@ require_once($CFG->dirroot . '/comment/lib.php');
  */
 class core_blog_privacy_testcase extends provider_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index 4903d3b..108015b 100644 (file)
@@ -45,11 +45,11 @@ class cachestore_apcu_test extends cachestore_tests {
         return 'cachestore_apcu';
     }
 
-    public function setUp() {
+    public function setUp(): void {
         if (!cachestore_apcu::are_requirements_met()) {
             $this->markTestSkipped('Could not test cachestore_apcu. Requirements are not met.');
         }
-        return parent::setUp();
+        parent::setUp();
     }
 
     /**
@@ -88,4 +88,4 @@ class cachestore_apcu_test extends cachestore_tests {
         $instance2->purge();
         $this->assertSame(1, $instance->get('test1'));
     }
-}
\ No newline at end of file
+}
index 7f8876e..b7178b3 100644 (file)
@@ -45,7 +45,7 @@ class cachestore_redis_compressor_test extends advanced_testcase {
     /**
      * Test set up
      */
-    public function setUp() {
+    public function setUp(): void {
         if (!cachestore_redis::are_requirements_met() || !defined('TEST_CACHESTORE_REDIS_TESTSERVERS')) {
             $this->markTestSkipped('Could not test cachestore_redis. Requirements are not met.');
         }
index 1fa3861..a4bd9f7 100644 (file)
@@ -54,13 +54,13 @@ class cachestore_redis_test extends cachestore_tests {
         return 'cachestore_redis';
     }
 
-    public function setUp() {
+    public function setUp(): void {
         if (!cachestore_redis::are_requirements_met() || !defined('TEST_CACHESTORE_REDIS_TESTSERVERS')) {
             $this->markTestSkipped('Could not test cachestore_redis. Requirements are not met.');
         }
         parent::setUp();
     }
-    protected function tearDown() {
+    protected function tearDown(): void {
         parent::tearDown();
 
         if ($this->store instanceof cachestore_redis) {
@@ -123,4 +123,4 @@ class cachestore_redis_test extends cachestore_tests {
         $this->assertFalse($store->release_lock('lock', '321'));
         $this->assertTrue($store->release_lock('lock', '123'));
     }
-}
\ No newline at end of file
+}
index 146efe3..b392114 100644 (file)
@@ -45,7 +45,7 @@ class core_cache_administration_helper_testcase extends advanced_testcase {
     /**
      * Set things back to the default before each test.
      */
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
         cache_factory::reset();
         cache_config_testing::create_default_configuration();
@@ -54,7 +54,7 @@ class core_cache_administration_helper_testcase extends advanced_testcase {
     /**
      * Final task is to reset the cache system
      */
-    public static function tearDownAfterClass() {
+    public static function tearDownAfterClass(): void {
         parent::tearDownAfterClass();
         cache_factory::reset();
     }
index faafe51..ffbb1df 100644 (file)
@@ -44,7 +44,7 @@ class core_cache_testcase extends advanced_testcase {
     /**
      * Set things back to the default before each test.
      */
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
         cache_factory::reset();
         cache_config_testing::create_default_configuration();
@@ -53,7 +53,7 @@ class core_cache_testcase extends advanced_testcase {
     /**
      * Final task is to reset the cache system
      */
-    public static function tearDownAfterClass() {
+    public static function tearDownAfterClass(): void {
         parent::tearDownAfterClass();
         cache_factory::reset();
     }
index 12b6291..8fa28d7 100644 (file)
@@ -44,7 +44,7 @@ class core_cache_config_writer_testcase extends advanced_testcase {
     /**
      * Set things back to the default before each test.
      */
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
         cache_factory::reset();
         cache_config_testing::create_default_configuration();
@@ -53,7 +53,7 @@ class core_cache_config_writer_testcase extends advanced_testcase {
     /**
      * Final task is to reset the cache system
      */
-    public static function tearDownAfterClass() {
+    public static function tearDownAfterClass(): void {
         parent::tearDownAfterClass();
         cache_factory::reset();
     }
index 0eee86e..b266f4e 100644 (file)
@@ -46,7 +46,7 @@ abstract class cachestore_tests extends advanced_testcase {
      * Sets up the fixture, for example, open a network connection.
      * This method is called before a test is executed.
      */
-    public function setUp() {
+    public function setUp(): void {
         $class = $this->get_class_name();
         if (!class_exists($class) || !$class::are_requirements_met()) {
             $this->markTestSkipped('Could not test '.$class.'. Requirements are not met.');
@@ -190,4 +190,4 @@ abstract class cachestore_tests extends advanced_testcase {
         $this->assertSame(3, $instance->delete_many(array('many2', 'many3', 'many4')));
         $this->assertSame(2, $instance->delete_many(array('many1', 'many5', 'many6')));
     }
-}
\ No newline at end of file
+}
index 859acc6..487010b 100644 (file)
@@ -58,7 +58,7 @@ class core_calendar_type_testcase extends advanced_testcase {
     /**
      * Test set up.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         // The user we are going to test this on.
         $this->user = self::getDataGenerator()->create_user();
         self::setUser($this->user);
index bc72391..7e00429 100644 (file)
@@ -47,7 +47,7 @@ class core_calendar_container_testcase extends advanced_testcase {
     /**
      * Test setup.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
         $this->setAdminUser();
     }
index f0799db..0238db8 100644 (file)
@@ -41,7 +41,7 @@ class core_calendar_events_related_objects_cache_testcase extends advanced_testc
     /**
      * Tests set up
      */
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest();
     }
 
index 76f8374..fdb852c 100644 (file)
@@ -48,7 +48,7 @@ class core_calendar_events_testcase extends advanced_testcase {
     /**
      * Test set up.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $USER;
         // The user we are going to test this on.
         $this->setAdminUser();
index 1d41b31..d39596d 100644 (file)
@@ -43,7 +43,7 @@ class core_calendar_externallib_testcase extends externallib_advanced_testcase {
     /**
      * Tests set up
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $CFG;
         require_once($CFG->dirroot . '/calendar/externallib.php');
     }
index 4467337..d390125 100644 (file)
@@ -38,7 +38,7 @@ class core_calendar_lib_testcase extends advanced_testcase {
     /**
      * Tests set up
      */
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest();
     }
 
index 70f56d9..987de7b 100644 (file)
@@ -40,7 +40,7 @@ class core_calendar_local_api_testcase extends advanced_testcase {
     /**
      * Tests set up
      */
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest();
     }
 
index 2b7157d..d2173b7 100644 (file)
@@ -47,7 +47,7 @@ class core_calendar_privacy_testcase extends provider_testcase {
     /**
      * Overriding setUp() function to always reset after tests.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index a6e2531..3af4655 100644 (file)
@@ -46,7 +46,7 @@ class core_calendar_rrule_manager_testcase extends advanced_testcase {
     /**
      * Set up method.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB;
         $this->resetAfterTest();
 
index dc9f25e..5d175ac 100644 (file)
@@ -38,7 +38,7 @@ class core_calendar_std_proxy_testcase extends advanced_testcase {
      */
     public $objects;
 
-    public function setUp() {
+    public function setUp(): void {
         $this->objects = [
             1 => (object) [
                 'member1' => 'Hello',
index 71a7080..c6c04bc 100644 (file)
@@ -42,7 +42,7 @@ class core_cohort_testcase extends provider_testcase {
     /**
      * Basic setup for these tests.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index b947392..ea79b8b 100644 (file)
@@ -44,7 +44,7 @@ class core_comment_externallib_testcase extends externallib_advanced_testcase {
     /**
      * Tests set up
      */
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest();
     }
 
index b2f7ba2..45e2093 100644 (file)
@@ -38,7 +38,7 @@ use \core_privacy\tests\provider_testcase;
  */
 class core_comment_privacy_testcase extends provider_testcase {
 
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest();
     }
 
index 83247d1..c2ddd7a 100644 (file)
@@ -96,7 +96,7 @@ class core_competency_external_testcase extends externallib_advanced_testcase {
     /**
      * Setup function- we will create a course and add an assign instance to it.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB, $CFG;
 
         $this->resetAfterTest(true);
index 7105753..4ba3a69 100644 (file)
@@ -48,7 +48,7 @@ use core_competency\privacy\provider;
  */
 class core_competency_privacy_testcase extends provider_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         global $PAGE;
         $this->resetAfterTest();
 
index af24c07..984393e 100644 (file)
@@ -38,7 +38,7 @@ class core_completion_api_testcase extends advanced_testcase {
     /**
      * Test setup.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index fded219..2ea3061 100644 (file)
@@ -38,7 +38,7 @@ class core_completion_progress_testcase extends advanced_testcase {
     /**
      * Test setup.
      */
-    public function setUp() {
+    public function setUp(): void {
         global $CFG;
 
         $CFG->enablecompletion = true;
index 6e737d6..5d833b1 100644 (file)
@@ -36,7 +36,7 @@ class format_weeks_observer_testcase extends advanced_testcase {
     /**
      * Test setup.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index 52af5f7..4a5cb9a 100644 (file)
@@ -37,7 +37,7 @@ use PHPUnit\Framework\MockObject\MockObject;
  */
 class core_course_category_hooks_testcase extends \advanced_testcase {
 
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest();
         $this->setAdminUser();
     }
index 8d8b26b..5d463de 100644 (file)
@@ -32,7 +32,7 @@ class core_course_category_testcase extends advanced_testcase {
 
     protected $roles;
 
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
         $this->resetAfterTest();
         $user = $this->getDataGenerator()->create_user();
index 2673208..64d5ba0 100644 (file)
@@ -40,7 +40,7 @@ class core_course_customfield_testcase extends advanced_testcase {
     /**
      * Set up
      */
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
         $this->resetAfterTest();
         $this->setAdminUser();
index 0a30f7e..c3a9618 100644 (file)
@@ -36,7 +36,7 @@ class core_course_events_testcase extends advanced_testcase {
     /**
      * Tests set up
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $CFG;
         require_once($CFG->dirroot . '/course/lib.php');
         $this->resetAfterTest();
index 015632f..648463d 100644 (file)
@@ -42,7 +42,7 @@ class core_course_externallib_testcase extends externallib_advanced_testcase {
     /**
      * Tests set up
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $CFG;
         require_once($CFG->dirroot . '/course/externallib.php');
     }
index 6d03d48..f438ee1 100644 (file)
@@ -53,7 +53,7 @@ class course_search_testcase extends advanced_testcase {
      */
     protected $customfieldareaid = null;
 
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
         set_config('enableglobalsearch', true);
 
index 4d94533..9a64267 100644 (file)
@@ -48,7 +48,7 @@ class customfield_checkbox_plugin_testcase extends advanced_testcase {
     /**
      * Tests set up.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
 
         $this->cfcat = $this->get_generator()->create_category();
index 7498722..4d52f3b 100644 (file)
@@ -48,7 +48,7 @@ class customfield_date_plugin_testcase extends advanced_testcase {
     /**
      * Tests set up.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
 
         $this->cfcat = $this->get_generator()->create_category();
@@ -206,4 +206,4 @@ class customfield_date_plugin_testcase extends advanced_testcase {
     public function test_delete() {
         $this->cfcat->get_handler()->delete_all();
     }
-}
\ No newline at end of file
+}
index 6def88c..937c2b8 100644 (file)
@@ -48,7 +48,7 @@ class customfield_select_plugin_testcase extends advanced_testcase {
     /**
      * Tests set up.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
 
         $this->cfcat = $this->get_generator()->create_category();
@@ -199,4 +199,4 @@ class customfield_select_plugin_testcase extends advanced_testcase {
     public function test_delete() {
         $this->cfcat->get_handler()->delete_all();
     }
-}
\ No newline at end of file
+}
index 8f62564..b52d5a0 100644 (file)
@@ -48,7 +48,7 @@ class customfield_text_plugin_testcase extends advanced_testcase {
     /**
      * Tests set up.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
 
         $this->cfcat = $this->get_generator()->create_category();
index a5cc62d..9c48ad6 100644 (file)
@@ -48,7 +48,7 @@ class customfield_textarea_plugin_testcase extends advanced_testcase {
     /**
      * Tests set up.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
 
         $this->cfcat = $this->get_generator()->create_category();
index 2619389..0b99cfc 100644 (file)
@@ -63,7 +63,7 @@ class enrol_flatfile_privacy_testcase extends provider_testcase {
     /**
      * Called before every test.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index b0b783a..bbf3e25 100644 (file)
@@ -47,7 +47,7 @@ class enrol_imsenterprise_testcase extends advanced_testcase {
     /**
      * Setup required for all tests.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest(true);
         $this->imsplugin = enrol_get_plugin('imsenterprise');
         $this->set_test_config();
index b01009e..48cb7f0 100644 (file)
@@ -49,7 +49,7 @@ class enrol_lti_data_connector_testcase extends advanced_testcase {
      *
      * This is executed before running any test in this file.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
 
         // Set this user as the admin.
index a601cf7..75a79cd 100644 (file)
@@ -48,7 +48,7 @@ class enrol_lti_helper_testcase extends advanced_testcase {
      *
      * This is executed before running any test in this file.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
 
         // Set this user as the admin.
index 3e47866..5e9873d 100644 (file)
@@ -45,7 +45,7 @@ class enrol_lti_testcase extends advanced_testcase {
      *
      * This is executed before running any tests in this file.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
         $this->setAdminUser();
     }
index 984684c..dfcaf07 100644 (file)
@@ -58,7 +58,7 @@ class enrol_lti_privacy_provider_testcase extends \core_privacy\tests\provider_t
     /**
      * Basic setup for these tests.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
 
         $this->course = $this->getDataGenerator()->create_course();
index f9061bf..e39ec7a 100644 (file)
@@ -59,7 +59,7 @@ class sync_members_testcase extends advanced_testcase {
     /** @var  ResourceLink $resourcelink */
     protected $resourcelink;
 
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
 
         // Set this user as the admin.
index 5e673ec..c05987b 100644 (file)
@@ -53,7 +53,7 @@ class tool_provider_testcase extends advanced_testcase {
      *
      * This is executed before running any tests in this file.
      */
-    public function setUp() {
+    public function setUp(): void {
         global $SESSION;
         $this->resetAfterTest();
 
index f107787..25b6304 100644 (file)
@@ -79,7 +79,7 @@ class enrol_paypal_privacy_provider_testcase extends \core_privacy\tests\provide
     /** @var stdClass A test course with 2 enrolments for student2 and student12. */
     protected $course3;
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB;
 
         $this->resetAfterTest();
index 6954ff3..11c4146 100644 (file)
@@ -54,7 +54,7 @@ class core_course_enrolment_manager_testcase extends advanced_testcase {
     /**
      * Tests set up
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $CFG;
         require_once($CFG->dirroot . '/enrol/locallib.php');
         $this->setAdminUser();
index 4009101..64b63f3 100644 (file)
@@ -35,7 +35,7 @@ class core_enrol_role_external_testcase extends externallib_advanced_testcase {
     /**
      * Tests set up
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $CFG;
         require_once($CFG->dirroot . '/enrol/externallib.php');
     }
@@ -142,4 +142,4 @@ class core_enrol_role_external_testcase extends externallib_advanced_testcase {
         $categories = core_role_external::unassign_roles(
             array('roleid' => 3, 'userid' => $USER->id, 'contextid' => $context->id));
     }
-}
\ No newline at end of file
+}
index b444007..cb56a43 100644 (file)
@@ -33,7 +33,7 @@ defined('MOODLE_INTERNAL') || die();
  */
 class component_favourite_service_testcase extends advanced_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index 7507689..d7fe912 100644 (file)
@@ -37,7 +37,7 @@ use \core_privacy\local\request\transform;
  */
 class privacy_test extends provider_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index b95451a..d06868a 100644 (file)
@@ -36,7 +36,7 @@ use \core_favourites\local\entity\favourite;
  */
 class favourite_repository_testcase extends advanced_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index be372f1..f44a638 100644 (file)
@@ -33,7 +33,7 @@ defined('MOODLE_INTERNAL') || die();
  */
 class user_favourite_service_testcase extends advanced_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index 529b9ec..fbb8a7b 100644 (file)
@@ -44,7 +44,7 @@ class filter_algebra_testcase extends basic_testcase {
 
     protected $filter;
 
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
         $this->filter = new filter_algebra(context_system::instance(), array());
     }
index 467118c..cb9b3c4 100644 (file)
@@ -36,7 +36,7 @@ require_once($CFG->dirroot.'/filter/displayh5p/filter.php');
  */
 class filter_displayh5p_testcase extends advanced_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
 
         $this->resetAfterTest(true);
@@ -96,4 +96,4 @@ class filter_displayh5p_testcase extends advanced_testcase {
                     "\"[^>]*?></iframe>#"]
         ];
     }
-}
\ No newline at end of file
+}
index 2dbefa2..65d8ce3 100644 (file)
@@ -34,7 +34,7 @@ defined('MOODLE_INTERNAL') || die();
  */
 class filter_multilang_filter_testcase extends advanced_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
 
         $this->resetAfterTest(true);
index 1b03725..02dc2b1 100644 (file)
@@ -40,7 +40,7 @@ class filter_tex_testcase extends advanced_testcase {
 
     protected $filter;
 
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
         $this->resetAfterTest(true);
         $this->filter = new filter_tex(context_system::instance(), array());
index 67ce658..b2e9792 100644 (file)
@@ -36,7 +36,7 @@ class ods_logging_events_testcase extends advanced_testcase {
     /**
      * Setup is called before calling test case.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
@@ -62,4 +62,4 @@ class ods_logging_events_testcase extends advanced_testcase {
         $this->assertEquals(context_course::instance($course->id), $event->get_context());
         $this->assertEquals('ods', $event->get_export_type());
     }
-}
\ No newline at end of file
+}
index e2e7d7b..3bc0b49 100644 (file)
@@ -36,7 +36,7 @@ class txt_logging_events_testcase extends advanced_testcase {
     /**
      * Setup is called before calling test case.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
@@ -62,4 +62,4 @@ class txt_logging_events_testcase extends advanced_testcase {
         $this->assertEquals(context_course::instance($course->id), $event->get_context());
         $this->assertEquals('txt', $event->get_export_type());
     }
-}
\ No newline at end of file
+}
index fcada0d..277481f 100644 (file)
@@ -36,7 +36,7 @@ class xls_logging_events_testcase extends advanced_testcase {
     /**
      * Setup is called before calling test case.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
@@ -62,4 +62,4 @@ class xls_logging_events_testcase extends advanced_testcase {
         $this->assertEquals(context_course::instance($course->id), $event->get_context());
         $this->assertEquals('xls', $event->get_export_type());
     }
-}
\ No newline at end of file
+}
index b88d953..2e82632 100644 (file)
@@ -36,7 +36,7 @@ class xml_logging_events_testcase extends advanced_testcase {
     /**
      * Setup is called before calling test case.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
@@ -62,4 +62,4 @@ class xml_logging_events_testcase extends advanced_testcase {
         $this->assertEquals(context_course::instance($course->id), $event->get_context());
         $this->assertEquals('xml', $event->get_export_type());
     }
-}
\ No newline at end of file
+}
index f05e8b5..b49d16e 100644 (file)
@@ -64,7 +64,7 @@ Bobby,Bunce,,"Moodle HQ","Rock on!",student5@example.com,75.00,,75.00,{exportdat
     /** @var array $columns The first row of the csv file. These are the columns of the import file.*/
     protected $columns;
 
-    public function tearDown() {
+    public function tearDown(): void {
         $this->csvimport = null;
     }
 
index 2be84b6..5c8b7a5 100644 (file)
@@ -42,7 +42,7 @@ class gradereport_grader_privacy_testcase extends \core_privacy\tests\provider_t
     /**
      * Basic setup for these tests.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index 1301c0d..592c019 100644 (file)
@@ -42,7 +42,7 @@ class gradereport_overview_externallib_testcase extends externallib_advanced_tes
     /**
      * Set up for every test
      */
-    public function setUp() {
+    public function setUp(): void {
         global $DB;
         $this->resetAfterTest(true);
 
index 7af1a71..f1ff2e8 100644 (file)
@@ -51,7 +51,7 @@ class gradereport_user_lib_testcase extends advanced_testcase {
      */
     private $tree;
 
-    public function setUp() {
+    public function setUp(): void {
         $this->user = $this->getDataGenerator()->create_user();
         $this->course = $this->getDataGenerator()->create_course();
         $this->tree = new \core_user\output\myprofile\tree();
index b44f8bf..49a5425 100644 (file)
@@ -40,7 +40,7 @@ class gradereport_user_privacy_testcase extends \core_privacy\tests\provider_tes
     /**
      * Basic setup for these tests.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index 5789fcd..1cf7e15 100644 (file)
@@ -46,7 +46,7 @@ class core_grade_events_test extends advanced_testcase {
      *
      * This is executed before running any test in this file.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
 
         $this->setAdminUser();
index 5c8a50b..0549428 100644 (file)
@@ -279,7 +279,7 @@ class store_test extends advanced_testcase {
      * @param int $maxvalue The max value of the forum
      * @param int $suppliedvalue The value that was submitted
      */
-    public function test_execute_store_out_of__range(int $maxvalue, int $suppliedvalue): void {
+    public function test_execute_store_out_of__range(int $maxvalue, float $suppliedvalue): void {
         $this->resetAfterTest();
 
         $forum = $this->get_forum_instance([
index d9c2d48..d721dee 100644 (file)
@@ -46,7 +46,7 @@ require_once($CFG->libdir . '/gradelib.php');
  */
 class core_grades_privacy_testcase extends provider_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         global $PAGE;
         $this->resetAfterTest();
         $PAGE->get_renderer('core');
index 13f46c9..04ab135 100644 (file)
@@ -46,7 +46,7 @@ class editor_ajax_testcase extends \advanced_testcase {
     /**
      * Set up function for tests.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
 
         autoloader::register();
index 1dd6b8c..8e3e220 100644 (file)
@@ -45,7 +45,7 @@ class editor_framework_testcase extends \advanced_testcase {
     /**
      * Set up function for tests.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
 
         autoloader::register();
index c2c65f5..89d8ca9 100644 (file)
@@ -42,7 +42,7 @@ class core_h5p_event_h5p_deleted_testcase extends advanced_testcase {
     /**
      * Setup test.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
         autoloader::register();
     }
index 75a7671..73eb264 100644 (file)
@@ -42,7 +42,7 @@ class core_h5p_event_h5p_viewed_testcase extends advanced_testcase {
     /**
      * Setup test.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
         autoloader::register();
     }
index 42f1aea..e2a842a 100644 (file)
@@ -46,7 +46,7 @@ use core_h5p\local\library\autoloader;
  */
 class core_h5p_external_testcase extends externallib_advanced_testcase {
 
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
         autoloader::register();
     }
index 13883c1..9e5c449 100644 (file)
@@ -44,7 +44,7 @@ class framework_testcase extends \advanced_testcase {
     /**
      * Set up function for tests.
      */
-    public function setUp() {
+    public function setUp(): void {
         $factory = new \core_h5p\factory();
         $this->framework = $factory->get_framework();
     }
index 05be967..b43df42 100644 (file)
@@ -43,7 +43,7 @@ class generator_testcase extends \advanced_testcase {
     /**
      * Tests set up.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
 
         autoloader::register();
index d441fdc..a1bcb2c 100644 (file)
@@ -40,7 +40,7 @@ defined('MOODLE_INTERNAL') || die();
  */
 class h5p_core_testcase extends \advanced_testcase {
 
-    protected function setup() {
+    protected function setUp(): void {
         global $CFG;
         parent::setUp();
 
index 8cce6c2..8199d35 100644 (file)
@@ -60,7 +60,7 @@ class h5p_file_storage_testcase extends \advanced_testcase {
     /** @var int $libraryid an id for the library. */
     protected $libraryid = 1;
 
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
         $this->resetAfterTest(true);
 
index d553bd8..f1a8a93 100644 (file)
@@ -35,7 +35,7 @@ require_once("{$CFG->dirroot}/iplookup/lib.php");
  * GeoIp data file parsing test.
  */
 class core_iplookup_geoip_testcase extends advanced_testcase {
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index 79466bc..3ebe8b4 100644 (file)
@@ -31,7 +31,7 @@ defined('MOODLE_INTERNAL') || die();
  */
 class core_iplookup_geoplugin_testcase extends advanced_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         global $CFG;
         require_once("$CFG->libdir/filelib.php");
         require_once("$CFG->dirroot/iplookup/lib.php");
index 87a39b7..4a2eca7 100644 (file)
@@ -29,7 +29,7 @@ class antivirus_clamav_scanner_testcase extends advanced_testcase {
     /** @var string temporary file used in testing */
     protected $tempfile;
 
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest();
 
         // Create tempfile.
@@ -38,7 +38,7 @@ class antivirus_clamav_scanner_testcase extends advanced_testcase {
         touch($this->tempfile);
     }
 
-    protected function tearDown() {
+    protected function tearDown(): void {
         @unlink($this->tempfile);
     }
 
index 852b9e9..35526ac 100644 (file)
@@ -31,7 +31,7 @@ class core_ddl_testcase extends database_driver_testcase {
     /** @var array table name => array of stdClass test records loaded into that table. Created in setUp. */
     private $records = array();
 
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
         $dbman = $this->tdb->get_manager(); // Loads DDL libs.
 
index e23a456..d55c809 100644 (file)
@@ -27,7 +27,7 @@ defined('MOODLE_INTERNAL') || die();
 
 class core_dml_testcase extends database_driver_testcase {
 
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
         $dbman = $this->tdb->get_manager(); // Loads DDL libs.
     }
index 690d128..53a2ff1 100644 (file)
@@ -44,7 +44,7 @@ class pgsql_native_recordset_testcase extends basic_testcase {
     /**
      * Creates a second db connection and a temp table with values in for testing.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB;
 
         parent::setUp();
@@ -87,7 +87,7 @@ class pgsql_native_recordset_testcase extends basic_testcase {
     /**
      * Gets rid of the second db connection.
      */
-    protected function tearDown() {
+    protected function tearDown(): void {
         if ($this->specialdb) {
             $table = new xmldb_table('silly_test_table');
             $this->specialdb->get_manager()->drop_table($table);
index bab7045..17b58df 100644 (file)
@@ -35,7 +35,7 @@ defined('MOODLE_INTERNAL') || die();
  */
 class core_recordset_walk_testcase extends advanced_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
         $this->resetAfterTest();
     }
index 8736d79..e5f08e8 100644 (file)
@@ -38,7 +38,7 @@ require_once($CFG->dirroot.'/lib/dml/sqlsrv_native_moodle_database.php');
  */
 class sqlsrv_native_moodle_database_testcase extends advanced_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
         $this->resetAfterTest();
     }
index 2959646..0641408 100644 (file)
@@ -59,7 +59,7 @@ class file_browser_testcase extends advanced_testcase {
     /**
      * Set up
      */
-    public function setUp() {
+    public function setUp(): void {
         global $DB;
         $this->resetAfterTest();
 
index 525d770..2465371 100644 (file)
@@ -43,7 +43,7 @@ class core_files_file_system_filedir_testcase extends advanced_testcase {
     /**
      * Shared test setUp.
      */
-    public function setUp() {
+    public function setUp(): void {
         // Reset the file storage so that subsequent fetches to get_file_storage are called after
         // configuration is prepared.
         get_file_storage(true);
@@ -52,7 +52,7 @@ class core_files_file_system_filedir_testcase extends advanced_testcase {
     /**
      * Shared teset tearDown.
      */
-    public function tearDown() {
+    public function tearDown(): void {
         // Reset the file storage so that subsequent tests will use the standard file storage.
         get_file_storage(true);
     }
index 387d047..4858c1c 100644 (file)
@@ -39,11 +39,11 @@ require_once($CFG->libdir . '/filestorage/file_system.php');
  */
 class core_files_file_system_testcase extends advanced_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         get_file_storage(true);
     }
 
-    public function tearDown() {
+    public function tearDown(): void {
         get_file_storage(true);
     }
 
index 4b36265..56fd859 100644 (file)
@@ -37,7 +37,7 @@ class core_files_zip_packer_testcase extends advanced_testcase implements file_p
      */
     protected $progress;
 
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
 
         $this->testfile = __DIR__.'/fixtures/test.txt';
index 1293414..9605220 100644 (file)
@@ -50,7 +50,7 @@ class core_form_dateselector_testcase extends advanced_testcase {
     /**
      * Initalize test wide variable, it is called in start of the testcase
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $CFG;
         parent::setUp();
 
index ec45212..6e635b3 100644 (file)
@@ -50,7 +50,7 @@ class core_form_datetimeselector_testcase extends advanced_testcase {
     /**
      * Initalize test wide variable, it is called in start of the testcase
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $CFG;
         parent::setUp();
 
index e2203a2..16050c8 100644 (file)
@@ -49,7 +49,7 @@ abstract class grade_base_testcase extends advanced_testcase {
     protected $courseid;
     protected $userid;
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $CFG;
         parent::setup();
 
index 63a99d4..a255208 100644 (file)
@@ -67,7 +67,7 @@ abstract class database_driver_testcase extends base_testcase {
         $this->setRunTestInSeparateProcess(false);
     }
 
-    public static function setUpBeforeClass() {
+    public static function setUpBeforeClass(): void {
         global $CFG;
         parent::setUpBeforeClass();
 
@@ -101,7 +101,7 @@ abstract class database_driver_testcase extends base_testcase {
         self::$extradb = $d;
     }
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $DB;
         parent::setUp();
 
@@ -112,7 +112,7 @@ abstract class database_driver_testcase extends base_testcase {
         }
     }
 
-    protected function tearDown() {
+    protected function tearDown(): void {
         // delete all test tables
         $dbman = $this->tdb->get_manager();
         $tables = $this->tdb->get_tables(false);
@@ -125,7 +125,7 @@ abstract class database_driver_testcase extends base_testcase {
         parent::tearDown();
     }
 
-    public static function tearDownAfterClass() {
+    public static function tearDownAfterClass(): void {
         if (self::$extradb) {
             self::$extradb->dispose();
             self::$extradb = null;
index bc80ae7..48d0412 100644 (file)
@@ -51,7 +51,7 @@ abstract class restore_date_testcase extends advanced_testcase {
     /**
      * Setup.
      */
-    public function setUp() {
+    public function setUp(): void {
         global $CFG;
 
         parent::setUp();
@@ -160,4 +160,4 @@ abstract class restore_date_testcase extends advanced_testcase {
         return ($this->restorestartdate - $this->startdate);
     }
 
-}
\ No newline at end of file
+}
index 04b75c5..9ef7e1d 100644 (file)
@@ -37,7 +37,7 @@ defined('MOODLE_INTERNAL') || die();
 class core_phpunit_basic_testcase extends basic_testcase {
     protected $testassertexecuted = false;
 
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
         if ($this->getName() === 'test_setup_assert') {
             $this->assertTrue(true);
index 18b4594..4ce4ca9 100644 (file)
@@ -29,7 +29,7 @@ class core_ajaxlib_testcase extends advanced_testcase {
     /** @var string Original error log */
     protected $oldlog;
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $CFG;
 
         parent::setUp();
@@ -38,7 +38,7 @@ class core_ajaxlib_testcase extends advanced_testcase {
         ini_set('error_log', "$CFG->dataroot/testlog.log");
     }
 
-    protected function tearDown() {
+    protected function tearDown(): void {
         ini_set('error_log', $this->oldlog);
         parent::tearDown();
     }
index 96fb1ce..fb92f8e 100644 (file)
@@ -29,7 +29,7 @@ require_once(__DIR__ . '/fixtures/testable_antivirus.php');
 class core_antivirus_testcase extends advanced_testcase {
     protected $tempfile;
 
-    protected function setUp() {
+    protected function setUp(): void {
         global $CFG;
         // Use our special testable fixture plugin.
         $CFG->antiviruses = 'testable';
@@ -42,7 +42,7 @@ class core_antivirus_testcase extends advanced_testcase {
         touch($this->tempfile);
     }
 
-    protected function tearDown() {
+    protected function tearDown(): void {
         @unlink($this->tempfile);
     }
 
index 1cf4552..4fbd0a6 100644 (file)
@@ -39,7 +39,7 @@ class core_blocklib_testcase extends advanced_testcase {
     protected $blockmanager;
     protected $isediting = null;
 
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
         $this->testpage = new moodle_page();
         $this->testpage->set_context(context_system::instance());
@@ -47,7 +47,7 @@ class core_blocklib_testcase extends advanced_testcase {
         $this->blockmanager = new testable_block_manager($this->testpage);
     }
 
-    protected function tearDown() {
+    protected function tearDown(): void {
         $this->testpage = null;
         $this->blockmanager = null;
         parent::tearDown();
index 6b09987..a880f9b 100644 (file)
@@ -39,7 +39,7 @@ class core_calendar_cron_task_testcase extends advanced_testcase {
     /**
      * Tests set up
      */
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest();
     }
 
index 7fb9edd..94604fe 100644 (file)
@@ -48,7 +48,7 @@ class core_collator_testcase extends advanced_testcase {
     /**
      * Prepares things for this test case.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $SESSION;
         if (isset($SESSION->lang)) {
             $this->initiallang = $SESSION->lang;
@@ -65,7 +65,7 @@ class core_collator_testcase extends advanced_testcase {
     /**
      * Cleans things up after this test case has run.
      */
-    protected function tearDown() {
+    protected function tearDown(): void {
         global $SESSION;
         parent::tearDown();
         if ($this->initiallang !== null) {
index f66a69d..0111d66 100644 (file)
@@ -38,7 +38,7 @@ class core_component_testcase extends advanced_testcase {
      */
     const SUBSYSTEMCOUNT = 71;
 
-    public function setUp() {
+    public function setUp(): void {
         $psr0namespaces = new ReflectionProperty('core_component', 'psr0namespaces');
         $psr0namespaces->setAccessible(true);
         $this->oldpsr0namespaces = $psr0namespaces->getValue(null);
@@ -47,7 +47,7 @@ class core_component_testcase extends advanced_testcase {
         $psr4namespaces->setAccessible(true);
         $this->oldpsr4namespaces = $psr4namespaces->getValue(null);
     }
-    public function tearDown() {
+    public function tearDown(): void {
         $psr0namespaces = new ReflectionProperty('core_component', 'psr0namespaces');
         $psr0namespaces->setAccessible(true);
         $psr0namespaces->setValue(null, $this->oldpsr0namespaces);
index c7e925b..5feb310 100644 (file)
@@ -40,7 +40,7 @@ class core_media_player_native_testcase extends advanced_testcase {
     /**
      * Pre-test setup.
      */
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
         $this->resetAfterTest();
     }
@@ -160,4 +160,4 @@ class core_media_player_native_testcase extends advanced_testcase {
         $this->assertNotContains('<source src="http://example.org/some_filename.mp4" />', $content);
         $this->assertNotContains('<source src="http://example.org/some_filename_hires.mp4" />', $content);
     }
-}
\ No newline at end of file
+}
index 55e84d1..7d7523b 100644 (file)
@@ -36,7 +36,7 @@ class core_csvclass_testcase extends advanced_testcase {
     protected $teststring3 = '';
     protected $teststring4 = '';
 
-    protected function setUp() {
+    protected function setUp(): void {
 
         $this->resetAfterTest();
 
index eaa2f3c..7dcd228 100644 (file)
@@ -103,7 +103,7 @@ class customcontext_testcase extends advanced_testcase {
     /**
      * Perform setup before every test. This tells Moodle's phpunit to reset the database after every test.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         parent::setUp();
         $this->resetAfterTest(true);
     }
index 0e381b9..70a9f58 100644 (file)
@@ -39,7 +39,7 @@ class contentbank_content_created_testcase extends \advanced_testcase {
     /**
      * Setup to ensure that fixtures are loaded.
      */
-    public static function setUpBeforeClass() {
+    public static function setUpBeforeClass(): void {
         global $CFG;
 
         require_once($CFG->dirroot . '/contentbank/tests/fixtures/testable_contenttype.php');
index 20c7517..5604bd8 100644 (file)
@@ -39,7 +39,7 @@ class contentbank_content_deleted_testcase extends \advanced_testcase {
     /**
      * Setup to ensure that fixtures are loaded.
      */
-    public static function setUpBeforeClass() {
+    public static function setUpBeforeClass(): void {
         global $CFG;
 
         require_once($CFG->dirroot . '/contentbank/tests/fixtures/testable_contenttype.php');
index b6aeae7..e5fc297 100644 (file)
@@ -39,7 +39,7 @@ class contentbank_content_updated_testcase extends \advanced_testcase {
     /**
      * Setup to ensure that fixtures are loaded.
      */
-    public static function setUpBeforeClass() {
+    public static function setUpBeforeClass(): void {
         global $CFG;
 
         require_once($CFG->dirroot . '/contentbank/tests/fixtures/testable_contenttype.php');
index 93e6d7d..8e1fe8d 100644 (file)
@@ -41,7 +41,7 @@ class contentbank_content_uploaded_testcase extends \advanced_testcase {
     /**
      * Setup to ensure that fixtures are loaded.
      */
-    public static function setUpBeforeClass() {
+    public static function setUpBeforeClass(): void {
         global $CFG;
 
         require_once($CFG->dirroot . '/contentbank/tests/fixtures/testable_contenttype.php');
index 3cf66de..e2bb786 100644 (file)
@@ -39,7 +39,7 @@ class contentbank_content_viewed_testcase extends \advanced_testcase {
     /**
      * Setup to ensure that fixtures are loaded.
      */
-    public static function setUpBeforeClass() {
+    public static function setUpBeforeClass(): void {
         global $CFG;
 
         require_once($CFG->dirroot . '/contentbank/tests/fixtures/testable_contenttype.php');
index b60f301..82154e7 100644 (file)
@@ -42,7 +42,7 @@ class core_event_profile_field_testcase extends advanced_testcase {
     /**
      * Test set up.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index b0eb084..e30e30a 100644 (file)
@@ -44,7 +44,7 @@ class core_event_user_graded_testcase extends advanced_testcase {
     /**
      * Tests set up.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index 9fc766a..fe1d24a 100644 (file)
@@ -34,7 +34,7 @@ class core_events_testcase extends advanced_testcase {
      *
      * This is executed before running any test in this file.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index b56cd13..7b22757 100644 (file)
@@ -39,7 +39,7 @@ class core_exporter_testcase extends advanced_testcase {
     protected $validdata = null;
     protected $invaliddata = null;
 
-    public function setUp() {
+    public function setUp(): void {
         $s = new stdClass();
         $this->validrelated = array(
             'simplestdClass' => $s,
index aa8101f..3d91acd 100644 (file)
@@ -32,11 +32,11 @@ require_once($CFG->libdir . '/externallib.php');
 class core_externallib_testcase extends advanced_testcase {
     protected $DB;
 
-    public function setUp() {
+    public function setUp(): void {
         $this->DB = null;
     }
 
-    public function tearDown() {
+    public function tearDown(): void {
         global $DB;
         if ($this->DB !== null) {
             $DB = $this->DB;
index 8875c3c..edc6d1d 100644 (file)
@@ -38,7 +38,7 @@ class core_gdlib_testcase extends basic_testcase {
 
     private $fixturepath = null;
 
-    public function setUp() {
+    public function setUp(): void {
         $this->fixturepath = __DIR__ . DIRECTORY_SEPARATOR . 'fixtures' . DIRECTORY_SEPARATOR;
     }
 
index 3502a1b..00ac618 100644 (file)
@@ -38,7 +38,7 @@ defined('MOODLE_INTERNAL') || die();
  */
 class h5p_get_content_types_task_testcase extends advanced_testcase {
 
-    protected function setup() {
+    protected function setup(): void {
         global $CFG;
         parent::setUp();
 
index 2064621..d7c3211 100644 (file)
@@ -39,7 +39,7 @@ class lock_testcase extends advanced_testcase {
     /**
      * Some lock types will store data in the database.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index 8b230c9..d92ba7a 100644 (file)
@@ -34,7 +34,7 @@ class core_medialib_testcase extends advanced_testcase {
     /**
      * Pre-test setup. Preserves $CFG.
      */
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
 
         // Reset $CFG and $SERVER.
@@ -496,4 +496,4 @@ class core_medialib_testcase extends advanced_testcase {
         }
         return $out;
     }
-}
\ No newline at end of file
+}
index bd68624..0159a6c 100644 (file)
@@ -37,7 +37,7 @@ class core_moodle_page_testcase extends advanced_testcase {
      */
     protected $testpage;
 
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
         $this->resetAfterTest();
         $this->testpage = new testable_moodle_page();
index 6cb5dc9..20476c0 100644 (file)
@@ -51,7 +51,7 @@ class core_myprofilelib_testcase extends advanced_testcase {
      */
     private $tree;
 
-    public function setUp() {
+    public function setUp(): void {
         // Set the $PAGE->url value so core_myprofile_navigation() doesn't complain.
         global $PAGE;
         $PAGE->set_url('/test');
index 50d5adc..33078fe 100644 (file)
@@ -42,7 +42,7 @@ class core_notification_testcase extends advanced_testcase {
      * This includes emptying the list of notifications on the session, resetting any session which exists, and setting
      * up a new moodle_page object.
      */
-    public function setUp() {
+    public function setUp(): void {
         global $PAGE, $SESSION;
 
         parent::setUp();
@@ -57,7 +57,7 @@ class core_notification_testcase extends advanced_testcase {
      * This includes emptying the list of notifications on the session, resetting any session which exists, and setting
      * up a new moodle_page object.
      */
-    public function tearDown() {
+    public function tearDown(): void {
         global $PAGE, $SESSION;
 
         $PAGE = null;
index 9446e2a..711903e 100644 (file)
@@ -34,7 +34,7 @@ global $CFG;
  */
 class core_persistent_testcase extends advanced_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         $this->make_persistent_table();
         $this->resetAfterTest();
     }
index 009c57e..41a7957 100644 (file)
@@ -34,7 +34,7 @@ require_once($CFG->dirroot.'/lib/tests/fixtures/testable_plugininfo_base.php');
  */
 class core_plugin_manager_testcase extends advanced_testcase {
 
-    public function tearDown() {
+    public function tearDown(): void {
         // The caches of the testable singleton must be reset explicitly. It is
         // safer to kill the whole testable singleton at the end of every test.
         testable_core_plugin_manager::reset_caches();
index d5b805f..e8f5566 100644 (file)
@@ -50,7 +50,7 @@ class base_testcase extends \advanced_testcase {
     /**
      * Tear down the testable plugin manager singleton between tests.
      */
-    public function tearDown() {
+    public function tearDown(): void {
         // The caches of the testable singleton must be reset explicitly. It is
         // safer to kill the whole testable singleton at the end of every test.
         testable_core_plugin_manager::reset_caches();
index 3b79c46..0ffa10c 100644 (file)
@@ -49,7 +49,7 @@ class core_questionlib_testcase extends advanced_testcase {
      *
      * This is executed before running any test in this file.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index c724f6a..0db975b 100644 (file)
@@ -40,7 +40,7 @@ class core_rsslib_testcase extends advanced_testcase {
     // The number of seconds tests should wait for the server to respond (high to prevent false positives).
     const TIMEOUT = 10;
 
-    protected function setUp() {
+    protected function setUp(): void {
         moodle_simplepie::reset_cache();
     }
 
index cadbc3b..46373d6 100644 (file)
@@ -47,7 +47,7 @@ class core_session_redis_testcase extends advanced_testcase {
     /** @var $redis The current testing redis connection */
     protected $redis = null;
 
-    public function setUp() {
+    public function setUp(): void {
         global $CFG;
 
         if (!extension_loaded('redis')) {
@@ -79,7 +79,7 @@ class core_session_redis_testcase extends advanced_testcase {
         $this->redis->connect(TEST_SESSION_REDIS_HOST);
     }
 
-    public function tearDown() {
+    public function tearDown(): void {
         if (!extension_loaded('redis') || !defined('TEST_SESSION_REDIS_HOST')) {
             return;
         }
index 7af80e0..7e45f9a 100644 (file)
@@ -52,7 +52,7 @@ class core_statslib_testcase extends advanced_testcase {
      * Setup function
      *   - Allow changes to CFG->debug for testing purposes.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $CFG, $DB;
         parent::setUp();
 
index d2c43b2..4ae0a66 100644 (file)
@@ -43,7 +43,7 @@ class task_database_logger_testcase extends advanced_testcase {
     /**
      * Setup to backup the database before mocking.
      */
-    public function setUp() {
+    public function setUp(): void {
         global $DB;
 
         $this->DB = $DB;
@@ -52,7 +52,7 @@ class task_database_logger_testcase extends advanced_testcase {
     /**
      * Tear down to unmock the database where it was mocked.
      */
-    public function tearDown() {
+    public function tearDown(): void {
         global $DB;
 
         $DB = $this->DB;
index d4f4e12..39b86cd 100644 (file)
@@ -43,7 +43,7 @@ class core_task_logmanager extends advanced_testcase {
     /**
      * Relevant tearDown for logging tests.
      */
-    public function tearDown() {
+    public function tearDown(): void {
         global $DB;
 
         // Ensure that any logging is always ended.
index d36f9b9..646bbf4 100644 (file)
@@ -44,7 +44,7 @@ class core_analytics_time_splittings_testcase extends advanced_testcase {
      *
      * @return void
      */
-    public function setUp() {
+    public function setUp(): void {
 
         $this->resetAfterTest(true);
 
index 6777f1f..ecfd86b 100644 (file)
@@ -34,7 +34,7 @@ class core_user_testcase extends advanced_testcase {
     /**
      * Setup test data.
      */
-    protected function setUp() {
+    protected function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index 28a3b00..9765e91 100644 (file)
@@ -34,7 +34,7 @@ class core_useragent_testcase extends advanced_testcase {
     /**
      * Restores the user agent to the default one.
      */
-    public function tearDown() {
+    public function tearDown(): void {
         core_useragent::instance(true);
     }
 
index 90e5000..e6547a9 100644 (file)
@@ -64,7 +64,7 @@ class post_statement_testcase extends externallib_advanced_testcase {
     /**
      * Setup test.
      */
-    public function setUp() {
+    public function setUp(): void {
         global $CFG;
         // We disable group actors on the test xapi_handler.
         $CFG->xapitestforcegroupactors = false;
index 7dfe8e4..15e0ccd 100644 (file)
@@ -36,7 +36,7 @@ class media_html5audio_testcase extends advanced_testcase {
     /**
      * Pre-test setup. Preserves $CFG.
      */
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
 
         // Reset $CFG and $SERVER.
index 3def9af..c4a20c1 100644 (file)
@@ -36,7 +36,7 @@ class media_html5video_testcase extends advanced_testcase {
     /**
      * Pre-test setup. Preserves $CFG.
      */
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
 
         // Reset $CFG and $SERVER.
index f7b053f..d072e33 100644 (file)
@@ -36,7 +36,7 @@ class media_swf_testcase extends advanced_testcase {
     /**
      * Pre-test setup. Preserves $CFG.
      */
-    public function setUp() {
+    public function setUp(): void {
         global $CFG;
         parent::setUp();
 
index aa8de9f..c906934 100644 (file)
@@ -36,7 +36,7 @@ class media_videojs_player_testcase extends advanced_testcase {
     /**
      * Pre-test setup. Preserves $CFG.
      */
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
 
         // Reset $CFG and $SERVER.
index 3f4dcbf..125e40f 100644 (file)
@@ -36,7 +36,7 @@ class media_vimeo_testcase extends advanced_testcase {
     /**
      * Pre-test setup. Preserves $CFG.
      */
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
 
         // Reset $CFG and $SERVER.
index 5c9d851..b5f298e 100644 (file)
@@ -36,7 +36,7 @@ class media_youtube_testcase extends advanced_testcase {
     /**
      * Pre-test setup. Preserves $CFG.
      */
-    public function setUp() {
+    public function setUp(): void {
         parent::setUp();
 
         // Reset $CFG and $SERVER.
index d0d3a54..a4fbbf8 100644 (file)
@@ -42,7 +42,7 @@ class message_airnotifier_external_testcase extends externallib_advanced_testcas
     /**
      * Tests set up
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $CFG;
         require_once($CFG->dirroot . '/message/output/airnotifier/externallib.php');
     }
index 96bd347..22cc63d 100644 (file)
@@ -38,7 +38,7 @@ class message_airnotifier_testcase extends provider_testcase {
     /**
      * Basic setup for these tests.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index dbfd9ba..700c919 100644 (file)
@@ -34,7 +34,7 @@ class message_email_testcase extends provider_testcase {
     /**
      * Basic setup for these tests.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index 99e16cf..1475b27 100644 (file)
@@ -36,7 +36,7 @@ class message_jabber_testcase extends provider_testcase {
     /**
      * Basic setup for these tests.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index 1d51f6a..069e5f6 100644 (file)
@@ -46,7 +46,7 @@ class message_popup_api_testcase extends advanced_testcase {
      *
      * This is executed before running any test in this file.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->preventResetByRollback(); // Messaging is not compatible with transactions.
         $this->messagesink = $this->redirectMessages();
         $this->resetAfterTest();
index 94fe0db..26da55f 100644 (file)
@@ -44,7 +44,7 @@ class message_popup_externallib_testcase extends advanced_testcase {
      *
      * This is executed before running any test in this file.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->preventResetByRollback(); // Messaging is not compatible with transactions.
         $this->messagesink = $this->redirectMessages();
         $this->resetAfterTest();
index 00a6b19..0ec24ea 100644 (file)
@@ -44,7 +44,7 @@ class core_message_events_testcase extends core_message_messagelib_testcase {
      *
      * This is executed before running any test in this file.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index 4fd7e6e..c21c311 100644 (file)
@@ -37,7 +37,7 @@ class core_message_externallib_testcase extends externallib_advanced_testcase {
     /**
      * Tests set up
      */
-    protected function setUp() {
+    protected function setUp(): void {
         global $CFG;
 
         require_once($CFG->dirroot . '/message/lib.php');
index 7b27d56..2bfa83e 100644 (file)
@@ -39,7 +39,7 @@ require_once($CFG->dirroot . '/message/tests/messagelib_test.php');
  */
 class core_message_helper_testcase extends advanced_testcase {
 
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
     }
 
index e07b2f7..6db7eb4 100644 (file)
@@ -38,7 +38,7 @@ class core_message_inbound_testcase extends advanced_testcase {
      * This includes:
      * * configuring the messageinbound_mailbox.
      */
-    public function setUp() {
+    public function setUp(): void {
         global $CFG;
 
         $this->resetAfterTest(true);
index 5a75739..7d195e7 100644 (file)
@@ -48,7 +48,7 @@ class core_message_messagelib_testcase extends advanced_testcase {
      *
      * This is executed before running any test in this file.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->preventResetByRollback(); // Messaging is not compatible with transactions.
         $this->messagesink = $this->redirectMessages();
         $this->resetAfterTest();
index a480e70..704396b 100644 (file)
@@ -44,7 +44,7 @@ class core_message_migrate_message_data_task_testcase extends advanced_testcase
      *
      * This is executed before running any test in this file.
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest();
     }
 
index c8f53ac..7e59a28 100644 (file)
@@ -45,7 +45,7 @@ class message_received_search_testcase extends advanced_testcase {
      * Setting up the test environment
      * @return void
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
         set_config('enableglobalsearch', true);
 
index eb42216..91a0a6e 100644 (file)
@@ -45,7 +45,7 @@ class message_sent_search_testcase extends advanced_testcase {
      * Setting up the test environment
      * @return void
      */
-    public function setUp() {
+    public function setUp(): void {
         $this->resetAfterTest(true);
         set_config('enableglobalsearch', true);
 
index 005e0e3..795ec77 100644 (file)
@@ -51,7 +51,7 @@ class mnetservice_enrol_privacy_testcase extends provider_testcase {
      *
      * This is executed before running any test in this file.
      */
-    public function setUp() {
+    public function setUp(): void {
         global $DB;
 
         // Add a mnet host.
@@ -340,4 +340,4 @@ class mnetservice_enrol_privacy_testcase extends provider_testcase {
         $course->rolename       = 'student';
         $DB->insert_record('mnetservice_enrol_courses', $course);
     }
-}
\ No newline at end of file
+}