Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 331368

Summary: Dependency from org.eclipse.jst.j2ee.tests to org.eclipse.jst.jsf.test.util should be removed
Product: [WebTools] WTP Java EE Tools Reporter: Roberto Sanchez Herrera <shr31223>
Component: jst.j2eeAssignee: Roberto Sanchez Herrera <shr31223>
Status: RESOLVED FIXED QA Contact: Chuck Bridgham <cbridgha>
Severity: normal    
Priority: P2 CC: ccc, dimitar.giormov, kaloyan
Version: 3.2Flags: cbridgha: review+
Target Milestone: 3.2.3   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Patch to remove the dependency of jsf tests
dimitar.giormov: iplog+
Zip used in the test none

Description Roberto Sanchez Herrera CLA 2010-11-29 13:07:08 EST 
Build Identifier: M-3.2.3-20101129083231

As part of the work made in bugzilla 323185, a dependency from plugin org.eclipse.jst.j2ee.tests to org.eclipse.jst.jsf.test.util was added, in order to call a method to unzip projects from a zip file. 

The plugin org.eclipse.jst.j2ee.tests already has a class that do that (take a look at org.eclipse.jst.j2ee.flexible.project.fvtests.J2EEFlexProjDeployablePerfTest, which uses org.eclipse.etools.common.test.apitools.ProjectUnzipUtil to unzip projects).

The class org.eclipse.jst.jee.model.web.tests.TestWebXmlModelAfterUpdate should be changed to use ProjectUnzipUtil  instead of org.eclipse.jst.jsf.test.util.TestUtil and the dependency should be removed

Reproducible: Always
Comment 1 Carl Anderson CLA 2010-11-29 13:10:42 EST 
The j2ee tests are utilized in an adopter product.  As such, we wish to maintain the j2ee tests dependencies along the same lines as the j2ee plugin dependencies.
Since the additional JSF tests dependencies could cause adoption issues, I am raising this to a P2.
Comment 2 Roberto Sanchez Herrera CLA 2010-11-29 14:59:18 EST 
Created attachment 184070 [details]
Patch to remove the dependency of jsf tests
Comment 3 Roberto Sanchez Herrera CLA 2010-11-29 15:02:15 EST 
Created attachment 184074 [details]
Zip used in the test

I had to work a little on this attachment, to include the "root" project folder. This file must be placed in folder TestData/WebEditDdExternal in plugin org.eclipse.jst.j2ee.tests
Comment 4 Chuck Bridgham CLA 2010-11-30 20:33:39 EST 
approve
Comment 5 Carl Anderson CLA 2010-12-01 19:26:03 EST 
Committed to R3_2_maintenance and HEAD for WTP 3.2.3 and WTP 3.3