search

conservancy / reimbursinator

Git Up GitHub: https://GiveUpGitHub.org
https://GiveUpGitHub.org
GNU Affero General Public License v3.0
7 stars 6 forks source link

Created tests for get_sections, get_fields, get_files, section, and fixed email confirmation bug #133

Closed danieldupriest closed 5 years ago

danieldupriest commented 5 years ago

This PR adds tests for the get_sections(), get_fields(), get_files() and section() functions. It skips send_email_to_admin() as there really isn't anything to unit test. It will have to be done manually. To test:

  1. Run pipenv shell in back/
  2. Run python manage.py test

Confirmation emails now work correctly. To test:

  1. Ensure you have correct settings in back/.env
  2. Edit back/reimbursinator/settings.py and make sure the values for LOGIN_URL and LOGIN_REDIRECT_URL both point to the correct ip/address for your front page.
  3. Start docker
  4. Create a new account using your own email address
  5. Check that you receive a confirmation email and click the link.
  6. Confirm that your email address is correct.
  7. Confirm that you are forwarded to the front page to log in.
danieldupriest commented 5 years ago

bob12345, daniel12345, frank12345 all work for me, as long as it's 8 letters long. I don't think it needs uppercase.