Closed Bug 1496377 Opened Last year Closed Last year

test_mediarecorder_state_transition.html is not being run in tests (and fails)

Categories

(Core :: Audio/Video: Recording, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla65
Tracking Status
firefox64 --- fixed
firefox65 --- fixed

People

(Reporter: standard8, Assigned: k.bumsik)

References

Details

Attachments

(1 file)

I picked this up from #introduction on irc from talking with :kbumsik about an issue running a test.

It seems that dom/media/test/test_mediarecorder_state_transition.html is never run, because it is not listed in mochitest.ini.

I did a bit of digging, and back in 2013 when we transitioned from Makefile.in to mochitest.ini, this file got missed:

https://hg.mozilla.org/mozilla-central/rev/7f083546aa95

Attempting to enable it currently shows a series of failures:

dom/media/test/test_mediarecorder_state_transition.html
  FAIL Failed transitions for start,pause,pause
    runStateTransitionTests@dom/media/test/test_mediarecorder_state_transition.html:130:9
    startTest/element.oncanplaythrough@dom/media/test/test_mediarecorder_state_transition.html:160:5
    EventHandlerNonNull*startTest@dom/media/test/test_mediarecorder_state_transition.html:158:3
    MediaTestManager/this.nextTest@dom/media/test/manifest.js:1833:7
    MediaTestManager/this.runTests/<@dom/media/test/manifest.js:1753:7
  FAIL Failed transitions for start,pause,requestData
    runStateTransitionTests@dom/media/test/test_mediarecorder_state_transition.html:130:9
    startTest/element.oncanplaythrough@dom/media/test/test_mediarecorder_state_transition.html:160:5
    EventHandlerNonNull*startTest@dom/media/test/test_mediarecorder_state_transition.html:158:3
    MediaTestManager/this.nextTest@dom/media/test/manifest.js:1833:7
    MediaTestManager/this.runTests/<@dom/media/test/manifest.js:1753:7
  FAIL No InvalidStateError for start,pause,resume
    runStateTransitionTests@dom/media/test/test_mediarecorder_state_transition.html:136:9
    startTest/element.oncanplaythrough@dom/media/test/test_mediarecorder_state_transition.html:160:5
    EventHandlerNonNull*startTest@dom/media/test/test_mediarecorder_state_transition.html:158:3
    MediaTestManager/this.nextTest@dom/media/test/manifest.js:1833:7
    MediaTestManager/this.runTests/<@dom/media/test/manifest.js:1753:7
  FAIL No InvalidStateError for start,pause,resume with timeslice 200
    runStateTransitionTests@dom/media/test/test_mediarecorder_state_transition.html:136:9
    startTest/element.oncanplaythrough@dom/media/test/test_mediarecorder_state_transition.html:160:5
    EventHandlerNonNull*startTest@dom/media/test/test_mediarecorder_state_transition.html:158:3
    MediaTestManager/this.nextTest@dom/media/test/manifest.js:1833:7
    MediaTestManager/this.runTests/<@dom/media/test/manifest.js:1753:7
Blocks: 1496383
No longer blocks: 1496383
Note: bug 1496383 may / may not be related.
See Also: → 1496383
Component: Audio/Video: Playback → Audio/Video: Recording
Depends on: 1496383
Priority: -- → P3
I uploaded a patch for bug 1496383. Apart from the test result, I think the test cases written in test_mediarecorder_state_transition.html are quite incorrect. I can correct them if anyone assign me to upload a patch.
Bumsik: You should be able to upload the patch without being assigned. I don't know if in this case it might be better to do the patch + test together, in any case I can assign you to this one.
Assignee: nobody → k.bumsik
This resolves two problems:

1. test_mediarecorder_state_transition.html was not in mochitest.ini.
2. The test set in the file has some wrong test cases, especially after D7910.

TO verify the state transition cases is trivial as the state error policy in the W3C spec is very simple:

1. start() can be called only when it is "inactive" state.
2. All other functions (pause(), resume(), stop(), requestData()) must not be called when it's "inactive" state, regardless of the order and how many number they are called.
Keywords: checkin-needed
Pushed by ebalazs@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f5577f9e95a9
Fix and add test_mediarecorder_state_transition.html in tests. r=jya
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/f5577f9e95a9
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Might as well take this test-only fix on Beta also since bug 1496383 landed there.
Whiteboard: [checkin-needed-beta]
Whiteboard: [checkin-needed-beta]
See Also: → 1505671
You need to log in before you can comment on or make changes to this bug.