looks like initializeModules would be cleaner if it just created the arraylist itself and returned it rather than modifying an empty one that's passed in.