Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #14970 from edx/ammar/educator-126-fix-js-flaky

Browse Source 
fix flaky js video test
This commit is contained in:
Muhammad Ammar
2017-04-27 00:05:56 +05:00
committed by GitHub
parent 1ab0825f2e 7da2091aaf
commit 3469cfc441
1 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
common/lib/xmodule/xmodule/js/spec/video/video_caption_spec.js
View File

@@ -68,22 +68,25 @@
expect($('.video')).toContainElement('.closed-captions');
});
// flaky test, see LEARNER-689
xit('fetch the transcript in HTML5 mode', function(done) {
it('fetch the transcript in HTML5 mode', function(done) {
var transcriptURL = '/transcript/translation/en',
transcriptCall;
state = jasmine.initializePlayer();
jasmine.waitUntil(function() {
return state.videoCaption.loaded;
}).then(function() {
expect($.ajaxWithPrefix).toHaveBeenCalledWith({
url: '/transcript/translation/en',
url: transcriptURL,
notifyOnError: false,
data: void(0),
success: jasmine.any(Function),
error: jasmine.any(Function)
});
expect($.ajaxWithPrefix.calls.mostRecent().args[0].data)
.toBeUndefined();
transcriptCall = $.ajaxWithPrefix.calls.all().find(function(call) {
return call.args[0].url === transcriptURL;
});
expect(transcriptCall.args[0].data).toBeUndefined();
}).always(done);
});