refactored the test loader to iterate once and be a bit more readable

Prevented the test loader from iterating twice (not really a performance win, but still unnecessary work) and made things a bit more readable. `requirejs.entries` and `entry` are easier to understand than `requirejs._eak_seen` and `key` (to me at least).