KeyValueBlobTransientStore

[Eric_Ace]

Nuxeo Folks: KeyValueBlobTransientStore.getBlobs can return Collections.emptyList which you can't add to. However, SimpleTransientStore.getBlobs returns an ArrayList and so a unit test that expects to be able to add to the return value from getBlobs can pass, but then fail if deployed over a K/V transient store...

[Florent_Guillau]

The Javadoc for TransientStore.getBlobs doesn't promise anything about mutability of the returned List, so you shouldn't assume that it's an ArrayList even though that's the case for the current implementation of SimpleTransientStore.

[Eric_Ace]

True. It is a shortcoming of Java that interface contracts don't explicitly state mutability. Lacking that, programmers tend to interpret the contract to be