Namespace child context under key "shortcuts"

Issue #1283
metasfresh · Nov 16, 2017 · 1e77a05 · 1e77a05
1 parent d69a136
commit 1e77a05
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 7 deletions.
diff --git a/src/components/Shortcuts/ShortcutProvider.js b/src/components/Shortcuts/ShortcutProvider.js
@@ -7,8 +7,10 @@ export default class ShortcutProvider extends Component {
     };
 
     static childContextTypes = {
-        subscribe: PropTypes.func.isRequired,
-        unsubscribe: PropTypes.func.isRequired
+        shortcuts: PropTypes.shape({
+            subscribe: PropTypes.func.isRequired,
+            unsubscribe: PropTypes.func.isRequired
+        })
     };
 
     hotkeys = {};
@@ -17,8 +19,10 @@ export default class ShortcutProvider extends Component {
 
     getChildContext() {
         return {
-            subscribe: this.subscribe,
-            unsubscribe: this.unsubscribe
+            shortcuts: {
+                subscribe: this.subscribe,
+                unsubscribe: this.unsubscribe
+            }
         };
     }
 

diff --git a/src/components/Shortcuts/ShortcutProvider.test.js b/src/components/Shortcuts/ShortcutProvider.test.js
@@ -7,7 +7,7 @@ import ShortcutProvider from './ShortcutProvider';
 
 chai.use(sinonChai);
 
-describe('Shortcuts', () => {
+describe('ShortcutProvider', () => {
     it('should be a React component', () => {
         expect(ShortcutProvider).to.be.an.instanceOf(Component.constructor);
     });
@@ -73,8 +73,10 @@ describe('Shortcuts', () => {
         const context = shortcutProvider.getChildContext();
 
         expect(context).to.deep.equal({
-            subscribe: shortcutProvider.subscribe,
-            unsubscribe: shortcutProvider.unsubscribe
+            shortcuts: {
+                subscribe: shortcutProvider.subscribe,
+                unsubscribe: shortcutProvider.unsubscribe
+            }
         });
     });