Removed conversion of IDomainResult to ActionResult<T> (#41)

src/Mvc/DomainResultTo200OkResult.cs

@@ -21,11 +21,11 @@ public static partial class DomainResultExtensions
 		/// <param name="errorAction"> Optional processing in case of an error </param>
 		public static IActionResult ToActionResult<T>(this IDomainResult<T> domainResult,
 													 Action<ProblemDetails, IDomainResult<T>>? errorAction = null)
-			=> ToActionResult(domainResult.Value, domainResult, errorAction, (value) => new OkObjectResult(value));
+			=> ToActionResult(domainResult.Value, domainResult, errorAction, value => new OkObjectResult(value));
 
 		/// <summary>
 		///		Returns HTTP code 200 (OK) with a value or a 4xx code in case of an error.
-		///		The reesult is wrapped in a <see cref="Task{V}"/>
+		///		The result is wrapped in a <see cref="Task{V}"/>
 		/// </summary>
 		/// <typeparam name="T"> The returned value type from the domain operation in <paramref name="domainResultTask"/> </typeparam>
 		/// <param name="domainResultTask"> Details of the operation results (<see cref="DomainResult{T}"/>) </param>
@@ -34,7 +34,7 @@ public static async Task<IActionResult> ToActionResult<T>(this Task<IDomainResul
 																  Action<ProblemDetails, IDomainResult<T>>? errorAction = null)
 		{
 			var domainResult = await domainResultTask;
-			return ToActionResult(domainResult.Value, domainResult, errorAction, (value) => new OkObjectResult(value));
+			return ToActionResult(domainResult.Value, domainResult, errorAction, value => new OkObjectResult(value));
 		}
 
 		/// <summary>
@@ -47,11 +47,11 @@ public static async Task<IActionResult> ToActionResult<T>(this Task<IDomainResul
 		public static IActionResult ToActionResult<V, R>(this (V, R) domainResult,
 														Action<ProblemDetails, R>? errorAction = null)
 														where R : IDomainResult
-			=> domainResult.ToCustomActionResult((value) => new OkObjectResult(value), errorAction);
+			=> domainResult.ToCustomActionResult(value => new OkObjectResult(value), errorAction);
 
 		/// <summary>
 		///		Returns HTTP code 200 (OK) with a value or a 4xx code in case of an error.
-		///		The reesult is wrapped in a <see cref="Task{T}"/>
+		///		The result is wrapped in a <see cref="Task{T}"/>
 		/// </summary>
 		/// <typeparam name="V"> The value type returned in a successful response </typeparam>
 		/// <typeparam name="R"> The type derived from <see cref="IDomainResult"/>, e.g. <see cref="DomainResult"/> </typeparam>
@@ -62,7 +62,7 @@ public static async Task<IActionResult> ToActionResult<V, R>(this Task<(V, R)> d
 																	 where R : IDomainResult
 		{
 			var domainResult = await domainResultTask;
-			return domainResult.ToCustomActionResult((value) => new OkObjectResult(value), errorAction);
+			return domainResult.ToCustomActionResult(value => new OkObjectResult(value), errorAction);
 		}
 	}
 }

src/Mvc/DomainResultTo204NoContentResult.cs

@@ -7,12 +7,11 @@
 
 namespace DomainResults.Mvc
 {
+	//
+	// Conversion to HTTP code 204 (NoContent)
+	//
 	public static partial class DomainResultExtensions
 	{
-		//
-		// Conversion to HTTP code 204 (NoContent)
-		//
-
 		/// <summary>
 		///		Returns HTTP code 204 (NoContent) or a 4xx code in case of an error
 		/// </summary>

tests/DomainResults.Tests/Mvc/To200OkResultTupleValueTests.cs

@@ -34,9 +34,9 @@ public void ValueResult_Converted_ToActionResult_Test<TValue>((TValue, IDomainRe
 
 		public static readonly IEnumerable<object[]> SuccessfulTestCases = new List<object[]>
 		{
-			new object[] { (10,  GetSuccess()) },
-			new object[] { ("1", GetSuccess()) },
-			new object[] { (new TestDto("1"), GetSuccess()) }
+			new object[] { (10,  IDomainResult.Success()) },
+			new object[] { ("1", IDomainResult.Success()) },
+			new object[] { (new TestDto("1"), IDomainResult.Success()) }
 		};
 		#endregion // Test of successful '(TValue, IDomainResult)' response conversion ------------
 
@@ -63,12 +63,10 @@ public async Task ValueResult_Task_Converted_ToActionResult_Test<TValue>(Task<(T
 
 		public static readonly IEnumerable<object[]> SuccessfulTaskTestCases = new List<object[]>
 		{
-			new object[] { Task.FromResult((10,  GetSuccess())) },
-			new object[] { Task.FromResult(("1", GetSuccess())) },
-			new object[] { Task.FromResult((new TestDto("1"), GetSuccess())) }
+			new object[] { Task.FromResult((10,  IDomainResult.Success())) },
+			new object[] { Task.FromResult(("1", IDomainResult.Success())) },
+			new object[] { Task.FromResult((new TestDto("1"), IDomainResult.Success())) }
 		};
 		#endregion // Test of successful 'Task<(TValue, IDomainResult)>' response conversion ------
-
-		private static IDomainResult GetSuccess() => IDomainResult.Success();
 	}
 }

tests/DomainResults.Tests/Mvc/To204NoContentResultSuccessTests.cs

@@ -35,31 +35,5 @@ public async Task DomainResult_Task_Converted_To_NoContent()
 			// THEN the response type is correct
 			Assert.IsType<NoContentResult>(actionRes);
 		}
-
-		[Fact]
-		public void DomainResultConverted_To_NoContentOfT()
-		{
-			// GIVEN a successful domain result
-			var domainRes = DomainResult.Success();
-
-			// WHEN convert it to ActionResult
-			var actionRes = domainRes.ToActionResultOfT();
-
-			// THEN the response type is correct
-			Assert.IsType<NoContentResult>(actionRes.Result);
-		}
-
-		[Fact]
-		public async Task DomainResult_Task_Converted_To_NoContentOfT()
-		{
-			// GIVEN a successful domain result
-			var domainRes = DomainResult.SuccessTask();
-
-			// WHEN convert it to ActionResult
-			var actionRes = await domainRes.ToActionResultOfT();
-
-			// THEN the response type is correct
-			Assert.IsType<NoContentResult>(actionRes.Result);
-		}
 	}
 }