search

edina / nbexchange

External exchange for nbgrader
Other
6 stars 2 forks source link

Nbex user never none #108

Closed perllaghu closed 3 years ago

perllaghu commented 3 years ago

Raises a ValueError if nbex_user does not have course details, and the handlers all manage that error.

codecov[bot] commented 3 years ago

Codecov Report

Merging #108 (a56457c) into master (7f5fb2d) will increase coverage by 0.19%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #108      +/-   ##
==========================================
+ Coverage   87.40%   87.59%   +0.19%     
==========================================
  Files          73       73              
  Lines        6024     6109      +85     
==========================================
+ Hits         5265     5351      +86     
+ Misses        759      758       -1
Impacted Files Coverage Δ
nbexchange/handlers/assignment.py 82.14% <ø> (ø)
nbexchange/handlers/collection.py 89.90% <ø> (ø)
nbexchange/handlers/feedback.py 83.73% <ø> (ø)
nbexchange/handlers/submission.py 84.14% <ø> (ø)
nbexchange/handlers/base.py 98.93% <100.00%> (+1.11%) :arrow_up:
nbexchange/tests/test_handlers_collection.py 82.09% <100.00%> (+1.43%) :arrow_up:
nbexchange/tests/test_handlers_collections.py 100.00% <100.00%> (ø)
nbexchange/tests/test_handlers_delete.py 94.91% <100.00%> (+0.36%) :arrow_up:
nbexchange/tests/test_handlers_feedback.py 95.23% <100.00%> (+0.45%) :arrow_up:
nbexchange/tests/test_handlers_fetch.py 100.00% <100.00%> (ø)
... and 4 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 7f5fb2d...a56457c. Read the comment docs.