[Asterisk-code-review] ARI: Add coverage for mailbox 404 errors. (testsuite[16])

Jenkins2 asteriskteam at digium.com
Wed Jul 25 12:58:34 CDT 2018


Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/9678 )

Change subject: ARI: Add coverage for mailbox 404 errors.
......................................................................

ARI: Add coverage for mailbox 404 errors.

Change-Id: I258197b14de9e8087ef5796ab1f6b641678978ef
---
M tests/rest_api/mailbox/baseline/mailbox_baseline.py
1 file changed, 8 insertions(+), 0 deletions(-)

Approvals:
  Joshua Colp: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved
  Jenkins2: Approved for Submit



diff --git a/tests/rest_api/mailbox/baseline/mailbox_baseline.py b/tests/rest_api/mailbox/baseline/mailbox_baseline.py
index 4e45462..de2c504 100644
--- a/tests/rest_api/mailbox/baseline/mailbox_baseline.py
+++ b/tests/rest_api/mailbox/baseline/mailbox_baseline.py
@@ -67,6 +67,14 @@
     LOGGER.info("Changed alice_mailbox. Confirmed new values.")
     LOGGER.info("Deleted bob_mailbox. Confirmed removal from list.")
 
+    ari.set_allow_errors(True)
+    resp = ari.get('mailboxes', 'bob_mailbox')
+    assert resp.status_code == 404
+
+    resp = ari.delete('mailboxes', 'bob_mailbox')
+    assert resp.status_code == 404
+    ari.set_allow_errors(False)
+
     # While this test doesn't actually use the channel for any of the ARI
     # operations that it does, its presence is necessary for triggering
     # the test and removing it is necessary for completing the test.

-- 
To view, visit https://gerrit.asterisk.org/9678
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: testsuite
Gerrit-Branch: 16
Gerrit-MessageType: merged
Gerrit-Change-Id: I258197b14de9e8087ef5796ab1f6b641678978ef
Gerrit-Change-Number: 9678
Gerrit-PatchSet: 1
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180725/175353d9/attachment-0001.html>


More information about the asterisk-code-review mailing list