L-electron-Rare/moodle
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MDL-78750 communication: Fix missing test properties

Browse Source
This commit is contained in:
Andrew Nicols
2023-09-27 14:12:53 +08:00
parent 9739399741
commit 9a4a2df0e4
2 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
communication/provider/customlink/tests/communication_feature_test.php
+2 -2
View File
@@ -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());
}
communication/provider/matrix/tests/communication_feature_test.php
+4 -4
View File
@@ -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',