feat: change autodetect output to a function

Author: reconbot

lib/bindings.test.ts

@@ -1,7 +1,7 @@
 import { assert, shouldReject } from '../test/assert'
 import { makeTestFeature } from '../test/makeTestFeature'
 import { BindingInterface, OpenOptions, PortInfo, SetOptions } from './binding-interface'
-import Binding, { AllBindingClasses } from './index'
+import { autoDetect, AllBindingClasses } from './index'
 import MockBinding from '@serialport/binding-mock'
 
 const defaultOpenOptions: OpenOptions = Object.freeze({
@@ -37,7 +37,7 @@ const listFields = ['path', 'manufacturer', 'serialNumber', 'pnpId', 'locationId
 const readyData = Buffer.from('READY')
 
 testBinding('mock', MockBinding, '/dev/exists')
-testBinding(process.platform, Binding, process.env.TEST_PORT)
+testBinding(process.platform, autoDetect(), process.env.TEST_PORT)
 
 function testBinding(bindingName: string, Binding: AllBindingClasses, testPort?: string) {
   const { testFeature, testHardware, describeHardware } = makeTestFeature(bindingName, testPort)
@@ -52,7 +52,6 @@ function testBinding(bindingName: string, Binding: AllBindingClasses, testPort?:
     describe('static method', () => {
       describe('.list', () => {
         it('returns an array', async () => {
-          console.log({ Binding })
           const ports = await Binding.list()
           assert.isArray(ports)
         })

lib/index.ts

@@ -12,19 +12,20 @@ export * from './binding-interface'
 
 export type AllBindingClasses = typeof WindowsBinding | typeof DarwinBinding | typeof LinuxBinding
 export type AllBindings = WindowsBinding | DarwinBinding | LinuxBinding
-let binding: AllBindingClasses
-switch (process.platform) {
-case 'win32':
-  debug('loading WindowsBinding')
-  binding = WindowsBinding
-  break
-case 'darwin':
-  debug('loading DarwinBinding')
-  binding = DarwinBinding
-  break
-default:
-  debug('loading LinuxBinding')
-  binding = LinuxBinding
-}
 
-export default binding
+/**
+ * This is an auto detected binding for your current platform
+ */
+export const autoDetect = (): AllBindingClasses => {
+  switch (process.platform) {
+  case 'win32':
+    debug('loading WindowsBinding')
+    return WindowsBinding
+  case 'darwin':
+    debug('loading DarwinBinding')
+    return DarwinBinding
+  default:
+    debug('loading LinuxBinding')
+    return LinuxBinding
+  }
+}