feat: all the tests

Author: nicosantangelo

src/ConnectionManager.ts

@@ -94,10 +94,10 @@ export class ConnectionManager {
     chainId: ChainId
   ): AbstractConnector {
     switch (providerType) {
-      case ProviderType.INJECTED:
-        return new InjectedConnector(chainId)
       case ProviderType.FORTMATIC:
         return new FortmaticConnector(chainId)
+      case ProviderType.INJECTED:
+        return new InjectedConnector(chainId)
       case ProviderType.WALLET_CONNECT:
         return new WalletConnectConnector(chainId)
       default:

test/ConnectionManager.spec.ts

@@ -3,6 +3,11 @@ import chaiAsPromised from 'chai-as-promised'
 import sinon from 'sinon'
 import { getConfiguration } from '../src/configuration'
 import { ConnectionManager, connection } from '../src/ConnectionManager'
+import {
+  FortmaticConnector,
+  InjectedConnector,
+  WalletConnectConnector
+} from '../src/connectors'
 import { LocalStorage } from '../src/storage'
 import { ChainId, ClosableConnector, ProviderType } from '../src/types'
 import { StubClosableConnector, StubConnector, StubStorage } from './utils'
@@ -238,6 +243,7 @@ describe('ConnectionManager', () => {
     })
 
     it('should throw if no successful connect occurred', () => {
+      connectionManager.connector = undefined
       expect(connectionManager.getProvider()).to.eventually.throw(
         new Error('No valid connector found. Please .connect() first')
       )
@@ -265,4 +271,40 @@ describe('ConnectionManager', () => {
       browser.window = undefined
     })
   })
+
+  describe('#getConnector', () => {
+    it('should throw if an invalid provider type is supplied', () => {
+      const providerType = 'Invalid Provider Type' as any
+      expect(() =>
+        connectionManager.getConnector(providerType, ChainId.MAINNET)
+      ).to.throw(`Invalid provider ${providerType}`)
+    })
+
+    it('should return an instance of FortmaticConnector for the supplied chain', () => {
+      const connector = connectionManager.getConnector(
+        ProviderType.FORTMATIC,
+        ChainId.KOVAN
+      )
+      expect(connector).to.be.instanceOf(FortmaticConnector)
+      expect(connector.getChainId()).to.eventually.eq(ChainId.KOVAN)
+    })
+
+    it('should return an instance of InjectedConnector for the supplied chain', () => {
+      const connector = connectionManager.getConnector(
+        ProviderType.INJECTED,
+        ChainId.KOVAN
+      )
+      expect(connector).to.be.instanceOf(InjectedConnector)
+      expect(connector.getChainId()).to.eventually.eq(ChainId.KOVAN)
+    })
+
+    it('should return an instance of WalletConnectConnector for the supplied chain', () => {
+      const connector = connectionManager.getConnector(
+        ProviderType.WALLET_CONNECT,
+        ChainId.KOVAN
+      )
+      expect(connector).to.be.instanceOf(WalletConnectConnector)
+      expect(connector.getChainId()).to.eventually.eq(ChainId.KOVAN)
+    })
+  })
 })

test/storage.spec.ts

@@ -7,7 +7,8 @@ describe('LocalStorage', () => {
   const browser: any = global
   const windowLocalStorage = {
     getItem: (_key: string) => {},
-    setItem: (_key: string, _value: any) => {}
+    setItem: (_key: string, _value: any) => {},
+    clear: () => {}
   }
   let mockStorage: Sinon.SinonMock
 
@@ -67,4 +68,13 @@ describe('LocalStorage', () => {
       localStorage.set(key, value)
     })
   })
+
+  describe('#clear', () => {
+    it('should call the window localStorage clear method', () => {
+      mockStorage.expects('clear').once()
+
+      const localStorage = new LocalStorage()
+      localStorage.clear()
+    })
+  })
 })