From 0ca3431ec342e9ad21d31af919f4d0028c8e1057 Mon Sep 17 00:00:00 2001 From: Paul Kaplan Date: Wed, 30 Jun 2021 11:26:54 -0400 Subject: [PATCH] test: fix the new test for studio removeManager edge case --- test/unit/redux/studio-member-actions.test.js | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/test/unit/redux/studio-member-actions.test.js b/test/unit/redux/studio-member-actions.test.js index 1738b1bc5..f242de702 100644 --- a/test/unit/redux/studio-member-actions.test.js +++ b/test/unit/redux/studio-member-actions.test.js @@ -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 () => {