[HBASE-4955] Use the official versions of surefire & junit

• Type: Improvement
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 0.94.0, 0.98.0, 0.96.0, 0.99.0
• Fix Version/s: 0.99.2
• Component/s: test

Environment

all

• Hadoop Flags: Reviewed
• Assignee: Alex Newman
• Reporter: Nicolas Liochon
• Created: 05/Dec/11 18:10
• Updated: 06/Apr/18 17:55
• Resolved: 21/Aug/14 08:51

Description

We currently use private versions for Surefire & JUnit since HBASE-4763.

This JIRA traks what we need to move to official versions.

Surefire 2.11 is just out, but, after some tests, it does not contain all what we need.

JUnit. Could be for JUnit 4.11. Issue to monitor:
https://github.com/KentBeck/junit/issues/359: fixed in our version, no feedback for an integration on trunk

Surefire: Could be for Surefire 2.12. Issues to monitor are:
329 (category support): fixed, we use the official implementation from the trunk
786 (@Category with forkMode=always): fixed, we use the official implementation from the trunk
791 (incorrect elapsed time on test failure): fixed, we use the official implementation from the trunk
793 (incorrect time in the XML report): Not fixed (reopen) on trunk, fixed on our version.
760 (does not take into account the test method): fixed in trunk, not fixed in our version
798 (print immediately the test class name): not fixed in trunk, not fixed in our version
799 (Allow test parallelization when forkMode=always): not fixed in trunk, not fixed in our version
800 (redirectTestOutputToFile not taken into account): not yet fix on trunk, fixed on our version

800 & 793 are the more important to monitor, it's the only ones that are fixed in our version but not on trunk.

Issue Links

• blocks: HBASE-7211 Improve hbase ref guide for the testing part.
• is depended upon by: HBASE-8256 Add category Flaky for tests which are flaky
• relates to: HBASE-4602 Make the suite run in at least half the time
• requires: HBASE-7554 TestZooKeeper depends on the execution order
• requires: HBASE-7556 TestAdmin depends on the execution order
• requires: HBASE-7553 TestServerCustomProtocol depends on the test methods execution order