Merge pull request ProjectMirador#81 from ProjectMirador/config-pattern

Adds the ability to add and update configuration/settings

minimal_redux_poc/__tests__/integration/mirador/config_updating_from_instance.test.js

@@ -0,0 +1,17 @@
+/* global miradorInstance */
+
+describe('Config updating from instance', () => {
+  beforeAll(async () => {
+    await page.goto('http://127.0.0.1:4488/__tests__/integration/mirador/');
+  });
+  it('can modify the config via api', async () => {
+    await page.evaluate(() => {
+      const a = miradorInstance.actions.updateConfig({ foo: 'bat' });
+      miradorInstance.store.dispatch(a);
+    });
+    const config = await page.evaluate(() => (
+      miradorInstance.store.getState().config
+    ));
+    await expect(config.foo).toBe('bat');
+  });
+});

minimal_redux_poc/__tests__/integration/mirador/index.html

@@ -10,7 +10,7 @@
     <div id="mirador"></div>
     <script>document.write("<script type='text/javascript' src='../../../dist/mirador.min.js?v=" + Date.now() + "'><\/script>");</script>
     <script type="text/javascript">
-      Mirador({
+      var miradorInstance = Mirador({
         id: 'mirador',
         plugins: ['HelloWorld']
       });

minimal_redux_poc/__tests__/src/reducers/config.test.js

@@ -0,0 +1,42 @@
+import reducer from '../../../src/reducers/config';
+import ActionTypes from '../../../src/action-types';
+
+describe('config reducer', () => {
+  describe('SET_CONFIG', () => {
+    it('should handle SET_CONFIG', () => {
+      expect(reducer({}, {
+        type: ActionTypes.SET_CONFIG,
+        config: { manifestVersion: 'v3' },
+      })).toEqual({
+        manifestVersion: 'v3',
+      });
+    });
+    it('does not deep merge', () => {
+      expect(reducer({ stuff: { foo: 'bar' } }, {
+        type: ActionTypes.SET_CONFIG,
+        config: { stuff: { foo: 'bat' } },
+      })).toEqual({
+        stuff: { foo: 'bat' },
+      });
+    });
+  });
+  describe('UPDATE_CONFIG', () => {
+    it('should handle UPDATE_CONFIG', () => {
+      expect(reducer({}, {
+        type: ActionTypes.UPDATE_CONFIG,
+        config: { manifestVersion: 'v3' },
+      })).toEqual({
+        manifestVersion: 'v3',
+      });
+    });
+    it('does a deep merge', () => {
+      expect(reducer({ stuff: { foo: 'bar' }, hello: 'world' }, {
+        type: ActionTypes.UPDATE_CONFIG,
+        config: { stuff: { foo: 'bat' } },
+      })).toEqual({
+        stuff: { foo: 'bat' },
+        hello: 'world',
+      });
+    });
+  });
+});