Closed Bug 1138942 Opened 9 years ago Closed 9 years ago

Add Autophone specific video test manifest

Categories

(Testing :: Mochitest, defect)

defect
Not set
normal

Tracking

(firefox40 fixed)

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: bc, Unassigned)

Details

Attachments

(1 file, 1 obsolete file)

Attached file autophone-video (obsolete) —
Autophone currently runs several mochitests via existing manifests in the tree:
https://wiki.mozilla.org/Auto-tools/Projects/AutoPhone/Autophone_for_developers#Tests

The intent is to test video and audio playback and the full sets of tests are not needed and actually take too long to execute on the available devices. This manifest is intended to provide a place for video tests to be added specifically for Autophone.

ted: I'm piggy backing on ahal's Makefile.in since we haven't removed it yet in favor of moz.build. Is this ok for now?

snorp: this is a preliminary patch with the tests you listed yesterday. Before I ask for a formal review, can we get a full list of tests? Perhaps you could invite some of the media folks to provide input on which tests we would like?
Attachment #8571958 - Flags: feedback?(ted)
Attachment #8571958 - Flags: feedback?(snorp)
Comment on attachment 8571958 [details]
autophone-video

I think 'autophone-media' is a better name for the file, but looks like a good start to me.

Anthony, can you or one of your folks let Bob know what we should put in here? The idea is for Autophone to only test the media bits that are significantly affected by hardware or OS differences.
Flags: needinfo?(ajones)
Attachment #8571958 - Flags: feedback?(snorp) → feedback+
Video reftests would probably be the best approach. This will test our ability to decode video on each different piece of hardware. We need to add a reftest for MP4.
Flags: needinfo?(ajones)
works in local testing on autophone.
Attachment #8571958 - Attachment is obsolete: true
Attachment #8571958 - Flags: feedback?(ted)
Attachment #8588142 - Flags: review?(jmaher)
Comment on attachment 8588142 [details] [diff] [review]
autophone-media.patch

Review of attachment 8588142 [details] [diff] [review]:
-----------------------------------------------------------------

it would be cool if we could use the new tagging that :ahal is working on.  It might be a bit premature to do it, but that seems like the right longer term solution. maybe file a bug to do that.  The work was done in bug 987360.
Attachment #8588142 - Flags: review?(jmaher) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/021f785c5e9b

I've also been testing using reftests for:

layout/reftests/ogg-video/reftest.list
layout/reftests/webm-video/reftest.list

Once the mochitest manifest merges, I'll run tests on treeherder staging and if all is well file a bug to get these on trychooser. Hopefully we can discontinue the old autophone mochitests early next week and turn these on.
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/021f785c5e9b
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: