IPojos.findContainerHierarchies() fails with ValidationException

[dwhitehurst-x]

Client integration tests fail with nested Hibernate error "not null property references a null or transient value" in PojoServiceTest?, method testFindContainerHierarchies rather than the expected Api usage exception.

Test marked as "broken"

try {
       results = iPojos.findContainerHierarchies(Dataset.class, ids, empty
                   .map());
           fail("Should fail");
       } catch (ApiUsageException e) {
           // ok.
       }

[jmoore]

Investigating this now. Has appeared in the wild.

[jmoore]

This issue was not was to track down. The implementation of findContainerHierarchies walks a graph returned from Hibernate and rearranges the entities in a read-only graph. However, to prevent Hibernate from saving the rearragements, first the entities are evicted from the session.

The exception that's being thrown here:

ome.conditions.ValidationException: not-null property references a null or transient value: ome.model.containers.DatasetImageLink.details.creati
onEvent; nested exception is org.hibernate.PropertyValueException: not-null property references a null or transient value: ome.model.containers.DatasetImageLink.detail
s.creationEvent
        at org.springframework.orm.hibernate3.SessionFactoryUtils.convertHibernateAccessException(SessionFactoryUtils.java:628)
        at org.springframework.orm.hibernate3.HibernateAccessor.convertHibernateAccessException(HibernateAccessor.java:412)
        at org.springframework.orm.hibernate3.HibernateTemplate.execute(HibernateTemplate.java:377)
        at org.springframework.orm.hibernate3.HibernateTemplate.execute(HibernateTemplate.java:338)
        at ome.security.basic.EventHandler.invoke(EventHandler.java:160)
...

just says that one of the read-only links created for the read-only graph is missing some values. That's fine, we expect as much. The question is why is it even checking. Because an object is still reachable that shouldn't be.

Turns out the cyclical relationship image->defaultPixels->image is again causing problems (this will probably be removed in the next version.)

Fixed by r1921.

[jmoore]

Included in milestone:3.0-Beta2.2.