-
Notifications
You must be signed in to change notification settings - Fork 4
Test coverage enhancement #344
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
Conversation
b5db5a3 to
8b8a8e7
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #344 +/- ##
==========================================
Coverage ? 31.26%
==========================================
Files ? 84
Lines ? 4158
Branches ? 1228
==========================================
Hits ? 1300
Misses ? 2826
Partials ? 32 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
8b8a8e7 to
49a5ed7
Compare
49a5ed7 to
06fe3b9
Compare
06fe3b9 to
35a7699
Compare
35a7699 to
9e6ffdf
Compare
9e6ffdf to
b5cec4e
Compare
b5cec4e to
d91af50
Compare
d91af50 to
4f1d9c0
Compare
4f1d9c0 to
e6da0ab
Compare
e6da0ab to
6064b79
Compare
6064b79 to
9df90d5
Compare
Backend coverage enhancement Additional tests for backend
9df90d5 to
b034257
Compare
RWEMAREMY
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good work
shebz2023
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
PR Description
This pr focuses on increasing the test coverage of the whole backend application
Functionality
How does this works, add some information that maybe needed in testing
How has this been tested?
npm run test
or
see circle Ci build reports of the pr
Screenshots
Before
After
For now
