Re: Testcase attrcreatetextnode2.xml in level1/core does not exercise the test

Not sure I understand why we have 4 tests -
Perhaps we should set the attribute to be something different
than what was originally in the file -- the use of  %ent1 was
an error, but rather than using &ent1;, which is what is in the
xml file already, we should set it to &ent2; - then it may be less
confusing.

What's the purpose of the additional tests.  Curt, did you create
attrcreatetextnode2, and forget to change the metadata info?  I
don't think that is a test that we created.  Could you please explain
the differences and the additional need for attrcreatetextnode2, 3, & 4?

I would think in general if a test has a bug, we should agree on a
fix and fix the original test rather than creating additional tests.  Is
there something more going on here than is stated in the description.
It does appear that you are trying to do something with nodevalue as
well -- could you provide a little more detail?

--Mary

----- Original Message -----
From: "Curt Arnold" <carnold@houston.rr.com>
To: <www-dom-ts@w3.org>
Sent: Sunday, April 21, 2002 7:29 PM
Subject: Fw: Testcase attrcreatetextnode2.xml in level1/core does not
exercise the test


> I've added a attrcreatetextnode3 and 4 that should come closer to the
stated
> intent of the the test and modified the descriptions for
attrcreatetextnode
> and attrcreatetextnode2.
>
> ----- Original Message -----
> From: "Curt Arnold" <carnold@houston.rr.com>
> To: "Nick Efthymiou" <NEFTH@pacbell.net>
> Sent: Saturday, April 20, 2002 11:05 AM
> Subject: Re: Testcase attrcreatetextnode2.xml in level1/core does not
> exercise the test
>
>
> > I went back and checked earlier sources and that seems to have been
> brought
> > over from the original NIST tests.  Definitely the actual test doesn't
> > exercise the intent, not only is the semi-colon missing but it uses a %
> > instead of a &.
> >
> > I think the right thing to do is to change the description and admit
that
> > the test isn't doing what it should be doing and to add additional tests
> > that might actually trigger the potentially flawed behavior.
> >
> > Was there a reason this message was sent off-list?  Could you resent
this
> or
> > the original message to the mailing list to get it in the official
record?
> >
> > ----- Original Message -----
> > From: "Nick Efthymiou" <NEFTH@pacbell.net>
> > To: <carnold@houston.rr.com>
> > Sent: Saturday, April 20, 2002 2:59 AM
> > Subject: Testcase attrcreatetextnode2.xml in level1/core does not
exercise
> > the test
> >
> >
> > >
> > > Please review
> > >
> >
>
http://dev.w3.org/cvsweb/2001/DOM-Test-Suite/tests/level1/core/attrcreatetex
> > > tnode2.xml?annotate=1.2
> > >
> > > Line 22 states the intention of the test:
> > > 22:   child of of the fourth employee and assign the "Y%ent1;"
> > >
> > > Line 44 sets the value as follows (note missing ";"):
> > > 44: <nodeValue obj="streetAttr" value='"Y%ent1"'/>
> > >
> > > Since no entity is set, the test case does not test what it is
intended
> > to.
> > >
> > > The same bug (test does not exercise what is intended) is apparently
> also
> > > true with attrcreatetextnode:
> > >
> >
>
http://dev.w3.org/cvsweb/2001/DOM-Test-Suite/tests/level1/core/attrcreatetex
> > > tnode.xml?annotate=1.7
> > >
> > > Same line numbers (22 vs. 44).
> > >
> > > Regards,
> > >
> > > - Nick -
> > >
> > >
> > >
> > >
> >
>
>
>

Received on Tuesday, 23 April 2002 09:26:00 UTC