This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 3584 - white space in fn-union-node-args-003
Summary: white space in fn-union-node-args-003
Status: CLOSED FIXED
Alias: None
Product: XML Query Test Suite
Classification: Unclassified
Component: XML Query Test Suite (show other bugs)
Version: unspecified
Hardware: PC Windows XP
: P2 normal
Target Milestone: ---
Assignee: Ying Lu
QA Contact: Mailing list for public feedback on specs from XSL and XML Query WGs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-08-07 12:10 UTC by David Carlisle
Modified: 2006-08-09 10:11 UTC (History)
0 users

See Also:


Attachments

Description David Carlisle 2006-08-07 12:10:43 UTC
The expected result file for fn-union-node-args-003 has extra white space.
white space children of document nodes do not form part of the infoset and should not appear in the input tree, and consequently should not be in the output tree.
the comparison used for this test is Fragment which means that these nodes are
made children of an element node before comparison, and so are significant.

<!-- this file is a copy of bib.xml; just adds a few comments and PI nodes for testing -->
<!-- Comment 1 -->
<?PI1 Processing Instruction 1?>
<bib>

should be all on one line, as should


</bib>
<!-- Comment 2 -->
<?PI2 Processing Instruction 2?>
<title>Data on the Web</title>
Comment 1 David Carlisle 2006-08-07 12:30:59 UTC
PS.
this is a recent (3 day) change to the expected result file in CVS, no reason is given in the CVS log why the change was made, but I believe the file should be reverted to the form that it was in last week.
Comment 2 Frans Englich 2006-08-07 17:16:46 UTC
Funny, there seems to be only one revision of this file. Could be that someone removed/added, instead of merely updating it. Another cause to these kind of regressions could be that people forget to update their local CVS checkouts, and then commit. Reassigning to Ravi.

RCS file: /sources/public/2006/xquery-test-suite/TestSuiteStagingArea/Queries/XQuery/Expressions/Operators/SeqOp/SeqUnion/fn-union-node-args-003.xq,v
Working file: ./TestSuiteStagingArea/Queries/XQuery/Expressions/Operators/SeqOp/SeqUnion/fn-union-node-args-003.xq
head: 1.1
branch:
locks: strict
access list:
symbolic names:
        XQTS_0_9_4: 1.1
        XQTS_0_9_0: 1.1
        XQTS_0_8_6: 1.1
        XQTS_0_8_4: 1.1
        XQTS_0_8_2: 1.1
        XQTS_0_8_0: 1.1
        XQTS_0_7_0: 1.1
        XQTS_0_6_0: 1.1
        XQTS_0_2_9: 1.1
keyword substitution: kv
total revisions: 1;     selected revisions: 1
description:
----------------------------
revision 1.1
date: 2005/06/15 00:43:09;  author: rchennoj;  state: Exp;
*** empty log message ***



Frans
Comment 3 Ying Lu 2006-08-07 19:38:34 UTC
Hi David,

I've updated fn-union-node-args-003.txt. If you are ok with the fix, please mark the bug as closed. 

Frans, it's the .txt file which has been changed, and there is a lot of revisions of that file.

Thanks,
Ying
Comment 4 David Carlisle 2006-08-08 10:43:39 UTC
Using
/XQTS_current.zip/1.32/Tue Aug  8 09:39:32 2006/-kb/
the expected result file is still not correct, it has a trailing newline at the end.
Comment 5 Ying Lu 2006-08-08 19:08:19 UTC
Andrew has updated the file and removed EOL at EOF. Please mark the bug as closed if the result is expected. Thanks.
Comment 6 David Carlisle 2006-08-09 10:11:37 UTC
Yo!!!
For the first time, in a year of running the test suite, xq2xsl does not report
any tests which require white space fix up in the expected result files.
Closing this, it's all looking good for a 1.0 release...