don't need instance variables - factory girl?