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.

Sign up for GitHub

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

Jump to bottom

E2E tests for API user routes #112

Merged
merged 4 commits into from
Apr 25, 2023
Merged

E2E tests for API user routes #112

merged 4 commits into from
Apr 25, 2023

Conversation

josecelano
Copy link
Member

@josecelano josecelano commented Apr 24, 2023 •
edited by da2ce7
Loading

E2E tests for API user routes.

@josecelano josecelano linked an issue Apr 24, 2023 that may be closed by this pull request
E2E tests: user routes #111
Closed
@josecelano josecelano mentioned this pull request Apr 24, 2023
Merged
@josecelano josecelano marked this pull request as ready for review April 25, 2023 11:11
@josecelano
Copy link
Member Author

Hi @da2ce7 @WarmBeer, I've not included tests for registration verification via email and token renewal during the allowed period for renewal (only one week before the expiration).

@josecelano
Copy link
Member Author

ACK 2b58923

da2ce7
da2ce7 approved these changes Apr 25, 2023
View reviewed changes
Copy link
Contributor

@da2ce7 da2ce7 left a comment

Choose a reason for hiding this comment

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

ACK 2b58923

@josecelano josecelano merged commit a2aba45 into torrust:develop Apr 25, 2023
@josecelano josecelano deleted the issue-111-e2e-tests-user-routes branch April 25, 2023 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@da2ce7 da2ce7 da2ce7 approved these changes

@mickvandijke mickvandijke Awaiting requested review from mickvandijke

Assignees
No one assigned
Labels
None yet
Projects
Torrust Solution
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

E2E tests: user routes
2 participants
@josecelano @da2ce7