Tibor17 edited a comment on issue #234: (doc) add static modifier to signatures
URL: https://github.com/apache/maven-surefire/pull/234#issuecomment-496024381
 
 
   @pzygielo 
   You are wrong. The `@Before` must not be static. Every test method must be 
isolated due to a new object creation every time the test method is called. The 
data across test instances must not be shared. That's important detail in unit 
tests. That's why `setUp` and `tearDown` methods are not static. The same works 
in JUnit4 and the same works in POJO. IMO our implementation shares data which 
is a bug but another bug. The TestNG breaks this paradighm and there is one 
instance. So if you like sharing data and have random behavior then I would 
recommend using TestNG.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to