W3C home > Mailing lists > Public > public-html-testsuite@w3.org > June 2011

Re: Submitted tests

From: Ms2ger <ms2ger@gmail.com>
Date: Tue, 14 Jun 2011 13:11:58 +0200
Message-ID: <4DF741FE.70005@gmail.com>
To: public-html-testsuite@w3.org
Hi James,

On 06/07/2011 11:22 PM, James Graham wrote:
> I just commited some Opera test submissions for the classList attribute,
> the history pushstate API and the time element.
>

I've looked over the classList test. I believe it should be approved, 
with the following nits addressed:

 > assert_true( elem.classList.constructor == window.DOMTokenList );
 > assert_true( getComputedStyle(elem,null).fontStyle != 'italic',
 > 'critical test; required by the testsuite' );
> assert_true( getComputedStyle(elem,null).fontStyle == 'italic',
 > 'critical test; required by the testsuite' );
 > assert_false( getComputedStyle(elem,null).fontStyle == 'italic' );
 > assert_true( getComputedStyle(elem,null).fontStyle == 'italic' );
 > assert_false( getComputedStyle(elem,null).fontStyle == 'italic' );

Should use assert_equals and assert_not_equals.

> /* the normative part of the spec states that:
> "unless the length is zero, in which case there are no supported property indices"
> ...
> "The term[...] supported property indices [is] used as defined in the WebIDL specification."
> WebIDL creates actual OwnProperties and then [] just acts as a normal property lookup */
> assert_equals( elem.classList[2], window.undefined );

The quote was copied too eagerly; the length isn't zero.

As a general comment, I'd like s/window.undefined/undefined/g, as the 
former looks rather silly.

Also, I'd like to see tests for classList.toString called explicitly, 
rather than through |classList + ""|, and for setting the readonly 
properties in ES5 strict mode.

Thanks
Ms2ger
Received on Tuesday, 14 June 2011 11:12:39 GMT

This archive was generated by hypermail 2.2.0+W3C-0.50 : Tuesday, 14 June 2011 11:12:40 GMT