Closed Bug 1638973 Opened 2 years ago Closed 2 years ago

Convert 'mach marionette-test' to run with Python 3

Categories

(Testing :: Marionette, task, P3)

Product:
Testing
Component:
Marionette
Type:
task

Tracking

(firefox78 wontfix, firefox87 fixed)

Status:
RESOLVED FIXED
Milestone:
87 Branch
Tracking Flags:
Tracking Status
firefox78 --- wontfix
firefox87 --- fixed

People

(Reporter: KrisWright, Assigned: whimboo)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

Bug 1638973 - enable mach marionette-test to run with Python 3.
47 bytes, text/x-phabricator-request
Details | Review
Bug 1638973 - mark mach marionette-test as Python 2 only again. r=gbrown
47 bytes, text/x-phabricator-request
Details | Review
Bug 1638973 - Run 'mach marionette-test' with Python 3.
48 bytes, text/x-phabricator-request
Details | Review

mach marionette-test still runs with python 2 [1]. We want to convert all mach commands to python 3.

To do this, any dependencies of the command may need to be converted as well as the command itself.

[1] https://searchfox.org/mozilla-central/rev/9193635dca8cfdcb68f114306194ffc860456044/testing/marionette/mach_commands.py

The driver and harness of Marionette would have to be finished porting to Python3 first. See bug 1355537 for the current status.

Depends on: 1355537
Assignee: nobody → bob
Status: NEW → ASSIGNED
Attachment #9151397 - Attachment description: Bug 1638973 - enable marionette-test to run with Python 3. → Bug 1638973 - enable mach marionette-test to run with Python 3.
Pushed by bclary@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6c9242930ed4
enable mach marionette-test to run with Python 3. r=marionette-reviewers,whimboo

https://hg.mozilla.org/mozilla-central/rev/6c9242930ed4

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox78: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla78

I am going to revert the change to mach due to multiple issues with android.

Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Pushed by bclary@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/31734da426f1
mark mach marionette-test as Python 2 only again. r=gbrown

I'm un-assigning myself until the time where we have more of the underlying infrastructure ported to support python3.

Assignee: bob → nobody

(In reply to Bob Clary [:bc] from comment #6)

I am going to revert the change to mach due to multiple issues with android.

I'm not sure why this has been reverted. Marionette tests only run on desktop. We do not support Fenix. As such we should be fine to run everything here with Python 3.

Assignee: nobody → hskupin
Status: REOPENED → ASSIGNED
Priority: -- → P3
Target Milestone: mozilla78 → ---
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2d603721703d
Run 'mach marionette-test' with Python 3. r=jgraham
Status: ASSIGNED → RESOLVED
Closed: 2 years ago → 2 years ago
status-firefox87: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 87 Branch
You need to log in before you can comment on or make changes to this bug.