MDL-78750 communication: Fix missing test properties

moodle · Sep 27, 2023 · 9a4a2df · 9a4a2df
1 parent 9739399
commit 9a4a2df
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/communication/provider/customlink/tests/communication_feature_test.php b/communication/provider/customlink/tests/communication_feature_test.php
@@ -117,9 +117,9 @@ protected function get_test_communication_processor(): processor {
     /**
      * Test if the selected provider is configured.
      *
-     * @return void
+     * @covers ::is_configured
      */
-    public function test_is_configured() {
+    public function test_is_configured(): void {
         $communicationprocessor = $this->get_test_communication_processor();
         $this->assertTrue($communicationprocessor->get_form_provider()->is_configured());
     }

diff --git a/communication/provider/matrix/tests/communication_feature_test.php b/communication/provider/matrix/tests/communication_feature_test.php
@@ -52,7 +52,7 @@ public function setUp(): void {
      *
      * @covers ::create_chat_room
      */
-    public function test_create_chat_room() {
+    public function test_create_chat_room(): void {
         // Set up the test data first.
         $communication = \core_communication\api::load_by_instance(
             component: 'communication_matrix',
@@ -296,7 +296,7 @@ public function test_update_room_avatar(
      *
      * @return array
      */
-    public function avatar_provider(): array {
+    public static function avatar_provider(): array {
         return [
             'Empty to avatar' => [
                 null,
@@ -543,9 +543,9 @@ protected function create_room(
     /**
      * Test if the selected provider is configured.
      *
-     * @return void
+     * @covers ::is_configured
      */
-    public function test_is_configured() {
+    public function test_is_configured(): void {
         $course = $this->get_course();
         $communicationprocessor = processor::load_by_instance(
             component: 'core_course',