Silly question but shouldn't the BeforeCompletionFailureTest create an entity with the same primary key twice? I would expect the second call to newEntity() to pass one instead of two.