Skip to content

Commit 865b7d0

Browse files
committed
Fix tests using isOpen
1 parent 068bf67 commit 865b7d0

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

__tests__/Menu-test.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,7 @@ describe('Menu', () => {
185185
<Menu opened={false}/>
186186
);
187187
instance._setOpened(true);
188-
expect(instance._isOpen()).toEqual(false);
188+
expect(instance.isOpen()).toEqual(false);
189189
expect(instance._getOpened()).toEqual(true);
190190
});
191191

@@ -194,7 +194,7 @@ describe('Menu', () => {
194194
<Menu/>
195195
);
196196
instance._setOpened(true);
197-
expect(instance._isOpen()).toEqual(true);
197+
expect(instance.isOpen()).toEqual(true);
198198
});
199199

200200
it('should be considered closed after unmount', () => {
@@ -204,9 +204,9 @@ describe('Menu', () => {
204204
<MenuOptions />
205205
</Menu>
206206
);
207-
expect(instance._isOpen()).toEqual(true);
207+
expect(instance.isOpen()).toEqual(true);
208208
instance.componentWillUnmount();
209-
expect(instance._isOpen()).toEqual(false);
209+
expect(instance.isOpen()).toEqual(false);
210210
expect(ctx.menuActions._notify).toHaveBeenCalled();
211211
});
212212

__tests__/MenuProvider-test.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,9 @@ describe('MenuProvider', () => {
3434
let opened = false;
3535
return {
3636
getName: ()=>name,
37+
isOpen: ()=>opened,
3738
_getOpened: ()=>opened,
3839
_setOpened: (value)=>opened=value,
39-
_isOpen: ()=>opened,
4040
_getTrigger: ()=>(<MenuTrigger/>),
4141
_getOptions: ()=>(<MenuOptions/>),
4242
props : {
@@ -180,10 +180,10 @@ describe('MenuProvider', () => {
180180
menuRegistry.subscribe(menu1);
181181
return menuActions.toggleMenu('menu1').then(() => {
182182
expect(menuActions.isMenuOpen()).toEqual(true);
183-
expect(menu1._isOpen()).toEqual(true);
183+
expect(menu1.isOpen()).toEqual(true);
184184
return menuActions.toggleMenu('menu1').then(() => {
185185
expect(menuActions.isMenuOpen()).toEqual(false);
186-
expect(menu1._isOpen()).toEqual(false);
186+
expect(menu1.isOpen()).toEqual(false);
187187
return menuActions.toggleMenu('menu1').then(() => {
188188
expect(menuActions.isMenuOpen()).toEqual(true);
189189
});

0 commit comments

Comments
 (0)