Assessments tests and minor fixes

[YurenSUN]

Assessments tests and minor fixes

♻️ Current situation & Problem

Currently, we only implement the assessments tab for #44 but we did not write any tests to test it. Therefore, this PR aims to add the UI and uni tests for the assessments tab, including each of the tasks and the visualization of results.
Currently, we only test creating the task for trail making and reaction time task as described below, but we reach the desired test coverage and 539 out of 589 lines (~91.5%) in the assessment folder are covered. This PR also includes some small fixes described below.

⚙️ Release Notes

• tests the assessment tasks. Currently, we run through the Stroop task during the UI task and only test creating tasks for (1) trail-making task as we are unable to click the buttons to make trails as they are not shown in the app but grouped into an Other element Tappable Buttons and (2) reaction time as currently there is no supports for the shake motion in testing. This should be acceptable as the process of the two tasks is also solely handled by the ResaerchKit, and we only wrote functions to create and parse the results.
• Add the additional unit task to the function parsing the trail-making task to ensure that no error is returned
• We are missing tests for parsing the reaction time result as the simulator does not support result files, which is required when parsing the reaction time result.

Other fixes

• Fix some colors for the dark mode for HealthKit visualizations.
• Fix the issue that the running task occurs on both the main screen and the sheet. Also change the toggle showingTestSheet to directly setting to true to avoid the sheet not re-rendering (always showing trail-making task) issue.
• Do not show chart legend for score/error count if those data are not collected.

📚 Documentation

Related commends are added to the codes.

✅ Testing

The PR adds tests to our codes.

📝 Code of Conduct & Contributing Guidelines

By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

• I agree to follow the Code of Conduct and Contributing Guidelines.

[codecov]

Codecov Report

Attention: Patch coverage is 87.69231% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 54.75%. Comparing base (3be7d9a) to head (b0ac9d4).

Additional details and impacted files

@@             Coverage Diff             @@
##             main      #56       +/-   ##
===========================================
+ Coverage   36.18%   54.75%   +18.57%     
===========================================
  Files          54       54               
  Lines        2977     2974        -3     
===========================================
+ Hits         1077     1628      +551     
+ Misses       1900     1346      -554     

Files	Coverage Δ
PICS/Assessment/Assessments.swift	100.00% <100.00%> (+95.53%)	⬆️
PICS/Assessment/ReactionTime.swift	56.17% <100.00%> (+56.17%)	⬆️
PICS/Assessment/ResultsViz.swift	96.90% <100.00%> (+96.90%)	⬆️
PICS/Assessment/StroopTest.swift	98.37% <100.00%> (+98.37%)	⬆️
PICS/HealthVisulization/HKVisualizationItem.swift	96.38% <100.00%> (ø)
PICS/Assessment/TrailMakingTest.swift	82.82% <69.24%> (+82.82%)	⬆️

... and 6 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3be7d9a...b0ac9d4. Read the comment docs.

[akanshyabhat]

Looks good to me! Thank you :)