fix: remove ansi escape sequences from escaped xunit output

[trieloff]

Requirements

• Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
• All new code requires tests to ensure against regressions.

Description of the Change

The utils.escape function is amended to strip the escape sequence using a regular expression.

Alternate Designs

• patch the underlying he library: dismissed because the direct fix is likely faster to become usable
• filter the output before running he: dismissed because the correct escaping in JavaScript is harder to find
• filter out all invalid XML characters: dismissed because this is not a problem observed with other characters

Why should this be in core?

The xunit reporter is in core and should not be buggy

Benefits

xunit reporter output can be consumed by tools that expect valid XML

Possible Drawbacks

Tools that process the XML as plain (unicode) text (unlikely) can no longer see the ANSI escape sequence.

Applicable issues

fixes #4526

This is a bug fix (patch release)

[coveralls]

Coverage remained the same at 94.143% when pulling d417538 on trieloff:issue/4526 into bc8ce05 on mochajs:master.

[outsideris]

Isn't there a way to escape or remove all invalid characters for XML?

[juergba]

There is a related xunit issue #3586.

[github-actions]

This PR hasn't had any recent activity, and I'm labeling it stale. Remove the label or comment or this PR will be closed in 14 days. Thanks for contributing to Mocha!

[trieloff]

This issue has been hanging around for 8 months now, and I'm unsure what I can do to get it moving again. There is an open thread that has multiple possible solutions to the problem, but no expression of preference from the Mocha team.

[JoshuaKGoldberg]

👋 coming back to this @trieloff, are you still interested in working on this PR? As of #5027 there are again maintainers who can review it now. No worries if you no longer have time - but if you do that'd be great!

[trieloff]

Hi @JoshuaKGoldberg – that's unexpected – but not unwelcome. I'm happy to rebase the PR to the latest main. Do you want to reopen this PR or have me create a new one?

[JoshuaKGoldberg]

Great! Let's go ahead and reopen this one.

We also discussed that we don't like the requirement of rebasing, so if you want to do a more casual merge that works too. Whatever's easiest on your end. 🙂

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: trieloff / name: Lars Trieloff (717474a, b4b7392, 9f3f8db, d417538, bfb62dc)

[trieloff]

Ok, let me get that CLA signed, then.

[trieloff]

• CLA has been signed (thanks, corporate overlords)
• master has been merged
• conflicts have been resolved