Last modified: 2013-07-30 16:56:15 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 T52982, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 50982 - Possible bug in selser changes generation
Possible bug in selser changes generation
Status: RESOLVED FIXED
Product: Parsoid
Classification: Unclassified
tests (Other open bugs)
unspecified
All All
: Normal normal
: ---
Assigned To: Arlo Breault
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-08 21:44 UTC by ssastry
Modified: 2013-07-30 16:56 UTC (History)
3 users (show)

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


Attachments

Description ssastry 2013-07-08 21:44:59 UTC
Investigate why some parser tests have an empty selser changes array.  This could be a bug in the selser changes generation code.  Some of the tests (redirect or php-only) wont have any selser changes, but unsure why some others dont have.  To be investigated.

[subbu@earth tests] egrep ":\[\]," selser.changes.json | grep -v php | wc
     29     287    2186

[subbu@earth tests] egrep ":\[\]," selser.changes.json | grep -v php | head
"Parsoid only: Quote balancing context should be restricted to td/th cells on the same wikitext line\n(Requires tidy for PHP parser output to be fixed up)":[],
"Parsoid: Pipe in external link target and content in template parameter":[],
"Templates: Dont escape already nowiki-escaped text in template parameters":[],
"Implicit <td> after a |-\n(PHP parser relies on Tidy to add the missing <td> tags)":[],
"Pres should be recognized in an explicit <td> context, but not in an implicit <td> context\n(PHP parser relies on Tidy to add the missing <td> tags)":[],
"Lists should be recognized in an implicit <td> context\n(PHP parser relies on Tidy to add the missing <td> tags)":[],
"Parsoid link trail escaping":[],
"Parsoid link prefix escaping":[],
"Redirect to category":[],
"Redirect to category page":[],
Comment 1 Gerrit Notification Bot 2013-07-29 22:07:45 UTC
Change 76636 had a related patch set uploaded by Arlolra:
Avoid filtering out selser tests.

https://gerrit.wikimedia.org/r/76636
Comment 2 Gerrit Notification Bot 2013-07-29 22:13:21 UTC
Change 76636 had a related patch set uploaded by Arlolra:
Avoid filtering out selser tests.

https://gerrit.wikimedia.org/r/76636
Comment 3 Gerrit Notification Bot 2013-07-30 16:55:31 UTC
Change 76636 merged by jenkins-bot:
(Bug 50982) Fix for empty selser changes arrays.

https://gerrit.wikimedia.org/r/76636

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


Navigation
Links