Use attr_readers in association declaration
Use attr_readers in association declaration

This commit separates the factory name from the overrides on initialize,
then uses attr_readers throughout instead of manipulating instance
variables.

This is a bit cleaner, and will make it easier to reused the
factory_name and overrides in a future code change.

thoughtbot/factory_girl