Issue Details (XML | Word | Printable)

Key: HARMONY-5224
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Stepan Mishura
Reporter: Vasily Zakharov
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
Harmony

[buildtest][gut] Geronimo Unit Tests update to v2.0.2

Created: 29/Nov/07 04:09 PM   Updated: 04/Apr/08 06:03 PM
Return to search
Component/s: build - test - ci
Affects Version/s: None
Fix Version/s: None

Time Tracking:
Not Specified

File Attachments:
  Size
File Licensed for inclusion in ASF works alltests.list 2007-11-30 06:18 PM Vasily Zakharov 11 kB
Text File Licensed for inclusion in ASF works Harmony-5224-Exclude.patch 2007-12-27 02:16 PM Vasily Zakharov 15 kB
Text File Licensed for inclusion in ASF works Harmony-5224-Exclude.patch 2007-12-02 04:11 AM Vasily Zakharov 14 kB
Text File Licensed for inclusion in ASF works Harmony-5224.patch 2007-12-02 03:55 AM Vasily Zakharov 11 kB
Issue Links:
Reference
 

Patch Info: Patch Available
Resolution Date: 25/Jan/08 07:13 AM


 Description  « Hide
For now, Harmony CC runs very old version of Geronimo Unit Tests, 1.2beta, while current version is 2.0.2.
I'm going to provide a patch to upgrade here.


 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Vasily Zakharov added a comment - 30/Nov/07 06:12 PM
The attached patch makes a number of interconnected changes, in particular:

1. Adjusts adaptors/gut/parameters.xml, adaptors/snapshot/adaptor.xml and tests/gut/gut.properties to use Geronimo v2.0.2 instead of 1.2-beta and Maven v2.0.6 instead of 2.0.4.

2. Adjusts adaptors/gut/adaptor.xml to store downloaded Maven and Geronimo in tests/gut, not in build/checkouts - thus the same downloaded copies are used if the test is run from root build.xml or from tests/gut/build.xml.

3. Adjusts tests/gut/build.xml for the number of things:

3.1. In Geronimo v2.x the source archive geronimo-<version>-src.tar.gz contains a directory named geronimo-<version>-src, not geronimo-<version> as it was before. It's taken into account, the directory is renamed to geronimo-<version> after exctracting.

3.2. Processing of exclude list is optimized, a number of new variables introduced to structurize the code.

3.2.1. Previously in target setup.non-empty.exclude.list it could remove the platform-specific exclude list before using it if the tests were run from tests/gut/build.xml, as basedir and work.dir were the same. Now the working exclude list is named simply exclude.list and doesn't overlap with existing files.

3.3. Results processing is optimized.

3.3.1. logs directory is abolished as duplicate of reports directory.

3.3.2. junitreport generation is added.

Vasily Zakharov added a comment - 30/Nov/07 06:17 PM
Known issues:

1. Geronimo v2.0.2 used old XStream v1.1.3, that causes some tests to fail (see HARMONY-5196 for details). Reconfiguring Geronimo to use latest XStream 1.2.2 is possible and resolves the problem, but is tricky and is not easy to be done automatically. So, until Geronimo 2.1 is out, those tests would have to be excluded.

2. I'm still working on updating the excluded lists. Some tests now pass with bugs in Geronimo fixed, some tests are just no longer there, and some new failures appeared, investigating.

Vasily Zakharov added a comment - 30/Nov/07 06:20 PM
The attached alltests.list file should be put to bt-2/tests/gut directory, I didn't include it to the patch because it only contains one very long line.

The file contains the list of all GUT tests, updated to v2.0.2.

Vasily Zakharov added a comment - 02/Dec/07 03:00 AM
Attached another patch, Harmony-5224-Exclude.patch that updates the exclude lists.

Vasily Zakharov added a comment - 02/Dec/07 03:25 AM
Updated both patches.

Please apply both Harmony-5224.patch and Harmony-5224-Exclude.patch, and also put alltests.list to bt-2/tests/gut directory.

Vasily Zakharov added a comment - 27/Dec/07 02:16 PM
Updated exclude list patch.

Stepan Mishura added a comment - 25/Jan/08 07:13 AM
Thanks Vasily - the patch was applied to BTI branch 2.0 at r615105.
Please check that the patch was applied as you expected.

Stepan Mishura added a comment - 28/Jan/08 05:28 AM
Starting with r615453 snapshot revision Geronimo Unit Tests 2.0.2 is used.

Vasily Zakharov added a comment - 14/Feb/08 02:47 PM
The updated GUT works fine in CC, so closing the issue.

Vasily Zakharov added a comment - 31/Mar/08 02:58 PM
r643013: committed a change:
Removed TSSConfigEditorTest from exclude lists as HARMONY-5626 is now fixed.

Vasily Zakharov added a comment - 04/Apr/08 06:03 PM
r644788 put CXFHandlerResolverTest to exclude lists on 64-bit platforms as HARMONY-5625 is now investigated and the test proved invalid.
Cleaned up the exclude lists a bit.

r644794 removed the descriptions of old GUT v1.2 test problems from exclude lists.

Enabling complete, 100% valid test pass rate achieved.