Permalink
Browse files

Merge pull request #716 from foolip/currentTime-test-names

Remove pass conditions from currentTime test names
  • Loading branch information...
2 parents 2950886 + f3b2ca1 commit 23b5482bb4f36045a3259319ab1ea40251033faf @jgraham jgraham committed Mar 5, 2014
@@ -8,30 +8,23 @@
test(function() {
var v = document.createElement('video');
assert_equals(v.currentTime, 0);
-}, 'currentTime is initially zero');
-
-// Negative test for the specified behavior prior to HTML r6580.
-test(function() {
- var v = document.createElement('video');
- assert_equals(v.readyState, v.HAVE_NOTHING);
- v.currentTime = 1;
-}, 'setting currentTime when readyState is HAVE_NOTHING does not throw');
+}, 'currentTime initial value');
test(function() {
var v = document.createElement('video');
v.controller = new MediaController();
assert_true(v.controller instanceof MediaController);
assert_throws('InvalidStateError', function() { v.currentTime = 1; });
assert_false(v.seeking);
-}, 'setting currentTime with a media controller present throws');
+}, 'setting currentTime with a media controller present');
test(function() {
var v = document.createElement('video');
assert_equals(v.readyState, v.HAVE_NOTHING);
v.currentTime = Number.MAX_VALUE;
assert_equals(v.currentTime, Number.MAX_VALUE);
assert_false(v.seeking);
-}, 'setting currentTime sets the default playback start position when readyState is HAVE_NOTHING');
+}, 'setting currentTime when readyState is HAVE_NOTHING');
async_test(function(t) {
var v = document.createElement('video');
@@ -43,5 +36,5 @@
assert_true(v.seeking);
t.done();
});
-}, 'setting currentTime seeks when readyState is not HAVE_NOTHING');
+}, 'setting currentTime when readyState is greater than HAVE_NOTHING');
</script>

0 comments on commit 23b5482

Please sign in to comment.