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

(#149) Posts service tests #165

Closed
wants to merge 27 commits into from
Closed

(#149) Posts service tests #165

wants to merge 27 commits into from

Conversation

SaintAngeLs
Copy link
Owner

Adding the posts service tests

Copy link

codecov bot commented May 20, 2024

Codecov Report

Attention: Patch coverage is 95.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 55.82%. Comparing base (33bbee9) to head (487c62a).

Files Patch % Lines
....Services.Posts.Application/Commands/CreatePost.cs 75.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #165      +/-   ##
==========================================
+ Coverage   46.63%   55.82%   +9.18%     
==========================================
  Files         120      165      +45     
  Lines        1561     2137     +576     
  Branches      139      191      +52     
==========================================
+ Hits          728     1193     +465     
- Misses        795      889      +94     
- Partials       38       55      +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@SaintAngeLs
Copy link
Owner Author

@olegkiprik, the current tests coverage is not sufficient. Please, get together and add more coverage.

@SaintAngeLs SaintAngeLs force-pushed the posts_service_tests branch from f8fa486 to 38f67b3 Compare May 26, 2024 20:10
@olegkiprik olegkiprik closed this May 26, 2024
@olegkiprik olegkiprik deleted the posts_service_tests branch May 26, 2024 20:55
@SaintAngeLs SaintAngeLs restored the posts_service_tests branch May 26, 2024 20:56
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