search

fac-14 / OSCEBossKey

Weeks 13-16 > Tech for Better project: An app to help medical students revise for their exams
https://oscebosskey.herokuapp.com/
MIT License
4 stars 2 forks source link

Body component #73

Closed njons closed 5 years ago

codecov-io commented 5 years ago

Codecov Report

Merging #73 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #73   +/-   ##
=======================================
  Coverage   89.74%   89.74%           
=======================================
  Files          11       11           
  Lines          39       39           
  Branches        2        2           
=======================================
  Hits           35       35           
  Misses          4        4
Impacted Files Coverage Δ
src/components/HistoryCaseRevision.js 50% <ø> (ø) :arrow_up:
src/components/Body.js 100% <100%> (ø) :arrow_up:
src/utils/HistoryCaseRevision.functions.js 100% <100%> (ø) :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 46be8ba...23ca39b. Read the comment docs.

martingaston commented 5 years ago

would recommend you test this if you haven't already: 

markComplete: (id, prevState) => {
    if (!prevState.markSchemeElements[id]) {
      throw new Error("element out of range");
    }
    const markSchemeElements = [...prevState.markSchemeElements];
    markSchemeElements[id].completed = !markSchemeElements.completed;
    return {
        markSchemeElements,
        markSchemeCompleted: prevState.markSchemeCompleted + markSchemeElements[id].completed ? 1 : -1
    }
  }