From c2046be0d8606ded9a328f392baf62cad802b98f Mon Sep 17 00:00:00 2001 From: Dan Finlay Date: Wed, 30 Mar 2016 19:15:49 -0700 Subject: Made configuration migrateable Abstract all configuration data into a singleton called `configManager`, who is responsible for reading and writing to the persisted storage (localStorage, in our case). Uses my new module [pojo-migrator](https://www.npmjs.com/package/pojo-migrator), and wraps it with the `ConfigManager` class, which we can hang any state setting or getting methods we need. By keeping all the persisted state in one place, we can stabilize its outward-facing API, making the interactions increasingly atomic, which will allow us to add features that require restructuring the persisted data in the long term without having to rewrite UI or even `background.js` code. All the restructuring and data-type management is kept in one neat little place. This should make it very easy to add new configuration options like user-configured providers, per-domain vaults, and more! I know this doesn't seem like a big user-facing feature, but we have a big laundry list of features that I think this will really help streamline. --- package.json | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'package.json') diff --git a/package.json b/package.json index f482fd4f6..be61f1d14 100644 --- a/package.json +++ b/package.json @@ -5,7 +5,8 @@ "private": true, "scripts": { "start": "gulp dev", - "test": "mocha --compilers js:babel-register --recursive" + "test": "mocha --require test/helper.js --compilers js:babel-register --recursive", + "watch": "mocha watch --compilers js:babel-register --recursive" }, "dependencies": { "async": "^1.5.2", @@ -21,6 +22,7 @@ "inject-css": "^0.1.1", "metamask-ui": "^1.5.0", "multiplex": "^6.7.0", + "pojo-migrator": "^2.1.0", "pumpify": "^1.3.4", "readable-stream": "^2.0.5", "through2": "^2.0.1", @@ -39,10 +41,10 @@ "gulp-util": "^3.0.7", "gulp-watch": "^4.3.5", "jsdom": "^8.1.0", + "jsdom-global": "^1.7.0", "jshint-stylish": "~0.1.5", "lodash.assign": "^4.0.6", "mocha": "^2.4.5", - "mocha-jsdom": "^1.1.0", "mocha-sinon": "^1.1.5", "sinon": "^1.17.3", "vinyl-buffer": "^1.0.0", -- cgit v1.2.3