From 9a4a2df0e4629d2169db128175f1f84445d77950 Mon Sep 17 00:00:00 2001
From: Andrew Nicols <andrew@nicols.co.uk>
Date: Wed, 27 Sep 2023 14:11:33 +0800
Subject: [PATCH] MDL-78750 communication: Fix missing test properties

---
 .../customlink/tests/communication_feature_test.php       | 4 ++--
 .../provider/matrix/tests/communication_feature_test.php  | 8 ++++----
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/communication/provider/customlink/tests/communication_feature_test.php b/communication/provider/customlink/tests/communication_feature_test.php
index 37663c3fd9d..136da680508 100644
--- a/communication/provider/customlink/tests/communication_feature_test.php
+++ b/communication/provider/customlink/tests/communication_feature_test.php
@@ -117,9 +117,9 @@ class communication_feature_test extends \advanced_testcase {
     /**
      * 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
index 496bd0e85d1..20ec6ee9ae0 100644
--- a/communication/provider/matrix/tests/communication_feature_test.php
+++ b/communication/provider/matrix/tests/communication_feature_test.php
@@ -52,7 +52,7 @@ class communication_feature_test extends \advanced_testcase {
      *
      * @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 @@ class communication_feature_test extends \advanced_testcase {
      *
      * @return array
      */
-    public function avatar_provider(): array {
+    public static function avatar_provider(): array {
         return [
             'Empty to avatar' => [
                 null,
@@ -543,9 +543,9 @@ class communication_feature_test extends \advanced_testcase {
     /**
      * 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',