Skip to content
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

#166639551 fix reset password tests #28

Merged
merged 1 commit into from
Jun 17, 2019

Conversation

katherine95
Copy link
Contributor

What does this PR do?

  • Fix and add reset password tests.

Description of Task to be completed?

  • Increase tests so as to increase test coverage.

How should this be manually tested?

  • Clone the repository:
    $ git clone https://github.com/andela/ah-the-jedi-frontend.git

  • change directory to the ah-the-jedi-frontend directory:
    $ cd ah-the-jedi-frontend/

  • Get all branches:
    $ git pull

  • Switch to the feature branch:
    $ bg-fix-reset-password-tests-166639551

  • Install the project dependencies
    $ npm install

  • Run the project
    $ npm run test:coverage

What are the relevant pivotal tracker stories?

#166639551

Any background context you want to add?

None

@katherine95 katherine95 changed the title #166639551 Fix reset password tests #166639551 fix reset password tests Jun 17, 2019
@katherine95 katherine95 added FTL Review Waiting FTL Review Peer Review Waiting for PR Review labels Jun 17, 2019
@jkamz
Copy link
Contributor

jkamz commented Jun 17, 2019

Hey @katherine95. Please check on some failing tests on Travis.

- reset password tests.

[Fixes #166639551]
@katherine95 katherine95 force-pushed the bg-fix-reset-password-tests-166639551 branch from 4e7bd88 to 35120d0 Compare June 17, 2019 16:37
@AnguleMathias AnguleMathias merged commit 60e0a1d into develop Jun 17, 2019
@AnguleMathias AnguleMathias added Approved and removed FTL Review Waiting FTL Review Peer Review Waiting for PR Review labels Jun 18, 2019
@katherine95 katherine95 temporarily deployed to ah-the-jedi-frontend-staging June 20, 2019 06:47 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants