HomePhabricator

CommentFormatterTest: Avoid re-serializing the HTML

Description

CommentFormatterTest: Avoid re-serializing the HTML

The code we're testing already produces a string of serialized HTML,
no need to parse and re-serialize it.

Also, we recently learned that the precise format matters here
(T274709), and now this test *actually* covers the fix for that bug.
Follow-up to 5b26e9664b723142cd9e0642641eb9b4c8bdf231.

As a downside, this test might now spuriously fail if the format of
the output of Parsoid's XMLSerializer changes. Hopefully that won't
happen too often.

Change-Id: I69b514f545e47dcb437fb39a83edb8e2f19ed99b

Details

Provenance
matmarexAuthored on Feb 27 2021, 12:15 AM
Parents
rEDTO6650eb56cd0f: Merge "Fix toolbar dialog placement"
Branches
Unknown
Tags
Unknown
References
refs/changes/14/667314/2
ChangeId
I69b514f545e47dcb437fb39a83edb8e2f19ed99b