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

Correctly support DateTime pattern as string in ExactMatcher #391

Merged
merged 1 commit into from
Dec 14, 2019

Conversation

StefH
Copy link
Collaborator

@StefH StefH commented Dec 13, 2019

@StefH StefH added the bug label Dec 13, 2019
@StefH StefH self-assigned this Dec 13, 2019
@codecov
Copy link

codecov bot commented Dec 13, 2019

Codecov Report

Merging #391 into master will decrease coverage by 0.07%.
The diff coverage is 42.85%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #391      +/-   ##
==========================================
- Coverage      80%   79.93%   -0.08%     
==========================================
  Files         121      121              
  Lines        4782     4789       +7     
==========================================
+ Hits         3826     3828       +2     
- Misses        956      961       +5
Impacted Files Coverage Δ
src/WireMock.Net/Util/BodyParser.cs 96.96% <100%> (-0.04%) ⬇️
src/WireMock.Net/ResponseBuilders/Response.cs 96.96% <100%> (ø) ⬆️
src/WireMock.Net/Server/FluentMockServer.Admin.cs 61.01% <16.66%> (-0.21%) ⬇️
src/WireMock.Net/Util/JsonUtils.cs 96.84% <50%> (-3.16%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b3c2af0...17d12b5. Read the comment docs.

@StefH StefH merged commit a8c17ce into master Dec 14, 2019
@StefH StefH deleted the ExactMatcher_DateTimePattern branch December 14, 2019 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant