Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Json result #11

Merged
merged 7 commits into from
Mar 10, 2020
Merged

Json result #11

merged 7 commits into from
Mar 10, 2020

Conversation

faddiv
Copy link
Collaborator

@faddiv faddiv commented Feb 19, 2020

Hy Kevin,

I added JsonResult assertions to the project. I thought it would be useful. Please review if this is fine.
This consist of the followings:
result.Should().BeJsonResult()
result.Should().BeJsonResult().WithContentType("content type");
result.Should().BeJsonResult().WithStatusCode(200);
result.Should().BeJsonResult().Value (For the JsonResult.Value)
result.Should().BeJsonResult().ValueAs() (Type checking and casting like in the ViewResult)
result.Should().BeJsonResult().SerializerSettings (For the JsonResult.SerializerSettings)

I am also interested doing further works on this project.

@faddiv faddiv requested a review from kevinkuszyk February 19, 2020 16:45
@kevinkuszyk
Copy link
Member

@faddiv hello and thanks for your email.

I missed the notifications on this project (I have been working on things outside the .net space recently) - apologies for that.

We have some merge conflicts here now. Would you mind to fix them?

Thanks for the offer of help here. After a few more contributions, I'd be happy to make you a maintainer.

# Conflicts:
#	src/FluentAssertions.AspNetCore.Mvc/FailureMessages.Designer.cs
#	src/FluentAssertions.AspNetCore.Mvc/FailureMessages.resx
#	src/FluentAssertions.AspNetCore.Mvc/FluentAssertions.AspNetCore.Mvc.csproj
@faddiv
Copy link
Collaborator Author

faddiv commented Mar 10, 2020

Finished resolving the conflict. Everything seems fine.

@kevinkuszyk kevinkuszyk merged commit c523d4b into fluentassertions:master Mar 10, 2020
@kevinkuszyk
Copy link
Member

Thanks again for the contribution. It's in v2.2.0 which should be up on nuget in the next hour or so.

@faddiv faddiv deleted the JsonResult branch March 16, 2020 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants