fix Fields beforeVerify and afterVerify Not Properly Handled in Verif…

…ySettings (#1373)
VerifyTests · Dec 19, 2024 · 946a5c6 · 946a5c6
1 parent 7e1deb1
commit 946a5c6
Show file tree

Hide file tree

Showing 5 changed files with 22 additions and 3 deletions.
diff --git a/docs/verify-options.md b/docs/verify-options.md
@@ -184,7 +184,7 @@ public Task OnCallbacks()
     return Verify("value", settings);
 }
 ```
-<sup><a href='/src/Verify.Tests/Tests.cs#L122-L150' title='Snippet source file'>snippet source</a> | <a href='#snippet-OnInstanceHandlers' title='Start of snippet'>anchor</a></sup>
+<sup><a href='/src/Verify.Tests/Tests.cs#L137-L165' title='Snippet source file'>snippet source</a> | <a href='#snippet-OnInstanceHandlers' title='Start of snippet'>anchor</a></sup>
 <!-- endSnippet -->
 
 
@@ -215,7 +215,7 @@ public Task OnFluentCallbacks() =>
                 return Task.CompletedTask;
             });
 ```
-<sup><a href='/src/Verify.Tests/Tests.cs#L152-L176' title='Snippet source file'>snippet source</a> | <a href='#snippet-OnFluentHandlers' title='Start of snippet'>anchor</a></sup>
+<sup><a href='/src/Verify.Tests/Tests.cs#L167-L191' title='Snippet source file'>snippet source</a> | <a href='#snippet-OnFluentHandlers' title='Start of snippet'>anchor</a></sup>
 <!-- endSnippet -->
 
 

diff --git a/readme.md b/readme.md
@@ -800,7 +800,7 @@ var result = await Verify(
     });
 Assert.Contains("Value To Check", result.Text);
 ```
-<sup><a href='/src/Verify.Tests/Tests.cs#L414-L423' title='Snippet source file'>snippet source</a> | <a href='#snippet-VerifyResult' title='Start of snippet'>anchor</a></sup>
+<sup><a href='/src/Verify.Tests/Tests.cs#L429-L438' title='Snippet source file'>snippet source</a> | <a href='#snippet-VerifyResult' title='Start of snippet'>anchor</a></sup>
 <!-- endSnippet -->
 
 If using `Verifier.Throws`, the resulting `Exception` will also be accessible

diff --git a/src/Verify.Tests/Tests.OnCallbacksTest.verified.txt b/src/Verify.Tests/Tests.OnCallbacksTest.verified.txt
@@ -0,0 +1 @@
+value
diff --git a/src/Verify.Tests/Tests.cs b/src/Verify.Tests/Tests.cs
@@ -119,6 +119,21 @@ public async Task OnVerifyMismatch()
         Assert.True(onVerifyMismatchCalled2);
     }
 
+    [Fact]
+    public async Task OnCallbacksTest()
+    {
+        var onVerifyBeforeCalled = false;
+        var onVerifyAfterCalled = false;
+        var settings = new VerifySettings();
+        settings.OnVerify(
+            before: () => onVerifyBeforeCalled = true,
+            after: () => onVerifyAfterCalled = true);
+
+        await Verify("value", settings);
+        Assert.True(onVerifyBeforeCalled);
+        Assert.True(onVerifyAfterCalled);
+    }
+
     #region OnInstanceHandlers
 
     [Fact]

diff --git a/src/Verify/VerifySettings.cs b/src/Verify/VerifySettings.cs
@@ -42,6 +42,9 @@ public VerifySettings(VerifySettings? settings)
         hashParameters = settings.hashParameters;
         parametersText = settings.parametersText;
         fileName = settings.fileName;
+        handleOnFirstVerify = settings.handleOnFirstVerify;
+        handleOnVerifyDelete = settings.handleOnVerifyDelete;
+        handleOnVerifyMismatch = settings.handleOnVerifyMismatch;
         UniquePrefixDisabled = settings.UniquePrefixDisabled;
         UseUniqueDirectorySplitMode = settings.UseUniqueDirectorySplitMode;
         Namer = new(settings.Namer);