Commit 524232e

committed

feat: throw 422 status code when validation fails

1 parent b841433 commit 524232eCopy full SHA for 524232e

File tree

-6

lines changed

-6

lines changed

Lines changed: 3 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -156,7 +156,7 @@ public virtual async Task<IActionResult> PostAsync([FromBody] T entity)`
`156`	`156`	`return Forbidden();`
`157`	`157`
`158`	`158`	`if (_jsonApiContext.Options.ValidateModelState && !ModelState.IsValid)`
`159`		`- return BadRequest(ModelState.ConvertToErrorCollection<T>(_jsonApiContext.ContextGraph));`
	`159`	`+ return UnprocessableEntity(ModelState.ConvertToErrorCollection<T>(_jsonApiContext.ContextGraph));`
`160`	`160`
`161`	`161`	`entity = await _create.CreateAsync(entity);`
`162`	`162`
`@@ -169,8 +169,9 @@ public virtual async Task<IActionResult> PatchAsync(TId id, [FromBody] T entity)`
`169`	`169`
`170`	`170`	`if (entity == null)`
`171`	`171`	`return UnprocessableEntity();`
	`172`	`+`
`172`	`173`	`if (_jsonApiContext.Options.ValidateModelState && !ModelState.IsValid)`
`173`		`- return BadRequest(ModelState.ConvertToErrorCollection<T>(_jsonApiContext.ContextGraph));`
	`174`	`+ return UnprocessableEntity(ModelState.ConvertToErrorCollection<T>(_jsonApiContext.ContextGraph));`
`174`	`175`
`175`	`176`	`var updatedEntity = await _update.UpdateAsync(id, entity);`
`176`	`177`

Lines changed: 4 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -200,8 +200,8 @@ public async Task PatchAsync_ModelStateInvalid_ValidateModelStateEnabled()`
`200`	`200`
`201`	`201`	`// assert`
`202`	`202`	`serviceMock.Verify(m => m.UpdateAsync(id, It.IsAny<Resource>()), Times.Never);`
`203`		`- Assert.IsType<BadRequestObjectResult>(response);`
`204`		`- Assert.IsType<ErrorCollection>(((BadRequestObjectResult) response).Value);`
	`203`	`+ Assert.IsType<UnprocessableEntityObjectResult>(response);`
	`204`	`+ Assert.IsType<ErrorCollection>(((UnprocessableEntityObjectResult) response).Value);`
`205`	`205`	`}`
`206`	`206`
`207`	`207`	`[Fact]`
`@@ -278,8 +278,8 @@ public async Task PostAsync_ModelStateInvalid_ValidateModelStateEnabled()`
`278`	`278`
`279`	`279`	`// assert`
`280`	`280`	`serviceMock.Verify(m => m.CreateAsync(It.IsAny<Resource>()), Times.Never);`
`281`		`- Assert.IsType<BadRequestObjectResult>(response);`
`282`		`- Assert.IsType<ErrorCollection>(((BadRequestObjectResult)response).Value);`
	`281`	`+ Assert.IsType<UnprocessableEntityObjectResult>(response);`
	`282`	`+ Assert.IsType<ErrorCollection>(((UnprocessableEntityObjectResult)response).Value);`
`283`	`283`	`}`
`284`	`284`
`285`	`285`	`[Fact]`

Comments

(0)