Well, no, the main point was to add the validation and be explicit in the tests where it matters what type of attachable is being used. A factory should generally be able to generate a valid instance by default. I guess we could make an exception for attachments if we don't trust ourselves to build the right thing in our tests, but I'd say this is the right way to go.