Bug 372514 - Need to enable to run DDLGenerationExtendTablesJUnitTestSuite with app servers
Summary: Need to enable to run DDLGenerationExtendTablesJUnitTestSuite with app servers
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: Nobody - feel free to take it CLA
QA Contact:
URL:
Whiteboard:
Keywords: test
Depends on:
Blocks:
 
Reported: 2012-02-24 10:47 EST by Edwin Tang CLA
Modified: 2022-06-09 10:19 EDT (History)
1 user (show)

See Also:


Attachments
patch file (2.01 KB, patch)
2012-02-24 11:38 EST, Edwin Tang CLA
no flags Details | Diff
patch file (871 bytes, patch)
2012-02-24 11:53 EST, Edwin Tang CLA
no flags Details | Diff
patch file (2.65 KB, patch)
2012-03-26 10:56 EDT, Edwin Tang CLA
no flags Details | Diff
patch file (2.67 KB, patch)
2012-03-29 15:07 EDT, Edwin Tang CLA
no flags Details | Diff
patch file (3.06 KB, patch)
2012-03-29 15:21 EDT, Edwin Tang CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Edwin Tang CLA 2012-02-24 10:47:21 EST
DDLGenerationExtendTablesJUnitTestSuite was added to test feature "DDL Alter Table Support". It is currently run in JSE environment. We need to enable to run it in JEE environment, i.e., with app servers.
Comment 1 Edwin Tang CLA 2012-02-24 11:38:57 EST
Created attachment 211584 [details]
patch file

Added target server-test-ddlgeneration-extendtables in the jpa\eclipselink.jpa.test\antbuild.xml, and included it in server-test-lrg
Comment 2 Edwin Tang CLA 2012-02-24 11:53:48 EST
Created attachment 211585 [details]
patch file

Reversed the previous patch, instead, added the suite into DDLGenerationServerTestSuite
Comment 3 Edwin Tang CLA 2012-03-26 10:56:14 EDT
Created attachment 213185 [details]
patch file

- Create a server test suite - DDLGenerationExtendTablesTestSuite, which extends DDLGenerationExtendTablesJUnitTestSuite, because persistence unit "MulitPU-1" must be used on servers
- Include the server test suite in DDLGenerationServerTestSuite
Comment 4 Edwin Tang CLA 2012-03-29 15:07:48 EDT
Created attachment 213352 [details]
patch file

- A neater solution without extending DDLGenerationExtendTablesJUnitTestSuite
Comment 5 Edwin Tang CLA 2012-03-29 15:21:12 EDT
Created attachment 213357 [details]
patch file

- Added comments based on feedback from Christopher Delahunt
Comment 6 Edwin Tang CLA 2012-03-29 15:27:23 EDT
Checked in trunk at revision: 11069
Code reviewed by: Christopher Delahunt
Comment 7 Eclipse Webmaster CLA 2022-06-09 10:19:48 EDT
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink