🐛 Add useEffect for action state sync

[ibolton336]

Resolves https://issues.redhat.com/browse/MTA-1826

• Uses a useEffect hook to synchronize the state of the action based on the assessment object. This way, whenever assessment changes, we can ensure that the action is recalculated.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ca600a3) 39.27% compared to head (521fe79) 39.27%.
Report is 1 commits behind head on main.

❗ Current head 521fe79 differs from pull request most recent head 15cf683. Consider uploading reports for the commit 15cf683 to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1600   +/-   ##
=======================================
  Coverage   39.27%   39.27%           
=======================================
  Files         146      146           
  Lines        4827     4827           
  Branches     1152     1152           
=======================================
  Hits         1896     1896           
  Misses       2917     2917           
  Partials       14       14           

Flag	Coverage Δ
client	39.27% <ø> (ø)
server	∅ <ø> (∅)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sjd78]

Just curious if you tried useMemo instead of useEffect?

It seems like you'd be able to just do the whole thing as a single memo w/o state:

  const action = useMemo(() => {
    if (!assessment) {
      return AssessmentAction.Take;
    } else if (assessment.status === "started") {
      return AssessmentAction.Continue;
    } else {
      return AssessmentAction.Retake;
    }
  }, [assessment]);