[asterisk-dev] [Code Review] 3118: Testsuite: Add tests for ARI control of external MWI (ARI mailboxes resource)

opticron reviewboard at asterisk.org
Thu Jan 16 10:47:01 CST 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3118/#review10601
-----------------------------------------------------------



/asterisk/trunk/tests/rest_api/mailbox/baseline/mailbox_baseline.py
<https://reviewboard.asterisk.org/r/3118/#comment20088>

    This could be simplified to:
    expected = [
        {"name": "alice_mailbox", "old_messages": 3, "new_messages": 4},
        {"name": "bob_mailbox", "old_messages": 5, "new_messages": 6}
    ]
    assert set(mailboxes) == set(expected)



/asterisk/trunk/tests/rest_api/mailbox/baseline/mailbox_baseline.py
<https://reviewboard.asterisk.org/r/3118/#comment20089>

    A similar operation could be performed here.



/asterisk/trunk/tests/rest_api/mailbox/baseline/test-config.yaml
<https://reviewboard.asterisk.org/r/3118/#comment20090>

    12.1.0


- opticron


On Jan. 9, 2014, 5:40 p.m., Jonathan Rose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3118/
> -----------------------------------------------------------
> 
> (Updated Jan. 9, 2014, 5:40 p.m.)
> 
> 
> Review request for Asterisk Developers, Kevin Harwell, Matt Jordan, and Mark Michelson.
> 
> 
> Repository: testsuite
> 
> 
> Description
> -------
> 
> Somewhat similar to kharwell's device state tests.
> 
> ARI is used to:
> * create a couple mailboxes
> * confirm that GET can be used to get one mailbox and that it matches expectations
> * confirm that GET can be used to get all the mailboxes and that they match expectations
> * modify one mailbox
> * delete another mailbox
> * verify that the mailbox that was deleted is no longer obtained by get
> * verify that the mailbox that was modified has the modified state.
> 
> The nature of these operations is simple enough that this seemed appropriate to be a single test.
> 
> 
> Diffs
> -----
> 
>   /asterisk/trunk/tests/rest_api/tests.yaml 4545 
>   /asterisk/trunk/tests/rest_api/mailbox/baseline/test-config.yaml PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/mailbox/baseline/mailbox_baseline.py PRE-CREATION 
>   /asterisk/trunk/tests/rest_api/mailbox/baseline/configs/ast1/extensions.conf PRE-CREATION 
> 
> Diff: https://reviewboard.asterisk.org/r/3118/diff/
> 
> 
> Testing
> -------
> 
> Ran the test. Varied the expectations from the results to cause failures for each substep.
> 
> 
> Thanks,
> 
> Jonathan Rose
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140116/3e0d5771/attachment-0001.html>


More information about the asterisk-dev mailing list