[Asterisk-code-review] Testsuite: batched rls subscription failure (testsuite[master])

[Jonathan Rose]

Jonathan Rose has posted comments on this change.

Change subject: Testsuite: batched rls subscription failure
......................................................................


Patch Set 1:

(2 comments)

https://gerrit.asterisk.org/#/c/1537/1/tests/channels/pjsip/subscriptions/rls/rls_element.py
File tests/channels/pjsip/subscriptions/rls/rls_element.py:

Line 902:         LOGGER.debug("Processing RLMI resource -- Inspecting number of " \
        :                      "'name' elements. Expecting: 1.")
What's the purpose of the backslash line continuation here?


Line 1007:             message = "Processing RLS packet -- Validation check failed. " \
         :                       "Received unexpected number of parts ({0}) in " \
         :                       "multipart body.".format(body_parts - 1)
Only pointing this out since I got hit on a similar issue with \ continuing quote blocks before, but it's not necessary. You can wrap it all up like

message = ("line 1"
           "line 2"
           "line 3").format(body_parts -1)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Icb8af439f5c3db3efb0a93d6402ab4dcacdf3731
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Ashley Sanders <asanders at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Jonathan Rose <jrose at digium.com>
Gerrit-HasComments: Yes