Merge branch 'develop' into fix/bridgechain-ux

__tests__/unit/components/App/AppSidemenu/AppSidemenu.spec.js

@@ -113,11 +113,19 @@ describe('AppSidemenu', () => {
     expect(wrapper.vm.isImportantNotificationVisible).toBe(false)
   })
 
-  it('should toggle & close plugin menu state', () => {
-    expect(wrapper.vm.isPluginMenuVisible).toBe(false)
-    wrapper.vm.toggleShowPluginMenu()
-    expect(wrapper.vm.isPluginMenuVisible).toBe(true)
-    wrapper.vm.closeShowPlugins()
-    expect(wrapper.vm.isPluginMenuVisible).toBe(false)
+  describe('Plugin Menu', () => {
+    it('should toggle & close plugin menu state', () => {
+      expect(wrapper.vm.isPluginMenuVisible).toBe(false)
+      wrapper.vm.toggleShowPluginMenu()
+      expect(wrapper.vm.isPluginMenuVisible).toBe(true)
+      wrapper.vm.closeShowPlugins()
+      expect(wrapper.vm.isPluginMenuVisible).toBe(false)
+    })
+
+    it('should activate the plugin menu item when \'setActive\' is true', () => {
+      expect(wrapper.vm.activeItem).toBe('mock_route')
+      wrapper.vm.closeShowPlugins(true)
+      expect(wrapper.vm.activeItem).toBe('plugin-pages')
+    })
   })
 })

__tests__/unit/components/Input/InputFee.spec.js

@@ -22,17 +22,21 @@ jest.mock('@/store', () => {
             return null
         }
       }),
-      'session/lastFeeByType': jest.fn(type => {
-        switch (type) {
-          case 0:
-            return 10000000
-          case 1:
-            return undefined
-          case 3:
-            return 100000000
-          default:
-            return null
+      'session/lastFeeByType': jest.fn((type, typeGroup) => {
+        if (typeGroup === 1) {
+          switch (type) {
+            case 0:
+              return 10000000
+            case 1:
+              return undefined
+            case 3:
+              return 100000000
+            default:
+              return null
+          }
         }
+
+        return null
       })
     }
   }
@@ -125,6 +129,18 @@ describe('InputFee', () => {
     expect(buttons).toHaveLength(5)
   })
 
+  it('should set the default chosen fee if available when created', () => {
+    const wrapper = mountComponent({
+      mocks: {
+        session_profile: {
+          defaultChosenFee: 'LAST'
+        }
+      }
+    })
+
+    expect(wrapper.vm.chosenFee).toBe('LAST')
+  })
+
   describe('maxV1fee', () => {
     it('should uses V1 configuration', () => {
       let wrapper = mountComponent({

__tests__/unit/components/PluginManager/PluginManagerGrid.spec.js

@@ -6,7 +6,26 @@ const i18n = useI18nGlobally()
 let wrapper
 
 const plugins = [
-  { id: 'test' }
+  {
+    id: 'test 4',
+    title: 'plugin 4'
+  },
+  {
+    id: 'test 3',
+    title: 'plugin 3'
+  },
+  {
+    id: 'test 2',
+    title: 'plugin 2'
+  },
+  {
+    id: 'test 1',
+    title: 'plugin 1'
+  },
+  {
+    id: 'first test',
+    title: 'first plugin'
+  }
 ]
 
 beforeEach(() => {
@@ -38,4 +57,17 @@ describe('PluginManagerGrid', () => {
       expect(wrapper.emitted('show-details', plugins[0])).toBeTruthy()
     })
   })
+
+  describe('computed', () => {
+    it('sortedPlugins', () => {
+      expect(wrapper.vm.sortedPlugins[0].title).toBe('first plugin')
+      expect(wrapper.vm.sortedPlugins).toEqual([
+        plugins[4],
+        plugins[3],
+        plugins[2],
+        plugins[1],
+        plugins[0]
+      ])
+    })
+  })
 })

__tests__/unit/components/Transaction/TransactionConfirm/TransactionConfirmMultiPayment.spec.js

@@ -1,5 +1,6 @@
 import { createLocalVue, mount } from '@vue/test-utils'
 import installI18n from '../../../__utils__/i18n'
+import CurrencyMixin from '@/mixins/currency'
 import TransactionConfirmMultiPayment from '@/components/Transaction/TransactionConfirm/TransactionConfirmMultiPayment'
 
 const localVue = createLocalVue()
@@ -34,6 +35,7 @@ const createWrapper = (component, transaction) => {
       transaction
     },
     mocks: {
+      currency_toBuilder: jest.fn(CurrencyMixin.methods.currency_toBuilder),
       formatter_networkCurrency: jest.fn((amount) => amount),
       wallet_formatAddress: jest.fn((address) => `formatted-${address}`),
       wallet_name: jest.fn(wallet => wallet)

__tests__/unit/services/plugin-manager/sandbox/storage-sandbox.spec.js

@@ -22,11 +22,15 @@ beforeAll(() => {
         }),
         'session/profileId': '1'
       },
-      dispatch: jest.fn((_, data) => {
-        if (!db[data.profileId]) {
-          db[data.profileId] = {}
+      dispatch: jest.fn((action, data) => {
+        if (action === 'plugin/setPluginOption') {
+          if (!db[data.profileId]) {
+            db[data.profileId] = {}
+          }
+          db[data.profileId][data.key] = data.value
+        } else if (action === 'plugin/deletePluginOptionsForProfile') {
+          delete db[data.profileId]
         }
-        db[data.profileId][data.key] = data.value
       })
     }
   }
@@ -55,54 +59,80 @@ describe('Storage Sandbox', () => {
     expect(walletApi.storage.getOptions).toBeTruthy()
     expect(walletApi.storage.get).toBeTruthy()
     expect(walletApi.storage.set).toBeTruthy()
+    expect(walletApi.storage.clear).toBeTruthy()
   })
 
-  it('should set a value to key', () => {
-    walletApi.storage.set(localOptions.key, localOptions.value)
-    expect(app.$store.dispatch).toHaveBeenCalledWith('plugin/setPluginOption', localOptions)
-  })
-
-  it('should set a value to global key', () => {
-    walletApi.storage.set(globalOptions.key, globalOptions.value, true)
-    expect(app.$store.dispatch).toHaveBeenCalledWith('plugin/setPluginOption', globalOptions)
-  })
-
-  it('should get the value of key', () => {
-    const result = walletApi.storage.get(localOptions.key)
-    expect(app.$store.getters['plugin/pluginOptions']).toHaveBeenCalledWith(plugin.config.id, '1')
-    expect(result).toBe(localOptions.value)
-  })
-
-  it('should get the value of global key from the same profile', () => {
-    const result = walletApi.storage.get(globalOptions.key, true)
-    expect(result).toBe(globalOptions.value)
-  })
+  describe('set', () => {
+    it('should set a value to key', () => {
+      walletApi.storage.set(localOptions.key, localOptions.value)
+      expect(app.$store.dispatch).toHaveBeenCalledWith('plugin/setPluginOption', localOptions)
+    })
 
-  it('should get the value of global key from a different profile', () => {
-    app.$store.getters['session/profileId'] = '2'
-    const result = walletApi.storage.get(globalOptions.key, true)
-    expect(result).toBe(globalOptions.value)
+    it('should set a value to global key', () => {
+      walletApi.storage.set(globalOptions.key, globalOptions.value, true)
+      expect(app.$store.dispatch).toHaveBeenCalledWith('plugin/setPluginOption', globalOptions)
+    })
   })
 
-  it('should NOT get the value of a local key from a different profile', () => {
-    app.$store.getters['session/profileId'] = '2'
-    const result = walletApi.storage.get(localOptions.key)
-    expect(result).toBe(undefined)
+  describe('get', () => {
+    it('should get the value of key', () => {
+      const result = walletApi.storage.get(localOptions.key)
+      expect(app.$store.getters['plugin/pluginOptions']).toHaveBeenCalledWith(plugin.config.id, '1')
+      expect(result).toBe(localOptions.value)
+    })
+
+    it('should get the value of global key from the same profile', () => {
+      const result = walletApi.storage.get(globalOptions.key, true)
+      expect(result).toBe(globalOptions.value)
+    })
+
+    it('should get the value of global key from a different profile', () => {
+      app.$store.getters['session/profileId'] = '2'
+      const result = walletApi.storage.get(globalOptions.key, true)
+      expect(result).toBe(globalOptions.value)
+    })
+
+    it('should NOT get the value of a local key from a different profile', () => {
+      app.$store.getters['session/profileId'] = '2'
+      const result = walletApi.storage.get(localOptions.key)
+      expect(result).toBe(undefined)
+    })
   })
 
-  it('should get all local values', () => {
-    app.$store.getters['session/profileId'] = '1'
-    let result = walletApi.storage.getOptions()
-    expect(Object.keys(result)).toHaveLength(1)
-    expect(result).toHaveProperty(localOptions.key, localOptions.value)
-    app.$store.getters['session/profileId'] = '2'
-    result = walletApi.storage.getOptions()
-    expect(Object.keys(result)).toHaveLength(0)
+  describe('getOptions', () => {
+    it('should get all local values', () => {
+      app.$store.getters['session/profileId'] = '1'
+      let result = walletApi.storage.getOptions()
+      expect(Object.keys(result)).toHaveLength(1)
+      expect(result).toHaveProperty(localOptions.key, localOptions.value)
+      app.$store.getters['session/profileId'] = '2'
+      result = walletApi.storage.getOptions()
+      expect(Object.keys(result)).toHaveLength(0)
+    })
+
+    it('should get all global values', () => {
+      const result = walletApi.storage.getOptions(true)
+      expect(Object.keys(result)).toHaveLength(1)
+      expect(result).toHaveProperty(globalOptions.key, globalOptions.value)
+    })
   })
 
-  it('should get all global values', () => {
-    const result = walletApi.storage.getOptions(true)
-    expect(Object.keys(result)).toHaveLength(1)
-    expect(result).toHaveProperty(globalOptions.key, globalOptions.value)
+  describe('clear', () => {
+    beforeAll(() => {
+      walletApi.storage.set(localOptions.key, localOptions.value)
+      walletApi.storage.set(globalOptions.key, globalOptions.value, true)
+    })
+
+    it('should clear all local values', () => {
+      expect(walletApi.storage.get(localOptions.key)).toBe(localOptions.value)
+      walletApi.storage.clear()
+      expect(walletApi.storage.get(localOptions.key)).toBe(undefined)
+    })
+
+    it('should clear all global values', () => {
+      expect(walletApi.storage.get(globalOptions.key, true)).toBe(globalOptions.value)
+      walletApi.storage.clear(true)
+      expect(walletApi.storage.get(globalOptions.key, true)).toBe(undefined)
+    })
   })
 })

__tests__/unit/services/plugin-manager/setup/menu-items-setup.spec.js

@@ -9,7 +9,7 @@ const plugin = new Plugin({
 
 plugin.routes = [
   {
-    name: 'test'
+    name: '1:test'
   }
 ]