a6071b56 by David LaPalomento

Media playlist 404s are network errors

Re-enable the test for media playlist 404s.
1 parent 8fdc9a07
...@@ -1769,11 +1769,7 @@ test('does not modify the media index for in-buffer seeking', function() { ...@@ -1769,11 +1769,7 @@ test('does not modify the media index for in-buffer seeking', function() {
1769 equal(requests.length, 1, 'did not abort the outstanding request'); 1769 equal(requests.length, 1, 'did not abort the outstanding request');
1770 }); 1770 });
1771 1771
1772 QUnit.skip('playlist 404 should trigger MEDIA_ERR_NETWORK', function() { 1772 test('playlist 404 should end stream with a network error', function() {
1773 var errorTriggered = false;
1774 player.on('error', function() {
1775 errorTriggered = true;
1776 });
1777 player.src({ 1773 player.src({
1778 src: 'manifest/media.m3u8', 1774 src: 'manifest/media.m3u8',
1779 type: 'application/vnd.apple.mpegurl' 1775 type: 'application/vnd.apple.mpegurl'
...@@ -1781,13 +1777,7 @@ QUnit.skip('playlist 404 should trigger MEDIA_ERR_NETWORK', function() { ...@@ -1781,13 +1777,7 @@ QUnit.skip('playlist 404 should trigger MEDIA_ERR_NETWORK', function() {
1781 openMediaSource(player); 1777 openMediaSource(player);
1782 requests.pop().respond(404); 1778 requests.pop().respond(404);
1783 1779
1784 equal(errorTriggered, 1780 equal(player.tech.hls.mediaSource.error_, 'network', 'set a network error');
1785 true,
1786 'Missing Playlist error event should trigger');
1787 equal(player.error().code,
1788 2,
1789 'Player error code should be set to MediaError.MEDIA_ERR_NETWORK');
1790 ok(player.error().message, 'included an error message');
1791 }); 1781 });
1792 1782
1793 test('segment 404 should trigger MEDIA_ERR_NETWORK', function () { 1783 test('segment 404 should trigger MEDIA_ERR_NETWORK', function () {
......