Closed Danwhy closed 6 years ago
Merging #675 into staging will decrease coverage by
6.28%
. The diff coverage is45.45%
.
@@ Coverage Diff @@
## staging #675 +/- ##
===========================================
- Coverage 94.84% 88.56% -6.29%
===========================================
Files 133 144 +11
Lines 3007 3357 +350
Branches 524 600 +76
===========================================
+ Hits 2852 2973 +121
- Misses 155 384 +229
Impacted Files | Coverage Δ | |
---|---|---|
server/lib/getModuleForStudent.js | 100% <ø> (ø) |
:arrow_up: |
src/js/reducers/module.js | 88.23% <ø> (ø) |
:arrow_up: |
server/lib/organiseModuleData.js | 88.88% <ø> (ø) |
:arrow_up: |
src/js/reducers/login.js | 69.23% <0%> (-20.77%) |
:arrow_down: |
src/js/reducers/register.js | 90% <0%> (-10%) |
:arrow_down: |
src/js/reducers/new-module.js | 93.47% <0%> (-4.25%) |
:arrow_down: |
server/lib/getUserByMoodleID.js | 100% <100%> (ø) |
|
server/plugins/modules.js | 97.24% <100%> (+0.05%) |
:arrow_up: |
server/plugins/authenticate-user.js | 97.77% <100%> (+1.55%) |
:arrow_up: |
server/lib/authentication/setSession.js | 100% <100%> (ø) |
|
... and 31 more |
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 1256d51...4c28c20. Read the comment docs.
Adds some improvements to the flow of signing up through Moodle, both as an existing and new user:
ref #670 , #671