When i run the below code, both test cases come true:
import static junit.framework.Assert.assertEquals;
import org.junit.Test;
public class MyTest{
private int count;
@Before
public void before(){
count=1;
}
@Test
public void test1(){
count++;
assertEquals(2, count);
}
@Test
public void test2(){
count++;
assertEquals(2, count);
}
}
EXPECTED BEHAVIOUR
ACTUAL BEHAVIOUR
Why junit is reinitializing class/variable
with each test method invocation.
It is a bug in junit or is provided intentionally.
JUnit creates a new instance of the test class before invoking each @Test method. This helps provide independence between test methods and avoids unintentional side effects in the test code. Because each test method runs on a new test class instance, we can't reuse instance variable values across test methods.
Create Test fixture before each test. @Before - method that is run before every test case. setUp( ) is the traditional name. @After - method that is run after every test case.
If a JUnit test method is declared as "private", it compiles successfully. But the execution will fail. This is because JUnit requires that all test methods must be declared as "public".
(i) onStart(): This method is invoked after the test class is instantiated and before any configuration method is called.
It is because of test isolation.
No test should depend on another.
MyTest
for each test method
For each test method a new instance of MyTest
will be created this is the behavior of Junit.
So in your case for both methods the variable count
will have value 1
, and thus the value of count++
will be 2
for both the test methods and hence the test cases pass.
public class MyTest{
public MyTest(){
// called n times
System.out.println("Constructor called for MyTest");
}
@Before //called n times
public void setUp(){
System.out.println("Before called for MyTest");
}
//n test methods
}
If you execute the code above with 2 test methods:
Output will be:
Constructor called for MyTest
Before called for MyTest
//test execution
Constructor called for MyTest
Before called for MyTest
Edit:
Test frameworks help you in doing the right thing, a very important property of unit tests is isolation.
By creating a new instance every test method, the dirty SUT is thrown away. So that we have a fresh state for every test.
Read about F.I.R.S.T principle of testing.
Look at the documentation of org.junit.runner.Runner
:
The default runner implementation guarantees that the instances of the test case class will be constructed immediately before running the test and that the runner will retain no reference to the test case instances, generally making them available for garbage collection.
Unit tests should be independant otherwise it becomes unmaintable. Note that the order of executed methods is not guaranteed (unless you use the annotation @FixMethodOrder
).
In JUnit5, this behavior is controlled using the @TestInstance
annotation. The annotation can take as value one of two lifecycle qualifiers:
@TestInstance(Lifecycle.PER_CLASS)
: The test class will be initialized once for all the methods in the class.@TestInstance(Lifecycle.PER_METHOD)
: The test class will be reinitialized before each test method (the behavior described in other answers).If a test class is not annotated with @TestInstance
, the default behavior is PER_METHOD
.
For more details, see Test Instance Lifecycle in the JUnit5 User Guide.
If you want to use test class's member variable for all tests , without it being reinitialised to null, then make it static.
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With