[Asterisk-code-review] Add local attended transfer tests with direct media, hold, a... (testsuite[master])

[John Bigelow]

John Bigelow has posted comments on this change.

Change subject: Add local attended transfer tests with direct media, hold, and REFER/Replaces.
......................................................................


Patch Set 3:

(4 comments)

Comments that were drafts.

https://gerrit.asterisk.org/#/c/483/3/tests/channels/pjsip/transfers/attended_transfer/nominal/packet_sniffer.py
File tests/channels/pjsip/transfers/attended_transfer/nominal/packet_sniffer.py:

Line 195: location = self.find_call(callid)
        :         if location is not None:
        :             return self.calls[location[0]][location[1]][callid]
> I would just say:
Already done in patch set 4.


Line 195: location = self.find_call(callid)
        :         if location is not None:
        :             return self.calls[location[0]][location[1]][callid]
> Although this will in fact return None in the event that location is None, 
Done


Line 259:                 self.get_call_state(callid) == "CONNECTING"):
> Since this starts a block, self.get_call_state... should be indented by 4 m
Done


Line 284:                     .format(reinvited_endpoints))
> PEP8 wants this aligned with "Endpoints...
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibd1f9a3ea8bf6b80fe95efe3fcd880a55d5a7cae
Gerrit-PatchSet: 3
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: John Bigelow <jbigelow at digium.com>
Gerrit-Reviewer: Ashley Sanders <asanders at digium.com>
Gerrit-Reviewer: John Bigelow <jbigelow at digium.com>
Gerrit-Reviewer: Samuel Galarneau <sgalarneau at digium.com>
Gerrit-HasComments: Yes