And in fact (and I hate to say this) since the caller is iterating over the map, we shouldn't return a HashMap because the iteration ordering of HashMap is undefined, and the last thing someone wants in a test framework is seemly-random problems caused by undefined behavior.

To be safe, we should replace all occurrences of `HashMap` with `LinkedHashMap`