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

Test Implementation and Basic User Creation Script #749

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

ParadigmMango2
Copy link

@ParadigmMango2 ParadigmMango2 commented May 1, 2024


[Issue #562 and Issue #747] | Implement metadata tests, login page navigation test, and a create test user script

Discord Username: @ParadigmMango

What type of PR is this? (select all that apply)

  • 🍕 Feature
  • 🐛 Bug Fix
  • 🚧 Breaking Change
  • 🧑‍💻 Code Refactor
  • 📝 Documentation Update

Description

This code partially addresses the unfinished tests by implementing a couple of short tests, specifically page metadata and navigating to the GitHub login. Unfortunately, every other test requires user authentication. To do so, I have tried many things and have failed. My most successful attempt so far has been to create a local testing user and authenticate with that for testing. Unfortunately, I have only managed to get the user creation script to work.

Related Tickets & Documents

QA Instructions, Screenshots, Recordings

This code was tested on a Linux laptop and cypress was run in the Firefox web browser.

UI accessibility concerns?

None.

Added/updated tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

[optional] Are there any post deployment tasks we need to perform?

No.

[optional] What gif best describes this PR or how it makes you feel?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant