Merge pull request #526 from ethereumjs/pass-common-statemanager-copy

Pass the Common object when copying a StateManager
ethereumjs · May 28, 2019 · 176058b · 176058b
2 parents 2e2e509 + 5c06175
commit 176058b
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/lib/state/stateManager.ts b/lib/state/stateManager.ts
@@ -58,7 +58,7 @@ export default class StateManager {
    * @method copy
    */
   copy(): StateManager {
-    return new StateManager({ trie: this._trie.copy() })
+    return new StateManager({ trie: this._trie.copy(), common: this._common })
   }
 
   /**

diff --git a/tests/api/state/stateManager.js b/tests/api/state/stateManager.js
@@ -1,6 +1,7 @@
 const promisify = require('util.promisify')
 const tape = require('tape')
 const util = require('ethereumjs-util')
+const Common = require('ethereumjs-common').default
 const { StateManager } = require('../../../dist/state')
 const { createAccount } = require('../utils')
 
@@ -160,4 +161,19 @@ tape('StateManager', (t) => {
       st.end()
     })
   })
+
+  t.test('should pass Common object when copying the state manager', st => {
+    const stateManager = new StateManager({
+      common: new Common('goerli', 'byzantium')
+    })
+
+    st.equal(stateManager._common.chainName(), 'goerli')
+    st.equal(stateManager._common.hardfork(), 'byzantium')
+
+    const stateManagerCopy = stateManager.copy()
+    st.equal(stateManagerCopy._common.chainName(), 'goerli')
+    st.equal(stateManagerCopy._common.hardfork(), 'byzantium')
+
+    st.end()
+  })
 })