Skip to content

Commit

Permalink
test: Fix edge case in player integration test
Browse files Browse the repository at this point in the history
If bufferBehind is set to 10, then having 10 seconds buffered behind
is fine.  It doesn't need to be strictly less than 10.

This test was previously failing only on Xbox One.

Change-Id: I6487e724753412614840b42eadfcd5edd71cc7bd
  • Loading branch information
joeyparrish committed Mar 30, 2021
1 parent 198a6d4 commit 572e512
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/player_integration.js
Original file line number Diff line number Diff line change
Expand Up @@ -756,7 +756,7 @@ describe('Player', () => {
// buffering goal so we append another segment.
player.configure('streaming.bufferingGoal', 40);
await waitUntilBuffered(40);
expect(getBufferedBehind()).toBeLessThan(10);
expect(getBufferedBehind()).toBeLessThanOrEqual(10);
});

function getBufferedAhead() {
Expand Down

0 comments on commit 572e512

Please sign in to comment.