Issues with unknown devices or errors are now handled

[manzanotti]

This replaces #78.

Unknown issues and devices are now handled by the code.

Move the issue code out into its own class, matching the future direction of the codebase.

Added tests to cover issues.

[manzanotti]

@GeoffAtHome Do you want to cast an eye over this commit, make sure it makes sense to you?

[GeoffAtHome]

Hi Paul,

Like the refactoring. Splitting the issues out to a new file gets my vote as does the changes to parsing arguments.

Still works on my system so I am pleased.

[manzanotti]

Hi Paul,

Like the refactoring. Splitting the issues out to a new file gets my vote as does the changes to parsing arguments.

Still works on my system so I am pleased.

Yeah, I first started splitting the code up into more class-based units three (!) years ago, though I didn't end up merging it in as it felt like too big a change. I ended up just adding tests for the current code base first. I still plan to move to that model though.

You can see the changes in the old PRs:

https://github.com/manzanotti/geniushub-client/pull/48/files
https://github.com/manzanotti/geniushub-client/pull/49/files