BREAKING: junit: Set default lifecycle to PER_TEST

This is a temporary reversal of the breaking change in 1ca007d until we support `TestInstancePostProcessor`s.
CodeIntelligenceTesting · Sep 13, 2023 · 7781986 · 7781986
1 parent 4540c3c
commit 7781986
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/examples/junit/src/test/java/com/example/PerExecutionLifecycleFuzzTest.java b/examples/junit/src/test/java/com/example/PerExecutionLifecycleFuzzTest.java
@@ -21,6 +21,7 @@
 import static java.util.Collections.unmodifiableList;
 
 import com.code_intelligence.jazzer.junit.FuzzTest;
+import com.code_intelligence.jazzer.junit.Lifecycle;
 import com.example.PerExecutionLifecycleFuzzTest.LifecycleCallbacks1;
 import com.example.PerExecutionLifecycleFuzzTest.LifecycleCallbacks2;
 import com.example.PerExecutionLifecycleFuzzTest.LifecycleCallbacks3;
@@ -79,7 +80,7 @@ void disabledFuzz(byte[] data) {
     throw new AssertionError("This test should not be executed");
   }
 
-  @FuzzTest(maxExecutions = RUNS)
+  @FuzzTest(maxExecutions = RUNS, lifecycle = Lifecycle.PER_EXECUTION)
   void lifecycleFuzz(byte[] data) {
     events.add("lifecycleFuzz");
     assertThat(beforeEachCalledOnInstance).isTrue();

diff --git a/examples/junit/src/test/java/com/example/PerTestLifecycleFuzzTest.java b/examples/junit/src/test/java/com/example/PerTestLifecycleFuzzTest.java
@@ -80,7 +80,7 @@ void disabledFuzz(byte[] data) {
     throw new AssertionError("This test should not be executed");
   }
 
-  @FuzzTest(maxExecutions = RUNS, lifecycle = Lifecycle.PER_TEST)
+  @FuzzTest(maxExecutions = RUNS)
   void lifecycleFuzz(byte[] data) {
     events.add("lifecycleFuzz");
     assertThat(beforeEachCalledOnInstance).isTrue();

diff --git a/src/main/java/com/code_intelligence/jazzer/junit/FuzzTest.java b/src/main/java/com/code_intelligence/jazzer/junit/FuzzTest.java
@@ -131,7 +131,7 @@
    * <p>During regression testing, fuzz tests always go through the full JUnit lifecycle for every
    * execution regardless of the value of this option.
    */
-  Lifecycle lifecycle() default Lifecycle.PER_EXECUTION;
+  Lifecycle lifecycle() default Lifecycle.PER_TEST;
 }
 
 // Internal use only.