Contacts/Appointments Testing

[akanshyabhat]

Contacts/Appointments Testing + Final Changes (Timeline)

♻️ Current situation & Problem

There were some untested lines of code in the Contacts and Appointments tabs.

⚙️ Release Notes

• Removed the left arrowhead from timeline based on feedback from PIs

✅ Testing

• Added a UI test to Contacts and a UI test to Appointments
• Added a functional test for the code in AppointmentInformation
• Reschedule the dates to also trigger the notification handling

📝 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

All modified and coverable lines are covered by tests ✅

Project coverage is 82.26%. Comparing base (f1b4e0c) to head (fb5e21b).

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #55      +/-   ##
==========================================
- Coverage   83.07%   82.26%   -0.80%     
==========================================
  Files          54       54              
  Lines        2993     2987       -6     
==========================================
- Hits         2486     2457      -29     
- Misses        507      530      +23     

Files	Coverage Δ
PICS/Appointment/TimelineView.swift	98.12% <100.00%> (-0.10%)	⬇️

... and 8 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 f1b4e0c...fb5e21b. Read the comment docs.

[YurenSUN]

The tests look great to me, maybe just need to revert the changes to the build scheme and maybe also revert changes in deleting the DEVELOPMENT_TEAM as we currently have all the tests (both Github PR tests and the beta deployment tests) and not sure whether changing them to an empty one might break any tests or deployment?