FAC-11 / AllAboutMe

An app to help young people at Anna Freud centre to easily transition to adult care
https://allaboutme-af.herokuapp.com/
10 stars 6 forks source link

Tests #134

Closed polyccon closed 6 years ago

polyccon commented 6 years ago

Added tests for the router endpoints and fixed bugs with password hashing and verification. Fixes #144 #106 #143

codecov-io commented 6 years ago

Codecov Report

Merging #134 into master will decrease coverage by 26.99%. The diff coverage is 93.33%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master     #134    +/-   ##
========================================
- Coverage   95.41%   68.42%   -27%     
========================================
  Files           8       26    +18     
  Lines         109      418   +309     
========================================
+ Hits          104      286   +182     
- Misses          5      132   +127
Impacted Files Coverage Δ
src/controllers/appointments.js 17.64% <ø> (ø)
src/controllers/index.js 100% <ø> (ø)
src/controllers/send.js 41.66% <ø> (ø)
src/controllers/login.js 89.47% <100%> (ø)
src/controllers/requireLogin.js 85.71% <100%> (ø)
src/controllers/signup.js 75% <100%> (ø)
src/controllers/reset.js 37.83% <50%> (ø)
src/controllers/passwordModule.js 100% <0%> (ø)
src/controllers/logout.js 33.33% <0%> (ø)
... and 15 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 a30e1bc...77e3d0e. Read the comment docs.

sohilpandya commented 6 years ago

Please resolve conflicts and I can merge in.

sohilpandya commented 6 years ago

Please resolve conflicts. :)