Re: Run-in tests in CSS2.1 test suite (harness)

Le 2014-06-01 21:57, Gérard Talbot a écrit :
> Le 2014-06-01 20:43, Gérard Talbot a écrit :
>> Le 2014-06-01 17:48, Peter Linss a écrit :
>>> On Jun 1, 2014, at 8:59 AM, Gérard Talbot 
>>> <css21testsuite@gtalbot.org> wrote:
>>> 
>>>> Peter,
>>>> 
>>>> Tests involving display: run-in should not be part of CSS2.1 test 
>>>> suite. Right now, the test harness with CSS2.1 (nightly-unstable) 
>>>> will have those.
>>> 
>>> A few weeks back I rewrote the build script to support the
>>> refactoring. Tests are now assigned to each test suite based entirely
>>> on their spec links, not their location in the repository. These 
>>> tests
>>> each have links to the CSS2.1 spec, so they're included in the test
>>> suite, but I agree they shouldn't be.
>>> 
>>> One fix would to correct the links in the tests, they need to refer 
>>> to
>>> the proper sections in level 3 test suites
>> 
>> That would have to be:
>> 
>> <link rel="help" href="http://www.w3.org/TR/css3-box/#run-in-boxes"
>> title="4.3. Run-in boxes">
>> 
>> I can correct links in my tests.
>> 
>> Gérard
>> 
>>> and have their links to
>>> CSS2.1 removed.
> 
> Links to section 9.2.4 The "display" property of CSS2.1 were removed.
> 
> If there are links to CSS2.1 features that don't yet
>>> have a level 3 module, then we need to come up with a system for
>>> managing cross-spec dependencies for the build system.
>>> 
>>> Peter
> 
> Done.
> 
> committed changeset 6393:146484fedea8
> 
> Gérard

Peter,

http://test.csswg.org/suites/css21_dev/nightly-unstable/html4/after-content-display-004.htm

is still available/in/inside the CSS2.1 test suite when using the test 
harness.

<link rel="help" 
href="http://www.w3.org/TR/CSS21/generate.html#before-after-content" 
title="12.1 The :before and :after pseudo-elements">
<link rel="help" href="http://www.w3.org/TR/css3-box/#run-in-boxes" 
title="4.3. Run-in boxes">

Just as you said, if 2 specs are listed, then the test appears in both 
test suites.

I believe now I should also remove the linked reference to 
/CSS21/generate.html in that after-content-display-004 test.

CSS2.1 uses 1 colon for pseudo-elements (after, before); CSS3 uses 2 
consecutive colons. And CSS3 Generated and Replaced Content Module spec
http://www.w3.org/TR/css3-content/
seems outdated, unmaintained too.


If we agree on this, then I will apply such kind of changes to all 
Microsoft-submitted tests on run-in: most of the *-applies-to-011 tests 
and a bunch of *-applies-to-004 tests


-------------

What I would do:

eg

In

http://test.csswg.org/suites/css2.1/nightly-unstable/html4/border-bottom-width-applies-to-011.htm

line 6: <link rel="help" 
href="http://www.w3.org/TR/CSS21/box.html#propdef-border-bottom-width">
line 7: <link rel="help" 
href="http://www.w3.org/TR/CSS21/box.html#border-width-properties">

would be entirely replaced with

line 6: <link rel="help" 
href="http://www.w3.org/TR/css3-background/#the-border-width" 
title="4.3. Line Thickness: the 'border-width' properties">
line 7: <link rel="help" 
href="http://www.w3.org/TR/css3-box/#run-in-boxes" title="4.3. Run-in 
boxes">

-------------

What I would do:

eg

In

http://test.csswg.org/suites/css2.1/nightly-unstable/html4/caption-side-applies-to-004.htm

   <link rel="help" 
href="http://www.w3.org/TR/CSS21/tables.html#propdef-caption-side">
   <link rel="help" 
href="http://www.w3.org/TR/CSS21/tables.html#caption-position">

would be entirely replaced with

<link rel="help" href="http://www.w3.org/TR/css3-box/#run-in-boxes" 
title="4.3. Run-in boxes">

because there is no CSS3 table spec right now.

Gérard
-- 
Web authors' contributions to CSS 2.1 test suite
http://www.gtalbot.org/BrowserBugsSection/css21testsuite/web-authors-contributions-css21-testsuite.html
CSS 2.1 Test suite RC6, March 23rd 2011
http://test.csswg.org/suites/css2.1/20110323/html4/toc.html

Received on Monday, 2 June 2014 17:16:34 UTC