Catalogue Browser
Showing 57 results
Lazily Evaluate Values
Motivation
Lazily evaluated attributes and values are only computed when they are needed.
In tests, some components are good candidates for being lazily evaluated, i.e., SUT dependencies and assertions.
For instance, assertions' message of failure that are expensive to compute may benefit from lazy evaluation.
Qualities
- Efficiency
Code Demonstration
assertEquals(expected, actual, message: slowComputation())message = () -> slowComputation()
assertEquals(expected, actual, message)Sources
Consolidate Multiple Assertions into a Fluent Assertion
Motivation
Test cases with many assertions are known as the assertion roulette test smell.
One way to fix it is replacing all assertions that have the same actual value with one single Fluent Assertion.
Fluent Assertions enables chaining together different condition checks as unary method invocations.
In comparison, traditional assertions take up to three parameters, therefore they are subject to confusion, e.g., failing a binary assertion with a custom message is expressed differently in JUnit 4 assertEquals(message, expected, actual), JUnit 5 assertEquals(expected, actual, message), and TestNG assertEquals(actual, expected, message)
Conversely, in AssertJ, the same assertion would look like assertThat(actual).withFailMessage(message).isEqualTo(expected).
Qualities
- Uniformity
Code Demonstration
assertEquals(9, actual.size())
assertTrue(actual.contains(element))assertThat(actual)
.hasSize(9)
.contains(expected)Instances
Sources
Group Multiple Assertions
Motivation
Test Methods with multiple assertions may have its execution interrupted prematurely, thus preventing the evaluation of the remaining assertions (Soares et al., 2023).
Grouping multiple assertions (a₁, a₂,...,aₙ) with JUnit 5's assertAll(()->a₁, ()->a₂,..., ()->aₙ) assures that they execute and their result are outputted to the test report before terminating the test suite execution.
Qualities
- Effectiveness
Code Demonstration
assert(a₁)
...
assert(aₙ)AssertAll(
() -> assert(a₁)
...
() -> assert(aₙ)
)Instances
Sources
Replace Assertion
Motivation
1. Having a conditional logic in the assertions using the not (!) operator affect their readability.
2. Forced conditional expressions into assertTrue/assertFalse methods to verify objects equality should be replaced.
3. Developers should use special assertions (e.g. assertNull) instead of passing reserved words.
Qualities
- Uniformity
- Effectiveness
- Simplicity
Code Demonstration
assert(!condition)
assert(actual == expected)
assert(actual == true)
assert(actual == null)assertFalse(condition)
assertEquals(expected, actual)
assertTrue(actual)
assertNull(actual)Instances
Sources
Enforce Use of Type Inference
Motivation
As shown by Kashiwa et al. (2021) Change Return Type and Add Parameter refactorings in production APIs are mainly responsible for breaking tests, and require 4-12 lines of changes to fix the tests they break.
A good practice to avoid or limit the impact of such breaking changes in tests is to utilize the var keyword (Java 10 feature) in the place of type references within the tests.
When the var keyword is used, JVM applies type inference to detect automatically the datatype of a variable based on the surrounding context.
This approach makes the tests more resilient to SUT type-related changes.
Qualities
- Brittleness
Code Demonstration
Type variable = new ConcreteType()
AnotherType result = methodCall()
List<SpecificType> list = getItems()var variable = new ConcreteType()
var result = methodCall()
var list = getItems() Instances
Sources
Abstract away SUT's Construction
Motivation
1. C# only, AutoFixture works as a generic Test Data Builder and an Auto-Mocking container (SO 2056602) built upon sensible defaults for different attribute types, e.g., DateTime.Now (SO 2622334).
2. Sometimes the same hard-to-build objects can be reused throughout the test suite; ObjectMother (Schuh and Punke, 2001; Fields, 2014) encapsulates the complex build logic into its static methods for reuse and code clone elimination.
3. Unlike ObjectMother, "Test Data Builder copes well with variation in test data" (Fields, 2014). By providing a customizable test object creation with sensible defaults, Test Data Builder favours transparency over conciseness.
Qualities
- Changeability
- Uniformity
- Reliability
- Brittleness
- Reusability
- Efficiency
Code Demonstration
@Test
void t() {
SUT sut = new SUT(new ComplexDependency(
new NestedDependency(...),
...))
} // Variant (1) - C# only
[Test]
void t() {
Fixture fixture = new Fixture()
SUT sut = fixture.Create<SUT>()
}
// Variant (2)
@Test
void t() {
SUT sut = ObjectMother.CreateDefaultSUT()
}
// Variant (3)
@Test
void t() {
SUT sut = new SutBuilder()
.WithParam1(expression)
.WithParam2(expression)
.Build()
} Instances
- StackOverflow #2056602 (AutoFixture)
- StackOverflow #2622334 (AutoFixture)
- StackOverflow #34284527 (AutoFixture)
- Commit: Kafka (Object Mother)
- StackOverflow #69918987 (Object Mother)
- StackOverflow #33080488 (Test Data Builder)
- StackOverflow #3735015 (Test Data Builder)
- StackOverflow #54156809 (Test Data Builder)
Sources
Extract Preconditions
Motivation
Preconditions can either be expressed through special assertions (e.g., JUnit 5 assumptions) or regular ones.
It all comes down to whether an unmet precondition should be interpreted as a test failure or not as JUnit 5 assumptions behave similarly to assertions except they abort tests without failing them.
Preconditions are a set of rules to determine whether it makes sense to continue the execution of specific test methods.
1. Placing preconditions in a class-level fixture is particularly useful for efficiency as it aborts the execution of all tests in a test class as soon as the preconditions fail once.
2. Custom Utility Assumption Methods assist developers keeping a consistent mechanism to skip specific test methods in a test class.
3. Tests can contain code that may or may not be executed due to branching logic. The branching logic may cause the test to finish without executing the intended assertion(s). Adding guard assertions (Meszaros, 2007) to the beginning of tests guarantee the test code will only execute when established preconditions are met.
4. Conditional Execution (Soares et al., 2023), introduced in JUnit 5, enables test skipping according to programmatic conditions. Unlike guard assertions, Conditional Execution are expressed as method annotations, which further separate test logic from its preconditions.
5. When conditions are not clear or easily computed, expecting exception could be a valid alternative. Try-catch statement with failing assumptions in the catch-block may be leveraged to make exceptions disable a test rather than failing or passing it. Such a workaround compensates for the lack of an Assumptions.doesNotThrow API in JUnit.
Qualities
- Efficiency
- Reliability
- Uniformity
Code Demonstration
@Test
void t() {
if (C) {
stmt
}
}// Variant (1)
@BeforeAll
static void setUpClass() {
assumeTrue(C)
}
@Test
void t() {
stmt
}
// Variant (2)
@Test
void t() {
assumePrecondition()
stmt
}
// Variant (3)
@Test
void t() {
assertTrue(C)
stmt
}
// Variant (4)
@Test
@EnabledIf("C")
void t() {
stmt
}
// Variant (5): Provided stmt throws E when C is false
@Test
void t() {
try {
stmt
} catch (E) {
assumeTrue(false)
}
}Instances
Sources
Reuse code with Fixture / Extract Fixture
Motivation
Test methods within a single test class may evolve and share many code similarities.
In that case, unless the duplication only affects few methods in the test class, it is advisable to place the duplicated code into a test fixture, e.g., set-up or tear-down methods.
The former comprises test data creation and configuration, whereas the latter should clean up the environment and destroy test objects safely.
If the desired test fixture type already exists in the test class, one can reuse it.
Otherwise, one extracts a new fixture containing the duplicated code.
Qualities
- Reliability
- Uniformity
- Reusability
- Changeability
- Simplicity
Code Demonstration
@Test
void t1() {
setUp
stmt
teardown
}
@Test
void t2() {
setUp
stmt'
teardown
}
@AfterEach
void tearDown() {
teardown'
}// Extract Fixture
@BeforeEach
void setUp() {
setUp
}
// Reuse Code with Fixture
@AfterEach
void tearDown() {
teardown
teardown'
}
@Test
void t1() {
stmt
}
@Test
void t2() {
stmt'
}Instances
Sources
Inline Fixture
Motivation
As new methods are added and others removed, test fixtures may become obsolete and less cohesive, i.e., General Fixture test smell.
As a consequence, one may inline the test fixture, introducing code duplication into the few test methods that depend on it.
Qualities
- Changeability
- Independence
Code Demonstration
@BeforeEach
void setUp() {
setUp
}
@AfterEach
void tearDown() {
teardown
}
@Test
void t1() {
stmt
}
@Test
void t2() {
stmt'
}
@Test
void t1() {
setUp
stmt
teardown
}
@Test
void t2() {
setUp
stmt'
teardown
} Sources
Minimize Fixture
Motivation
Diverging context of test methods in a single class leads to a General Fixture, i.e., a fixture whose statements inconsistently target different subsets of the test methods.
"Minimizing fixtures make tests better suitable as documentation and less sensitive to changes" (Deursen et al., 2001)
Qualities
- Independence
Code Demonstration
@BeforeEach
void setUp() {
setUp
setUp'
setUp''
}
@Test
void t1() {
stmt
}
@Test
void t2() {
stmt'
} @BeforeEach
void setUp() {
setUp
}
@Test
void t1() {
setUp'
stmt
}
@Test
void t2() {
setUp''
stmt'
}Sources
Override Fixtures
Motivation
Despite the risk of introducing redundancy and inter-test dependency, inheritance is extensively used in Java tests.
For reusing tests under different environment settings, testers extract subclasses that override test fixtures.
However, JUnit 5 changed how overriding test fixtures are executed.
While in JUnit 4 test fixtures implemented in subclasses are always executed, JUnit 5 requires the subclass's fixture to be annotated with one of the test fixture annotations (i.e., @BeforeEach, @BeforeAll, @AfterEach, or @AfterAll) even when that particular test fixture overrides a test fixture in the superclass that is annotated as such (Travis, 2018).
Qualities
- Changeability
- Reusability
Code Demonstration
class C1 {
@BeforeEach
void setUp() {
setUp
}
}
class C2 {
void setUp() {
setUp
setUp'
}
}class C1 {
@BeforeEach
void setUp() {
setUp
}
}
class C2 extends C1 {
@Override
@BeforeEach
void setUp() {
super.setUp()
setUp'
}
}Instances
Sources
Inject SUT Dependency
Motivation
1. Complex SUT dependencies are good mocking candidates: injecting such dependencies into the SUT constructor often enables to evaluate the interaction with the mocked dependency.
2. Some dependency injection containers, e.g., Spring and Guice, support constructor injection, but test classes usually have a setup method (a test fixture) rather than a constructor. To take advantage of Dependency Injection, your test fixture should be replaced.
Qualities
- Reliability
- Uniformity
- Brittleness
Code Demonstration
class C {
@BeforeEach
void setUp() {
var dep = new DependencyImpl()
sut = new SUT(dep)
}
}// Variant (1)
class C {
@BeforeEach
void setUp() {
Dependency dep = mock(Dependency.class)
SUT sut = new SUT(dep)
}
}
// Variant (2)
class C {
@Autowired
C(Dependency dep) {
sut = new SUT(dep)
}
}Instances
Sources
Replace Class Fixture with Method Fixture
Motivation
The components defined in a test fixture should not be modified during the test execution unless the test fixture runs once per test method.
Qualities
- Independence
Code Demonstration
@BeforeAll
void setUp() { }@BeforeEach
void setUp() { }Instances
Sources
Replace Method Fixture with Class Fixture
Motivation
Sometimes a test fixture does not have to execute once for each test method, thus, constraining its execution to once for all methods in a test suite can speed-up test execution.
Qualities
- Efficiency
Code Demonstration
@BeforeEach
void setUp() { }@BeforeAll
void setUp() { }Instances
Sources
Split Fixture
Motivation
When performance is a concern, testers may prefer a setupClass method rather than a setup method, but there is a caveat, not all statements are safe for setupClass.
Then, developers can split the time-consuming part of a fixture, which goes into setupClass, from the part that require execution before each test, which goes into setup.
Qualities
- Efficiency
Code Demonstration
@BeforeEach
void setUp() {
stmt
stmt'
}@BeforeAll
void setUpClass() {
stmt
}
@BeforeEach
void setUp() {
stmt'
}Sources
Merge Fixture
Motivation
Merge separate fixture methods to simplify setup logic when separation is unnecessary.
Qualities
- Changeability
- Simplicity
Code Demonstration
@BeforeAll
void setUpClass() {
stmt
}
@BeforeEach
void setUp() {
stmt'
}@BeforeEach
void setUp() {
stmt
stmt'
}Instances
Sources
Parameterize Test with Framework Support
Motivation
1. The logic of many test methods may share similarities, i.e., test code duplication. When multiple methods only differ in terms of data input, it is a good candidate for parameterization.
2. JUnit 5’s and TestNG’s parameterized test class have at least one data provider, but having many may compromise the tests’ cohesion and maintainability.
3. To accommodate variation in the reuse of mostly similar test methods, parameterized tests may include flag parameters to enable/disable test methods with assumptions or assertions guarded by if-statements.
4. When multiple implementations with similar interfaces and behaviour (e.g., sorting algorithm) must be tested for multiple data, one may parameterize the test and extract test subclasses to build each implementation in a setup fixture.
Qualities
- Reusability
- Reliability
- Uniformity
Code Demonstration
// Variant (1)
@Test
void t1() {
stmt¹¹(P₁₁)
...
stmt¹ⁿ(P₁ₙ)
}
...
@Test
void t2() {
stmtᵐ¹(Pₘ₁)
...
stmtᵐⁿ(Pₘₙ)
}
// Variant (2)
@CsvSource({
P₁₁, ..., P₁ₙ
})
void t1() { }
@CsvSource({
Pₘ₁, ..., Pₘₙ
})
void t2() { } // Variant (1) and (2)
@ParameterizedTest
@CsvSource({
P₁₁, ..., P₁ₙ
...
Pₘ₁, ..., Pₘₙ
})
void t(P₁, ..., Pₙ) {
stmt¹(P₁)
...
stmtⁿ(Pₙ)
}void t() {
List actual = sut.method()
assertEquals(expcted, actual)
}
void t2() {
int[] actual = sut.method()
assertArrayEquals(expcted, actual)
}@ParameterizedTest
@MethodSource
void t(SUT sut) {
actual = sut.method()
if (P₁ instanceof P) {
assertEquals(expcted, actual)
}
else {
assertArrayEquals(expcted, actual)
}
...
assumeTrue(Pₙ)
stmtⁿ(Pₙ)
}class AlgorithmATest {
Algorithm algo = new AlgorithmA()
@Test
void t() {
algo.process(d')
}
@Test
void t2() {
algo.process(d'')
}
}
class AlgorithmBTest {
Algorithm algo = new AlgorithmB()
@Test
void t1() {
algo.process(d')
}
@Test
void t2() {
algo.process(d'')
}
}abstract class BaseTest {
Algorithm algo
@ParameterizedTest
@MethodSource("data")
void sharedTests(InputType input) {
Result result = algo.process(input)
assertBehavior(result)
}
static Stream<Arguments> data() {
return Stream.of(
Arguments.of(d'),
Arguments.of(d'')
)
}
}
class AlgorithmATest extends BaseTest {
@BeforeEach
void setup() { algo = new AlgorithmA() }
}
class AlgorithmBTest extends BaseTest {
@BeforeEach
void setup() { algo = new AlgorithmB() }
}Instances
- StackOverflow #33080488 (Common Logic)
- StackOverflow #34284527 (Common Logic)
- StackOverflow #3437962 (Common Logic)
- StackOverflow #38812937 (Common Logic)
- StackOverflow #58063106 (Common Logic)
- StackOverflow #69010737 (Common Logic)
- StackOverflow #69918987 (Common Logic)
- Commit: HBase (Common Logic)
- Commit: AWS SDK (Common Logic)
- Commit: ElasticSearch (Common Logic)
- Commit: Haikunator (Common Logic)
- PR Commit: HAPI FHIR (Common Logic)
- Commit: OpenTripPlanner (Common Logic)
- PR Commit: HTSJDK (Common Logic)
- Commit: HTSJDK (Merge Data Provider)
- StackOverflow #12051087 (Conditional)
- Commit: Hadoop (Conditional)
- Commit: HTSJDK (Conditional)
- Commit: SQLg (Conditional)
- Commit: Camel (Conditional)
- StackOverflow #10431090 (Inheritance and Fixture Overrides)
- StackOverflow #48579508 (Inheritance and Fixture Overrides)
- StackOverflow #73011622 (Inheritance and Fixture Overrides)
- Commit: Hadoop (Inheritance and Fixture Overrides)
Sources
Dependency-free Test Parameterization
Motivation
The logic of many test methods may share similarities, i.e., test code duplication. When test cases differ in the Arrange phase (setup), developers may choose to parameterize their fixture or utility methods. However, support is still incipient in most test frameworks (see junit5#878). As alternatives, developers leverage method and superclass extractions to achieve the same behaviour.
Qualities
- Reusability
- Uniformity
Code Demonstration
@Test
void t1() {
setUp(expression)
actual = sut.method()
assertEquals(expected, actual)
}
@Test
void t2() {
setUp(expression')
actual' = sut.method()
assertEquals(expected', actual')
}void t(expression, expected) {
setUp(expression)
actual = sut.method()
assertEquals(expected, actual)
}
@Test
void t1() {
t(expression, expected)
}
@Test
void t2() {
t(expression', expected')
}Instances
Sources
Enhance Test Report
Motivation
1. Good assertions' and exceptions' error messages are useful for debugging a failing test case.
2. Test frameworks rely on method names to identify tests in a test report; conversely, parameterized tests may benefit from custom names to distinguish the execution report of each one of its parameters.
3. Having more than one assertion with no explanation in a test method is known as the Assertion Roulette test smell. The name of tests suffering with that smell may not explain well the failing assertion.
4. Testers may misplace the expected and obtained value parameters of assertions due to conflicting standards in assertions' parameters order among Java test libraries, which incurs in misleading test reports.
5. The name of test methods and classes must indicate context, trigger, and behaviour of the SUT. Projects and communities may also enforce compliance to certain naming idioms. When one of those elements changes, the names should be updated to reflect it.
6. Incorrect use of built-in exception types may introduce ambiguity. Testers disambiguate exception testing by extracting a subclass of the ambiguous exception.
7. Like test methods, data providers should be named after the data they provide and their intended purpose. Despite different syntax, all major Java test frameworks offer a powerful name parameter for data provider annotation, in which developers may describe it in human readable format.
Qualities
- Effectiveness
- Uniformity
Code Demonstration
public class AccountService {
public void withdraw(double amount) {
if (amount <= 0) {
throw new IllegalArgumentException("Invalid amount")
}
}
}public class AccountService {
public void withdraw(double amount) {
if (amount <= 0) {
throw new IllegalArgumentException(
String.format("Withdrawal amount must be positive. Got: %.2f", amount)
)
}
}
}@ParameterizedTest
@ValueSource()
void t(params) { }@ParameterizedTest(name = "Descriptive name {arguments}")
@ValueSource()
void t(params) { }assertTrue(condition)
assertNotNull(obj)assertTrue("Condition should hold", condition)
assertNotNull("Object must be initialized", obj)assertEquals(actual, expected)assertEquals(expected, actual)@Test
void test1() { }@Test
void descriptiveTestName() { }@Test(expected = GeneralException.class)
void t() { }@Test(expected = SpecificException.class)
void t() { }@DataProvider(name = "The data provider")
static Object[] source() { }
@Test(dataProvider = "The data provider")
void t(params) { }@DataProvider(name = "Positive cases data provider")
static Object[] source() { }
@Test(dataProvider = "Positive cases data provider")
void t(params) { }Instances
- StackOverflow #10814807 (Error Msg)
- StackOverflow #42551015 (Error Msg)
- StackOverflow #33099441 (Param Name)
- StackOverflow #40960930 (Param Name)
- Commit: Kafka (Param Name)
- StackOverflow #4599177 (Explanation)
- StackOverflow #39765540 (Order Param)
- Commit: Hawkular (Order Param)
- Commit: HubSpot (Order Param)
- Commit: Iluwatar (Order Param)
- Commit: Rapidoid (Order Param)
- Commit: Rapidoid (Order Param)
- Commit: Strimzi (Order Param)
- Commit: Weld (Order Param)
- StackOverflow #39765540 (Order Param)
- Commit: CloudFoundry (Rename Test)
- Commit: cqframework (Rename Test)
- Commit: Hadoop (Exception)
- Commit: htsjdk (Rename Provider)
Sources
Integration Test to Unit Test
Motivation
There are inherent trade-offs between integration testing and unit testing (Aniche, 2022; Osherove, 2009; Vocke, 2018). Unit tests are perfect for high-churn logic and test-driven development because they are excellent at validating isolated components using mocked dependencies, offer quick execution, accurate failure localization, and require little maintenance. Their limited scope, however, runs the risk of ignoring systemic defects brought on by component interactions. On the other hand, integration tests confirm the interoperability of external systems and end-to-end workflows, revealing emergent problems like dataflow errors or protocol mismatches. However, this comes at the expense of slower execution, higher infrastructure requirements, and increased flakiness because of external dependencies. Integration tests reduce the risk of architectural misalignment, but they come with a higher operational complexity and delayed debugging than unit tests, which place more emphasis on agility and detailed feedback. The testing pyramid model (Vocke, 2018) is in line with a balanced strategy that prioritizes unit tests for core logic and saves integration tests for crucial interfaces. Breaking integration tests into unit tests consists in promoting isolation through mocking and proper set-up/clean-up between consecutive test executions. This ensures robust coverage of isolated behaviours while balancing risk mitigation with quick feedback.
Qualities
- Brittleness
- Independence
- Efficiency
Code Demonstration
setupDatabaseConnection()
User user = userService.create("test@example.com")
teardownDatabaseConnection()
assertTrue(userExistsInDB("test@example.com"))InMemoryUserRepository fakeRepository = new InMemoryUserRepository()
UserService service = new UserService(fakeRepository)
User user = service.create("test@example.com")
assertTrue(fakeRepository.contains("test@example.com"))Instances
Sources
Replace Duplicate Assert with Repeat Tests
Motivation
When a test repeatedly evaluates the same assertion for a fixed number of times, the code can be simplified with @RepeatedTest.
Qualities
- Simplicity
- Changeability
Code Demonstration
@Test
void t() {
stmt // 1ˢᵗ repetition
assertEquals(expected, actual)
...
stmt // mᵗʰ repetition
assertEquals(expected, actual)
}@RepeatedTest(m)
void t() {
stmt
assertEquals(expected, actual)
}Instances
No instances recorded.
Sources
Reuse Test Methods
Motivation
1. Oftentimes, unit tests simply differ to integration test due to mocked dependencies; one may find useful to generalize the test code so real instances and mocks can be used interchangeably.
2. Including third-party test suites in a custom TestSuite facilitates seamless integration and execution of external test cases when using third-party plug-in components.
Qualities
- Reusability
- Reliability
- Independence
Code Demonstration
@Test
void unitT() {
Dependency mock = mock(Dependency.class)
SUT sut = new SUT(mock)
stmt(sut)
verify(mock).expectedCall()
}
@Test
void integrationT() {
Dependency real = new Dependency()
SUT sut = new SUT(real)
assertEqual(expected, stmt(sut))
}abstract class BaseTest {
abstract Dependency dependency()
@Test
void t() {
SUT sut = new SUT(dependency())
stmt(sut)
assertOrVerify(expected)
}
}
class UnitC extends BaseTest {
Dependency dependency() {
return mock(Dependency.class)
}
}
class IntegrationC extends BaseTest {
Dependency dependency() {
return new Dependency()
}
}Introduce Test Parameter Object
Motivation
Like production code methods, parameterized tests with many parameters may become hard to read. Introducing parameter objects (Fowler et al., 2012) encapsulates all parameters into one object.
Qualities
- Uniformity
Code Demonstration
@ParameterizedTest
@CsvSource({
P₁₁, ..., P₁ₙ
...
Pₘ₁, ..., Pₘₙ
})
void t(P₁, ..., Pₙ) { } @ParameterizedTest
@MethodSource("data")
void t(ParamObj P) { }
public static StreamArguments data() {
return Stream.of(
Arguments.of(new ParamObj(P₁₁, ..., P₁ₙ)),
...
Arguments.of(new ParamObj(Pₘ₁, ..., Pₘₙ))
)
}Sources
Extract Utility Method
Motivation
After identifying common code that can be shared between two or more test classes, utility methods may be extracted.
1. Standardize a complex MUT (Method-Under-Test) invocation throughout the test code.
2. Combines many assertion invocations into a custom higher-level assertion method with semantically accurate name useful failure messages.
3. Encapsulates environment configuration and clean up, through data creation, deletion, and transformation without depending on test framework features and conventions, which promote higher flexibility and reusability at the cost of conciseness.
4. SUTs that share public methods with the same signatures but no common interface can hinder reusability within tests. Testers may choose to extract the assertions into an utility method that expects a lambda to invoke the MUT (Method-Under-Test).
Qualities
- Reliability
- Uniformity
- Changeability
- Reusability
Code Demonstration
@Test
void t1() {
stmt
actual = sutA.method()
assertEquals(expected, actual)
stmt'
}
@Test
void t2() {
stmt
actual = sutA.method()
assertEquals(expected, actual)
stmt'
}
@Test
void t3() {
actual = sutA.method()
assertEquals(expected, actual)
actual = sutB.method()
assertEquals(expected, actual)
}// Variant (1)
void setUp() {
stmt
}
void tearDown() {
stmt'
}
// Variant (2)
void act() {
actual = sutA.method()
}
// Variant (3)
void customAssert() {
assertEquals(expected, actual)
}
@Test
void t1() {
setUp()
act()
customAssert()
tearDown()
}
@Test
void t2() {
setUp()
act()
customAssert()
tearDown()
}
// Variant (4)
@Test
void TSutA() {
testMutBehavior(() -> sutA().method())
}
@Test
void TSutB() {
testMutBehavior(() -> sutB().method())
}
void testMutBehavior(Fuc<?> func) {
actual = func()
assertEquals(expected, actual)
}Instances
- Commit: Hadoop (Act)
- Commit: Druid (Assertion)
- Commit: Druid (Assertion)
- Commit: Hadoop (Assertion)
- Commit: Hadoop (Assertion)
- Commit: Log4j2 (Assertion)
- StackOverflow #23349464 (Assertion)
- StackOverflow #3796812 (Assertion)
- StackOverflow #63475079 (Assertion)
- StackOverflow #64128102 (Assertion)
- Commit: Hadoop (Fixture)
- Commit: HAPI FHIR (Fixture)
- Commit: HBase (Fixture)
- Commit: SonarQube (Fixture)
- Commit: Strimzi (Fixture)
- Commit: Qt (Fixture)
- StackOverflow #24782783 (Fixture)
- StackOverflow #26910976 (Fixture)
- StackOverflow #45316571 (Fixture)
- StackOverflow #60211054 (Fixture)
- StackOverflow #55946747 (Fixture)
- StackOverflow #22710085 (Delegate Disjoint SUTs)
Sources
Categorize Test Method
Motivation
Test suites may have test methods with common features to another, i.e., execution time, code size, targeted quality aspect, and technical debt. Developers may want to execute those groups separately and in a specific order (test prioritization).
Qualities
- Efficiency
Code Demonstration
class C {
@Test
void fastTest() { }
@Test
void largeDatasetTest() { }
}// Target Test
@Tag("Fast")
@Test
void fastTest() { }
// Test Priorization
@Suite
@SelectClasses({
FastTests.class,
LargeDatasetTests.class
})
class FastTests {
@Test
void fastTest() { }
}
@Tag("LargeDataset")
class LargeDatasetTests {
@Test
void largeDatasetTest() { }
}
// Method Order
@TestMethodOrder(OrderAnnotation.class)
class C {
@Test
@Order(1)
void fastTest() { }
@Test
@Order(3)
void largeDatasetTest() { }
}Instances
Sources
Extract Utility Class
Motivation
1. After identifying common code that can be shared between two or more test classes, utility methods (e.g., custom assertions, data creation, and transformation methods) can be extracted and collected into a General Utility Class.
2. Using named Expectations in JMockit allows you to set up custom shared mocks in a base class, streamlining repetitive setup across test classes.
3. Sharing utility methods among few, related test classes is possible through a common utility superclass to which utility methods can be moved.
Qualities
- Uniformity
- Independence
Code Demonstration
class C1 {
@Test
void t1() {
commonUtilityMethod()
}
private void commonUtilityMethod() { }
}
class C2 {
@Test
void t2() {
commonUtilityMethod()
}
private void commonUtilityMethod() { }
}class UtilsClass {
public void commonUtilityMethod() { }
}
@Test
void t1() {
commonUtilityMethod()
}
@Test
void t2() {
commonUtilityMethod()
}@Test
public void t1() {
assertThat(expression, allOf(
assert',
assert''
))
}
@Test
public void t2() {
assertThat(expression', not(allOf(
assert',
assert''
)))
}public static Matcher<String> myMatcher() {
return allOf(
assert',
assert''
)
}
@Test
public void t1() {
assertThat(expression, myMatcher())
}
@Test
public void t2() {
assertThat(expression', not(myMatcher()))
}abstract class BaseTest { }
class C1 extends BaseTest {
private void utilityMethod() { }
}abstract class BaseTest {
protected void utilityMethod() { }
}
class C1 extends BaseTest { }Instances
Sources
Split Test Method
Motivation
Test methods may evolve and accumulate multiple responsibilities, which is associated with eager test and assertion roulette test smells as well as lower cohesion. Moreover, some test frameworks (e.g., MSpec) enforce the one-Assertion per method rule. In those cases, test methods can be separated in multiple pure test methods to improve the execution trace for debugging a failure.
Qualities
- Uniformity
Code Demonstration
@Test
void t() {
assert(A)
assert(B)
}@Test
void t1() { assert(A) }
@Test
void t2() { assert(B) }Instances
Sources
Merge Test Method
Motivation
Test suites can accumulate redundancies over time, e.g., tests with redundant test statements. When such redundancy is scattered throughout multiple test methods, we can eliminate code duplication by merging them.
Qualities
- Changeability
- Efficiency
- Simplicity
Code Demonstration
@BeforeEach
void setUp() {
sut = new SUT()
}
@Test
void t1() {
actual = sut.method()
assertEquals(expected, actual)
}
@Test
void t2() {
actual' = sut.method'()
assertEquals(expected, actual')
}@Test
void t() {
sut = new SUT()
actual = sut.method()
assertEquals(expected, actual)
actual' = sut.method'()
assertEquals(expected, actual')
}Instances
Sources
Migrate Categories
Motivation
In Java, test prioritization and test filtering are achieved through test frameworks' annotations.
Although TestNG's @Test(groups=), JUnit 4's @Category, and JUnit 5's @Tags are equivalent features, they differ in both flexibility and ease of use.
Tags and groups take string parameters, whereas categories expect marker interfaces.
While we can pass many parameters to groups and categories, test methods with many tags are represented with multiple annotations.
Lastly, tags integrate with other JUnit 5 features, such as custom display name for improving the test execution report.
Qualities
- Simplicity
- Effectiveness
Code Demonstration
// JUnit 4 Category Marker
public interface FastTests {}
@Test
@Category(FastTests.class)
void t() { }
// TestNG Group
@Test(groups = "security")
void securityTest() { }// JUnit 5 Tag Migration
@Tag("fast")
@Test
void t() { }
@Tag("security")
@Test
void securityTest() { }Instances
Sources
Migrate Runner
Motivation
JUnit supports extending and modifying its core functionalities by overriding the default test runner (JUnit 4) or registering extensions (JUnit 5). Many projects using JUnit develop custom runners or leverage third-party ones. Popular third-party runners often enhance integration with dependency injection, mocking, or web frameworks. Others backport later JUnit features, such as parameterized and exception testing. Often, popular JUnit 4 runners have corresponding JUnit 5 extensions, simplifying migration. However, migrating tests with custom runners may require adapting JUnit 4’s Runner API to JUnit 5’s Extension API
Qualities
- Reliability
Code Demonstration
@RunWith(SpringJUnit4ClassRunner.class)
class SpringTest {}@ExtendWith(SpringExtension.class)
class SpringTest {}Instances
Sources
Migrate Mock
Motivation
1. Ease of use and active development are among the reasons some projects migrate their mocks to Mockito.
2. Earlier versions of mock frameworks (i.e., Mockito, EasyMock, and JMock) had limited support for static methods mocking. Powermock is a reflection-powered extension to EasyMock that optionally integrates to Mockito through its Powermockito class. Among its features, it includes support for mocking static methods.
Qualities
- Changeability
Code Demonstration
@ExtendWith(EasyMockRunner.class)
class C {
@Mock Dependency dependency
@Test
void t() {
expect(mock.method()).andReturn(expected)
replay(mock)
SUT sut = new SUT(mock)
assertEquals(expected, sut.method())
verify(mock)
}
}@ExtendWith(MockitoExtension.class)
class C {
@Mock Dependency dependency
@InjectMocks SUT sut
@Test
void t() {
when(dependency.method()).thenReturn(expected)
SUT sut = new SUT(mock)
assertEquals(expected, sut.method())
}
}@Test
void t() {
DatabaseUtils dbUtils = new DatabaseUtils()
Connection conn = dbUtils.getConnection()
assertTrue(conn.isValid(0))
}@RunWith(PowerMockRunner.class)
@PrepareForTest(DatabaseUtils.class)
public class DatabaseUtilsTest {
@Test
public void testDatabaseConnection() {
PowerMockito.mockStatic(DatabaseUtils.class)
Connection mockConn = Mockito.mock(Connection.class)
Mockito.when(mockConn.isValid(0)).thenReturn(true)
DatabaseUtils dbUtils = new DatabaseUtils()
Connection conn = dbUtils.getConnection()
assertTrue(conn.isValid(0))
PowerMockito.verifyStatic(DatabaseUtils.class)
DatabaseUtils.getConnection()
}
}Instances
Sources
Migrate Fixture
Motivation
Test fixtures are key test framework features for reusing test code. Generally, test frameworks provide two class-fixtures and two method-fixtures. While class-fixtures run once before (or after) all tests in a class, method-fixtures run once before (or after) each test. Unlike JUnit 5, TestNG (and JUnit 4 with @RunWith(Parameterized.class)) allow test fixture to access tests' parameters.
Qualities
- Reusability
Code Demonstration
@Before
void setUp() { }
@BeforeClass
static void setUpClass() { }
@After
void tearDown() { }
@AfterClass
static void tearDownClass() { }@BeforeEach
void setUp() { }
@BeforeAll
static void setUpClass() { }
@AfterEach
void tearDown() { }
@AfterAll
static void tearDownClass() { }Instances
- Commit: Pitest
- Commit: CQRS
- Commit: Cassandra
- Commit: Clerezza
- Commit: Plc4x
- Commit: Plc4x
- Commit: Storm
- Commit: Baralga
- Commit: Qpp
- Commit: Dbmaintain
- Commit: Enmasse
- Commit: Hawkular
- Commit: Baragon
- Commit: Hypergraphdb
- Commit: Java Design Patterns
- Commit: Orianna
- Commit: Rapidoid
- Commit: Rapidoid
- Commit: Thunder
- Commit: Olingo
- Commit: Spring Integration
- Commit: Spring Integration
- Commit: Orientdb
Sources
Migrate Assertion
Motivation
An adopted test library may lack some desired features available in other libraries, e.g., fluent API assertions, modular fixtures, parameterized tests, and parallel test execution.
1. Hamcrest is an external dependency of JUnit 4, which used to provide a delegate method to Hamcrest's MatcherAssert.assertThat. While JUnit 5 avoids external dependencies, its tests are compatible with third-party assertion libraries.
2. ReflectionAssert (part of Unitils) has stalled maintenance (discontinued in 2022). Projects migrate away from it to ensure long-term support.
3. AssertJ is a fluent assertions library compatible with most test frameworks. Beyond its syntax, it also ships with multiple unique, advanced assertions.
4. RSpec's expect syntax with the "change" matcher eliminates the need for multiple assertions (i.e., to validate pre- and post-conditions) by encapsulating state transitions in a single check. Therefore, it supports the principle of "one expectation per test".
5. JUnit 5 introduces subtle changes to JUnit 4's assertions set, including three new assertion types (assertThrows, assertTimeout, and assertTimeoutPreemptively), message parameter position (last parameter in JUnit 5), and support for lazily evaluated message parameters.
Qualities
- Uniformity
Code Demonstration
assertEquals(expected, actual)assertThat(expected, is(equalTo(actual)))ReflectionAssert.assertReflectionEquals(expected, actual)assertEquals(expected, actual)assertEquals(message, expected, actual)assertThat(actual).withFailMessage(message).isEqualTo(expected)assertEquals("Failure message", expected, actual)assertEquals(expected, actual, () -> "Failure message")Instances
- Commit: Neoemf (Hamcrest)
- Commit: Event (Hamcrest)
- Commit: Enmasse (Hamcrest)
- Commit: Analytics (Hamcrest)
- Commit: Nakadi (ReflectionAssert)
- Commit: Flink (AssertJ)
- Commit: Flink (AssertJ)
- Commit: Plc4x (AssertJ)
- Commit: Plc4x (AssertJ)
- Commit: Plc4x (AssertJ)
- Commit: Gedcomx (AssertJ)
- Commit: Hypergraphdb (AssertJ)
- Commit: Strata (AssertJ)
- Commit: Strata (AssertJ)
- Commit: Strata (AssertJ)
- Commit: Strata (AssertJ)
- Commit: Strata (AssertJ)
- Commit: Orientdb (AssertJ)
- Commit: Orient73011622db (AssertJ)
- StackOverflow #11493881 (RSpec)
- Commit: Storm (JUnit 4 to 5)
- Commit: Robozonky (JUnit 4 to 5)
Sources
Migrate Parameterized Test
Motivation
1. Both TestNG and JUnit 5 provide built-in support for test parameterization. However, TestNG relies exclusively on data provider methods, whereas JUnit 5 offers additional argument providers, e.g., external CSV files, inline CSV values, and null/empty values.
2. JUnit 4 includes test parameterization as an optional feature enabled through @RunWith(Parameterized.class), which restricts the use of other third-party runners, e.g., Mockito. Conversely, JUnit 5 supports parameterization by default, allows multiple argument providers within a single class and includes over ten provider types, such as MethodSource (equivalent to JUnit 4's only provider).
3. In JUnit 5, Parameterized test (and Repeated test) is the specialization of another feature, Test Template, which enables providing multiple invocation contexts to methods.
Qualities
- Simplicity
- Reusability
Code Demonstration
@Test(dataProvider = "data")
void t(P₁, ..., Pₙ) { }
@DataProvider
Object[][] data() {
return new Object[][]{{
{ P₁₁, ..., P₁ₙ },
...
{ Pₘ₁, ..., Pₘₙ }
}}
}@ParameterizedTest
@CsvSource({
P₁₁, ..., P₁ₙ,
...
Pₘ₁, ..., Pₘₙ
})
void t(P₁, ..., Pₙ) { }@RunWith(Parameterized.class)
class LegacyTest {
@Parameters
public static Collection<Object[]> data() {
return Arrays.asList(new Object[][]{{P}, ..., {Pₙ}})
}
public legacyTest(P₁, ..., Pₙ) { }
@Test
public void t() { }
}class ModernTest {
@ParameterizedTest
@MethodSource("data")
void t(ParamObj P) { }
public static StreamArguments data() {
return Stream.of(
Arguments.of(new ParamObj(P₁₁, ..., P₁ₙ)),
...
Arguments.of(new ParamObj(Pₘ₁, ..., Pₘₙ))
)
}
}Instances
Sources
Migrate Expected Exception
Motivation
Using try-catch inside a test case to catch an expected exception can lead to unpredictable behavior, e.g., assertions inside the try-block may not execute if the exception is thrown earlier than expected.
Furthermore, the optional @Test annotation attribute expected indicates an exception should be thrown anywhere inside a method body, which leads to false positives due to the lack of fine-grained control over which specific statement should trigger the exception.
Moreover, in JUnit 4.7+, test execution is interrupted as soon as an exception is thrown even if a rule is set to expect an exception.
Therefore, JUnit 5 assertions (backported to JUnit 4.13) can catch exceptions thrown by specific statements while preserving the execution of remaining statements, allowing for post-exception verification.
Qualities
- Simplicity
Code Demonstration
// Try-fail-catch Pattern
@Test
void t() {
try {
stmt
fail(M)
} catch (E) {
}
stmt'
}// JUnit4 Annotation Base
@Test(expected = E)
void t() {
stmt
stmt'
}Instances
- Commit: Fest
- Commit: Flink
- Commit: Flink
- Commit: Cassandra
- Commit: Clerezza
- Commit: Commons
- PR: Jmeter
- Commit: Plc4x
- Commit: Plc4x
- Commit: Assertj
- Commit: Eclipse
- Commit: Enmasse
- Commit: Java
- Commit: Exchange
- Commit: Neo4j
- Commit: Strata
- Commit: Rapidoid
- Commit: Thunder
- Commit: Olingo
- Commit: Problem
- Commit: Zanata
- StackOverflow #7035137
Sources
Replace Test Annotation between Class and Method
Motivation
TestNG’s @Test annotation can either be used on test method or test classes. When used in test classes, public methods become test methods (i.e., they execute as if they had @Test annotation) and non-public methods become utility methods (i.e., they must be invoked to execute). Since using class test annotation is encouraged, migrations to or from TestNG often involve this refactoring.
Qualities
- Simplicity
- Uniformity
Code Demonstration
class C {
@Test
public void t1() { }
@Test
public void t2() { }
public void utility() { }
} @Test // Public methods are tests
class C {
public void t1() { }
public void t2() { }
private void utility() { }
} Instances
Sources
Migrate Test Timeout
Motivation
Unlike TestNG and JUnit 4, JUnit 5 provides a separate annotation to set up timeout for a test case. Developers may apply such a change either to tests already undergoing a migration or to adhere with JUnit 5's multiple annotation design. Such a design is more flexible as it promotes stacking multiple annotations and it enables projects to compose their annotations into one single custom annotation.
Qualities
- Changeability
- Uniformity
Code Demonstration
@Test(timeout = 5000)
void t() {}@Timeout(value = 5)
@Test
void t() {}Instances
Sources
Replace Loop
Motivation
1. When a test repeatedly evaluates the same assertion for a fixed number of times, the code can be simplified with @RepeatedTest.
2. When testing permutation of values with nested for-each loop cases, extracting a parameterized test guarantees each permutation can be tested and reported independently.
Qualities
- Uniformity
- Changeability
Code Demonstration
@Test
void t() {
for (int i = 0; i < 5; i++) {
stmt
assertEquals(expected, actual)
}
} @RepeatedTest(5)
void t() {
stmt
assertEquals(expected, actual)
} Instances
Sources
Replace Mystery Guest with @TempDir Parameter
Motivation
Using external resources (e.g., creating temporary files or writing to existing ones) in test code is known as a symptom of the Mystery Guest test smell. The consequences are test no longer being self-contained and resource use being risky and requiring Try-Finally statements for execution consistency.
Qualities
- Independence
Code Demonstration
@Test
void t() {
stmt
File.createTempFile(params)
stmt'
}@Test
void t(@TempDir File D) {
stmt
D.createTempFile(params)
stmt'
}Instances
Sources
Solve Race Condition with Resource Lock
Motivation
Running test cases concurrently can be a challenge when the test suite relies on external resources, such as environment variables and system properties. Using specialized test framework features or OS locks to synchronize resource acquisition can ease workaround race condition.
Qualities
- Independence
Code Demonstration
@Execution(CONCURRENT)
Class C {
@Test
void t() { }
}@Execution(CONCURRENT)
Class C {
@Test
@ResourceLock(value=SYS_PROPS, mode=READ_WRITE)
void t() { }
}Sources
Inline Resource
Motivation
Tests that use or create external resources are not self-contained. To remove the dependency between a test method and some external resource, we incorporate that resource in the test code by setting up a fixture in the test code that holds the same contents as the resource.
Qualities
- Uniformity
- Changeability
- Effectiveness
Code Demonstration
@BeforeEach
void setUp() {
res = loadResource("external.txt")
value = res.read()
} @BeforeEach
void setUp() {
value = """
INLINED_DATA
"""
}Instances
No instances recorded.
Sources
Setup External Resource
Motivation
Test code that makes optimistic assumptions about the existence (or absence) and state of external resources (such as particular directories or database tables) can cause non-deterministic behaviour in test outcomes. If it is necessary for a test to rely on external resources, such as directories, databases, or files, make sure the test that uses them explicitly creates or allocates these resources before testing, and releases them when done.
Qualities
- Reliability
Code Demonstration
@Test
void t() {
actual = processResource("data.csv")
assertEquals(expected, actual)
} @BeforeEach
void setUp() {
createResource("data.csv")
}
@Test
void t() {
actual = processResource("data.csv")
assertEquals(expected, actual)
}
@AfterEach
void tearDown() {
deleteResource("data.csv")
} Instances
No instances recorded.
Sources
Make Resource Unique
Motivation
Problems originate from the use of overlapping resource names (Deursen et al., 2001), either between different tests run done by the same user or between simultaneous test runs done by different users. For example, tests writing to a shared file may overwrite each other’s data or crash due to concurrent access. Such problems can easily be prevented (or repaired) by using unique identifiers for all resources that are allocated, for example by including a timestamp.
Qualities
- Reliability
Code Demonstration
@Test
void t() {
r = createResource("shared-name.txt")
} class C {
private String uniqueResourceName
@BeforeEach
void setUp() {
uniqueResourceName = "resource-" + System.currentTimeMillis()
createResource(uniqueResourceName)
}
@Test
void t() {
actual = loadResource(uniqueResourceName)
}
@AfterEach
void tearDown() {
deleteResource(uniqueResourceName)
}
} Instances
No instances recorded.
Sources
Introduce Equality Method
Motivation
Test Methods with multiple assertions are hard to debug and may have its execution interrupted prematurely. When verifying an object’s state, one should compare it with an expected object rather than their properties separately. Otherwise, the number of assertions explodes impacting the readability.
Qualities
- Changeability
- Uniformity
- Brittleness
- Simplicity
Code Demonstration
@Test
void t() {
actual = method()
assertEquals(expected1, actual.getProp1())
assertEquals(expected2, actual.getProp2())
assertEquals(expected3, actual.getProp3())
} class Result {
@Override
bool equals() { }
}
class C {
@Test
void t() {
actual = sut.act()
assertEquals(expected, actual)
}
}Instances
Sources
Mock Time Input
Motivation
Testing software that relies on the clock time to operate is challenging, e.g., scheduled events, timer, and calendars. When testing, it is a good practice to mock the time to speed-up tests and avoid synchronization problems.
Qualities
- Efficiency
- Effectiveness
Code Demonstration
@Test
void t() {
Cache cache = new Cache(expirationTime: 5, TimeUnit.SECONDS)
cache.put("key", "value")
Thread.sleep(5000)
assertNull(cache.get("key"))
} @Test
void t() {
Clock mockClock = Clock.fixed(Instant.now(), ZoneId.systemDefault())
Cache cache = new Cache(expirationTime: 5, TimeUnit.SECONDS, mockClock)
cache.put("key", "value")
Instant expiredTime = Instant.now(mockClock).plusSeconds(5)
mockClock = Clock.fixed(expiredTime, ZoneId.systemDefault())
cache.setClock(mockClock)
assertNull(cache.get("key"))
} Instances
Sources
Replace Mocks with Real Instances
Motivation
Mocks are not real objects, thus they might behave differently to the objects they try to simulate and prevent tests from catching some bug types, i.e., bugs in the integration of different components. And the simpler a dependency is, fewer are the benefits one would get from mocking it. Besides, too much mocking can increase maintenance effort as reflecting production changes on mocks become more frequent. Replacing mock with real instance can be beneficial if the latter is performs satisfactorily.
Qualities
- Uniformity
Code Demonstration
@Test
void t() {
Dependency mockDep = mock(Dependency.class)
when(mockDep.method())
.thenReturn(expected)
SUT sut = new SUT(mockDep)
sut.process()
verify(mockDep).method()
} @Test
void t() {
Dependency realDep = new LightweightDependency()
realDep.configure(expected)
SUT sut = new SUT(realDep)
sut.process()
assertEquals(expected, sut.getResult())
} Instances
Sources
Replace Real Instances with Mocks
Motivation
Compared to real objects, mocks offer four advantages (Aniche, 2022). Mocks are efficient, thus are good replacements for slow dependencies. Mocks are easy to write, which makes them appropriate solutions for objects of a class that can’t be created easily. Mocks are flexible, which enables simulating complex real world scenarios efficiently. Mocks isolate components, which can be particularly useful to prevent undesired access to database and external APIs.
Qualities
- Independence
- Efficiency
- Reliability
Code Demonstration
@Test
void t() {
CloudDatabase db = new CloudDatabase(config)
db.method()
SUT sut = new SUT(db)
Result result = sut.process()
assertEquals(expected, result)
} @Test
void t() {
CloudDatabase mockDep = mock(CloudDatabase.class)
when(mockDep.method())
.thenReturn(expected)
SUT sut = new SUT(mockDep)
sut.process()
verify(mockDep).method()
} Instances
Sources
Reuse Mock
Motivation
When specific features are heavily depended on code bases, it is common that many unit tests require mocking it. One may extract a Fake or Stub class that is used across many tests in the project.
Qualities
- Reusability
- Changeability
Code Demonstration
@Test
void t() {
Dependency dep = mock(Dependency.class)
when(dep.method(args)).thenReturn(value)
stmt
verify(dep).method(args)
stmt'
}@Test
void t() {
Dependency dep = new FakeC(args, value)
stmt
verify(dep).method(args)
stmt'
}
class FakeDependency extends Dependency {
FakeDependency(args, value) {
when(method(args)).thenReturn(value)
}
}Sources
Replace Inheritance by Mocking API
Motivation
Inheritance requires the developer to manually craft additional attributes/features in the subclass for tracking the execution of the mock objects. The logic behind the attributes is implicit, and may blur the testing logic.
Qualities
- Simplicity
Code Demonstration
@Test
class T extends MockBaseClass {
Dependency dep
ClassUnderTest obj
@Override
void setUp() {
super.setUp()
dep = createDependencyMock()
obj = new ClassUnderTest(dep)
}
void t() {
stubDependencyBehavior(dep)
obj.performAction()
assertCustomTracking(dep)
}
}@Test
class T {
@Mock
Dependency dep
@InjectMocks
ClassUnderTest obj
@Before
void setUp() {
MockitoAnnotations.initMocks(this)
}
void t() {
when(dep.specificMethod()).thenReturn(mockValue)
obj.performAction()
verify(dep).specificMethod()
}
}Instances
No instances recorded.
Sources
Replace Mocking API with Anonymous Subclass
Motivation
Java's anonymous classes are commonly used for creating stubs. Wrapping them with mocking framework's Spies can be redundant, thus it might be worth dropping the mocking API and replace it with purely manual implementation of its logic. Like so developers may keep stubs' customizability while simplifying the test code and potentially dropping the dependency on mocking framework.
Qualities
- Reusability
Code Demonstration
@Test
void t() {
Dependency obj = MockingFramework.spy(new Dependency() {
@Override
ReturnType method(ParamType args) {
methodCalled = true
return value
}
})
MockingFramework.when(obj.method(args)).thenReturn(value)
stmt
MockingFramework.verify(obj).method(args)
stmt'
}@Test
void t() {
Dependency obj = new Dependency() {
public boolean methodCalled = false
@Override
ReturnType method(ParamType args) {
methodCalled = true
return value
}
}
stmt
assertTrue(obj.methodCalled)
stmt'
}Instances
Sources
Replace Test Double Type
Motivation
There are five different test doubles (i.e., dummy, stub, fake, spy, and mock). Dummy is the simplest instance to fill parameter lists. Stub is a test-only subclass overriding methods for behaviour customization. Fakes are working simplified implementations. Spies and Mocks provide mechanisms for behaviour verification.
Qualities
- Efficiency
- Simplicity
Code Demonstration
@Test
void t() {
Dependency stub = new Dependency() {
@Override
Result method() { return expected }
}
SUT sut = new SUT(stub)
assertEquals(expected, sut.process())
} @Test
void t() {
when(mockDep.method())
.thenReturn(expected)
SUT sut = new SUT(mockDep)
sut.process()
verify(mockDep).method()
}SUT sut = new SUT(null)class FakeDependency implements Dependency {
void method() { }
}
SUT sut = new SUT(new FakeDependency()) class SpyDependency extends RealDependency {
boolean called = false
@Override
void method() {
called = true
super.method()
}
} @Spy RealDependency spyDep
@Test
void t() {
verify(spyDep).method()
}Instances
Sources
Replace Test Double with ObjectMother
Motivation
When we have many tests using similar but not identical mocks, we need to recreate the mock for each test method. Alternatively, we can create an Object Mother to simplify tests and centralize complex object construction. Object Mother is a Test Utility Class whose methods provide specific versions of complex type instances required by multiple tests.
Qualities
- Uniformity
- Efficiency
Code Demonstration
@Test
void t1() {
Dependency mockDep = mock(Dependency.class)
when(mockDep.method())
.thenReturn(expected)
SUT sut = new SUT(mockDep)
assertEquals(expected, sut.process())
}
@Test
void t2() {
Dependency mockDep = mock(Dependency.class)
when(mockDep.method())
.thenReturn(expected')
SUT sut = new SUT(mockDep)
assertEquals(expected', sut.analyze())
} class DependencyMother {
static Dependency createConfiguredDependency() {
Dependency dep = new Dependency()
return dep
}
}
@Test
void t1() {
SUT sut = new SUT(DependencyMother.createConfiguredDependency())
assertEquals(expected, sut.process())
}
@Test
void t2() {
SUT sut = new SUT(DependencyMother.createConfiguredDependency())
assertEquals(expected', sut.analyze())
} Instances
Sources
Standardize Test Utilities for Project-Wide Reuse
Motivation
Test utility classes often become rigid or isolated. Standardizing them promotes reusability, efficiency, and consistency across the entire project or organization.
1. Test utility classes may have methods with strict API, which hinders reuse for new test cases. Developers may generalize utility methods' return type and parameters set by prioritizing interfaces or super classes over concrete classes.
2. Some large communities and organizations take one step further creating an in-house Test Framework to share common test logic among many of their projects.
3. By organizing repetitive, shared setup and teardown logic in a global scope, developers can avoid duplicating test fixtures across multiple files.
4. Reusing cached Spring contexts across tests ensures that contexts with identical configurations are loaded only once, thus avoiding the repeated initialization of the application context.
5. To simplify test data setup for complex scenarios while maintaining test isolation, real database interactions can be recorded during an initial execution and subsequently replayed in later test runs.
Qualities
- Reusability
- Efficiency
Code Demonstration
class UtilsClass {
static boolean helper(ArrayList<?> p) { }
}
class C {
void t() {
var sut = new SUT()
var collection = sut.getSet()
var list = convertToList(collection)
assertTrue(UtilsClass.helper(list))
}
}class UtilsClass {
static boolean helper(Collection<?> p) { }
}
class C {
void t() {
var sut = new SUT()
var collection = sut.getSet()
assertTrue(UtilsClass.helper(collection))
}
}class ProjectUtilsClass {
static void validateResult(Result r)
{ }
}
class C {
void t() {
ProjectUtilsClass.validateResult(r)
}
}package com.inhouse.framework
class TestAssertions {
static void validateResult(Result r) { }
}
class C {
void t() {
com.inhouse.framework.TestAssertions.validateResult(r)
}
}class C {
@BeforeEach
void setUp() {
stmt
}
}
class C2 {
@BeforeEach
void setUp() {
stmt
}
}class GlobalTestFixture {
@BeforeAll
static void globalSetup() { }
}
@ExtendWith(GlobalTestFixture.class)
class C { }
@ExtendWith(GlobalTestFixture.class)
class C1 { }@ContextConfiguration(classes="classpath:1.xml")
class C1 { }
@ContextConfiguration(classes="classpath:2.xml")
class C2 { }@ContextConfiguration(classes="classpath:C.xml")
class C1 { }
@ContextConfiguration(classes="classpath:C.xml")
class C2 { }Result r = database.query("SELECT ...")
assertEquals(expected, r.hasData())Result r = DatabaseReplayer.replay("SELECT ...")
assertEquals(expected, r.hasData())Instances
- Commit: Camunda (Generalize)
- Commit: Camunda (Generalize)
- Commit: CXF (Generalize)
- Commit: Flink (Generalize)
- Commit: Flink (Generalize)
- Commit: Elasticsearch (in-house Test Framework)
- Commit: OrientDB (in-house Test Framework)
- StackOverflow #24782783 (in-house Test Framework)
- StackOverflow #64128102 (in-house Test Framework)
- StackOverflow #17317839 (Reuse Fixtures)
- StackOverflow #74425543 (Reuse Fixtures)
- StackOverflow #23220524 (Cached Context)
- StackOverflow #55614606 (Cached Context)
- StackOverflow #8501975 (Cached Context)
- StackOverflow #8412853 (Cached DB Query)
Sources
Custom Runner
Motivation
Unlike JUnit 5's extension model, only one JUnit 4 Test Runner can be applied to a Test Class at once. There are two strategies one can adopt to further customize the Test Framework behaviour for a Test Class that already relies on special Test Runners: using JUnit 4 rules or extracting a subclass of the special Test Runner.
Qualities
- Reusability
- Changeability
Code Demonstration
@RunWith(SpringJUnit4ClassRunner.class)
@ContextConfiguration(classes = {
DatabaseConfig.class
})
public class C {
@Test
public void t() { }
} @RunWith(CustomIntegrationTestRunner.class)
public class C {
@Test
public void t() { }
}
public class CustomIntegrationTestRunner extends SpringJUnit4ClassRunner {
public CustomIntegrationTestRunner(Class<?> clazz)
throws InitializationError {
super(clazz)
}
@Override
protected TestContextManager createTestContextManager(Class<?> clazz) {
return new TestContextManager(clazz) {
@Override
public void prepareTestInstance(Object testInstance) {
registerConfigurationClasses(DatabaseConfig.class)
super.prepareTestInstance(testInstance)
}
}
}
} Instances
Sources
Custom Annotation
Motivation
Custom annotations are often employed to temporarily tag test method for third-party library integration (e.g., Architectural test), to customize both test and production code with the same JUnit-like, annotation-based "extension model," or to attach metadata to tests (e.g., issue fixed and method-under-test).
Qualities
- Uniformity
Code Demonstration
@Test
@Timeout(value = 5, unit = TimeUnit.SECONDS)
@Tag("integration")
void t() {
stmt
}@Retention(RetentionPolicy.RUNTIME)
@Target(ElementType.METHOD)
@Test
@Timeout(value = 5, unit = TimeUnit.SECONDS)
@Tag("integration")
public @interface IntegrationTest {}
@IntegrationTest
void t() {
stmt
}Sources
Restructure Test Suite
Motivation
1. Nested test classes are best used when tests need to be organized into logical, hierarchical groups.
2. Test methods are moved to other classes to reflect changes in the production code or as part of a systematic test suite reorganization.
3. When nested classes evolves beyond the scope of their outer class, one may denest the outer class by moving its nested class to their own separate files.
Qualities
- Uniformity
Code Demonstration
class FlatTest {
@BeforeEach
void setUp() {
setUp
}
@Test
void T1() {
setUp'
assert(E1)
}
@Test
void T2() {
setUp''
assert(E2)
}
} class OuterTest {
@BeforeEach
void setUp() {
setUp
}
@Nested
class C1 {
@BeforeEach
void setUp'() { }
@Test
void T1() {
assertEquals(expected, actual)
}
}
@Nested
class C2 {
@BeforeEach
void setUp''() { }
@Test
void T2() {
assertEquals(expected, actual'')
}
}
} class C1 {
@Test
void t() {
assertEquals(expected, ClassB.method())
}
}
class C2 { }class C1 { }
class C2 {
@Test
void t() {
assertEquals(expected, ClassB.method())
}
} class OuterTest {
@Nested
class C { }
} class OuterTest { }
class C { } Instances
Sources
Extend Framework
Motivation
1. JUnit 5 supports an extension model and JUnit 4 supports rule-based extension. Both features prioritize composition over inheritance.
2. ArchUnit allows teams to define architectural rules in Java, creating a shared understanding of code structure that adapts as projects evolve.
Qualities
- Uniformity
- Reusability
- Changeability
Code Demonstration
@Test
void t() {
Resource res = acquireResource()
try {
actual = res.method()
assertEquals(expected, actual)
} finally {
releaseResource(res)
}
} @ExtendWith(ResourceExtension.class)
@Test
void t(Resource res) {
actual = res.method()
assertEquals(expected, actual)
}
class ResourceExtension implements BeforeEachCallback, AfterEachCallback {
public void beforeEach(ExtensionContext ctx) {
ctx.getStore().put("resource", acquireResource())
}
public void afterEach(ExtensionContext ctx) {
releaseResource(ctx.getStore().get("resource", Resource.class))
}
} @Test
void t() {
var violatingFiles = Files.walk(Paths.get("src"))
.filter(p -> p.endsWith(".java"))
.flatMap(p -> Files.readAllLines(p))
.filter(s -> s.contains("package" + basePackage) && s.contains("import" + forbiddenImport))
.collect(toList())
assertFalse(violatingFiles.isEmpty())
}@ArchTest
void t() {
noClasses()
.that().resideOutsideOfPackage(basePackage)
.should().accessClassesThat()
.resideInAnyPackage(forbiddenImport)
}