[Asterisk-code-review] sipp_iterator: converting iterator into pluggable module (testsuite[18])

[Michael Bradeen]

Attention is currently required from: N A, George Joseph.

Michael Bradeen has posted comments on this change. ( https://gerrit.asterisk.org/c/testsuite/+/19462 )

Change subject: sipp_iterator: converting iterator into pluggable module
......................................................................


Patch Set 5:

(4 comments)

Commit Message:

https://gerrit.asterisk.org/c/testsuite/+/19462/comment/e044c63e_b0a6d215 
PS1, Line 14: The two existing tests that ued the iterator have been moved to
> used
Done


File lib/python/asterisk/sipp_iterator.py:

https://gerrit.asterisk.org/c/testsuite/+/19462/comment/173f5f1f_cb6e3e38 
PS3, Line 23:     scenarios:
            :         - {'Name': 'alice-registers.xml', 'port': '5061', 'target': '127.0.0.1', 'ordered-args': {'-aa'}}
            :         - {'Name': 'alice-is-notified-1.xml', 'port': '5061', 'target': '127.0.0.1'}
            :         - {'Name': 'alice-is-notified-2.xml', 'port': '5061', 'target': '127.0.0.1'}
            :         - {'Name': 'alice-is-notified-3.xml', 'port': '5061', 'target': '127.0.0.1'}
            :         - {'Name': 'alice-is-notified-4.xml', 'port': '5061', 'target': '127.0.0.1'}
            :         - {'Name': 'done'}
            :     actions:
            :         - {'Action': 'none'}
            :         - {'Action': 'MWIUpdate', 'Mailbox': 'alice', 'NewMessages':'2', 'OldMessages':'0'}
            :         - {'Action': 'MWIUpdate', 'Mailbox': 'alice', 'NewMessages':'1', 'OldMessages':'1'}
            :         - {'Action': 'MWIUpdate', 'Mailbox': 'alice', 'NewMessages':'0', 'OldMessages':'2'}
            :         - {'Action': 'MWIDelete', 'Mailbox': 'alice'}
            :         - {'Action': 'UserEvent', 'UserEvent': 'testComplete'}
            : 
> I think it might be clearer to make this one list instead of two... […]
Done


https://gerrit.asterisk.org/c/testsuite/+/19462/comment/6301c872_07ace0e7 
PS3, Line 39:     scenarios:
            :         - {'Name': 'register', 'sequence': [
            :             {'Name': 'alice-registers.xml', 'port': '5061', 'target': '127.0.0.1', 'ordered-args': {'-aa'}},
            :             {'Name': 'bob-registers.xml', 'port': '5062', 'target': '127.0.0.1', 'ordered-args': {'-aa'}} ]}
            :         - {'Name': 'mailbox_a', 'sequence': [
            :             {'Name': 'alice-is-notified-1.xml', 'port': '5061', 'target': '127.0.0.1'},
            :             {'Name': 'bob-is-notified-1.xml', 'port': '5062', 'target': '127.0.0.1'} ]}
            :         - {'Name': 'mailbox_b', 'sequence': [
            :             {'Name': 'alice-is-notified-2.xml', 'port': '5061', 'target': '127.0.0.1'},
            :             {'Name': 'bob-is-notified-2.xml', 'port': '5062', 'target': '127.0.0.1'} ]}
            :         - {'Name': 'done'}
            : 
            :     actions:
            :         - {'Messages': [
            :             {'Action': 'UserEvent', 'UserEvent': 'testStarted'} ]}
            :         - {'Messages': [
            :             {'Action': 'MWIUpdate', 'Mailbox': 'mailbox_a', 'NewMessages':'2', 'OldMessages':'1'} ]}
            :         - {'Messages': [
            :             {'Action': 'MWIUpdate', 'Mailbox': 'mailbox_b', 'NewMessages':'3', 'OldMessages':'3'} ]}
            :         - {'Messages': [
            :             {'Action': 'UserEvent', 'UserEvent': 'testComplete'} ]}
            : 
> Same here
Done


https://gerrit.asterisk.org/c/testsuite/+/19462/comment/d8506a3f_f879fe72 
PS3, Line 78: 
> An actual sequence description would be good for both types for instance for multi... […]
Done



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

Gerrit-Project: testsuite
Gerrit-Branch: 18
Gerrit-Change-Id: I57ed85d2a2a044207da23a750e433dcc38eb93ff
Gerrit-Change-Number: 19462
Gerrit-PatchSet: 5
Gerrit-Owner: Michael Bradeen <mbradeen at sangoma.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: N A <asterisk at phreaknet.org>
Gerrit-Attention: N A <asterisk at phreaknet.org>
Gerrit-Attention: George Joseph <gjoseph at digium.com>
Gerrit-Comment-Date: Mon, 31 Oct 2022 23:19:28 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: N A <asterisk at phreaknet.org>
Comment-In-Reply-To: George Joseph <gjoseph at digium.com>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20221031/f0122893/attachment-0001.html>