[Asterisk-code-review] testsuite: Python 3 compatibility inital commit (testsuite[development/16/python3])

[Joshua Colp]

Attention is currently required from: Michael Bradeen, George Joseph.
Joshua Colp has posted comments on this change. ( https://gerrit.asterisk.org/c/testsuite/+/18528 )

Change subject: testsuite: Python 3 compatibility inital commit
......................................................................


Patch Set 12: Code-Review-1

(7 comments)

Commit Message:

https://gerrit.asterisk.org/c/testsuite/+/18528/comment/0aca8f08_beeff9d9 
PS12, Line 9: Multiple changes for compatibility.  Requires latest version of all
This commit message needs a clean up as it's actually two meshed together.


File contrib/scripts/install_prereq:

https://gerrit.asterisk.org/c/testsuite/+/18528/comment/df457fe6_4055796e 
PS12, Line 269: 	echo "Testsuite requires python version >= 3.8"
Should this be 3.6.8?


File lib/python/asterisk/extension_bank.py:

https://gerrit.asterisk.org/c/testsuite/+/18528/comment/6be9fc3d_383a71f0 
PS12, Line 2: Copyright (C) 2022, Sangoma Corp
Sangoma Technologies Corporation


File lib/python/asterisk/pcap.py:

https://gerrit.asterisk.org/c/testsuite/+/18528/comment/c7efcd74_a92ddda3 
PS12, Line 712:         #interpreted_packet = self._packet_factories.SIPPacketFactory.interpret_packet(packet)
Should this just be removed? Is it no longer needed?


File runtests.py:

https://gerrit.asterisk.org/c/testsuite/+/18528/comment/87ef73fe_e6e9f120 
PS12, Line 72: #ast_config = test_suite_utils.get_asterisk_conf()
What's the story of the change here?


File tests/rest_api/chunked-transfer/run-test:

https://gerrit.asterisk.org/c/testsuite/+/18528/comment/9c665fb8_2fb44c95 
PS12, Line 35: 
Can be removed


https://gerrit.asterisk.org/c/testsuite/+/18528/comment/0d81d04a_c7ef596f 
PS12, Line 67: #                data=bytes(chunkizer(b"{ b'variable': b'foo', b'value': b'bar'' }"),"utf-8"),
Can be removed



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

Gerrit-Project: testsuite
Gerrit-Branch: development/16/python3
Gerrit-Change-Id: I55caad3cb2a156b8e3f2f24dd10db5ebe67910d2
Gerrit-Change-Number: 18528
Gerrit-PatchSet: 12
Gerrit-Owner: Michael Bradeen <mbradeen at sangoma.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-CC: Mark Petersen <bugs.digium.com at zombie.dk>
Gerrit-Attention: Michael Bradeen <mbradeen at sangoma.com>
Gerrit-Attention: George Joseph <gjoseph at digium.com>
Gerrit-Comment-Date: Wed, 22 Jun 2022 10:51:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220622/09f8d9bb/attachment-0001.html>