Closed Bug 1469718 Opened 4 years ago Closed 4 years ago

Update socorro.external folder for Python 3


(Socorro :: General, task, P2)


(Not tracked)



(Reporter: cmiller, Assigned: willkg)




(2 files)

Update all files in socorro.external and their tests to run in Python 3 test runs.
No longer depends on: 1467308
I'd rescope this to a specific subdirectory of socorro/external because that directory tree is huge. Perhaps socorro/external/rabbitmq/ first?
Commits pushed to master at
bug 1469718: all the changes for sub dir socorro.external.boto

In testing the raw dump data there was an error in the test value
returning as a binary string. This value was failing in comparison to
the expected unicode value for python 3. Placing print statements in
class level functions didn’t tell us much.So we traced the function from
top to bottom and bottom to top. This pushed over the deep end as we
found ourselves in the boto service source code. We couldn’t find the
exact time this conversion of the test value occurred. To solve this
issue we made the assumption data being used in raw dump is binary. So
we change the test value to be a binary string and also changed the
expected comparison to be a binary string as well. The implementation of
the function iteritems() changed from python 2 to 3. Similar to the
range function the change in implementation went from generating a list
to using an iterator. Lastly objects of type ‘dict_values’ is not JSON
serializable. To solve this we wrap the call to values with a call to
Merge pull request #4508 from ceddy-cedd/bug-1469718-Update-socorro.external

bug 1469718: all the changes for sub dir socorro.external.boto
Assignee: nobody → cmiller
Grabbing this to do soon.

Looks like the following are done now:

* socorro/unittest/external/boto/test_*.py
* socorro/unittest/external/es/test_*.py
* socorro/unittest/external/postgresql/test_*.py
* socorro/unittest/external/rabbitmq/test_*.py

Leaving this one:

* socorro/unittest/external/fs/test_*.py
Assignee: cmiller → willkg
Priority: -- → P2
Commits pushed to master at
fix bug 1469718: convert socorro/external/ to Python 3
Merge pull request #4699 from willkg/1469718-crashstorage-base

fix bug 1469718: convert socorro/external/ to Python 3
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.