Dear Vlad, I saw the PR to solve this, but IMO it is not sufficient. That's just a workaround to a small part of the real problem, that's metamodel classes uniqueness. I added another test case to check for both uniqueness (by identity) and equivalence of ManagedTypes. Actually, on 5.2.12:
- managedTypesSelfConsistenceByEquivalenceTest: pass
- managedTypesFromAttributesConsistenceByEquivalenceTest: fail
- managedTypesFromAttributesConsistenceByIdentityTest: fail
- entityTypesFromAttributesConsistenceByEquivalenceTest: pass
- entityTypesFromAttributesConsistenceByIdentityTest: pass
- embeddableTypesFromAttributesConsistenceByEquivalenceTest: fail
- embeddableTypesFromAttributesConsistenceByIdentityTest: fail
I think that for each Persistence Class (annotated with @Entity, @Embeddable, ...) there should be one (and only one) ManagedType to be retrieved from Metamodel (itself, or retrived by Attribute types). This is true only for Entity types. Thanks |