Fix warning messages

[ghost]

This PR contains:

• [x] bugfix
• [ ] feature
• [ ] refactor
• [ ] documentation
• [ ] other

Are tests included?

• [x] yes (bugfixes and features will not be merged without tests)
• [ ] no

Breaking Changes?

• [ ] yes (breaking changes will not be merged unless absolutely necessary)
• [x] no

List any relevant issue numbers:

4

Description

Fixes #4, this time with a unit test.

@petabyteboy please include tests next time you contribute to a rollup project. I was moving too quick today to see that you had checked the box but did not include tests. That's not cool.

It was not obvious to me that this meant automated unit tests, sorry. I thought it would be sufficient to run manual tests and confirm that the problem is fixed.

[shellscape]

It was not obvious to me that this meant automated unit tests, sorry. I thought it would be sufficient to run manual tests and confirm that the problem is fixed.

Thank you for taking the time to open a new PR. I'll update out template to make that more clear, thank you for pointing that out.

[kermage]

Any update here?