Last modified: 2013-08-15 21:27:46 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T54899, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 52899 - parsertests should support wgUseTidy in options
parsertests should support wgUseTidy in options
Status: NEW
Product: MediaWiki
Classification: Unclassified
Parser (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: C. Scott Ananian
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-08-15 20:55 UTC by C. Scott Ananian
Modified: 2013-08-15 21:27 UTC (History)
1 user (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description C. Scott Ananian 2013-08-15 20:55:45 UTC
Currently all parserTests are run with $wgUseTidy=false, but the PHP parser's behavior changes when $wgUseTidy=true (see bug 52760 and bug 17663).  Parsoid attempts to maintain equivalence with the $wgUseTidy=true behavior of the parser.

We should add a wgUseTidy option to the parserTests.  When running parserTests in the PHP parser without tidy installed, tests with wgUseTidy=true would be skipped.  When running parserTests in Parsoid, tests with wgUseTidy=false would be skipped.

This would allow us to document places where parser output changes with and without tidy, and verify that parsoid remains consistent with the wgUseTidy=true output of the PHP parser.
Comment 1 Gabriel Wicke 2013-08-15 21:27:46 UTC
Tidy has been part of the production pipeline since 2004, so current content has been written against MediaWiki with tidy enabled. Parsoid strives to preserve the semantics of existing content, which is why we are shooting for MediaWiki + tidy.

Adding tidy support to parserTests would indeed be useful, and would allow us to share some tests that are currently marked as parsoid-only with the PHP parser.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links