Dependency update

- FIXED: tests did not check for devices that were already connected.
- Moved away from deprecated usb-detection and using the usb module instead.
- Added note in README.md about future feature to allow multiple devices.

Author: hopejr

README.md

@@ -95,6 +95,10 @@ By default, the udev system adds ShuttleXpress, ShuttlePro V1, and ShuttlePro V2
 Then reboot your computer.
 
 
+# Future Features
+I'm looking at allowing multiple devices to be connected at once. Stay tuned for this feature. It will likely result in a change to the API and will be a breaking change.
+
+
 ## Licence
 MIT
 

lib/Shuttle.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const hid = require('node-hid')
-const usbDetect = require('usb-detection')
+const { usb } = require('usb')
 const shuttleDevices = require('./ShuttleDefs')
 const EventEmitter = require('events').EventEmitter
 
@@ -21,8 +21,7 @@ class Shuttle extends EventEmitter {
 
   start () {
     if (!this._connected) {
-      usbDetect.startMonitoring()
-      usbDetect.on('add', (d) => {
+      usb.on('attach', (d) => {
         // Delay connection by 1 second because
         // it takes a second to load on macOS
         setTimeout(() => {
@@ -35,7 +34,7 @@ class Shuttle extends EventEmitter {
   }
 
   stop () {
-    usbDetect.stopMonitoring()
+    usb.unrefHotplugEvents()
     if (this._connected) {
       if (this._hid !== null) {
         this._hid.close()