scratchfoundation/scratch-www
1
0
Fork 0
mirror of https://github.com/scratchfoundation/scratch-www.git synced 2024-11-23 15:47:53 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

test: fix the new test for studio removeManager edge case

Browse source
This commit is contained in:
Paul Kaplan 2021-06-30 11:26:54 -04:00
parent e66a31abd8
commit 0ca3431ec3
1 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
test/unit/redux/studio-member-actions.test.js
View file

@ -186,12 +186,11 @@ describe('removeManager', () => {
const state = store.getState();
// Manager count should still be updated
expect(selectStudioManagerCount(state)).toBe(98);
expect(selectStudioManagerCount(state)).toBe(2);
// The removed manager isn't the current user, so manager permission should be unchanged
expect(state.studio.manager).toBe(false);
expect(state.studio.manager).toBe(true);
// No change to the manager items list
expect(managers.selector(state).items.length).toBe(1);
expect(managers.selector(state).items[0].username).toBe('user1');
expect(managers.selector(state).items.length).toBe(3);
});
test('on error, promise rejects without any changing count or list', async () => {