New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#165273526: Password reset enhancement #29
#165273526: Password reset enhancement #29
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some files could not be reviewed due to errors:
The configured stylelint version is not supported.
The configured stylelint version is not supported. See a list of supported versions on our docs page.
}; | ||
/** | ||
* This is a reducer for confirming a user's email address | ||
* @param {object} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected JSDoc for 'state' but found 'object' valid-jsdoc
Missing JSDoc parameter type for 'object' valid-jsdoc
passwordErrors: [], | ||
tokenErrors: [], | ||
}; | ||
/** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing JSDoc for parameter 'action' valid-jsdoc
Missing JSDoc return description valid-jsdoc
const newState = passwordResetReducer(undefined, { type: PASSWORD_RESET_FAILURE }); | ||
expect(newState.isLoading).toBe(false); | ||
expect(newState.isPassordResetSuccess).toBe(false); | ||
expect(newState.isPasswordResetError).toBe(true); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'expect' is not defined no-undef
it('Should have failed', () => { | ||
const newState = passwordResetReducer(undefined, { type: PASSWORD_RESET_FAILURE }); | ||
expect(newState.isLoading).toBe(false); | ||
expect(newState.isPassordResetSuccess).toBe(false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'expect' is not defined no-undef
|
||
it('Should have failed', () => { | ||
const newState = passwordResetReducer(undefined, { type: PASSWORD_RESET_FAILURE }); | ||
expect(newState.isLoading).toBe(false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'expect' is not defined no-undef
const newState = passwordResetReducer(undefined, {}); | ||
expect(newState.isLoading).toBe(false); | ||
expect(newState.isPassordResetSuccess).toBe(false); | ||
expect(newState.isPasswordResetError).toBe(false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'expect' is not defined no-undef
it('Should return the initial state', () => { | ||
const newState = passwordResetReducer(undefined, {}); | ||
expect(newState.isLoading).toBe(false); | ||
expect(newState.isPassordResetSuccess).toBe(false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'expect' is not defined no-undef
describe('cpassowd reset Reducer', () => { | ||
it('Should return the initial state', () => { | ||
const newState = passwordResetReducer(undefined, {}); | ||
expect(newState.isLoading).toBe(false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'expect' is not defined no-undef
}; | ||
/** | ||
* This is a reducer for confirming a user's email address | ||
* @param {object} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected JSDoc for 'state' but found 'object' valid-jsdoc
Missing JSDoc parameter type for 'object' valid-jsdoc
isConfirmEmailSuccess: false, | ||
isConfirmEmailError: false, | ||
}; | ||
/** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing JSDoc for parameter 'action' valid-jsdoc
Missing JSDoc return description valid-jsdoc
• Created an email confirmation form • Created a password reset form [Finishes #165273526]
598eaca
to
d4fc0fa
Compare
const store = mockStore(initialState); | ||
|
||
const wrapper = mount( | ||
<MemoryRouter> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Parsing error: Unexpected token <
What does this PR do?
• Redirects to the landing page on the successful reset of the password.
• Adds a success toast on the successful reset of the password.
Description of Task to be completed?
• A user should be able to reset their password in case they forget it.
• As long as a user can confirm their password, they should be able to reset their password.
How should this be manually tested?
Check out the preview app
Any background context you want to provide?
• Users should not have to create a new account once they lose their credentials.
What are the relevant pivotal tracker stories?
165273526
Screenshots (if appropriate)