Increase test coverage

[rinchen]

Is your feature request related to a problem? Please describe.
Increase test coverage so developers can run this prior to finalizing a PR.

[gnuamua]

My plan is to start easy and simple e.g. USB Serial test, BLE test, Wireless test and then expand from there as the testing would get more in-depth and require more involved / advanced Test fixture construction & taking down

[rinchen]

#19 will add in a basic test scaffold to use. Hopefully that makes this issue easier.

[rinchen]

One thing that would really help, if it can be automated, is testing of the builds. At the very least, before someone submits a PR we would want them to resolve all build (the binary apps) errors. If you build on macos, you get a warning when building win and linux...and it's ok to ignore that one. This isn't needed for this ticket unless you really want to.

[rinchen]

The error is

(node:92504) [DEP0190] DeprecationWarning: Passing args to a child process with shell option true can lead to security vulnerabilities, as the arguments are not escaped, only concatenated.

It's not coming from the electron builder nor package.json.

This is an internal warning triggered by electron-builder itself. When you run npm run dist:mac, you are executing electron-builder --mac. Under the hood, electron-builder performs various system operations—such as calling codesign, security, or hdiutil—and in certain versions, these internal calls are made using shell: true, which triggers the Node.js deprecation notice in newer runtimes.

[rinchen]

I found some errors so I added #75 to do basic format and lint.

[rinchen]

I've added more tests as part of fixing bugs and adding features. There is still more to be done.

[rinchen]

We're up to over 1000 test cases now.

[rinchen]

We now have a test coverage report, and it sits at an average of 50%. It's only viewable on pull requests.