Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests: USB: Fix Python 3 compatibility #11546

Merged
merged 2 commits into from
Sep 30, 2019

Conversation

fkjagodzinski
Copy link
Member

@fkjagodzinski fkjagodzinski commented Sep 23, 2019

Description

This is a follow-up to #11520.

Use integer division explicitly in basic test.
Convert bytes to str for Python 3 in serial test.

After the update, this patch also bumps the version of six module to 1.12.0.

Pull request type

[ ] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[x] Test update
[ ] Breaking change

Reviewers

@OPpuolitaival @madchutney

Release Notes

@ciarmcom ciarmcom requested review from madchutney, OPpuolitaival and a team September 23, 2019 09:00
@ciarmcom
Copy link
Member

@fkjagodzinski, thank you for your changes.
@OPpuolitaival @madchutney @ARMmbed/mbed-os-maintainers please review.

TESTS/host_tests/usb_device_serial.py Outdated Show resolved Hide resolved
@0xc0170
Copy link
Contributor

0xc0170 commented Sep 24, 2019

CI started

@mbed-ci
Copy link

mbed-ci commented Sep 24, 2019

Test run: SUCCESS

Summary: 4 of 4 test jobs passed
Build number : 1
Build artifacts

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 25, 2019

@fkjagodzinski What shall we do with six, do you want to update it and use it here?

Filip Jagodzinski added 2 commits September 30, 2019 10:20
This version provides a few handy functions: six.ensure_binary,
six.ensure_text, and six.ensure_str.
Use integer division explicitly in basic test.
Convert bytes to str for Python 3 in serial test.
@fkjagodzinski fkjagodzinski force-pushed the test_update-usb-python3 branch from 96581ff to 2b4b64f Compare September 30, 2019 09:13
@fkjagodzinski
Copy link
Member Author

I decided to update this patch to use the new version of six. The changelog looks rather straight forward for this release, so I don't expect any issues.

@fkjagodzinski
Copy link
Member Author

@0xc0170, could you re-trigger the CI? Apart from that, do I need to add a release note, or notify anyone?

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 30, 2019

CI restarted

@mbed-ci
Copy link

mbed-ci commented Sep 30, 2019

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 2
Build artifacts

@0xc0170 0xc0170 removed the needs: CI label Sep 30, 2019
@0xc0170 0xc0170 merged commit c3aedbf into ARMmbed:master Sep 30, 2019
@fkjagodzinski fkjagodzinski deleted the test_update-usb-python3 branch September 30, 2019 13:22
@jeromecoutant
Copy link
Collaborator

jeromecoutant commented Oct 11, 2019

Hi
Got new issue:

@jeromecoutant
Copy link
Collaborator

OK... I just see the six version requirement...
NB: I thought mbed scripts was checking the requirements.... :-(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants