From 622533ef4831616041c04cf744d7eb0514aab24f Mon Sep 17 00:00:00 2001 From: Anthony Ramine Date: Fri, 29 Sep 2017 19:10:08 +0200 Subject: [PATCH] Properly fail in readyState_during_playing.html --- tests/wpt/metadata/MANIFEST.json | 2 +- .../readyState_during_playing.html.ini | 3 +-- .../readyState_during_playing.html | 18 ++++++++---------- 3 files changed, 10 insertions(+), 13 deletions(-) diff --git a/tests/wpt/metadata/MANIFEST.json b/tests/wpt/metadata/MANIFEST.json index 1cfbb3ffb84..50d48f18c2e 100644 --- a/tests/wpt/metadata/MANIFEST.json +++ b/tests/wpt/metadata/MANIFEST.json @@ -569845,7 +569845,7 @@ "testharness" ], "html/semantics/embedded-content/media-elements/readyState_during_playing.html": [ - "081d656eb789fa49e09fe0e5936eb10658321aa4", + "663689445bebed727f7fe181ace408ecb2254dcb", "testharness" ], "html/semantics/embedded-content/media-elements/readyState_initial.html": [ diff --git a/tests/wpt/metadata/html/semantics/embedded-content/media-elements/readyState_during_playing.html.ini b/tests/wpt/metadata/html/semantics/embedded-content/media-elements/readyState_during_playing.html.ini index ce7cf868e7a..21176eb3c40 100644 --- a/tests/wpt/metadata/html/semantics/embedded-content/media-elements/readyState_during_playing.html.ini +++ b/tests/wpt/metadata/html/semantics/embedded-content/media-elements/readyState_during_playing.html.ini @@ -1,6 +1,5 @@ [readyState_during_playing.html] type: testharness - expected: TIMEOUT [audio.readyState should be >= HAVE_FUTURE_DATA during playing event] - expected: NOTRUN + expected: FAIL diff --git a/tests/wpt/web-platform-tests/html/semantics/embedded-content/media-elements/readyState_during_playing.html b/tests/wpt/web-platform-tests/html/semantics/embedded-content/media-elements/readyState_during_playing.html index 92a1b7f50a6..5e8809851ec 100644 --- a/tests/wpt/web-platform-tests/html/semantics/embedded-content/media-elements/readyState_during_playing.html +++ b/tests/wpt/web-platform-tests/html/semantics/embedded-content/media-elements/readyState_during_playing.html @@ -17,26 +17,24 @@ test(function() { var t = async_test("audio.readyState should be >= HAVE_FUTURE_DATA during playing event", {timeout:5000}); var a = document.getElementById("a"); - a.addEventListener("playing", function() { - t.step(function() { - assert_greater_than_equal(a.readyState, a.HAVE_FUTURE_DATA); - }); + a.addEventListener("error", t.unreached_func()); + a.addEventListener("playing", t.step_func(function() { + assert_greater_than_equal(a.readyState, a.HAVE_FUTURE_DATA); t.done(); a.pause(); - }, false); + }), false); a.src = getAudioURI("/media/sound_5") + "?" + new Date() + Math.random(); }, "audio events - readyState property during playing"); test(function() { var t = async_test("video.readyState should be >= HAVE_FUTURE_DATA during playing event", {timeout:5000}); var v = document.getElementById("v"); - v.addEventListener("playing", function() { - t.step(function() { - assert_greater_than_equal(v.readyState, v.HAVE_FUTURE_DATA); - }); + v.addEventListener("error", t.unreached_func()); + v.addEventListener("playing", t.step_func(function() { + assert_greater_than_equal(v.readyState, v.HAVE_FUTURE_DATA); t.done(); v.pause(); - }, false); + }), false); v.src = getVideoURI("/media/movie_5") + "?" + new Date() + Math.random(); }, "video events - readyState property during playing");