Skip to content

Commit

Permalink
MDL-77123 message_airnotifier: Don't test configured providers
Browse files Browse the repository at this point in the history
  • Loading branch information
cameron1729 committed Feb 8, 2023
1 parent 00ccb3a commit c56edaa
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions message/output/airnotifier/tests/manager_test.php
Expand Up @@ -46,7 +46,6 @@ public function test_check_configuration_default() {
$this->assertEquals(\core\check\result::OK, $checks[3]->get_status()); // Airnotifier URL available.
$this->assertEquals(\core\check\result::ERROR, $checks[4]->get_status()); // Missing access key.
$this->assertEquals(\core\check\result::WARNING, $checks[5]->get_status()); // Only a few of default mobile notifications.
$this->assertEquals(\core\check\result::ERROR, $checks[6]->get_status()); // No registered devices yet.
}

/** Test check_configuration with token **/
Expand All @@ -69,7 +68,6 @@ public function test_check_configuration_with_token() {
$this->assertEquals(\core\check\result::OK, $checks[3]->get_status()); // Airnotifier URL available.
// The original function fourth check (access key valid in the remote Airnotifier server) is not mockable.
$this->assertEquals(\core\check\result::WARNING, $checks[4]->get_status()); // Only a few of default mobile notifications.
$this->assertEquals(\core\check\result::ERROR, $checks[5]->get_status()); // No registered devices yet.
}

/** Test check_configuration bad settings **/
Expand All @@ -94,7 +92,6 @@ public function test_check_configuration_incorrect_settings() {
$this->assertEquals(\core\check\result::OK, $checks[4]->get_status()); // Invalid setting (empty space).
// The original function fifth check (access key valid in the remote Airnotifier server) is not mockable.
$this->assertEquals(\core\check\result::WARNING, $checks[5]->get_status()); // Only a few of default mobile notifications.
$this->assertEquals(\core\check\result::ERROR, $checks[6]->get_status()); // No registered devices yet.
}

/** Test has_enabled_devices **/
Expand Down

0 comments on commit c56edaa

Please sign in to comment.