fix/issue 1080

[kristof-mattei]

• fix: fixing bug identified in Issue #1080
• chore: re-running fmt with nightly
• chore: re-re-running fmt with nightly
• chore: switching out unwrap() with expect() per linter
• chore: swapping to plain array per linter
• chore: adding additional scenarios in to cover missing code lines
• fix: address PR comments

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 47.77%. Comparing base (0a69719) to head (af5ccca).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1457      +/-   ##
==========================================
+ Coverage   47.45%   47.77%   +0.32%     
==========================================
  Files          24       24              
  Lines        2131     2144      +13     
==========================================
+ Hits         1011     1024      +13     
  Misses       1120     1120              

Flag	Coverage Δ
unit-tests	47.77% <100.00%> (+0.32%)	⬆️

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.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.