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

Add e2e test for signup flow in auth + user services #2093

Merged
merged 2 commits into from
Apr 13, 2021

Conversation

humphd
Copy link
Contributor

@humphd humphd commented Apr 7, 2021

We're getting really close to being able to support login + signup in the microservices. I wanted to get a complete e2e test written that would allow us to see if it's working properly.

This test won't pass yet, since it requires a few more pieces (authenticated routes, #2058, etc). However, I wanted to get it written before I forgot and while I had time.

@humphd humphd added type: test Creation and development of test Blocked Can't do this, until something else is done type: security Security concerns area: microservices labels Apr 7, 2021
@humphd humphd added this to the 1.9.5 Release milestone Apr 7, 2021
@humphd humphd self-assigned this Apr 7, 2021
@humphd
Copy link
Contributor Author

humphd commented Apr 9, 2021

This needs a rebase on #2103

@humphd humphd modified the milestones: 1.9.5 Release, 2.0 Release Apr 9, 2021
@humphd
Copy link
Contributor Author

humphd commented Apr 11, 2021

Depends on #2116.

Copy link
Contributor

@chrispinkney chrispinkney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since it won't pass just yet I'm fine re-approaching this after the missing pieces land. The tests themselves seem fine though 😀

I'm excited to see this starting to come together.

@humphd humphd removed the Blocked Can't do this, until something else is done label Apr 13, 2021
@humphd
Copy link
Contributor Author

humphd commented Apr 13, 2021

This is not blocked anymore, I have fixed it.

@humphd humphd merged commit 896133f into Seneca-CDOT:master Apr 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: microservices type: security Security concerns type: test Creation and development of test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants