Bug 1016090 - Duplicated test case - JBWS3084TestCase.java
Summary: Duplicated test case - JBWS3084TestCase.java
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: Web Services
Version: 6.2.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: DR2
: EAP 6.3.0
Assignee: Rebecca Searls
QA Contact: Rostislav Svoboda
Russell Dickenson
URL:
Whiteboard:
Depends On: 1061340
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-10-07 13:18 UTC by Rostislav Svoboda
Modified: 2014-07-03 23:44 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-06-28 15:24:34 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Rostislav Svoboda 2013-10-07 13:18:09 UTC
[rsvoboda@steve jbossws-cxf-4.2.1.Final]$ find . | grep org.jboss.test.ws.saaj.jbws3084.JBWS3084TestCase
./modules/testsuite/shared-tests/src/test/java/org/jboss/test/ws/saaj/jbws3084/JBWS3084TestCase.java
./modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/saaj/jbws3084/JBWS3084TestCase.java

I think JBWS3084TestCase should be merged and placed just into one module.
Content of test and support classes is different.

Test from shared-tests module seems to be more complex.

Comment 1 Rebecca Searls 2013-11-05 21:25:27 UTC
Investigating deeper into these 2 tests they are significantly different
in their setUp and tearDown requirements.  The test file in the shared-tests
directory does not define any setUp and tearDown functionality, however the file 
in directory cxf-tests does.  There is significant setUp and tearDown code in 
class JBossWSCXFTestSetup, which (cxf-tests) JBWS3084TestCase is returning from
method JBWS3084TestCase.suite().  Because of this it is not practical or desirable to combine these 2 files.  

I would recommend the solution for the dup file names is to rename file
modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/saaj/jbws3084/JBWS3084TestCase.java to JBWS3084CxfTestCase.java

Comment 2 Rostislav Svoboda 2013-11-06 07:53:41 UTC
Agree with renaming, please proceed.

We need unique testcase / test-report names because of Jenkins reporting.

Comment 3 Rebecca Searls 2013-11-06 14:04:42 UTC
JBWS3084TestCase.java renamed to JBWS3084CxfTestCase.java
   svn Committed revision 18064.

Comment 4 Rostislav Svoboda 2013-11-08 07:42:30 UTC
https://source.jboss.org/changelog/JBossWS?cs=18064

Comment 6 Kabir Khan 2014-04-04 14:53:23 UTC
Setting to ON_QA since upgrade should fix this

Comment 7 Petr Sakař 2014-04-08 10:39:53 UTC
Verified for EAP 6.3.0.ER1


Note You need to log in before you can comment on or make changes to this bug.