Fix the tests -- is it worth merging this?

SteveGilham · Dec 1, 2021 · 98c61ec · 98c61ec
1 parent 2dc4d98
commit 98c61ec
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/AltCover.Tests/Expecto.fs b/AltCover.Tests/Expecto.fs
@@ -270,6 +270,7 @@ module ExpectoTestManifest =
        Tests.AltCoverRunnerTests.ThresholdViolationShouldBeReported,
        "Runner.ThresholdViolationShouldBeReported"
        Tests.AltCoverRunnerTests.TryGetValueHandlesNull, "Runner.TryGetValueHandlesNull"
+       Tests.AltCoverRunnerTests.CanExtractStringArray, "Runner.CanExtractStringArray"
        Tests.AltCoverTests.ReportFileShouldBeCorrectlySuffixed,
        "Tests.ReportFileShouldBeCorrectlySuffixed"
        Tests.AltCoverTests.CanSwitchSampling, "Tests.CanSwitchSampling"

diff --git a/AltCover.Tests/Runner.Tests.fs b/AltCover.Tests/Runner.Tests.fs
@@ -6587,7 +6587,11 @@ module AltCoverRunnerTests =
       assembly.MainModule.GetType("Tests.AltCoverRunnerTests")
 
     let m = Runner.J.getMethod instance "someData"
-    let data = Runner.J.getStrings m
+
+    let data =
+      Runner.J.getStrings m
+      // allow for instrumentation injection
+      |> Seq.filter (fun x -> someData () |> Seq.exists (fun s -> s = x))
 
     someData ()
     |> Seq.zip data