Merge pull request #170 from LinusU/expose-is-open

Expose isOpen function on Menu component

Author: sodik82

__tests__/Menu-test.js

@@ -185,7 +185,7 @@ describe('Menu', () => {
       <Menu opened={false}/>
     );
     instance._setOpened(true);
-    expect(instance._isOpen()).toEqual(false);
+    expect(instance.isOpen()).toEqual(false);
     expect(instance._getOpened()).toEqual(true);
   });
 
@@ -194,7 +194,7 @@ describe('Menu', () => {
       <Menu/>
     );
     instance._setOpened(true);
-    expect(instance._isOpen()).toEqual(true);
+    expect(instance.isOpen()).toEqual(true);
   });
 
   it('should be considered closed after unmount', () => {
@@ -204,9 +204,9 @@ describe('Menu', () => {
         <MenuOptions />
       </Menu>
     );
-    expect(instance._isOpen()).toEqual(true);
+    expect(instance.isOpen()).toEqual(true);
     instance.componentWillUnmount();
-    expect(instance._isOpen()).toEqual(false);
+    expect(instance.isOpen()).toEqual(false);
     expect(ctx.menuActions._notify).toHaveBeenCalled();
   });
 

__tests__/MenuProvider-test.js

@@ -34,9 +34,9 @@ describe('MenuProvider', () => {
     let opened = false;
     return {
       getName: ()=>name,
+      isOpen: ()=>opened,
       _getOpened: ()=>opened,
       _setOpened: (value)=>opened=value,
-      _isOpen: ()=>opened,
       _getTrigger: ()=>(<MenuTrigger/>),
       _getOptions: ()=>(<MenuOptions/>),
       props : {
@@ -180,10 +180,10 @@ describe('MenuProvider', () => {
     menuRegistry.subscribe(menu1);
     return menuActions.toggleMenu('menu1').then(() => {
       expect(menuActions.isMenuOpen()).toEqual(true);
-      expect(menu1._isOpen()).toEqual(true);
+      expect(menu1.isOpen()).toEqual(true);
       return menuActions.toggleMenu('menu1').then(() => {
         expect(menuActions.isMenuOpen()).toEqual(false);
-        expect(menu1._isOpen()).toEqual(false);
+        expect(menu1.isOpen()).toEqual(false);
         return menuActions.toggleMenu('menu1').then(() => {
           expect(menuActions.isMenuOpen()).toEqual(true);
         });