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

TE-468 JS test: Use the correct array for validating onSeek logs

Browse Source 
This particular test is failing in Solano workers; there the log
is written in a different order, which was causing objects not to
be found.
This commit is contained in:
Ben Patterson
2014-09-11 08:26:47 -04:00
parent 32071af4b6
commit ef383891f2
1 changed files with 14 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
common/lib/xmodule/xmodule/js/spec/video/video_player_spec.js
View File

@@ -395,9 +395,21 @@ function (VideoPlayer) {
}, 'currentTime is less than 2 seconds', WAIT_TIMEOUT);
runs(function () {
var args = state.videoPlayer.log.calls[0].args;
// Depending on the browser, the object of arrays may list the
// arrays in a different order. Find the array that is relevent
// to onSeek. Fail if that is not found.
var seekVideoArgIndex
for(var i = 0; i < state.videoPlayer.log.calls.length; i++){
if (state.videoPlayer.log.calls[i].args[0] == 'seek_video') {
seekVideoArgIndex = i
break;
}
}
expect(seekVideoArgIndex).toBeDefined;
var args = state.videoPlayer.log.calls[seekVideoArgIndex].args;
expect(args[0]).toBe('seek_video');
expect(args[1].old_time).toBeLessThan(2);
expect(args[1].new_time).toBe(2);
expect(args[1].type).toBe('onSlideSeek');