[asterisk-dev] Change in testsuite[master]: stasis: set a channel variable on websocket disconnect error

[Ashley Sanders (Code Review)]

Ashley Sanders has posted comments on this change.

Change subject: stasis: set a channel variable on websocket disconnect error
......................................................................


Patch Set 5:

(6 comments)

https://gerrit.asterisk.org/#/c/18/5/tests/rest_api/applications/stasisstatus/ari_client.py
File tests/rest_api/applications/stasisstatus/ari_client.py:

Line 103:         if self.__ari is not None:
> There should be an else-statement here, to clear the list manually if the a
Done.


Line 273:         app                   --  The name of the Stasis app. (optional)
> There is extra punctuation in the middle of this sentence.
Done.


https://gerrit.asterisk.org/#/c/18/5/tests/rest_api/applications/stasisstatus/configs/ast1/sip.conf
File tests/rest_api/applications/stasisstatus/configs/ast1/sip.conf:

Line 9: [acme](!)
> In fact, I don't think SIP is being used at all for your tests, so you shou
You are correct, sir. It has been removed in the latest patch.


https://gerrit.asterisk.org/#/c/18/5/tests/rest_api/applications/stasisstatus/monitor.py
File tests/rest_api/applications/stasisstatus/monitor.py:

Line 29:         variable               -- The AMI variable to monitor (optional)
> The description should read: "The name of the channel variable to monitor (
Done.


Line 48:         return self.__class__.__name__ + '[' + self.name + ']:'
> This should be handled in the base type, ObservableObject. If it is, this m
Done.


https://gerrit.asterisk.org/#/c/18/5/tests/rest_api/applications/stasisstatus/test_case.py
File tests/rest_api/applications/stasisstatus/test_case.py:

Line 170:                                 None,
> None is over 1 too many spaces.
Done.


-- 
To view, visit https://gerrit.asterisk.org/18
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0f7dadfd429bd30e9f07a531f47884d8c923fc13
Gerrit-PatchSet: 5
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Ashley Sanders <asanders at digium.com>
Gerrit-Reviewer: Ashley Sanders <asanders at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-HasComments: Yes