Skip to content

Commit

Permalink
If play is failing I suspect some attributes such as isDynamic to be …
Browse files Browse the repository at this point in the history
…wrong making wrong assumptions in the functional tests. Query dynamic state directly via player API.
  • Loading branch information
dsilhavy committed Jul 22, 2022
1 parent 1ff9797 commit 97b58a8
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
3 changes: 2 additions & 1 deletion test/functional/tests/pause.js
Original file line number Diff line number Diff line change
Expand Up @@ -62,8 +62,9 @@ exports.register = function(stream) {
await command.sleep(PAUSE_DELAY * 1000);

const time = await command.execute(player.getTime);
const isDynamic = await command.execute(player.isDynamic);
utils.log(NAME, 'Playback time = ' + time);
if (stream.dynamic) {
if (isDynamic) {
// For dynamic streams, when paused, current time is progressing backward
assert.isAtMost(time, (pauseTime - PAUSE_DELAY + 1)); // +1 for 1 sec tolerance
} else {
Expand Down
6 changes: 4 additions & 2 deletions test/functional/tests/playFromTime.js
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,8 @@ exports.register = function (stream) {
utils.log(NAME, 'Load stream');
command = remote.get(intern.config.testPage);
originalUrl = stream.url;
if (!stream.dynamic) {
const isDynamic = await command.execute(player.isDynamic);
if (!isDynamic) {
let period = stream.periods[stream.periods.length - 1];
startTime = period.start + Math.min(TIME_OFFSET, period.duration - 5);
stream.url += '#t=' + startTime;
Expand All @@ -50,7 +51,8 @@ exports.register = function (stream) {
utils.log(NAME, 'Play');
const playing = await command.executeAsync(player.isPlaying, [constants.EVENT_TIMEOUT]);
assert.isTrue(playing);
const time = await command.execute(stream.dynamic ? player.getTimeAsUTC : player.getTime);
const isDynamic = await command.execute(player.isDynamic);
const time = await command.execute(isDynamic ? player.getTimeAsUTC : player.getTime);
utils.log(NAME, 'Playback time: ' + time);
assert.isAtLeast(time, startTime);
assert.isAtMost(time, (startTime + 5));
Expand Down

0 comments on commit 97b58a8

Please sign in to comment.