Add Another constructor in the Factory class of Cocktail.Contrib

Using the Factory in a SampleDataProvider makes no sense. The factory creates "new" entities where as a SampleDataProvider fakes existing entities. You see the difference in how they get added to the EntityManager. The factory calls AddEntity where as in a SampleDataProvider you use AttachEntity. The difference is if you use AddEntity, the entity will be in the added state ready to be inserted into the database. AttachEntity on the other hand sets the entity state to unchanged. This is important, because sample data is supposed to fake a real database where the entities have already been inserted are are therefore in unchanged state.

you are right, I totally forgot about the AddEntity/AttachEntity issue. I focused in the different task I do in the factory to create/initialize the entities and wanted to be sure the entities created in the SampleDataProvider followed the same rules as in the factory. Often when we modify the Factory we don't always remember to modify the SampleDataProvider too.