Removed some TODOs that actually aren't a great idea..

sdcondon · May 6, 2024 · fb63de1 · fb63de1
1 parent 8c676e9
commit fb63de1
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 4 deletions.
diff --git a/src/FlUnit/TestActionOutcome.cs b/src/FlUnit/TestActionOutcome.cs
@@ -5,8 +5,6 @@ namespace FlUnit
     /// <summary>
     /// Container for the outcome of the invoking the "When" clause of a test.
     /// </summary>
-    // TODO-BREAKING-V2: is this still needed now that When clauses are async by default (so have a ValueTask/Task)?
-    // ThrowIfEx/NoEx become static methods on TestFailureException? Changes the args of Then(..) - breaking change.
     public sealed class TestActionOutcome
     {
         internal TestActionOutcome() { }

diff --git a/src/FlUnit/TestFunctionOutcome.cs b/src/FlUnit/TestFunctionOutcome.cs
@@ -6,8 +6,6 @@ namespace FlUnit
     /// Container for the outcome of the invoking the "When" clause of a test.
     /// </summary>
     /// <typeparam name="T">The return type of test function.</typeparam>
-    // TODO-BREAKING-V2: is this still needed now that When clauses are async by default (so have a ValueTask/Task)?
-    // ThrowIfEx/NoEx become static methods on TestFailureException? Changes the args of Then(..) - breaking change.
     public sealed class TestFunctionOutcome<T>
     {
         private readonly T result;