Bug 319134 - FetchGroups testsuite should run on application servers
Summary: FetchGroups testsuite should run on application 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: 2010-07-07 10:22 EDT by Kevin Yuan CLA
Modified: 2022-06-09 10:33 EDT (History)
1 user (show)

See Also:

Attachments
patch for bug fix (57.32 KB, patch)
2010-07-07 10:38 EDT, Kevin Yuan CLA 		no flags Details | Diff
patch for bug fix (523.81 KB, patch)
2010-07-15 11:48 EDT, Kevin Yuan CLA 		no flags Details | Diff
patch for bug fix (524.76 KB, patch)
2010-07-16 10:45 EDT, Kevin Yuan CLA 		no flags Details | Diff
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 Kevin Yuan CLA 2010-07-07 10:22:34 EDT 
Currently all jpa.fetchgroups testing only run on JavaSE, they should be migrated to run on application servers, e.g. WLS, WAS, JBoss, Glassfish, etc.
Comment 1 Kevin Yuan CLA 2010-07-07 10:38:48 EDT 
Created attachment 173657 [details]
patch for bug fix
Comment 2 Kevin Yuan CLA 2010-07-07 10:44:25 EDT 
The attached are all jpa.fetchgroups tests migrated to run on both JavaSE and application servers, also verified on jdk1.6.0_20, WLS 10.3.3, JBoss-5.1.0.GA, and GlassFish-v3.
Comment 3 Kevin Yuan CLA 2010-07-15 11:48:16 EDT 
Created attachment 174413 [details]
patch for bug fix

Based on the feed back from Andrei Ilitchev, some migrations had been modified, and created this new patch.
Comment 4 Kevin Yuan CLA 2010-07-16 10:45:44 EDT 
Created attachment 174502 [details]
patch for bug fix

Add runBare() for "FetchGroup Weaving Requirement", but will not determine if isWeavingEnabled() on server.
Comment 5 Kevin Yuan CLA 2010-07-16 11:28:46 EDT 
Verified on both javaSE (1.6.0_20) and javaEE (WLS/JBoss/GlassFish) by running jpa-lrg with no-weaving/static-weaving/dynamic-weaving.

Code reviewed by Andrei Ilitchev
Checked in trunk, revision: 7840.
Backport to 2.1.1, revision: 7841
Comment 6 Eclipse Webmaster CLA 2022-06-09 10:33:50 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink