Skip to content

Commit b8697cc

Browse files
committed
Added And method after checking for model state (#13)
1 parent ba498c6 commit b8697cc

12 files changed

+18
-114
lines changed
Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
namespace MyWebApi.Builders.Actions
22
{
33
using System.Web.Http.Results;
4-
using And;
5-
using Contracts.And;
64

75
/// <summary>
86
/// Class containing methods for testing NotFoundResult.
@@ -13,10 +11,9 @@ public partial class ActionResultTestBuilder<TActionResult>
1311
/// <summary>
1412
/// Tests whether action result is NotFoundResult.
1513
/// </summary>
16-
public IAndTestBuilder<TActionResult> ShouldReturnNotFound()
14+
public void ShouldReturnNotFound()
1715
{
1816
this.ShouldReturn<NotFoundResult>();
19-
return this.NewAndTestBuilder();
2017
}
2118
}
2219
}

MyWebApi/Builders/Actions/ShouldReturnStatusCode.cs

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
using System.Net;
44
using System.Web.Http.Results;
55
using Common.Extensions;
6-
using Contracts.And;
76
using Exceptions;
87

98
/// <summary>
@@ -15,17 +14,16 @@ public partial class ActionResultTestBuilder<TActionResult>
1514
/// <summary>
1615
/// Tests whether action result is StatusCodeResult.
1716
/// </summary>
18-
public IAndTestBuilder<TActionResult> ShouldReturnStatusCode()
17+
public void ShouldReturnStatusCode()
1918
{
2019
this.ShouldReturn<StatusCodeResult>();
21-
return this.NewAndTestBuilder();
2220
}
2321

2422
/// <summary>
2523
/// Tests whether action result is StatusCodeResult and is the same as provided HttpStatusCode.
2624
/// </summary>
2725
/// <param name="statusCode">HttpStatusCode enumeration.</param>
28-
public IAndTestBuilder<TActionResult> ShouldReturnStatusCode(HttpStatusCode statusCode)
26+
public void ShouldReturnStatusCode(HttpStatusCode statusCode)
2927
{
3028
var statusCodeResult = this.GetReturnObject<StatusCodeResult>();
3129
if (statusCodeResult.StatusCode != statusCode)
@@ -39,8 +37,6 @@ public IAndTestBuilder<TActionResult> ShouldReturnStatusCode(HttpStatusCode stat
3937
(int)statusCodeResult.StatusCode,
4038
statusCodeResult.StatusCode));
4139
}
42-
43-
return this.NewAndTestBuilder();
4440
}
4541
}
4642
}

MyWebApi/Builders/And/AndContinuityTestBuilder.cs

Lines changed: 0 additions & 19 deletions
This file was deleted.

MyWebApi/Builders/And/AndContinuityTestBuilder{TActionResult}.cs

Lines changed: 0 additions & 21 deletions
This file was deleted.
Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,22 @@
11
namespace MyWebApi.Builders.And
22
{
33
using System.Web.Http;
4+
using Actions;
45
using Base;
6+
using Contracts.Actions;
57
using Contracts.And;
68

7-
public class AndTestBuilder : BaseTestBuilder, IAndTestBuilder
9+
public class AndTestBuilder<TActionResult> : BaseTestBuilderWithActionResult<TActionResult>,
10+
IAndTestBuilder<TActionResult>
811
{
9-
protected AndTestBuilder(ApiController controller, string actionName)
10-
: base(controller, actionName)
12+
public AndTestBuilder(ApiController controller, string actionName, TActionResult actionResult)
13+
: base(controller, actionName, actionResult)
1114
{
1215
}
1316

14-
public IAndContinuityTestBuilder And()
17+
public IActionResultTestBuilder<TActionResult> And()
1518
{
16-
return new AndContinuityTestBuilder(this.Controller, this.ActionName);
19+
return new ActionResultTestBuilder<TActionResult>(this.Controller, this.ActionName, this.ActionResult);
1720
}
1821
}
1922
}

MyWebApi/Builders/And/AndTestBuilder{TActionResult}.cs

Lines changed: 0 additions & 21 deletions
This file was deleted.

MyWebApi/Builders/Contracts/Actions/IActionResultTestBuilder.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,18 +41,18 @@ public interface IActionResultTestBuilder<out TActionResult> : IBaseTestBuilderW
4141
/// <summary>
4242
/// Tests whether action result is StatusCodeResult.
4343
/// </summary>
44-
IAndTestBuilder<TActionResult> ShouldReturnStatusCode();
44+
void ShouldReturnStatusCode();
4545

4646
/// <summary>
4747
/// Tests whether action result is StatusCodeResult and is the same as provided HttpStatusCode.
4848
/// </summary>
4949
/// <param name="statusCode">HttpStatusCode enumeration.</param>
50-
IAndTestBuilder<TActionResult> ShouldReturnStatusCode(HttpStatusCode statusCode);
50+
void ShouldReturnStatusCode(HttpStatusCode statusCode);
5151

5252
/// <summary>
5353
/// Tests whether action result is NotFoundResult.
5454
/// </summary>
55-
IAndTestBuilder<TActionResult> ShouldReturnNotFound();
55+
void ShouldReturnNotFound();
5656

5757
/// <summary>
5858
/// Tests whether action result is BadRequestResult, InvalidModelStateResult or BadRequestErrorMessageResult.

MyWebApi/Builders/Contracts/And/IAndContinuityTestBuilder.cs

Lines changed: 0 additions & 9 deletions
This file was deleted.

MyWebApi/Builders/Contracts/And/IAndContinuityTestBuilder{TActionResult}.cs

Lines changed: 0 additions & 11 deletions
This file was deleted.
Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,9 @@
11
namespace MyWebApi.Builders.Contracts.And
22
{
3-
public interface IAndTestBuilder
3+
using Actions;
4+
5+
public interface IAndTestBuilder<out TActionResult>
46
{
5-
IAndContinuityTestBuilder And();
7+
IActionResultTestBuilder<TActionResult> And();
68
}
79
}

0 commit comments

Comments
 (0)