Remove mocha-junit-reporter package by relying on Mocha's xUnit module (#3593)

This important because the removed package (`mocha-junit-reporter`)
is quite out-of-date and pulls in a vulnerable dependency.
Funnily, we're still producing JUnit-style XML test results,
which is a very curious bug in Mocha itself.

Author: andyleejordan