Commit c882d163 authored by peterhegman's avatar peterhegman

Refactor spec to use `describe.each`

Per reviewer feedback
parent 7c57de4e
...@@ -60,27 +60,19 @@ describe('Members Utils', () => { ...@@ -60,27 +60,19 @@ describe('Members Utils', () => {
}); });
describe('when `canManageMembers` argument is `false`', () => { describe('when `canManageMembers` argument is `false`', () => {
describe('when member is not the current user', () => { describe.each`
it('sets `show` to `false` for 2FA badge', () => { description | memberIsCurrentUser | expectedBadgeToBeShown
${'is not the current user'} | ${false} | ${false}
${'is the current user'} | ${true} | ${true}
`('when member is $description', ({ memberIsCurrentUser, expectedBadgeToBeShown }) => {
it(`sets 'show' to '${expectedBadgeToBeShown}' for 2FA badge`, () => {
const badges = generateBadges({ const badges = generateBadges({
member: member2faEnabled, member: member2faEnabled,
isCurrentUser: false, isCurrentUser: memberIsCurrentUser,
canManageMembers: false, canManageMembers: false,
}); });
expect(badges.find((badge) => badge.text === '2FA').show).toBe(false); expect(badges.find((badge) => badge.text === '2FA').show).toBe(expectedBadgeToBeShown);
});
});
describe('when member is the current user', () => {
it('sets `show` to `false` for 2fA badge', () => {
const badges = generateBadges({
member: member2faEnabled,
isCurrentUser: true,
canManageMembers: false,
});
expect(badges.find((badge) => badge.text === '2FA').show).toBe(true);
}); });
}); });
}); });
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment