Summary: | AggregateTestModel using SubstituteSequencingWithReturningPolicyAdapter failed | ||||||
---|---|---|---|---|---|---|---|
Product: | z_Archived | Reporter: | Edwin Tang <edwin.tang> | ||||
Component: | Eclipselink | Assignee: | Andrei Ilitchev <andrei.ilitchev> | ||||
Status: | RESOLVED FIXED | QA Contact: | |||||
Severity: | normal | ||||||
Priority: | P2 | CC: | andrei.ilitchev, eclipselink.foundation-inbox, tom.ware | ||||
Version: | unspecified | Keywords: | test | ||||
Target Milestone: | --- | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Whiteboard: | |||||||
Attachments: |
|
Description
Edwin Tang
2008-11-28 13:19:18 EST
Setting to 1.1x pending information about if this is a regression. If it is, target will set to 1.1. It is a regression. It started to fail on Nov 20th, 2008. This was caused by Andrei's checkin - "Bug 248293: JPA 2.0 Element Collections." in revision 2794. Steps for running "AggregateTestModel using SubstituteSequencingWithReturningPolicyAdapter", 1. From trunk, execute "ant build" to compiles all components and tests (excluding Oracle extensions) 2. From trunk, execute "ant build-oracle-extension" to compile Oracle extensions and tests 3. From trunk/foundation/eclipselink.extension.oracle.test, execute "ant test-browser" to start the testing browser 4. The model is under Oracle Tests -> OracleTestModel -> ReturningPolicyTestModel in the testing browser Created attachment 119679 [details]
The patch.
The patch fixes both ReturningPolicy and tests.
The changes in both places due to the fact that AggregateCollectionMapping now clones the reference descriptor - and initializes the clone (possibly overriding tables and fields).
ReturningPolicy had to use buildField that returns back the corresponding field in descriptor (that may have a different name - the one supplied to AggregateCollectionMapping).
The test has to create trigger for the new table (added through AggregateCollectionMapping and overriding the original one), too.
Fixed. The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink |