Fixes broken seed overlay trigger

Resolves brave#14001 Auditors: Test Plan:
NejcZdovc · May 3, 2018 · 6f6fe9b · 6f6fe9b
1 parent 4ae26c3
commit 6f6fe9b
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 0 deletions.
diff --git a/app/browser/api/ledger.js b/app/browser/api/ledger.js
@@ -1688,6 +1688,7 @@ const getStateInfo = (state, parsedData) => {
     }
   } catch (e) {
     console.error(e)
+    state = ledgerState.setAboutProp(state, 'status', ledgerStatuses.CORRUPTED_SEED)
   }
 
   state = ledgerState.mergeInfoProp(state, newInfo)

diff --git a/test/unit/app/browser/api/ledgerTest.js b/test/unit/app/browser/api/ledgerTest.js
@@ -12,6 +12,7 @@ const appActions = require('../../../../../js/actions/appActions')
 const batPublisher = require('bat-publisher')
 const ledgerMediaProviders = require('../../../../../app/common/constants/ledgerMediaProviders')
 const fs = require('fs')
+const ledgerStatuses = require('../../../../../app/common/constants/ledgerStatuses')
 
 describe('ledger api unit tests', function () {
   let ledgerApi
@@ -161,6 +162,10 @@ describe('ledger api unit tests', function () {
       isReadyToReconcile: () => {}
     }
     window.getWalletPassphrase = (parsedData) => {
+      if (walletPassphraseReturn === 'error') {
+        throw TypeError('Invalid entropy')
+      }
+
       return walletPassphraseReturn
     }
     ledgerClient.prototype.boolion = function (value) { return false }
@@ -2002,6 +2007,29 @@ describe('ledger api unit tests', function () {
           }))
         assert.deepEqual(result.toJS(), expectedState.toJS())
       })
+
+      it('seed is broken', function () {
+        const wrongSeed = Immutable
+          .fromJS(seedData)
+          .setIn(['properties', 'wallet', 'keyinfo', 'seed'], Buffer.from([0, 0]))
+          .toJS()
+
+        walletPassphraseReturn = 'error'
+
+        const result = ledgerApi.getStateInfo(defaultAppState, wrongSeed)
+        const expectedState = defaultAppState
+          .setIn(['ledger', 'info'], Immutable.fromJS({
+            'created': true,
+            'creating': false,
+            'paymentId': '21951877-5998-4acf-9302-4a7b101c9188',
+            'reconcileFrequency': 30,
+            'reconcileStamp': 1
+          }))
+          .setIn(['ledger', 'about', 'status'], ledgerStatuses.CORRUPTED_SEED)
+
+        assert.deepEqual(result.toJS(), expectedState.toJS())
+        assert(getWalletPassphraseSpy.withArgs(wrongSeed).calledOnce)
+      })
     })
   })