1374633 - Add Firefox UI Tests jobs for Taskcluster on OS X

Status: RESOLVED FIXED

(Testing :: Firefox UI Tests, enhancement)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Similar to bug 1371404 which added Firefox UI test jobs recently for Taskcluster on Windows we can now also enable them on OS X.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

There are overlapping code changes with bug 1372094, so I will wait once that got merged to m-c.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1374633 - Add Firefox UI Tests jobs for Taskcluster on OS X.

Review commit: https://reviewboard.mozilla.org/r/150892/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/150892/

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Comment on attachment 8879570 [details]
Bug 1374633 - Add Firefox UI Tests jobs for Taskcluster on OS X.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/150892/diff/1-2/

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Comment on attachment 8879570 [details]
Bug 1374633 - Add Firefox UI Tests jobs for Taskcluster on OS X.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/150892/diff/2-3/

Comment 5

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8879570 [details]
Bug 1374633 - Add Firefox UI Tests jobs for Taskcluster on OS X.

https://reviewboard.mozilla.org/r/150892/#review156222

consider using default instead of linux, but that is optional.  this is great to see

::: taskcluster/ci/test/tests.yml:215
(Diff revision 3)
>              by-test-platform:
>                  windows.*:
>                      - firefox_ui_tests/taskcluster_windows.py
> -                default:
> +                macosx.*:
> +                    - firefox_ui_tests/taskcluster.py
> +                linux.*:

typically we use 'default', I know we only run on windows/osx/linux- so I am fine with this- but it goes against the standards in the file.

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Comment on attachment 8879570 [details]
Bug 1374633 - Add Firefox UI Tests jobs for Taskcluster on OS X.

https://reviewboard.mozilla.org/r/150892/#review156222

> typically we use 'default', I know we only run on windows/osx/linux- so I am fine with this- but it goes against the standards in the file.

No, not for the mozharness config settings. All test jobs are using `linux.*` here.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5f41ffc410de
Add Firefox UI Tests jobs for Taskcluster on OS X. r=jmaher

Comment 8

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/5f41ffc410de