all test warnings fixed

[braxex]

Summary

There were several warnings popping up on unit test run. This PR resolves them.

Type of change

• [x] Bug fix (non-breaking change which fixes an issue)

How To Test

n/a (all tests pass)

Checklist:

• [x] My code follows the style guidelines of this project and has no linting errors
• [x] I have performed a self-review of my code
• [x] I have commented my code, particularly in hard-to-understand areas
• [x] I have made corresponding changes to the documentation, including any applicable ADRs
• [x] My changes generate no new warnings
• [x] I have added tests that fail without these changes
• [x] New and existing tests (unit, integration, accessibility) pass locally
• [x] Documentation updated
• [x] If there are security concerns they are addressed or ticketed after being discussed

[codeclimate[bot]]

Code Climate has analyzed commit 6ea150e7 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (80% is the threshold).

This pull request will bring the total coverage in the repository to 95.1% (0.0% change).

View more on Code Climate.