From 9f6b209e2e9463fb5163dd973ebc91563c7df58a Mon Sep 17 00:00:00 2001
From: Ronald Holshausen <ronald.holshausen@smartbear.com>
Date: Thu, 14 Nov 2024 15:58:48 +1100
Subject: [PATCH] chore: Add a test + update docs on JUnit 4 report dir default
 #1836

---
 provider/junit/README.md                                     | 3 ++-
 .../dius/pact/provider/junit/target/HttpTargetSpec.groovy    | 5 ++++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/provider/junit/README.md b/provider/junit/README.md
index 43e4f2cc5c..6b314e096a 100644
--- a/provider/junit/README.md
+++ b/provider/junit/README.md
@@ -616,7 +616,8 @@ JUnit mechanism. Additional reports can be generated from the tests.
 ## Enabling additional reports via annotations on the test classes
 
 A `@VerificationReports` annotation can be added to any pact test class which will control the verification output. The
-annotation takes a list report types and an optional report directory (defaults to "target/pact/reports").
+annotation takes a list report types and an optional report directory (defaults to "target/pact/reports" for Maven
+builds and "build/pact/reports" with Gradle).
 The currently supported report types are `console`, `markdown` and `json`.
 
 For example:
diff --git a/provider/junit/src/test/groovy/au/com/dius/pact/provider/junit/target/HttpTargetSpec.groovy b/provider/junit/src/test/groovy/au/com/dius/pact/provider/junit/target/HttpTargetSpec.groovy
index 9fe5eb2153..0b5fa94fe7 100644
--- a/provider/junit/src/test/groovy/au/com/dius/pact/provider/junit/target/HttpTargetSpec.groovy
+++ b/provider/junit/src/test/groovy/au/com/dius/pact/provider/junit/target/HttpTargetSpec.groovy
@@ -44,7 +44,10 @@ class HttpTargetSpec extends Specification {
     httpTarget.setupReporters(verifier)
 
     then:
-    1 * verifier.setReporters { r ->  r*.class*.simpleName == ['AnsiConsoleReporter', 'MarkdownReporter'] }
+    1 * verifier.setReporters { r ->
+      r*.class*.simpleName == ['AnsiConsoleReporter', 'MarkdownReporter']
+      r*.reportDir == ['build/pacts/reports' as File, 'build/pacts/reports' as File]
+    }
   }
 
   @SuppressWarnings('ClosureStatementOnOpeningLineOfMultipleLineClosure')