Last modified: 2014-10-15 12:49:46 UTC
ExternalChangesList has no documentation on what it's supposed to be and class-orientated (ie everything is static). It also has a 120+ line method that should probably be split up.
It also has no tests.
From the looks of it, it's a utility class for constructing changes entries on on RecentChanges and the watchlist. Unit testing this will probably just be a bunch of regular expressions against the output, but we should have that anyway.
Yes. Looks like this code has not been improved upon. Also still lacks tests for basically everything.