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 trial user E2E Testing & ensure redirect correct after creating first application #2352

Merged
merged 4 commits into from
Jul 4, 2023

Conversation

joepavitt
Copy link
Contributor

@joepavitt joepavitt commented Jun 28, 2023

Description

  • Removes unnecessary loading = false just before the redirect, as that was triggering a re-mount of the InstanceForm and in turn redirecting to /billing
  • Adds testing infrastructure for checking platform state with trial users.
    • Moves the stripe mocking into a central location for forge unit tests, and the new ee E2E tests.
    • Adds a new terry user who will be a user on trial for 5 days.
    • Adds tests to ensure message is shown when Terry logs in, that they can create an Application/Instance and that they're redirected to /billing when trying to create a second

Related Issue(s)

Closes #2285

Checklist

@joepavitt
Copy link
Contributor Author

joepavitt commented Jun 28, 2023

Currently blocked as can't work out how to configure a trial user in our front-end E2E tests

@joepavitt joepavitt requested a review from Pezmc June 28, 2023 17:57
@joepavitt joepavitt changed the title Ensure user (on trial) directed to Application view after first created Application Add trial user E2E Testing & ensure redirect correct after creating first application Jun 28, 2023
test/e2e/frontend/environments/standard.js Outdated Show resolved Hide resolved
Co-authored-by: Pez Cuckow <email@pezcuckow.com>
@joepavitt joepavitt merged commit 6a93095 into main Jul 4, 2023
4 checks passed
@joepavitt joepavitt deleted the 2285-billing-redirect branch July 4, 2023 07:39
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.

User directed to /settings/billing after creating first Application
2 participants