Page MenuHomePhabricator

Better testing around scrubWikitext
Closed, ResolvedPublic

Description

Today's incident raised the issue that, since currently scrubWikitext mainly only applies to new content, our roundtrip tests aren't really exercising that code.

3:04:20 PM - subbu: but, i am thinking that this was a hole in our testing that we somehow missed this.
3:04:50 PM - subbu: y'day i was thinking that we probably need a 'force normalization' option so we can turn it on in rt-testing with that option.
3:04:55 PM - subbu: and this happened today.
3:05:27 PM - subbu: but, then, it will introduce noise in our rt-testing results.
3:05:52 PM - subbu: but, assuming scrubbable-wikitext is rare in production pages, it shouldn't.

Event Timeline

Arlolra raised the priority of this task from to Medium.
Arlolra updated the task description. (Show Details)
Arlolra added projects: Parsoid, Parsoid-Tests.
Arlolra subscribed.
ssastry claimed this task.