Skip to content

Commit 884788a

Browse files
committed
refactor(mfa): rename idle input state
1 parent 33e37cf commit 884788a

4 files changed

Lines changed: 8 additions & 8 deletions

File tree

src/components/MultifactorAuthentication/machine/mfaMachine.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -226,21 +226,21 @@ const MFAMachine = setup({
226226
// while the challenge request is in flight is dropped instead of emailing a
227227
// code the pending submission ignores.
228228
[MFA_STATE.AWAITING_VALIDATE_CODE]: {
229-
initial: MFA_STATE.IDLE,
229+
initial: MFA_STATE.AWAITING_INPUT,
230230
on: {
231231
VALIDATE_CODE_ENTERED: {target: MFA_STATE.REQUESTING_REGISTRATION_CHALLENGE, actions: 'submitValidateCode'},
232-
RESEND_VALIDATE_CODE: {target: `.${MFA_STATE.IDLE}`, actions: 'requestValidateCode'},
232+
RESEND_VALIDATE_CODE: {target: `.${MFA_STATE.AWAITING_INPUT}`, actions: 'requestValidateCode'},
233233
},
234234
states: {
235-
[MFA_STATE.IDLE]: {},
235+
[MFA_STATE.AWAITING_INPUT]: {},
236236
// The backend rejected the submitted code. The screen shows the
237237
// inline error exactly while this state is active, so every way out
238238
// (typing, a resend, a new submission) drops the error by
239239
// construction and nothing stale can outlive the screen.
240240
[MFA_STATE.INVALID_CODE]: {
241241
tags: 'showsInvalidCodeError',
242242
on: {
243-
VALIDATE_CODE_CHANGED: MFA_STATE.IDLE,
243+
VALIDATE_CODE_CHANGED: MFA_STATE.AWAITING_INPUT,
244244
},
245245
},
246246
},

src/libs/MultifactorAuthentication/shared/VALUES.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -225,7 +225,7 @@ const MFA_STATE = {
225225
CHECKING_SOFT_PROMPT_ACCEPTANCE: 'checkingSoftPromptAcceptance',
226226
MAGIC_CODE: 'magicCode',
227227
AWAITING_VALIDATE_CODE: 'awaitingValidateCode',
228-
IDLE: 'idle',
228+
AWAITING_INPUT: 'awaitingInput',
229229
INVALID_CODE: 'invalidCode',
230230
REQUESTING_REGISTRATION_CHALLENGE: 'requestingRegistrationChallenge',
231231
PROMPT: 'prompt',

tests/unit/components/MultifactorAuthentication/machine/graphTraversal/viewMatchesMachine.test.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,7 @@ const testConfig = {
218218
expect(screen.getByText(translateLocal('multifactorAuthentication.letsVerifyItsYou'))).toBeOnTheScreen();
219219
expect(state.context.error).toBeUndefined();
220220
},
221-
[`${MFA_STATE.OPEN}.${MFA_STATE.MAGIC_CODE}.${MFA_STATE.AWAITING_VALIDATE_CODE}.${MFA_STATE.IDLE}`]: () => {
221+
[`${MFA_STATE.OPEN}.${MFA_STATE.MAGIC_CODE}.${MFA_STATE.AWAITING_VALIDATE_CODE}.${MFA_STATE.AWAITING_INPUT}`]: () => {
222222
expect(screen.queryByText(translateLocal('validateCodeForm.error.incorrectSecurityCode'))).not.toBeOnTheScreen();
223223
},
224224
[`${MFA_STATE.OPEN}.${MFA_STATE.MAGIC_CODE}.${MFA_STATE.AWAITING_VALIDATE_CODE}.${MFA_STATE.INVALID_CODE}`]: () => {

tests/unit/components/MultifactorAuthentication/machine/validateCodeTransition.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@ describe('MFA magic code and registration decision', () => {
122122
actor.send({type: 'RESEND_VALIDATE_CODE'});
123123

124124
const result = actor.getSnapshot();
125-
expect(result.matches({[MFA_STATE.OPEN]: {[MFA_STATE.MAGIC_CODE]: {[MFA_STATE.AWAITING_VALIDATE_CODE]: MFA_STATE.IDLE}}})).toBe(true);
125+
expect(result.matches({[MFA_STATE.OPEN]: {[MFA_STATE.MAGIC_CODE]: {[MFA_STATE.AWAITING_VALIDATE_CODE]: MFA_STATE.AWAITING_INPUT}}})).toBe(true);
126126
expect(result.hasTag('showsInvalidCodeError')).toBe(false);
127127
expect(requestValidateCodeActionMock).toHaveBeenCalledTimes(1);
128128

@@ -249,7 +249,7 @@ describe('MFA magic code and registration decision', () => {
249249
actor.send({type: 'VALIDATE_CODE_CHANGED'});
250250

251251
const result = actor.getSnapshot();
252-
expect(result.matches({[MFA_STATE.OPEN]: {[MFA_STATE.MAGIC_CODE]: {[MFA_STATE.AWAITING_VALIDATE_CODE]: MFA_STATE.IDLE}}})).toBe(true);
252+
expect(result.matches({[MFA_STATE.OPEN]: {[MFA_STATE.MAGIC_CODE]: {[MFA_STATE.AWAITING_VALIDATE_CODE]: MFA_STATE.AWAITING_INPUT}}})).toBe(true);
253253
expect(result.hasTag('showsInvalidCodeError')).toBe(false);
254254

255255
actor.stop();

0 commit comments

Comments
 (0)