Add event testing

[emmyoop]

resolves #50

Description

• Adds testing for event system

Additional fixes found while adding tests

• Fixes missing coverage setup
• Fixes CI to actually fail when tests fail

Checklist

• I have read the contributing guide and understand what's expected of me
• I have signed the CLA
• I have run this code in development and it appears to resolve the stated issue
• I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
• I have run changie new to create a changelog entry

[emmyoop]

Tests are expected to fail until #49 is merged

[github-actions]

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7aa35e5) 51.47% compared to head (ec10b29) 52.31%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #51      +/-   ##
==========================================
+ Coverage   51.47%   52.31%   +0.84%     
==========================================
  Files          49       49              
  Lines        2856     2856              
==========================================
+ Hits         1470     1494      +24     
+ Misses       1386     1362      -24     

Flag	Coverage Δ
unit	52.31% <ø> (+0.84%)	⬆️

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.

[MichelleArk]

just a couple small nits. otherwise looks good. thank you!