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 2, 2023
1 parent da216a5 commit d92966f
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::ERROR, $checks[3]->get_status()); // Airnotifier NOT configured, missing key.
$this->assertEquals(\core\check\result::OK, $checks[4]->get_status()); // Airnotifier URL available.
$this->assertEquals(\core\check\result::ERROR, $checks[5]->get_status()); // Missing access key.
$this->assertEquals(\core\check\result::OK, $checks[6]->get_status()); // Enough default mobile notifications.
$this->assertEquals(\core\check\result::ERROR, $checks[7]->get_status()); // No registered devices yet.
}

Expand All @@ -70,7 +69,6 @@ public function test_check_configuration_with_token() {
$this->assertEquals(\core\check\result::OK, $checks[3]->get_status()); // Airnotifier configured.
$this->assertEquals(\core\check\result::OK, $checks[4]->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::OK, $checks[5]->get_status()); // Enough default mobile notifications.
$this->assertEquals(\core\check\result::ERROR, $checks[6]->get_status()); // No registered devices yet.
}

Expand Down Expand Up @@ -98,7 +96,6 @@ public function test_check_configuration_incorrect_settings() {
$this->assertEquals(\core\check\result::ERROR, $checks[4]->get_status()); // Airnotifier URL available.
$this->assertEquals(\core\check\result::OK, $checks[5]->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::OK, $checks[6]->get_status()); // Enough default mobile notifications.
$this->assertEquals(\core\check\result::ERROR, $checks[7]->get_status()); // No registered devices yet.
}

Expand Down

0 comments on commit d92966f

Please sign in to comment.