1334925 - Update mozharness config to expose twitch external media tests

Status: RESOLVED FIXED

(Release Engineering :: Applications: MozharnessCore, defect)

Description

[Bryce Seager van Dyk [:bryce] (he/him) - Not reading bugmail]

Bug 1331271 added twitch test to the external media tests. The mozharness media test scripts need to be updated to expose this functionality in the same way as it exposes the existing base, and youtube tests.

Comment 1

[Bryce Seager van Dyk [:bryce] (he/him) - Not reading bugmail]

Attachment: Bug 1334925 - Driveby update firefox_media_test scripts to use format() interpolation.

Change instances of old style python string interpolation in media test files
to the newer, format() style. format() appears to be the more common style
in the harness code, and this makes the media test code more consistent in
that regard.

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

Comment 2

[Bryce Seager van Dyk [:bryce] (he/him) - Not reading bugmail]

Attachment: Bug 1334925 - Add twitch test suite support to FF mozharness script.

Expand firefox_media_tests.py to include support for twitch.tv tests added to
the external media tests.

Other changes:
- Improve -h docs to mention more clearly available test suites.
- Remove duplate test suite config from platform specific configs. The different
  test suites can all be configured the same, independant of platform, so this
  has been lifted into the firefox_media_tests.py file to reduce duplication.
- Use symbolic constans for test suite names to avoid magic strings and reduce
  risk of typos.

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

Comment 3

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

Comment on attachment 8831677 [details]
Bug 1334925 - Driveby update firefox_media_test scripts to use format() interpolation.

https://reviewboard.mozilla.org/r/108226/#review109326

Comment 4

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

Comment on attachment 8831678 [details]
Bug 1334925 - Add twitch test suite support to FF mozharness script.

https://reviewboard.mozilla.org/r/108228/#review109354

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/01883302b8ea
Driveby update firefox_media_test scripts to use format() interpolation. r=maja_zf
https://hg.mozilla.org/integration/autoland/rev/37d919b2c97a
Add twitch test suite support to FF mozharness script. r=maja_zf

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/01883302b8ea
https://hg.mozilla.org/mozilla-central/rev/37d919b2c97a