Bug 374302 - Need to alter the existing Extensibility test suite to use DDL
Summary: Need to alter the existing Extensibility test suite to use DDL
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Eclipselink (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Edwin Tang CLA
QA Contact:
URL:
Whiteboard:
Keywords: test
Depends on:
Blocks:
 
Reported: 2012-03-14 15:38 EDT by Edwin Tang CLA
Modified: 2022-06-09 10:07 EDT (History)
2 users (show)

See Also:

Attachments
patch file (1.38 KB, patch)
2012-04-02 10:31 EDT, Edwin Tang CLA 		no flags Details | Diff
patch file (3.21 KB, patch)
2012-04-04 10:29 EDT, Edwin Tang CLA 		no flags Details | Diff
patch file (2.46 KB, patch)
2012-04-04 13:01 EDT, Edwin Tang CLA 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Edwin Tang CLA 2012-03-14 15:38:45 EDT 
In order to test the integration of the feature "DDL Alter Table Support" and Extensibility, we need to alter the existing Extensibility test suite to use DDL.
Comment 1 Edwin Tang CLA 2012-04-02 10:31:44 EDT 
Created attachment 213451 [details]
patch file

- Fix ExtensibilityTests.testExistingEntityManagerAfterRefresh(). Now it passes when it is run individually.
Comment 2 Edwin Tang CLA 2012-04-04 10:29:28 EDT 
Created attachment 213586 [details]
patch file

- Alter ExtensibilityTests to use DDL instead of TableCreater
Comment 3 Edwin Tang CLA 2012-04-04 13:01:44 EDT 
Created attachment 213596 [details]
patch file

- Based on feedback from Christ
- Still use ExtensibilityTableCreator().replaceTables(serverSession) to drop and create tables in testSetup(), to make sure the tables are dropped before running any tests
- In persistence.xml files, have the following line
            <property name="eclipselink.ddl-generation" value="create-or-extend-tables"/>
Comment 4 Edwin Tang CLA 2012-04-04 13:09:34 EDT 
Checked in the patches into trunk.
1st patch reviewed by: Tom Ware
2nd patch reviewed by: Christopher Delahunt
Comment 5 Eclipse Webmaster CLA 2022-06-09 10:07:23 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink