Closed Rupeshiya closed 4 years ago
@devesh-verma Please review the PR.
Merging #117 into development will increase coverage by
6.26%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## development #117 +/- ##
===============================================
+ Coverage 57.94% 64.21% +6.26%
===============================================
Files 22 20 -2
Lines 428 394 -34
Branches 60 53 -7
===============================================
+ Hits 248 253 +5
+ Misses 154 122 -32
+ Partials 26 19 -7
Impacted Files | Coverage Δ | |
---|---|---|
app/routes/user.js | 100.00% <ø> (ø) |
|
views/emailTemplate.js | ||
app/middleware/email.js | ||
app/controllers/organization.js | 57.89% <0.00%> (+1.64%) |
:arrow_up: |
app/controllers/user.js | 54.92% <0.00%> (+38.02%) |
:arrow_up: |
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 2419309...aea0838. Read the comment docs.
name: Org REST API TESTS about: Tested organisatin REST API title: ''Organisatin REST API tests" labels: ''feature assignees: ''@Rupeshiya
Problem
Github Issue Number: #116 Adde unit tests for Org REST API_
Solution of problem
Used jest and supertest to write unit test
In Case of UI Changes
No UI changes are required
Type of Change
[ ] Bug fix [✓] New Feature [ ] Development of UI/UX prototypes [✓] Small refactor [ ] Change in Documentation
Checklist
[✓] My code follows the same style as the codebase [ ] My Code change requires a change in documentation [ ] I have updated the Readme accordingly [✓] I made PR against development branch [✓] I have run the test cases locally and it's passing. [✓] I have squashed my commits