Fix xml hex entity parse

[johnjaylward]

fixes #537

What problem does this code solve?

Corrects bug in XML.unescape where it was incorrectly passing entities to be unescaped without lower-casing first as the tokener does internally. This corrects the issue at the tokener level by not requiring the entity to be lowercase before processing.

Risks

None. Bug fix

Changes to the API?

No

Will this require a new release?

Yes

Should the documentation be updated?

No need

Does it break the unit tests?

No. New unit tests have been added showing the tokener properly parsing, but the direct call failing, then succeeding after the code change

Was any code refactored in this commit?

Just some test calls to make error reporting easier to read:
changed some calls that looked like this:

// hard to read in error logs
assertTrue("some message", actual.equals(expected));

to

// easier to read in error logs
assertEquals("some message", expected, actual);

Review status
APPROVED

[stleary]

Starting 3-day comment window.