mirror of
https://github.com/enso-org/enso.git
synced 2024-12-22 04:31:40 +03:00
37cc980082
#### Tl;dr Closes: enso-org/cloud-v2#1283 This PR significantly reimplements Offline mode <details><summary>Demo Presentation</summary> <p> https://github.com/enso-org/enso/assets/61194245/752d0423-9c0a-43ba-91e3-4a6688f77034 </p> </details> --- #### Context: Offline mode is one of the core features of the dashboard. Unfortunately, after adding new features and a few refactoring, we lost the ability to work offline. This PR should bring this functionality back, with a few key differences: 1. We require users to sign in before using the dashboard even in local mode. 2. Once a user is logged in, we allow him to work with local files 3. If a user closes the dashboard, and then open it, he can continue using it in offline mode #### This Change: What does this change do in the larger context? Specific details to highlight for review: 1. Reimplements `<AuthProvider />` functionality, now it implemented on top of `<Suspense />` and ReactQuery 2. Reimplements Backend module flow, now remote backend is always created, You no longer need to check if the RemoteBackend is present 3. Introduces new `<Suspense />` component, which is aware of offline status 4. Introduce new offline-related hooks 5. Add a banner to the form if it's unable to submit it offline 6. Refactor `InviteUserDialog` to the new `<Form />` component 7. Fixes redirect bug when the app doesn't redirect a user to the dashboard after logging in 8. Fixes strange behavior when `/users/me` could stuck into infinite refetch 9. Redesign the Cloud table for offline mode. 10. Adds blocking UI dialog when a user clicks "log out" button #### Test Plan: This PR requires thorough QA on the login flow across the browser and IDE. All redirect logic must stay unchanged. ---
121 lines
4.4 KiB
TypeScript
121 lines
4.4 KiB
TypeScript
/** @file Test the login flow. */
|
|
import * as test from '@playwright/test'
|
|
|
|
import * as actions from './actions'
|
|
|
|
// =================
|
|
// === Constants ===
|
|
// =================
|
|
|
|
const EMAIL = 'example.email+1234@testing.org'
|
|
const NAME = 'a custom user name'
|
|
const ORGANIZATION_ID = 'some testing organization id'
|
|
|
|
// =============
|
|
// === Tests ===
|
|
// =============
|
|
|
|
// Note: This does not check that the organization ID is sent in the correct format for the backend.
|
|
// It only checks that the organization ID is sent in certain places.
|
|
test.test('sign up with organization id', async ({ page }) => {
|
|
await page.goto('/')
|
|
await page.waitForLoadState('domcontentloaded')
|
|
await page.goto(
|
|
'/registration?' + new URLSearchParams([['organization_id', ORGANIZATION_ID]]).toString()
|
|
)
|
|
const api = await actions.mockApi({ page })
|
|
api.setCurrentUser(null)
|
|
|
|
// Sign up
|
|
await actions.locateEmailInput(page).fill(actions.VALID_EMAIL)
|
|
await actions.locatePasswordInput(page).fill(actions.VALID_PASSWORD)
|
|
await actions.locateConfirmPasswordInput(page).fill(actions.VALID_PASSWORD)
|
|
await actions.locateRegisterButton(page).click()
|
|
|
|
// Log in
|
|
await actions.locateEmailInput(page).fill(actions.VALID_EMAIL)
|
|
await actions.locatePasswordInput(page).fill(actions.VALID_PASSWORD)
|
|
await actions.locateLoginButton(page).click()
|
|
await actions.passTermsAndConditionsDialog({ page })
|
|
|
|
// Set username
|
|
await actions.locateUsernameInput(page).fill('arbitrary username')
|
|
await actions.locateSetUsernameButton(page).click()
|
|
|
|
test
|
|
.expect(api.currentUser()?.organizationId, 'new user has correct organization id')
|
|
.toBe(ORGANIZATION_ID)
|
|
})
|
|
|
|
test.test('sign up without organization id', async ({ page }) => {
|
|
await page.goto('/')
|
|
await page.waitForLoadState('domcontentloaded')
|
|
await page.goto('/registration')
|
|
const api = await actions.mockApi({ page })
|
|
api.setCurrentUser(null)
|
|
|
|
// Sign up
|
|
await actions.locateEmailInput(page).fill(actions.VALID_EMAIL)
|
|
await actions.locatePasswordInput(page).fill(actions.VALID_PASSWORD)
|
|
await actions.locateConfirmPasswordInput(page).fill(actions.VALID_PASSWORD)
|
|
await actions.locateRegisterButton(page).click()
|
|
|
|
// Log in
|
|
await actions.locateEmailInput(page).fill(actions.VALID_EMAIL)
|
|
await actions.locatePasswordInput(page).fill(actions.VALID_PASSWORD)
|
|
await actions.locateLoginButton(page).click()
|
|
await actions.passTermsAndConditionsDialog({ page })
|
|
|
|
// Set username
|
|
await actions.locateUsernameInput(page).fill('arbitrary username')
|
|
await actions.locateSetUsernameButton(page).click()
|
|
|
|
test
|
|
.expect(api.currentUser()?.organizationId, 'new user has correct organization id')
|
|
.toBe(api.defaultOrganizationId)
|
|
})
|
|
|
|
test.test('sign up flow', ({ page }) =>
|
|
actions.mockAll({ page }).then(
|
|
async ({ pageActions, api }) =>
|
|
await pageActions
|
|
.do(() => {
|
|
api.setCurrentUser(null)
|
|
|
|
// These values should be different, otherwise the email and name may come from the defaults.
|
|
test.expect(EMAIL).not.toStrictEqual(api.defaultEmail)
|
|
test.expect(NAME).not.toStrictEqual(api.defaultName)
|
|
})
|
|
.loginAsNewUser(EMAIL, actions.VALID_PASSWORD)
|
|
.do(async thePage => {
|
|
await actions.passTermsAndConditionsDialog({ page: thePage })
|
|
})
|
|
.setUsername(NAME)
|
|
.do(async thePage => {
|
|
await test.expect(actions.locateUpgradeButton(thePage)).toBeVisible()
|
|
await test.expect(actions.locateDriveView(thePage)).not.toBeVisible()
|
|
})
|
|
.do(() => {
|
|
// Logged in, and account enabled
|
|
const currentUser = api.currentUser()
|
|
test.expect(currentUser).toBeDefined()
|
|
if (currentUser != null) {
|
|
// This is required because `UserOrOrganization` is `readonly`.
|
|
// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, no-restricted-syntax, no-extra-semi
|
|
;(currentUser as { isEnabled: boolean }).isEnabled = true
|
|
}
|
|
})
|
|
.openUserMenu()
|
|
.userMenu.logout()
|
|
.login(EMAIL, actions.VALID_PASSWORD)
|
|
.do(async () => {
|
|
await test.expect(actions.locateNotEnabledStub(page)).not.toBeVisible()
|
|
await test.expect(actions.locateDriveView(page)).toBeVisible()
|
|
})
|
|
.do(() => {
|
|
test.expect(api.currentUser()?.email, 'new user has correct email').toBe(EMAIL)
|
|
test.expect(api.currentUser()?.name, 'new user has correct name').toBe(NAME)
|
|
})
|
|
)
|
|
)
|