keyring - synchronous getState

app/scripts/keyring-controller.js

@@ -2,6 +2,7 @@ const ethUtil = require('ethereumjs-util')
 const BN = ethUtil.BN
 const bip39 = require('bip39')
 const EventEmitter = require('events').EventEmitter
+const extend = require('xtend')
 const ObservableStore = require('obs-store')
 const filter = require('promise-filter')
 const encryptor = require('browser-passworder')
@@ -32,6 +33,7 @@ class KeyringController extends EventEmitter {
     super()
     const initState = opts.initState || {}
     this.store = new ObservableStore(initState)
+    this.memStore = new ObservableStore({})
     this.configManager = opts.configManager
     this.ethStore = opts.ethStore
     this.encryptor = encryptor
@@ -74,30 +76,27 @@ class KeyringController extends EventEmitter {
   // in this class, but will need to be Promisified when we move our
   // persistence to an async model.
   getState () {
-    return Promise.all(this.keyrings.map(this.displayForKeyring))
+    const state = this.store.getState()
-    .then((displayKeyrings) => {
+    // old wallet
-      const state = this.store.getState()
+    const wallet = this.configManager.getWallet()
-      // old wallet
+    const memState = this.memStore.getState()
-      const wallet = this.configManager.getWallet()
+    return extend(memState, {
-      return {
+      // computed
-        // computed
+      isInitialized: (!!wallet || !!state.vault),
-        isInitialized: (!!wallet || !!state.vault),
+      isUnlocked: (!!this.password),
-        isUnlocked: (!!this.password),
+      // hard coded
-        keyrings: displayKeyrings,
+      keyringTypes: this.keyringTypes.map(krt => krt.type),
-        // hard coded
+      // memStore
-        keyringTypes: this.keyringTypes.map(krt => krt.type),
+      identities: this.identities,
-        // memStore
+      // configManager
-        identities: this.identities,
+      seedWords: this.configManager.getSeedWords(),
-        // configManager
+      isDisclaimerConfirmed: this.configManager.getConfirmedDisclaimer(),
-        seedWords: this.configManager.getSeedWords(),
+      currentFiat: this.configManager.getCurrentFiat(),
-        isDisclaimerConfirmed: this.configManager.getConfirmedDisclaimer(),
+      conversionRate: this.configManager.getConversionRate(),
-        currentFiat: this.configManager.getCurrentFiat(),
+      conversionDate: this.configManager.getConversionDate(),
-        conversionRate: this.configManager.getConversionRate(),
+      // messageManager
-        conversionDate: this.configManager.getConversionDate(),
+      unconfMsgs: messageManager.unconfirmedMsgs(),
-        // messageManager
+      messages: messageManager.getMsgList(),
-        unconfMsgs: messageManager.unconfirmedMsgs(),
-        messages: messageManager.getMsgList(),
-      }
     })
   }
 
@@ -164,6 +163,7 @@ class KeyringController extends EventEmitter {
   setLocked () {
     this.password = null
     this.keyrings = []
+    this._updateMemStoreKeyrings()
     return this.fullUpdate()
   }
 
@@ -639,6 +639,13 @@ class KeyringController extends EventEmitter {
     this.identities = {}
   }
 
+  _updateMemStoreKeyrings() {
+    Promise.all(this.keyrings.map(this.displayForKeyring))
+    .then((keyrings) => {
+      this.memStore.updateState({ keyrings })
+    })
+  }
+
 }
 
 module.exports = KeyringController

app/scripts/metamask-controller.js

@@ -169,22 +169,19 @@ module.exports = class MetamaskController extends EventEmitter {
   //
 
   getState () {
-    return this.keyringController.getState()
+    return extend(
-    .then((keyringControllerState) => {
+      this.state,
-      return extend(
+      this.ethStore.getState(),
-        this.state,
+      this.configManager.getConfig(),
-        this.ethStore.getState(),
+      this.txManager.getState(),
-        this.configManager.getConfig(),
+      this.keyringController.getState(),
-        this.txManager.getState(),
+      this.preferencesController.store.getState(),
-        keyringControllerState,
+      this.noticeController.getState(),
-        this.preferencesController.store.getState(),
+      {
-        this.noticeController.getState(),
+        shapeShiftTxList: this.configManager.getShapeShiftTxList(),
-        {
+        lostAccounts: this.configManager.getLostAccounts(),
-          shapeShiftTxList: this.configManager.getShapeShiftTxList(),
+      }
-          lostAccounts: this.configManager.getLostAccounts(),
+    )
-        }
-      )
-    })
   }
 
   //
@@ -199,7 +196,7 @@ module.exports = class MetamaskController extends EventEmitter {
 
     return {
       // etc
-      getState:              nodeify(this.getState.bind(this)),
+      getState:              (cb) => cb(null, this.getState()),
       setRpcTarget:          this.setRpcTarget.bind(this),
       setProviderType:       this.setProviderType.bind(this),
       useEtherscanProvider:  this.useEtherscanProvider.bind(this),
@@ -295,11 +292,8 @@ module.exports = class MetamaskController extends EventEmitter {
     )
   }
 
-  sendUpdate () {
+  sendUpdate () {  
-    this.getState()
+    this.emit('update', this.getState())
-    .then((state) => {
-      this.emit('update', state)
-    })
   }
 
   //

test/unit/idStore-migration-test.js

@@ -83,15 +83,10 @@ describe('IdentityStore to KeyringController migration', function() {
   describe('entering a password', function() {
     it('should identify an old wallet as an initialized keyring', function(done) {
       keyringController.configManager.setWallet('something')
-      keyringController.getState()
+      const state = keyringController.getState()
-      .then((state) => {
+      assert(state.isInitialized, 'old vault counted as initialized.')
-        assert(state.isInitialized, 'old vault counted as initialized.')
+      assert(!state.lostAccounts, 'no lost accounts')
-        assert(!state.lostAccounts, 'no lost accounts')
+      done()
-        done()
-      })
-      .catch((err) => {
-        done(err)
-      })
     })
   })
 })