Skip to content

Commit 9cc8bcd

Browse files
committed
. t Add logging to test
1 parent f24b268 commit 9cc8bcd

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

approvaltests/src/main/java/org/approvaltests/integrations/junit5/JupiterApprovals.java

+8-2
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,11 @@
11
package org.approvaltests.integrations.junit5;
22

3+
import com.spun.util.logger.SimpleLogger;
34
import org.approvaltests.SafetyCheckBeforeVerify;
45
import org.approvaltests.Approvals;
56
import org.approvaltests.core.Experimental;
67
import org.approvaltests.core.Options;
8+
import org.junit.jupiter.api.DisplayNameGenerator;
79
import org.junit.jupiter.api.DynamicTest;
810
import org.lambda.actions.Action1;
911

@@ -20,9 +22,13 @@ public static DynamicTest dynamicTest(String displayName, Action1<Options> actio
2022
}
2123
private static void checkOptionsWasUsed(Options actual, Options expected)
2224
{
23-
if (!actual.forFile().getNamer().getAdditionalInformation()
24-
.startsWith(expected.forFile().getNamer().getAdditionalInformation()))
25+
String actualAdditionalInformation = actual.forFile().getNamer().getAdditionalInformation();
26+
String expectedAdditionalInformation = expected.forFile().getNamer().getAdditionalInformation();
27+
if (!actualAdditionalInformation
28+
.startsWith(expectedAdditionalInformation))
2529
{
30+
SimpleLogger.variable("Actual additional information", actualAdditionalInformation);
31+
SimpleLogger.variable("Expected additional information", expectedAdditionalInformation);
2632
String helpMessage = "When using dynamic tests and Approvals, all calls to verify() must use the original Options or a derivative: \n"
2733
+ " wrong: o -> Approvals.verify(result); \n" + " right: o -> Approvals.verify(result, o); \n"
2834
+ " More at: https://github.com/approvals/ApprovalTests.Java/blob/master/approvaltests/docs/how_to/UseTestFactory.md";

0 commit comments

Comments
 (0)