Moar tests

[bizob2828]

Proposed Release Notes

• Refactored code to make it easier to test
• Add full suite of unit tests

Links

Closes #223

Details

[codecov]

Codecov Report

Merging #228 (56dce51) into main (4386ff0) will increase coverage by 9.62%.
The diff coverage is 96.00%.

@@            Coverage Diff             @@
##             main     #228      +/-   ##
==========================================
+ Coverage   87.57%   97.19%   +9.62%     
==========================================
  Files           4        5       +1     
  Lines         700      713      +13     
==========================================
+ Hits          613      693      +80     
+ Misses         87       20      -67     

Flag	Coverage Δ
integration-tests-14.x-linux	84.57% <73.50%> (+1.42%)	⬆️
integration-tests-14.x-windows-2019	78.96% <56.00%> (+2.24%)	⬆️
integration-tests-16.x-linux	84.57% <73.50%> (+1.42%)	⬆️
integration-tests-16.x-windows-2019	78.96% <56.00%> (+2.24%)	⬆️
integration-tests-18.x-linux	84.29% <73.50%> (+1.43%)	⬆️
integration-tests-18.x-windows-2019	78.68% <56.00%> (+2.25%)	⬆️
integration-tests-18.x-windows-latest	78.68% <56.00%> (+2.25%)	⬆️
unit-tests-14.x-linux	87.93% <93.50%> (+39.93%)	⬆️
unit-tests-14.x-windows-2019	87.79% <93.50%> (+39.79%)	⬆️
unit-tests-16.x-linux	87.93% <93.50%> (+39.93%)	⬆️
unit-tests-16.x-windows-2019	87.79% <93.50%> (+39.79%)	⬆️
unit-tests-18.x-linux	87.65% <93.50%> (+39.94%)	⬆️
unit-tests-18.x-windows-2019	87.51% <93.50%> (+39.80%)	⬆️
unit-tests-18.x-windows-latest	87.51% <93.50%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
lib/pre-build.js	96.96% <93.87%> (+18.78%)	⬆️
lib/gyp-utils.js	98.01% <98.01%> (ø)
lib/common.js	100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[jmartin4563]

you're gonna have to merge main in to get the required checks, but this is good to go!