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 28416 - [XT30] bug-0701 serialized html indented output
Summary: [XT30] bug-0701 serialized html indented output
Status: CLOSED FIXED
Alias: None
Product: XPath / XQuery / XSLT
Classification: Unclassified
Component: XSLT 3.0 Test Suite (show other bugs)
Version: Last Call drafts
Hardware: PC Windows NT
: P2 normal
Target Milestone: ---
Assignee: Michael Kay
QA Contact: Mailing list for public feedback on specs from XSL and XML Query WGs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-07 01:37 UTC by Abel Braaksma
Modified: 2015-05-06 21:25 UTC (History)
0 users

See Also:


Attachments

Description Abel Braaksma 2015-04-07 01:37:50 UTC
The expected serialized output of this test ends with (on one line):

   </ul></font><p><font size="1" face="Verdana, Arial, Helvetica, sans-serif" color="#000000">Some text in a paragraph</font></p>

and our output ends with (newline after </ul>):

   </ul>
</font><p><font size="1" face="Verdana, Arial, Helvetica, sans-serif" color="#000000">Some text in a paragraph</font></p>

I believe these are both valid renditions according to the serialization standard for HTML and indent="yes", which is what this test seems to test.

Indentation is always tricky to test. In this case, I would simply solve it by adding an alternative outcome, though that is not ideal, because another processor might do it differently, or a later version of an existing processor might change its indentation and whitespace generation algorithm.
Comment 1 Abel Braaksma 2015-04-07 01:38:18 UTC
Currently fixed as proposed, but leaving it open for comments for now.
Comment 2 Michael Kay 2015-04-07 08:06:24 UTC
This test case wasn't designed to test serialization. In such cases it's usually best to replace the expected result with a logical test on the result tree contents rather than a serialization assertion. Alternatively, remove the indent="yes", which makes the results a bit more predictable.
Comment 3 Abel Braaksma 2015-04-07 20:56:36 UTC
Thanks, I wasn't sure because it seemed a test originating from an internal bug report.

Fixed as proposed. Indentation is now off, resulting in easier comparison.
Comment 4 Abel Braaksma 2015-05-06 21:25:16 UTC
Was resolved > 30 days ago, closing.