Closed Bug 1235070 Opened 8 years ago Closed 8 years ago

Implement FxAPOM Module in Marionette Tests

Categories

(Cloud Services :: QA: Test Automation, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Silne30, Assigned: Silne30)

References

Details

We have a bit of a hackish way to sign up for FxA accounts. FxAPOM provides a cleaner way to create/reuse FxA accounts for automation purposes.
Blocks: 1226638
Assignee: sphilp → jdorlus
Status: NEW → ASSIGNED
Depends on: 1240253
This also depends on a github issue. https://github.com/mozilla/idea-town/issues/391
Depends on: 1247829
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
John, out of interest.... where has this been fixed? In case of marking a bug as fixed you should specify the code location as reference. I cannot find it on any of the recently closed bugs.
Flags: needinfo?(jdorlus)
Hmm. I posted the link to our services test repo. At a hotel this week so perhaps that didn't post. 

https://github.com/mozilla-services/marionette-wrapper/commit/d0c3365c24be5431f455935730c8665758eb7325

tests/functional/testpilot/conftest.py

tests/functional/pocket/conftest.py
Flags: needinfo?(jdorlus)
(In reply to Henrik Skupin (:whimboo) from comment #2)
> John, out of interest.... where has this been fixed? In case of marking a
> bug as fixed you should specify the code location as reference. I cannot
> find it on any of the recently closed bugs.

Just to be on the same page, this was not to be implemented for firefox-ui-tests but for services marionette tests.
Flags: needinfo?(hskupin)
I know that this was not for firefox-ui-tests, but I wanted to be up2date given that our code will most likely be shared in the future again when fx-ui-tests also make use of Selenium and not Marionette harness.
Flags: needinfo?(hskupin)
So thanks for the update and the references.
You need to log in before you can comment on or make changes to this bug.