fix: JUNIT 5 - Successful test result was being published after state change method failed #1058

Author: Ronald Holshausen

provider/pact-jvm-provider-junit5/src/main/kotlin/au/com/dius/pact/provider/junit5/PactJUnit5VerificationProvider.kt

@@ -63,7 +63,7 @@ data class PactVerificationContext @JvmOverloads constructor(
   var providerInfo: ProviderInfo = ProviderInfo(),
   val consumerName: String,
   val interaction: Interaction,
-  internal var testExecutionResult: TestResult = TestResult.Ok
+  var testExecutionResult: TestResult = TestResult.Ok
 ) {
   val stateChangeHandlers: MutableList<Any> = mutableListOf()
   var executionContext: Map<String, Any>? = null
@@ -291,8 +291,9 @@ class PactVerificationStateChangeExtension(
       testContext.executionContext = mapOf("providerState" to providerStateContext)
     } catch (e: Exception) {
       logger.error(e) { "Provider state change callback failed" }
-      testResultAccumulator.updateTestResult(pact, interaction, TestResult.Failed(description = "Provider state change callback failed",
-        results = listOf(mapOf("exception" to e))))
+      testContext.testExecutionResult = TestResult.Failed(description = "Provider state change callback failed",
+        results = listOf(mapOf("exception" to e)))
+      throw AssertionError("Provider state change callback failed", e)
     }
   }
 

provider/pact-jvm-provider-junit5/src/test/groovy/au/com/dius/pact/provider/junit5/PactVerificationStateChangeExtensionSpec.groovy

@@ -113,6 +113,7 @@ class PactVerificationStateChangeExtensionSpec extends Specification {
     def context = Mock(ExtensionContext) {
       getStore(_) >> store
       getRequiredTestClass() >> TestClass
+      getRequiredTestInstance() >> testInstance
     }
     def target = Mock(TestTarget)
     IProviderVerifier verifier = Mock()
@@ -128,7 +129,8 @@ class PactVerificationStateChangeExtensionSpec extends Specification {
     verificationExtension.beforeTestExecution(context)
 
     then:
-    1 * testResultAcc.updateTestResult(_, interaction, { it instanceof TestResult.Failed })
+    thrown(AssertionError)
+    verificationContext.testExecutionResult instanceof TestResult.Failed
   }
 
 }