WebSVN – Moodle – Diff – /message/tests/api_test.php

 namespace core_message;
-use core_message\tests\helper as testhelper;
-defined('MOODLE_INTERNAL') || die();
-global $CFG;
-require_once($CFG->dirroot . '/message/tests/messagelib_test.php');
+use core_message\tests\helper as testhelper;
 /**
  * Test message API.
+ *
  * @package core_message
+ * @category test
- * @category test
+ * @copyright 2016 Mark Nelson <markn@moodle.com>
- * @copyright 2016 Mark Nelson <markn@moodle.com>
+ * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ * @covers \core_message\api
+ */
- * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
+final class api_test extends \advanced_testcase {
-class api_test extends messagelib_test {
+    public function test_mark_all_read_for_user_touser(): void {
+        $this->resetAfterTest();
-    public function test_mark_all_read_for_user_touser() {
         $sender = $this->getDataGenerator()->create_user(array('firstname' => 'Test1', 'lastname' => 'User1'));
         $recipient = $this->getDataGenerator()->create_user(array('firstname' => 'Test2', 'lastname' => 'User2'));
-        $this->send_fake_message($sender, $recipient, 'Notification', 1);
+        testhelper::send_fake_message($sender, $recipient, 'Notification', 1);
-        $this->send_fake_message($sender, $recipient, 'Notification', 1);
+        testhelper::send_fake_message($sender, $recipient, 'Notification', 1);
-        $this->send_fake_message($sender, $recipient, 'Notification', 1);
+        testhelper::send_fake_message($sender, $recipient, 'Notification', 1);
-        $this->send_fake_message($sender, $recipient);
+        testhelper::send_fake_message($sender, $recipient);
-        $this->send_fake_message($sender, $recipient);
+        testhelper::send_fake_message($sender, $recipient);
-        $this->send_fake_message($sender, $recipient);
+        testhelper::send_fake_message($sender, $recipient);
         $this->assertEquals(1, api::count_unread_conversations($recipient));
+        api::mark_all_notifications_as_read($recipient->id);
         api::mark_all_messages_as_read($recipient->id);
         api::mark_all_notifications_as_read($recipient->id);
-        api::mark_all_messages_as_read($recipient->id);
+        // We've marked all of recipients conversation messages as read.
+        $this->assertEquals(0, api::count_unread_conversations($recipient));
+    }
+    public function test_mark_all_read_for_user_touser_with_fromuser(): void {
-        // We've marked all of recipients conversation messages as read.
+        $this->resetAfterTest();
-        $this->assertEquals(0, api::count_unread_conversations($recipient));
-    }
+        $sender1 = $this->getDataGenerator()->create_user(array('firstname' => 'Test1', 'lastname' => 'User1'));
+        $sender2 = $this->getDataGenerator()->create_user(array('firstname' => 'Test3', 'lastname' => 'User3'));
-    public function test_mark_all_read_for_user_touser_with_fromuser() {
+        $recipient = $this->getDataGenerator()->create_user(array('firstname' => 'Test2', 'lastname' => 'User2'));
-        $sender1 = $this->getDataGenerator()->create_user(array('firstname' => 'Test1', 'lastname' => 'User1'));
+        $sender1 = $this->getDataGenerator()->create_user(['firstname' => 'Test1', 'lastname' => 'User1']);
-        $sender2 = $this->getDataGenerator()->create_user(array('firstname' => 'Test3', 'lastname' => 'User3'));
+        $sender2 = $this->getDataGenerator()->create_user(['firstname' => 'Test3', 'lastname' => 'User3']);
-        $recipient = $this->getDataGenerator()->create_user(array('firstname' => 'Test2', 'lastname' => 'User2'));
+        $recipient = $this->getDataGenerator()->create_user(['firstname' => 'Test2', 'lastname' => 'User2']);
-        $this->send_fake_message($sender1, $recipient, 'Notification', 1);
+        testhelper::send_fake_message($sender1, $recipient, 'Notification', 1);
-        $this->send_fake_message($sender1, $recipient, 'Notification', 1);
+        testhelper::send_fake_message($sender1, $recipient, 'Notification', 1);
-        $this->send_fake_message($sender1, $recipient, 'Notification', 1);
+        testhelper::send_fake_message($sender1, $recipient, 'Notification', 1);
-        $this->send_fake_message($sender1, $recipient);
+        testhelper::send_fake_message($sender1, $recipient);
-        $this->send_fake_message($sender1, $recipient);
+        testhelper::send_fake_message($sender1, $recipient);
-        $this->send_fake_message($sender1, $recipient);
+        testhelper::send_fake_message($sender1, $recipient);
+    }
     /**
      * Test count_blocked_users.
      */
-    public function test_count_blocked_users() {
+    public function test_count_blocked_users(): void {
+        global $USER;
+        $this->resetAfterTest();
         global $USER;
         // Set this user as the admin.
+     *
      * This test verifies that any contacts are returned, as well as any non-contacts whose profile we can view.
      * If checks this by placing some users in the same course, where default caps would permit a user to view another user's
      * profile.
      */
-    public function test_message_search_users_messagingallusers_disabled() {
+    public function test_message_search_users_messagingallusers_disabled(): void {
         global $DB;
         $this->resetAfterTest();
         // Create some users.
      * Tests searching for users when site-wide messaging is enabled.
+     *
      * This test verifies that any contacts are returned, as well as any non-contacts,
      * provided the searching user can view their profile.
      */
-    public function test_message_search_users_messagingallusers_enabled() {
+    public function test_message_search_users_messagingallusers_enabled(): void {
         global $DB;
         $this->resetAfterTest();
         // Create some users.
+    }
     /**
      * Verify searching for users find themselves when they have self-conversations.
      */
-    public function test_message_search_users_self_conversations() {
+    public function test_message_search_users_self_conversations(): void {
         $this->resetAfterTest();
         // Create some users.
+    }
     /**
      * Verify searching for users works even if no matching users from either contacts, or non-contacts can be found.
      */
-    public function test_message_search_users_with_empty_result() {
+    public function test_message_search_users_with_empty_result(): void {
         $this->resetAfterTest();
         // Create some users, but make sure neither will match the search term.
+    }
     /**
      * Test verifying that limits and offsets work for both the contacts and non-contacts return data.
      */
-    public function test_message_search_users_limit_offset() {
+    public function test_message_search_users_limit_offset(): void {
         $this->resetAfterTest();
         // Create 20 users.
+    }
     /**
      * Tests searching users as a user having the 'moodle/user:viewdetails' capability.
      */
-    public function test_message_search_users_with_cap() {
+    public function test_message_search_users_with_cap(): void {
         $this->resetAfterTest();
         global $DB;
+    }
     /**
      * Tests searching users with messaging disabled.
      */
-    public function test_message_search_users_messaging_disabled() {
+    public function test_message_search_users_messaging_disabled(): void {
         $this->resetAfterTest();
+    }
     /**
      * Tests getting conversations between 2 users.
      */
+    public function test_get_conversations_between_users(): void {
+        $this->resetAfterTest();
     public function test_get_conversations_between_users() {
         // Create some users.
         $user1 = new \stdClass();
         $user1->firstname = 'User';
         $user1->lastname = 'One';
+    }
     /**
      * Tests getting self-conversations.
      */
+    public function test_get_self_conversation(): void {
+        $this->resetAfterTest();
     public function test_get_self_conversation() {
         // Create some users.
         $user1 = new \stdClass();
         $user1->firstname = 'User';
         $user1->lastname = 'One';
+    }
     /**
      * Tests searching messages.
      */
-    public function test_search_messages() {
+    public function test_search_messages(): void {
         $this->resetAfterTest();
         // Create some users.
         // Send some messages back and forth.
         $time = 1;
         testhelper::send_fake_message_to_conversation($user1, $sc->id, 'Test message to self!', $time);
         testhelper::send_fake_message_to_conversation($user1, $gc->id, 'My hero!', $time + 1);
-        $this->send_fake_message($user3, $user1, 'Don\'t block me.', 0, $time + 2);
+        testhelper::send_fake_message($user3, $user1, 'Don\'t block me.', 0, $time + 2);
-        $this->send_fake_message($user1, $user2, 'Yo!', 0, $time + 3);
+        testhelper::send_fake_message($user1, $user2, 'Yo!', 0, $time + 3);
-        $this->send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 4);
+        testhelper::send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 4);
-        $this->send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 5);
+        testhelper::send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 5);
-        $this->send_fake_message($user2, $user1, 'Word.', 0, $time + 6);
+        testhelper::send_fake_message($user2, $user1, 'Word.', 0, $time + 6);
         $convid = api::get_conversation_between_users([$user1->id, $user2->id]);
+    }
     /**
      * Test verifying that favourited conversations can be retrieved.
      */
+    public function test_get_favourite_conversations(): void {
+        $this->resetAfterTest();
     public function test_get_favourite_conversations() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
         // Only self-conversation created.
         $this->assertCount(1, api::get_conversations($user1->id));
         // Create some conversations for user1.
         $time = 1;
-        $this->send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
+        testhelper::send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
-        $this->send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
+        testhelper::send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
-        $this->send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
+        testhelper::send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
-        $messageid1 = $this->send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
+        $messageid1 = testhelper::send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
-        $this->send_fake_message($user1, $user3, 'Booyah', 0, $time + 5);
+        testhelper::send_fake_message($user1, $user3, 'Booyah', 0, $time + 5);
-        $this->send_fake_message($user3, $user1, 'Whaaat?', 0, $time + 6);
+        testhelper::send_fake_message($user3, $user1, 'Whaaat?', 0, $time + 6);
-        $this->send_fake_message($user1, $user3, 'Nothing.', 0, $time + 7);
+        testhelper::send_fake_message($user1, $user3, 'Nothing.', 0, $time + 7);
-        $messageid2 = $this->send_fake_message($user3, $user1, 'Cool.', 0, $time + 8);
+        $messageid2 = testhelper::send_fake_message($user3, $user1, 'Cool.', 0, $time + 8);
-        $this->send_fake_message($user1, $user4, 'Hey mate, you see the new messaging UI in Moodle?', 0, $time + 9);
+        testhelper::send_fake_message($user1, $user4, 'Hey mate, you see the new messaging UI in Moodle?', 0, $time + 9);
-        $this->send_fake_message($user4, $user1, 'Yah brah, it\'s pretty rad.', 0, $time + 10);
+        testhelper::send_fake_message($user4, $user1, 'Yah brah, it\'s pretty rad.', 0, $time + 10);
-        $messageid3 = $this->send_fake_message($user1, $user4, 'Dope.', 0, $time + 11);
+        $messageid3 = testhelper::send_fake_message($user1, $user4, 'Dope.', 0, $time + 11);
         // Favourite the first 2 conversations for user1.
         $convoids = [];
+    }
     /**
      * Tests retrieving favourite conversations with a limit and offset to ensure pagination works correctly.
      */
+    public function test_get_favourite_conversations_limit_offset(): void {
+        $this->resetAfterTest();
     public function test_get_favourite_conversations_limit_offset() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
         // Only self-conversation created.
         $this->assertCount(1, api::get_conversations($user1->id));
         // Create some conversations for user1.
         $time = 1;
-        $this->send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
+        testhelper::send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
-        $this->send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
+        testhelper::send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
-        $this->send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
+        testhelper::send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
-        $messageid1 = $this->send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
+        $messageid1 = testhelper::send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
-        $this->send_fake_message($user1, $user3, 'Booyah', 0, $time + 5);
+        testhelper::send_fake_message($user1, $user3, 'Booyah', 0, $time + 5);
-        $this->send_fake_message($user3, $user1, 'Whaaat?', 0, $time + 6);
+        testhelper::send_fake_message($user3, $user1, 'Whaaat?', 0, $time + 6);
-        $this->send_fake_message($user1, $user3, 'Nothing.', 0, $time + 7);
+        testhelper::send_fake_message($user1, $user3, 'Nothing.', 0, $time + 7);
-        $messageid2 = $this->send_fake_message($user3, $user1, 'Cool.', 0, $time + 8);
+        $messageid2 = testhelper::send_fake_message($user3, $user1, 'Cool.', 0, $time + 8);
-        $this->send_fake_message($user1, $user4, 'Hey mate, you see the new messaging UI in Moodle?', 0, $time + 9);
+        testhelper::send_fake_message($user1, $user4, 'Hey mate, you see the new messaging UI in Moodle?', 0, $time + 9);
-        $this->send_fake_message($user4, $user1, 'Yah brah, it\'s pretty rad.', 0, $time + 10);
+        testhelper::send_fake_message($user4, $user1, 'Yah brah, it\'s pretty rad.', 0, $time + 10);
-        $messageid3 = $this->send_fake_message($user1, $user4, 'Dope.', 0, $time + 11);
+        $messageid3 = testhelper::send_fake_message($user1, $user4, 'Dope.', 0, $time + 11);
         // Favourite the all conversations for user1.
         $convoids = [];
+    }
     /**
      * Tests retrieving favourite conversations when a conversation contains a deleted user.
      */
+    public function test_get_favourite_conversations_with_deleted_user(): void {
+        $this->resetAfterTest();
     public function test_get_favourite_conversations_with_deleted_user() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
         // Send some messages back and forth, have some different conversations with different users.
         $time = 1;
-        $this->send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
+        testhelper::send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
-        $this->send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
+        testhelper::send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
-        $this->send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
+        testhelper::send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
-        $this->send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
+        testhelper::send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
-        $this->send_fake_message($user1, $user3, 'Booyah', 0, $time + 5);
+        testhelper::send_fake_message($user1, $user3, 'Booyah', 0, $time + 5);
-        $this->send_fake_message($user3, $user1, 'Whaaat?', 0, $time + 6);
+        testhelper::send_fake_message($user3, $user1, 'Whaaat?', 0, $time + 6);
-        $this->send_fake_message($user1, $user3, 'Nothing.', 0, $time + 7);
+        testhelper::send_fake_message($user1, $user3, 'Nothing.', 0, $time + 7);
-        $this->send_fake_message($user3, $user1, 'Cool.', 0, $time + 8);
+        testhelper::send_fake_message($user3, $user1, 'Cool.', 0, $time + 8);
         // Favourite the all conversations for user1.
+    }
     /**
      * Test confirming that conversations can be marked as favourites.
      */
+    public function test_set_favourite_conversation(): void {
+        $this->resetAfterTest();
     public function test_set_favourite_conversation() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
         // Send some messages back and forth, have some different conversations with different users.
         $time = 1;
-        $this->send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
+        testhelper::send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
-        $this->send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
+        testhelper::send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
-        $this->send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
+        testhelper::send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
-        $this->send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
+        testhelper::send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
-        $this->send_fake_message($user1, $user3, 'Booyah', 0, $time + 5);
+        testhelper::send_fake_message($user1, $user3, 'Booyah', 0, $time + 5);
-        $this->send_fake_message($user3, $user1, 'Whaaat?', 0, $time + 6);
+        testhelper::send_fake_message($user3, $user1, 'Whaaat?', 0, $time + 6);
-        $this->send_fake_message($user1, $user3, 'Nothing.', 0, $time + 7);
+        testhelper::send_fake_message($user1, $user3, 'Nothing.', 0, $time + 7);
-        $this->send_fake_message($user3, $user1, 'Cool.', 0, $time + 8);
+        testhelper::send_fake_message($user3, $user1, 'Cool.', 0, $time + 8);
+    }
     /**
      * Test verifying that trying to mark a non-existent conversation as a favourite, results in an exception.
      */
+    public function test_set_favourite_conversation_nonexistent_conversation(): void {
+        $this->resetAfterTest();
     public function test_set_favourite_conversation_nonexistent_conversation() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         // Try to favourite a non-existent conversation.
         $this->expectException(\moodle_exception::class);
         api::set_favourite_conversation(0, $user1->id);
+    }
     /**
      * Test verifying that a conversation cannot be marked as favourite unless the user is a member of that conversation.
+     */
+    public function test_set_favourite_conversation_non_member(): void {
-     */
+        $this->resetAfterTest();
     public function test_set_favourite_conversation_non_member() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
         // Send some messages back and forth, have some different conversations with different users.
         $time = 1;
-        $this->send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
+        testhelper::send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
-        $this->send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
+        testhelper::send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
-        $this->send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
+        testhelper::send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
-        $this->send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
+        testhelper::send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
-        $this->send_fake_message($user1, $user3, 'Booyah', 0, $time + 5);
+        testhelper::send_fake_message($user1, $user3, 'Booyah', 0, $time + 5);
-        $this->send_fake_message($user3, $user1, 'Whaaat?', 0, $time + 6);
+        testhelper::send_fake_message($user3, $user1, 'Whaaat?', 0, $time + 6);
-        $this->send_fake_message($user1, $user3, 'Nothing.', 0, $time + 7);
+        testhelper::send_fake_message($user1, $user3, 'Nothing.', 0, $time + 7);
-        $this->send_fake_message($user3, $user1, 'Cool.', 0, $time + 8);
+        testhelper::send_fake_message($user3, $user1, 'Cool.', 0, $time + 8);
         // Try to favourite the first conversation as user 3, who is not a member.
         $conversationid1 = api::get_conversation_between_users([$user1->id, $user2->id]);
         $this->expectException(\moodle_exception::class);
         api::set_favourite_conversation($conversationid1, $user3->id);
     }
+    /**
      * Test confirming that those conversations marked as favourites can be unfavourited.
-    /**
+     */
-     * Test confirming that those conversations marked as favourites can be unfavourited.
+    public function test_unset_favourite_conversation(): void {
-     */
+        $this->resetAfterTest();
     public function test_unset_favourite_conversation() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
         // Send some messages back and forth, have some different conversations with different users.
         $time = 1;
-        $this->send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
+        testhelper::send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
-        $this->send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
+        testhelper::send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
-        $this->send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
+        testhelper::send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
-        $this->send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
+        testhelper::send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
-        $this->send_fake_message($user1, $user3, 'Booyah', 0, $time + 5);
+        testhelper::send_fake_message($user1, $user3, 'Booyah', 0, $time + 5);
-        $this->send_fake_message($user3, $user1, 'Whaaat?', 0, $time + 6);
+        testhelper::send_fake_message($user3, $user1, 'Whaaat?', 0, $time + 6);
+    }
     /**
      * Test verifying that a valid conversation cannot be unset as a favourite if it's not marked as a favourite.
      */
+    public function test_unset_favourite_conversation_not_favourite(): void {
+        $this->resetAfterTest();
     public function test_unset_favourite_conversation_not_favourite() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         // Send some messages back and forth, have some different conversations with different users.
         $time = 1;
-        $this->send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
+        testhelper::send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
-        $this->send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
+        testhelper::send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
-        $this->send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
+        testhelper::send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
-        $this->send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
+        testhelper::send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
         // Now try to unfavourite the conversation as user 1.
         $conversationid1 = api::get_conversation_between_users([$user1->id, $user2->id]);
         $this->expectException(\moodle_exception::class);
         api::unset_favourite_conversation($conversationid1, $user1->id);
+    }
+    /**
+     * Test verifying that a non-existent conversation cannot be unset as a favourite.
-    /**
+     */
-     * Test verifying that a non-existent conversation cannot be unset as a favourite.
+    public function test_unset_favourite_conversation_non_existent_conversation(): void {
-     */
+        $this->resetAfterTest();
     public function test_unset_favourite_conversation_non_existent_conversation() {
+    }
     /**
      * Test verifying get_conversations when no limits, offsets, type filters or favourite restrictions are used.
      */
+    public function test_get_conversations_no_restrictions(): void {
+        $this->resetAfterTest();
     public function test_get_conversations_no_restrictions() {
         global $DB;
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test verifying that html format messages are supported, and that message_format_message_text() is being called appropriately.
      */
-    public function test_get_conversations_message_format() {
+    public function test_get_conversations_message_format(): void {
+        global $DB;
+        $this->resetAfterTest();
         global $DB;
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test verifying get_conversations identifies if a conversation is muted or not.
      */
+    public function test_get_conversations_some_muted(): void {
+        $this->resetAfterTest();
     public function test_get_conversations_some_muted() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Tests retrieving conversations with a limit and offset to ensure pagination works correctly.
      */
+    public function test_get_conversations_limit_offset(): void {
+        $this->resetAfterTest();
     public function test_get_conversations_limit_offset() {
         // Get a bunch of conversations, some group, some individual and in different states.
         list($user1, $user2, $user3, $user4, $ic1, $ic2, $ic3,
             $gc1, $gc2, $gc3, $gc4, $gc5, $gc6) = $this->create_conversation_test_data();
+    }
     /**
      * Test verifying the type filtering behaviour of the
      */
+    public function test_get_conversations_type_filter(): void {
+        $this->resetAfterTest();
     public function test_get_conversations_type_filter() {
         // Get a bunch of conversations, some group, some individual and in different states.
         list($user1, $user2, $user3, $user4, $ic1, $ic2, $ic3,
             $gc1, $gc2, $gc3, $gc4, $gc5, $gc6) = $this->create_conversation_test_data();
+    }
     /**
      * Tests retrieving conversations when a 'self' conversation exists.
      */
-    public function test_get_conversations_self_conversations() {
+    public function test_get_conversations_self_conversations(): void {
+        global $DB;
+        $this->resetAfterTest();
         global $DB;
         // Create a conversation between one user and themself.
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Tests retrieving conversations when a conversation contains a deleted user.
      */
+    public function test_get_conversations_with_deleted_user(): void {
+        $this->resetAfterTest();
     public function test_get_conversations_with_deleted_user() {
         // Get a bunch of conversations, some group, some individual and in different states.
         list($user1, $user2, $user3, $user4, $ic1, $ic2, $ic3,
             $gc1, $gc2, $gc3, $gc4, $gc5, $gc6) = $this->create_conversation_test_data();
+    }
     /**
      * Test confirming the behaviour of get_conversations() when users delete all messages.
      */
+    public function test_get_conversations_deleted_messages(): void {
+        $this->resetAfterTest();
     public function test_get_conversations_deleted_messages() {
         // Get a bunch of conversations, some group, some individual and in different states.
         list($user1, $user2, $user3, $user4, $ic1, $ic2, $ic3,
             $gc1, $gc2, $gc3, $gc4, $gc5, $gc6) = $this->create_conversation_test_data();
     /**
      * Test verifying the behaviour of get_conversations() when fetching favourite conversations with only a single
      * favourite.
      */
+    public function test_get_conversations_favourite_conversations_single(): void {
+        $this->resetAfterTest();
     public function test_get_conversations_favourite_conversations_single() {
         // Get a bunch of conversations, some group, some individual and in different states.
         list($user1, $user2, $user3, $user4, $ic1, $ic2, $ic3,
             $gc1, $gc2, $gc3, $gc4, $gc5, $gc6) = $this->create_conversation_test_data();
+    }
     /**
      * Test verifying the behaviour of get_conversations() when fetching favourite conversations.
      */
+    public function test_get_conversations_favourite_conversations(): void {
+        $this->resetAfterTest();
     public function test_get_conversations_favourite_conversations() {
         // Get a bunch of conversations, some group, some individual and in different states.
         list($user1, $user2, $user3, $user4, $ic1, $ic2, $ic3,
             $gc1, $gc2, $gc3, $gc4, $gc5, $gc6) = $this->create_conversation_test_data();
     /**
      * Test verifying get_conversations when there are users in a group and/or individual conversation. The reason this
      * test is performed is because we do not need as much data for group conversations (saving DB calls), so we want
      * to confirm this happens.
      */
-    public function test_get_conversations_user_in_group_and_individual_chat() {
+    public function test_get_conversations_user_in_group_and_individual_chat(): void {
+        $this->resetAfterTest();
         $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test verifying that group linked conversations are returned and contain a subname matching the course name.
      */
-    public function test_get_conversations_group_linked() {
+    public function test_get_conversations_group_linked(): void {
+        global $CFG, $DB;
+        $this->resetAfterTest();
         global $CFG, $DB;
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test that creation can't create the same conversation twice for 1:1 conversations.
      */
-    public function test_create_conversation_duplicate_conversations() {
+    public function test_create_conversation_duplicate_conversations(): void {
+        global $DB;
+        $this->resetAfterTest();
         global $DB;
         $user1 = $this::getDataGenerator()->create_user();
      * @dataProvider get_conversations_mixed_provider
      * @param array $usersdata The list of users to create for this test.
      * @param array $messagesdata The list of messages to create.
      * @param array $expectations The list of expected outcomes.
      */
-    public function test_get_conversations_mixed($usersdata, $contacts, $messagesdata, $expectations) {
+    public function test_get_conversations_mixed($usersdata, $contacts, $messagesdata, $expectations): void {
         global $DB;
+        $this->resetAfterTest();
+        $this->redirectMessages();
         // Create all of the users.
         $users = array();
         foreach ($usersdata as $username) {
             $from       = $users[$messagedata['from']];
             $to         = $users[$messagedata['to']];
             $subject    = $messagedata['subject'];
             if (isset($messagedata['state']) && $messagedata['state'] == 'unread') {
-                $messageid = $this->send_fake_message($from, $to, $subject);
+                $messageid = testhelper::send_fake_message($from, $to, $subject);
             } else {
                 // If there is no state, or the state is not 'unread', assume the message is read.
                 $messageid = message_post_message($from, $to, $subject, FORMAT_PLAIN);
+    }
     /**
      * Tests retrieving user contacts.
      */
+    public function test_get_user_contacts(): void {
+        $this->resetAfterTest();
     public function test_get_user_contacts() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Tests retrieving conversation messages.
      */
+    public function test_get_conversation_messages(): void {
+        $this->resetAfterTest();
     public function test_get_conversation_messages() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Tests retrieving group conversation messages.
      */
+    public function test_get_group_conversation_messages(): void {
+        $this->resetAfterTest();
     public function test_get_group_conversation_messages() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test verifying the sorting param for get_conversation_messages is respected().
      */
+    public function test_get_conversation_messages_sorting(): void {
+        $this->resetAfterTest();
     public function test_get_conversation_messages_sorting() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test retrieving conversation messages by providing a minimum timecreated value.
      */
+    public function test_get_conversation_messages_time_from_only(): void {
+        $this->resetAfterTest();
     public function test_get_conversation_messages_time_from_only() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test retrieving conversation messages by providing a maximum timecreated value.
      */
+    public function test_get_conversation_messages_time_to_only(): void {
+        $this->resetAfterTest();
     public function test_get_conversation_messages_time_to_only() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test retrieving conversation messages by providing a minimum and maximum timecreated value.
      */
+    public function test_get_conversation_messages_time_from_and_to(): void {
+        $this->resetAfterTest();
     public function test_get_conversation_messages_time_from_and_to() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
     /**
      * Test retrieving conversation messages by providing a limitfrom value.
+     */
+    public function test_get_conversation_messages_limitfrom_only(): void {
-     */
+        $this->resetAfterTest();
     public function test_get_conversation_messages_limitfrom_only() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test retrieving conversation messages by providing a limitnum value.
      */
+    public function test_get_conversation_messages_limitnum(): void {
+        $this->resetAfterTest();
     public function test_get_conversation_messages_limitnum() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Tests retrieving most recent conversation message.
      */
+    public function test_get_most_recent_conversation_message(): void {
+        $this->resetAfterTest();
     public function test_get_most_recent_conversation_message() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Tests checking if a user can mark all messages as read.
      */
+    public function test_can_mark_all_messages_as_read(): void {
+        $this->resetAfterTest();
     public function test_can_mark_all_messages_as_read() {
         // Set as the admin.
         $this->setAdminUser();
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
         // Send some messages back and forth.
         $time = 1;
-        $this->send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
+        testhelper::send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
-        $this->send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
+        testhelper::send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
-        $this->send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
+        testhelper::send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
-        $this->send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
+        testhelper::send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
+    }
     /**
      * Tests checking if a user can delete a conversation.
      */
+    public function test_can_delete_conversation(): void {
+        $this->resetAfterTest();
     public function test_can_delete_conversation() {
         // Set as the admin.
         $this->setAdminUser();
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         // Send some messages back and forth.
         $time = 1;
-        $this->send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
+        testhelper::send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
-        $this->send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
+        testhelper::send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
-        $this->send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
+        testhelper::send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
-        $this->send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
+        testhelper::send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
+    }
     /**
      * Tests deleting a conversation by conversation id.
      */
-    public function test_delete_conversation_by_id() {
+    public function test_delete_conversation_by_id(): void {
+        global $DB;
+        $this->resetAfterTest();
         global $DB;
         // Create some users.
         $sc1 = api::get_self_conversation($user1->id);
         $sc2 = api::get_self_conversation($user2->id);
         // Send some messages back and forth.
         $time = 1;
-        $m1id = $this->send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
+        $m1id = testhelper::send_fake_message($user1, $user2, 'Yo!', 0, $time + 1);
-        $m2id = $this->send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
+        $m2id = testhelper::send_fake_message($user2, $user1, 'Sup mang?', 0, $time + 2);
-        $m3id = $this->send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
+        $m3id = testhelper::send_fake_message($user1, $user2, 'Writing PHPUnit tests!', 0, $time + 3);
-        $m4id = $this->send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
+        $m4id = testhelper::send_fake_message($user2, $user1, 'Word.', 0, $time + 4);
         $m5id = testhelper::send_fake_message_to_conversation($user1, $sc1->id, 'Hi to myself!', $time + 5);
         $m6id = testhelper::send_fake_message_to_conversation($user2, $sc2->id, 'I am talking with myself', $time + 6);
+    }
     /**
      * Tests counting unread conversations.
      */
-    public function test_count_unread_conversations() {
+    public function test_count_unread_conversations(): void {
         $this->resetAfterTest(true);
         // Create some users.
         // The person wanting the conversation count.
         $this->setUser($user1);
         // Send some messages back and forth, have some different conversations with different users.
-        $this->send_fake_message($user1, $user2, 'Yo!');
+        testhelper::send_fake_message($user1, $user2, 'Yo!');
-        $this->send_fake_message($user2, $user1, 'Sup mang?');
+        testhelper::send_fake_message($user2, $user1, 'Sup mang?');
-        $this->send_fake_message($user1, $user2, 'Writing PHPUnit tests!');
+        testhelper::send_fake_message($user1, $user2, 'Writing PHPUnit tests!');
-        $this->send_fake_message($user2, $user1, 'Word.');
+        testhelper::send_fake_message($user2, $user1, 'Word.');
-        $this->send_fake_message($user1, $user3, 'Booyah');
+        testhelper::send_fake_message($user1, $user3, 'Booyah');
-        $this->send_fake_message($user3, $user1, 'Whaaat?');
+        testhelper::send_fake_message($user3, $user1, 'Whaaat?');
-        $this->send_fake_message($user1, $user3, 'Nothing.');
+        testhelper::send_fake_message($user1, $user3, 'Nothing.');
-        $this->send_fake_message($user3, $user1, 'Cool.');
+        testhelper::send_fake_message($user3, $user1, 'Cool.');
-        $this->send_fake_message($user1, $user4, 'Hey mate, you see the new messaging UI in Moodle?');
+        testhelper::send_fake_message($user1, $user4, 'Hey mate, you see the new messaging UI in Moodle?');
-        $this->send_fake_message($user4, $user1, 'Yah brah, it\'s pretty rad.');
+        testhelper::send_fake_message($user4, $user1, 'Yah brah, it\'s pretty rad.');
-        $this->send_fake_message($user1, $user4, 'Dope.');
+        testhelper::send_fake_message($user1, $user4, 'Dope.');
+    }
     /**
      * Tests counting unread conversations where one conversation is disabled.
      */
-    public function test_count_unread_conversations_disabled() {
+    public function test_count_unread_conversations_disabled(): void {
         $this->resetAfterTest(true);
         // Create some users.
         // The person wanting the conversation count.
         $this->setUser($user1);
         // Send some messages back and forth, have some different conversations with different users.
-        $this->send_fake_message($user1, $user2, 'Yo!');
+        testhelper::send_fake_message($user1, $user2, 'Yo!');
-        $this->send_fake_message($user2, $user1, 'Sup mang?');
+        testhelper::send_fake_message($user2, $user1, 'Sup mang?');
-        $this->send_fake_message($user1, $user2, 'Writing PHPUnit tests!');
+        testhelper::send_fake_message($user1, $user2, 'Writing PHPUnit tests!');
-        $this->send_fake_message($user2, $user1, 'Word.');
+        testhelper::send_fake_message($user2, $user1, 'Word.');
-        $this->send_fake_message($user1, $user3, 'Booyah');
+        testhelper::send_fake_message($user1, $user3, 'Booyah');
-        $this->send_fake_message($user3, $user1, 'Whaaat?');
+        testhelper::send_fake_message($user3, $user1, 'Whaaat?');
-        $this->send_fake_message($user1, $user3, 'Nothing.');
+        testhelper::send_fake_message($user1, $user3, 'Nothing.');
-        $this->send_fake_message($user3, $user1, 'Cool.');
+        testhelper::send_fake_message($user3, $user1, 'Cool.');
-        $this->send_fake_message($user1, $user4, 'Hey mate, you see the new messaging UI in Moodle?');
+        testhelper::send_fake_message($user1, $user4, 'Hey mate, you see the new messaging UI in Moodle?');
-        $this->send_fake_message($user4, $user1, 'Yah brah, it\'s pretty rad.');
+        testhelper::send_fake_message($user4, $user1, 'Yah brah, it\'s pretty rad.');
-        $this->send_fake_message($user1, $user4, 'Dope.');
+        testhelper::send_fake_message($user1, $user4, 'Dope.');
+    }
     /**
      * Tests deleting a conversation.
      */
+    public function test_get_all_message_preferences(): void {
+        $this->resetAfterTest();
     public function test_get_all_message_preferences() {
         $user = self::getDataGenerator()->create_user();
         $this->setUser($user);
+    }
     /**
      * Tests the user can send a message.
      */
+    public function test_can_send_message(): void {
+        $this->resetAfterTest();
     public function test_can_send_message() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Tests the user can't send a message without proper capability.
      */
-    public function test_can_send_message_without_sendmessage_cap() {
+    public function test_can_send_message_without_sendmessage_cap(): void {
+        global $DB;
+        $this->resetAfterTest();
         global $DB;
         // Create some users.
+    }
     /**
      * Tests the user can send a message when they are contact.
      */
+    public function test_can_send_message_when_contact(): void {
+        $this->resetAfterTest();
     public function test_can_send_message_when_contact() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
     /**
      * Tests the user can't send a message if they are not a contact and the user
      * has requested messages only from contacts.
      */
+    public function test_can_send_message_when_not_contact(): void {
+        $this->resetAfterTest();
     public function test_can_send_message_when_not_contact() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Tests the user can't send a message if they are blocked.
      */
+    public function test_can_send_message_when_blocked(): void {
+        $this->resetAfterTest();
     public function test_can_send_message_when_blocked() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
     /**
      * Tests the user can send a message when site-wide messaging setting is enabled,
      * even if they are not a contact and are not members of the same course.
      */
+    public function test_can_send_message_site_messaging_setting(): void {
+        $this->resetAfterTest();
     public function test_can_send_message_site_messaging_setting() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Tests the user with the messageanyuser capability can send a message.
      */
-    public function test_can_send_message_with_messageanyuser_cap() {
+    public function test_can_send_message_with_messageanyuser_cap(): void {
+        global $DB;
+        $this->resetAfterTest();
         global $DB;
         // Create some users.
         $teacher1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Tests the user when blocked will not be able to send messages if they are blocked.
      */
-    public function test_can_send_message_even_if_blocked() {
+    public function test_can_send_message_even_if_blocked(): void {
         $this->resetAfterTest();
     /**
      * Tests the user will be able to send a message even if they are blocked as the user
      * has the capability 'moodle/site:messageanyuser'.
      */
-    public function test_can_send_message_even_if_blocked_with_message_any_user_cap() {
+    public function test_can_send_message_even_if_blocked_with_message_any_user_cap(): void {
         global $DB;
     /**
      * Tests the user will be able to send a message even if they are blocked as the user
      * has the capability 'moodle/site:readallmessages'.
      */
-    public function test_can_send_message_even_if_blocked_with_read_all_message_cap() {
+    public function test_can_send_message_even_if_blocked_with_read_all_message_cap(): void {
         global $DB;
+    }
     /**
      * Tests the user can not always send a message if they are blocked just because they share a course.
      */
-    public function test_can_send_message_even_if_blocked_shared_course() {
+    public function test_can_send_message_even_if_blocked_shared_course(): void {
         $this->resetAfterTest();
         // Create some users.
     /**
      * Tests the user can always send a message even if they are blocked because they share a course and
      * have the capability 'moodle/site:messageanyuser' at the course context.
      */
-    public function test_can_send_message_even_if_blocked_shared_course_with_message_any_user_cap() {
+    public function test_can_send_message_even_if_blocked_shared_course_with_message_any_user_cap(): void {
         global $DB;
+    }
     /**
      * Verify the expected behaviour of the can_send_message_to_conversation() method for authenticated users with default settings.
      */
+    public function test_can_send_message_to_conversation_basic(): void {
+        $this->resetAfterTest();
     public function test_can_send_message_to_conversation_basic() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Verify the behaviour of can_send_message_to_conversation() for authenticated users without the sendmessage capability.
      */
-    public function test_can_send_message_to_conversation_sendmessage_cap() {
+    public function test_can_send_message_to_conversation_sendmessage_cap(): void {
+        global $DB;
+        $this->resetAfterTest();
         global $DB;
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Verify the behaviour of can_send_message_to_conversation() for authenticated users without the messageanyuser capability.
      */
-    public function test_can_send_message_to_conversation_messageanyuser_cap() {
+    public function test_can_send_message_to_conversation_messageanyuser_cap(): void {
+        global $DB;
+        $this->resetAfterTest();
         global $DB;
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test verifying that users cannot send messages to conversations they are not a part of.
      */
+    public function test_can_send_message_to_conversation_non_member(): void {
+        $this->resetAfterTest();
     public function test_can_send_message_to_conversation_non_member() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test verifying the behaviour of the can_send_message_to_conversation method when privacy is set to contacts only.
      */
+    public function test_can_send_message_to_conversation_privacy_contacts_only(): void {
+        $this->resetAfterTest();
     public function test_can_send_message_to_conversation_privacy_contacts_only() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test verifying the behaviour of the can_send_message_to_conversation method when privacy is set to contacts / course members.
      */
+    public function test_can_send_message_to_conversation_privacy_contacts_course(): void {
+        $this->resetAfterTest();
     public function test_can_send_message_to_conversation_privacy_contacts_course() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test verifying the behaviour of the can_send_message_to_conversation method when privacy is set to any user.
      */
+    public function test_can_send_message_to_conversation_privacy_sitewide(): void {
+        $this->resetAfterTest();
     public function test_can_send_message_to_conversation_privacy_sitewide() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test verifying the behaviour of the can_send_message_to_conversation method when a user is blocked.
      */
+    public function test_can_send_message_to_conversation_when_blocked(): void {
+        $this->resetAfterTest();
     public function test_can_send_message_to_conversation_when_blocked() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Tests get_user_privacy_messaging_preference method.
      */
+    public function test_get_user_privacy_messaging_preference(): void {
+        $this->resetAfterTest();
     public function test_get_user_privacy_messaging_preference() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /*
      * Tes get_message_processor api.
      */
+    public function test_get_message_processor(): void {
+        $this->resetAfterTest();
     public function test_get_message_processor() {
         $processors = get_message_processors(true);
         if (empty($processors)) {
             $this->markTestSkipped("No message processors found");
+    }
     /**
      * Test method update_processor_status.
      */
+    public function test_update_processor_status(): void {
+        $this->resetAfterTest();
     public function test_update_processor_status() {
         $processors = get_message_processors();
         if (empty($processors)) {
             $this->markTestSkipped("No message processors found");
+        }
+    }
     /**
      * Test returning blocked users.
      */
-    public function test_get_blocked_users() {
+    public function test_get_blocked_users(): void {
+        global $USER;
+        $this->resetAfterTest();
         global $USER;
         // Set this user as the admin.
+    }
     /**
      * Test marking a message as read.
      */
-    public function test_mark_message_as_read() {
+    public function test_mark_message_as_read(): void {
+        global $DB;
-        global $DB;
+        $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
-        $this->send_fake_message($user1, $user2);
+        testhelper::send_fake_message($user1, $user2);
-        $m2id = $this->send_fake_message($user1, $user2);
+        $m2id = testhelper::send_fake_message($user1, $user2);
-        $this->send_fake_message($user2, $user1);
+        testhelper::send_fake_message($user2, $user1);
-        $m4id = $this->send_fake_message($user2, $user1);
+        $m4id = testhelper::send_fake_message($user2, $user1);
+    }
     /**
      * Test marking a notification as read.
      */
-    public function test_mark_notification_as_read() {
+    public function test_mark_notification_as_read(): void {
+        global $DB;
-        global $DB;
+        $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
-        $this->send_fake_message($user1, $user2, 'Notification 1', 1);
+        testhelper::send_fake_message($user1, $user2, 'Notification 1', 1);
-        $n2id = $this->send_fake_message($user1, $user2, 'Notification 2', 1);
+        $n2id = testhelper::send_fake_message($user1, $user2, 'Notification 2', 1);
-        $this->send_fake_message($user2, $user1, 'Notification 3', 1);
+        testhelper::send_fake_message($user2, $user1, 'Notification 3', 1);
-        $n4id = $this->send_fake_message($user2, $user1, 'Notification 4', 1);
+        $n4id = testhelper::send_fake_message($user2, $user1, 'Notification 4', 1);
+    }
     /**
      * Test a conversation is not returned if there is none.
      */
+    public function test_get_conversation_between_users_no_conversation(): void {
+        $this->resetAfterTest();
     public function test_get_conversation_between_users_no_conversation() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $this->assertFalse(api::get_conversation_between_users([$user1->id, $user2->id]));
+    }
     /**
+     * Test count_conversation_members for non existing conversation.
+     */
-     * Test count_conversation_members for non existing conversation.
+    public function test_count_conversation_members_no_existing_conversation(): void {
+        $this->resetAfterTest();
-     */
+        $this->assertEquals(
-    public function test_count_conversation_members_no_existing_conversation() {
+,
-        $this->assertEquals(0,
+            api::count_conversation_members(0)
-            api::count_conversation_members(0));
+        );
+    }
+    /**
+     * Test count_conversation_members for existing conversation.
-    /**
+     */
-     * Test count_conversation_members for existing conversation.
+    public function test_count_conversation_members_existing_conversation(): void {
-     */
+        $this->resetAfterTest();
     public function test_count_conversation_members_existing_conversation() {
+    }
     /**
      * Test add_members_to_conversation for an individual conversation.
      */
+    public function test_add_members_to_individual_conversation(): void {
+        $this->resetAfterTest();
     public function test_add_members_to_individual_conversation() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test add_members_to_conversation for existing conversation.
      */
+    public function test_add_members_to_existing_conversation(): void {
+        $this->resetAfterTest();
     public function test_add_members_to_existing_conversation() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test add_members_to_conversation for non existing conversation.
      */
+    public function test_add_members_to_no_existing_conversation(): void {
+        $this->resetAfterTest();
     public function test_add_members_to_no_existing_conversation() {
         $user1 = self::getDataGenerator()->create_user();
         // Throw dml_missing_record_exception for non existing conversation.
         $this->expectException('dml_missing_record_exception');
         api::add_members_to_conversation([$user1->id], 0);
+    }
     /**
+     * Test add_member_to_conversation for non existing user.
+     */
-     * Test add_member_to_conversation for non existing user.
+    public function test_add_members_to_no_existing_user(): void {
-     */
+        $this->resetAfterTest();
     public function test_add_members_to_no_existing_user() {
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test add_members_to_conversation for current conversation member.
      */
+    public function test_add_members_to_current_conversation_member(): void {
+        $this->resetAfterTest();
     public function test_add_members_to_current_conversation_member() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test add_members_to_conversation for multiple users.
      */
+    public function test_add_members_for_multiple_users(): void {
+        $this->resetAfterTest();
     public function test_add_members_for_multiple_users() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test add_members_to_conversation for multiple users, included non existing and current conversation members
      */
+    public function test_add_members_for_multiple_not_valid_users(): void {
+        $this->resetAfterTest();
     public function test_add_members_for_multiple_not_valid_users() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test remove_members_from_conversation for individual conversation.
      */
+    public function test_remove_members_from_individual_conversation(): void {
+        $this->resetAfterTest();
     public function test_remove_members_from_individual_conversation() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test remove_members_from_conversation for existing conversation.
      */
+    public function test_remove_members_from_existing_conversation(): void {
+        $this->resetAfterTest();
     public function test_remove_members_from_existing_conversation() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test remove_members_from_conversation for non existing conversation.
      */
+    public function test_remove_members_from_no_existing_conversation(): void {
+        $this->resetAfterTest();
     public function test_remove_members_from_no_existing_conversation() {
         $user1 = self::getDataGenerator()->create_user();
         // Throw dml_missing_record_exception for non existing conversation.
         $this->expectException('dml_missing_record_exception');
         api::remove_members_from_conversation([$user1->id], 0);
+    }
     /**
+     * Test remove_members_from_conversation for non existing user.
+     */
-     * Test remove_members_from_conversation for non existing user.
+    public function test_remove_members_for_no_existing_user(): void {
-     */
+        $this->resetAfterTest();
     public function test_remove_members_for_no_existing_user() {
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test remove_members_from_conversation for multiple users.
      */
+    public function test_remove_members_for_multiple_users(): void {
+        $this->resetAfterTest();
     public function test_remove_members_for_multiple_users() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test remove_members_from_conversation for multiple non valid users.
      */
+    public function test_remove_members_for_multiple_no_valid_users(): void {
+        $this->resetAfterTest();
     public function test_remove_members_for_multiple_no_valid_users() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test count_conversation_members for empty conversation.
      */
+    public function test_count_conversation_members_empty_conversation(): void {
+        $this->resetAfterTest();
     public function test_count_conversation_members_empty_conversation() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test can create a contact request.
      */
-    public function test_can_create_contact_request() {
+    public function test_can_create_contact_request(): void {
+        global $CFG;
-        global $CFG;
+        $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test creating a contact request.
      */
-    public function test_create_contact_request() {
+    public function test_create_contact_request(): void {
+        global $DB;
-        global $DB;
+        $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test confirming a contact request.
      */
-    public function test_confirm_contact_request() {
+    public function test_confirm_contact_request(): void {
+        global $DB;
-        global $DB;
+        $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test declining a contact request.
      */
-    public function test_decline_contact_request() {
+    public function test_decline_contact_request(): void {
+        global $DB;
-        global $DB;
+        $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test retrieving contact requests.
      */
-    public function test_get_contact_requests() {
+    public function test_get_contact_requests(): void {
+        global $PAGE;
-        global $PAGE;
+        $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test the get_contact_requests() function when the user has blocked the sender of the request.
      */
+    public function test_get_contact_requests_blocked_sender(): void {
-    public function test_get_contact_requests_blocked_sender() {
+        $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test getting contact requests when there are none.
      */
-    public function test_get_contact_requests_no_requests() {
+    public function test_get_contact_requests_no_requests(): void {
         $this->resetAfterTest();
+    }
     /**
      * Test getting contact requests with limits.
      */
-    public function test_get_contact_requests_with_limits() {
+    public function test_get_contact_requests_with_limits(): void {
         $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test adding contacts.
      */
-    public function test_add_contact() {
+    public function test_add_contact(): void {
+        global $DB;
-        global $DB;
+        $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test removing contacts.
      */
-    public function test_remove_contact() {
+    public function test_remove_contact(): void {
+        global $DB;
-        global $DB;
+        $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test blocking users.
      */
-    public function test_block_user() {
+    public function test_block_user(): void {
+        global $DB;
-        global $DB;
+        $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test unblocking users.
      */
-    public function test_unblock_user() {
+    public function test_unblock_user(): void {
+        global $DB;
-        global $DB;
+        $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test muting a conversation.
      */
-    public function test_mute_conversation() {
+    public function test_mute_conversation(): void {
+        global $DB;
-        global $DB;
+        $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test unmuting a conversation.
      */
-    public function test_unmute_conversation() {
+    public function test_unmute_conversation(): void {
+        global $DB;
-        global $DB;
+        $this->resetAfterTest();
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test if a conversation is muted.
      */
+    public function test_is_conversation_muted(): void {
+        $this->resetAfterTest();
     public function test_is_conversation_muted() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test is contact check.
      */
+    public function test_is_contact(): void {
+        $this->resetAfterTest();
     public function test_is_contact() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test get contact.
      */
+    public function test_get_contact(): void {
+        $this->resetAfterTest();
     public function test_get_contact() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test is blocked checked.
      */
+    public function test_is_blocked(): void {
+        $this->resetAfterTest();
     public function test_is_blocked() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test the contact request exist check.
      */
+    public function test_does_contact_request_exist(): void {
+        $this->resetAfterTest();
     public function test_does_contact_request_exist() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test the get_received_contact_requests_count() function.
      */
+    public function test_get_received_contact_requests_count(): void {
+        $this->resetAfterTest();
     public function test_get_received_contact_requests_count() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test the get_received_contact_requests_count() function when the user has blocked the sender of the request.
      */
+    public function test_get_received_contact_requests_count_blocked_sender(): void {
+        $this->resetAfterTest();
     public function test_get_received_contact_requests_count_blocked_sender() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test the get_contact_requests_between_users() function.
      */
+    public function test_get_contact_requests_between_users(): void {
+        $this->resetAfterTest();
     public function test_get_contact_requests_between_users() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test the user in conversation check.
      */
+    public function test_is_user_in_conversation(): void {
+        $this->resetAfterTest();
     public function test_is_user_in_conversation() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test the user in conversation check when they are not.
      */
+    public function test_is_user_in_conversation_when_not(): void {
+        $this->resetAfterTest();
     public function test_is_user_in_conversation_when_not() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test can create a group conversation.
      */
-    public function test_can_create_group_conversation() {
+    public function test_can_create_group_conversation(): void {
+        global $CFG;
-        global $CFG;
+        $this->resetAfterTest();
         $student = self::getDataGenerator()->create_user();
+    }
     /**
      * Test creating an individual conversation.
      */
+    public function test_create_conversation_individual(): void {
+        $this->resetAfterTest();
     public function test_create_conversation_individual() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test creating a group conversation.
      */
+    public function test_create_conversation_group(): void {
+        $this->resetAfterTest();
     public function test_create_conversation_group() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test creating an invalid conversation.
      */
-    public function test_create_conversation_invalid() {
+    public function test_create_conversation_invalid(): void {
         $this->expectException('moodle_exception');
         api::create_conversation(3, [1, 2, 3]);
+    }
     /**
      * Test creating an individual conversation with too many members.
      */
-    public function test_create_conversation_individual_too_many_members() {
+    public function test_create_conversation_individual_too_many_members(): void {
         $this->expectException('moodle_exception');
         api::create_conversation(api::MESSAGE_CONVERSATION_TYPE_INDIVIDUAL, [1, 2, 3]);
+    }
     /**
+     * Test create message conversation with area.
+     */
-     * Test create message conversation with area.
+    public function test_create_conversation_with_area(): void {
-     */
+        $this->resetAfterTest();
     public function test_create_conversation_with_area() {
         $contextid = 111;
         $itemid = 222;
+    }
     /**
      * Test get_conversation_by_area.
      */
+    public function test_get_conversation_by_area(): void {
+        $this->resetAfterTest();
     public function test_get_conversation_by_area() {
         $contextid = 111;
         $itemid = 222;
         $name = 'Name of conversation';
         $createconversation = api::create_conversation(
+    }
     /**
      * Test enable_conversation.
      */
-    public function test_enable_conversation() {
+    public function test_enable_conversation(): void {
+        global $DB;
-        global $DB;
+        $this->resetAfterTest();
         $name = 'Name of conversation';
+    }
     /**
      * Test disable_conversation.
      */
-    public function test_disable_conversation() {
+    public function test_disable_conversation(): void {
+        global $DB;
+        $this->resetAfterTest();
         global $DB;
         $name = 'Name of conversation';
+    }
     /**
      * Test update_conversation_name.
      */
-    public function test_update_conversation_name() {
+    public function test_update_conversation_name(): void {
+        global $DB;
-        global $DB;
+        $this->resetAfterTest();
         $conversation = api::create_conversation(api::MESSAGE_CONVERSATION_TYPE_GROUP, []);
+    }
     /**
      * Test returning members in a conversation with no contact requests.
      */
+    public function test_get_conversation_members(): void {
+        $this->resetAfterTest();
     public function test_get_conversation_members() {
         $lastaccess = new \stdClass();
         $lastaccess->lastaccess = time();
+    }
     /**
      * Test returning members in a conversation with contact requests.
      */
+    public function test_get_conversation_members_with_contact_requests(): void {
+        $this->resetAfterTest();
     public function test_get_conversation_members_with_contact_requests() {
         $lastaccess = new \stdClass();
         $lastaccess->lastaccess = time();
+    }
     /**
      * Test returning members of a self conversation.
      */
+    public function test_get_conversation_members_with_self_conversation(): void {
+        $this->resetAfterTest();
     public function test_get_conversation_members_with_self_conversation() {
         $lastaccess = new \stdClass();
         $lastaccess->lastaccess = time();
+    }
     /**
      * Test verifying that messages can be sent to existing individual conversations.
      */
+    public function test_send_message_to_conversation_individual_conversation(): void {
+        $this->resetAfterTest();
     public function test_send_message_to_conversation_individual_conversation() {
         // Get a bunch of conversations, some group, some individual and in different states.
         list($user1, $user2, $user3, $user4, $ic1, $ic2, $ic3,
             $gc1, $gc2, $gc3, $gc4, $gc5, $gc6) = $this->create_conversation_test_data();
+    }
     /**
      * Test verifying that messages can be sent to existing group conversations.
      */
+    public function test_send_message_to_conversation_group_conversation(): void {
-    public function test_send_message_to_conversation_group_conversation() {
+        $this->resetAfterTest();
         // Get a bunch of conversations, some group, some individual and in different states.
         list($user1, $user2, $user3, $user4, $ic1, $ic2, $ic3,
             $gc1, $gc2, $gc3, $gc4, $gc5, $gc6) = $this->create_conversation_test_data();
+    }
     /**
      * Test verifying that messages can be sent to existing linked group conversations.
      */
-    public function test_send_message_to_conversation_linked_group_conversation() {
+    public function test_send_message_to_conversation_linked_group_conversation(): void {
+        global $CFG, $PAGE;
-        global $CFG, $PAGE;
+        $this->resetAfterTest();
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
+    }
     /**
      * Test verifying that messages cannot be sent to conversations that don't exist.
      */
+    public function test_send_message_to_conversation_non_existent_conversation(): void {
+        $this->resetAfterTest();
     public function test_send_message_to_conversation_non_existent_conversation() {
         // Get a bunch of conversations, some group, some individual and in different states.
         list($user1, $user2, $user3, $user4, $ic1, $ic2, $ic3,
             $gc1, $gc2, $gc3, $gc4, $gc5, $gc6) = $this->create_conversation_test_data();
+    }
     /**
      * Test verifying that messages cannot be sent to conversations by users who are not members.
      */
+    public function test_send_message_to_conversation_non_member(): void {
+        $this->resetAfterTest();
     public function test_send_message_to_conversation_non_member() {
         // Get a bunch of conversations, some group, some individual and in different states.
         list($user1, $user2, $user3, $user4, $ic1, $ic2, $ic3,
             $gc1, $gc2, $gc3, $gc4, $gc5, $gc6) = $this->create_conversation_test_data();
+    }
     /**
      * Test verifying that messages cannot be sent to conversations by users who are not members.
      */
+    public function test_send_message_to_conversation_blocked_user(): void {
+        $this->resetAfterTest();
     public function test_send_message_to_conversation_blocked_user() {
         // Get a bunch of conversations, some group, some individual and in different states.
         list($user1, $user2, $user3, $user4, $ic1, $ic2, $ic3,
             $gc1, $gc2, $gc3, $gc4, $gc5, $gc6) = $this->create_conversation_test_data();
+    }
     /**
      * Test the get_conversation() function with a muted conversation.
      */
-    public function test_get_conversation_with_muted_conversation() {
+    public function test_get_conversation_with_muted_conversation(): void {
         $this->resetAfterTest();
         $deletemessages,
         $arguments,
         $expectedcounts,
         $expectedunreadcounts,
         $deletedusers
-    ) {
+    ): void {
+        $this->resetAfterTest();
         $generator = $this->getDataGenerator();
         $users = [
             $generator->create_user(),
             $generator->create_user(),
             $generator->create_user(),
+    }
     /**
      * Test the count_contacts() function.
      */
+    public function test_count_contacts(): void {
+        $this->resetAfterTest();
     public function test_count_contacts() {
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();
         $deletemessages,
         $arguments,
         $expectedcounts,
         $expectedunreadcounts,
         $deletedusers
-    ) {
+    ): void {
         $this->resetAfterTest();
         $generator = $this->getDataGenerator();
         $users = [
             $generator->create_user(),
             $generator->create_user(),
             $counts['types'][api::MESSAGE_CONVERSATION_TYPE_GROUP]);
         $this->assertEquals($expectedunreadcounts['types'][api::MESSAGE_CONVERSATION_TYPE_SELF],
             $counts['types'][api::MESSAGE_CONVERSATION_TYPE_SELF]);
+    }
-    public function test_delete_all_conversation_data() {
+    public function test_delete_all_conversation_data(): void {
         global $DB;
+    }
     /**
      * Tests the user can delete message for all users as a teacher.
      */
-    public function test_can_delete_message_for_all_users_teacher() {
+    public function test_can_delete_message_for_all_users_teacher(): void {
         global $DB;
         $this->resetAfterTest(true);
+    }
     /**
      * Tests the user can delete message for all users as a student.
      */
-    public function test_can_delete_message_for_all_users_student() {
+    public function test_can_delete_message_for_all_users_student(): void {
         $this->resetAfterTest(true);
+    }
     /**
      * Tests tdelete message for all users in group conversation.
      */
-    public function test_delete_message_for_all_users_group_conversation() {
+    public function test_delete_message_for_all_users_group_conversation(): void {
         global $DB;
         $this->resetAfterTest(true);
+    }
     /**
      * Tests delete message for all users in private conversation.
      */
-    public function test_delete_message_for_all_users_individual_conversation() {
+    public function test_delete_message_for_all_users_individual_conversation(): void {
         global $DB;
         $this->resetAfterTest(true);
     /**
      * Test retrieving conversation messages by providing a timefrom higher than last message timecreated. It should return no
      * messages but keep the return structure to not break when called from the ws.
      */
+    public function test_get_conversation_messages_timefrom_higher_than_last_timecreated(): void {
+        $this->resetAfterTest();
     public function test_get_conversation_messages_timefrom_higher_than_last_timecreated() {
         // Create some users.
         $user1 = self::getDataGenerator()->create_user();
         $user2 = self::getDataGenerator()->create_user();
         $user3 = self::getDataGenerator()->create_user();

Proyectos de Subversion Moodle

(root)/message/tests/api_test.php – Rev 1 → 11