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

tests/unittest.cpp : added unit tests #213

Merged
merged 3 commits into from
Aug 31, 2021

Conversation

hg333
Copy link
Contributor

@hg333 hg333 commented Aug 30, 2021

Added unit tests corresponding to #212, issue:#21?.

@The-EDev
Copy link
Member

Thanks for your work, but checking the Content-Type header is more important IMO

@hg333 hg333 requested a review from The-EDev August 30, 2021 12:53
@The-EDev
Copy link
Member

Sorry, seems there's a problem with coveralls reporting. Nothing related to your PR, just gotta get it fixed.

@The-EDev
Copy link
Member

Coveralls responded, seems like the issue was caused by some downtime they experienced over the weekend. Should be fine in time, I'll merge this now.

@The-EDev The-EDev merged commit deff822 into CrowCpp:master Aug 31, 2021
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