I don't think we should set the association to a user in the sit factory itself. This will lead to a lot of unnecessary objects being created when all we want to do is test a sit and don't need to interact with users at all.