@atproto/pds 0.5.1 → 0.5.3

package/tests/account-manager.test.ts

@@ -18,7 +18,7 @@ describe('account manager', () => {
       // For debugging:
       // headless: false,
       // devtools: true,
-      // slowMo: 150,
+      // slowMo: 25,
     })
 
     network = await TestNetworkNoAppView.create({
@@ -34,6 +34,10 @@ describe('account manager', () => {
     })
   })
 
+  afterEach(async () => {
+    await network.processAll()
+  })
+
   afterAll(async () => {
     await network?.close()
     await browser?.close()
@@ -44,7 +48,7 @@ describe('account manager', () => {
 
     await page.goto(new URL('/account', network.pds.url))
 
-    await page.assertTitle(`S'identifier`)
+    await page.assertTitle(`Se connecter`)
 
     await page.clickOnText('Créer un nouveau compte')
 
@@ -55,12 +59,14 @@ describe('account manager', () => {
     await page.typeInInput('email', 'bob@test.com')
     await page.typeInInput('password', 'bob-pass')
 
-    await page.clickOnText("S'inscrire")
+    await page.clickOnText('Inscription')
 
-    await page.assertTitle(`Compte utilisateur`)
+    await page.waitForNetworkIdle()
 
     await page.ensureTextVisibility('bob.test', 'span')
     await page.ensureTextVisibility('Votre compte Atmosphère est hébergé chez')
+
+    await page.assertTitle('Mon compte Atmosphère')
   })
 
   it('allows switching accounts', async () => {
@@ -68,7 +74,7 @@ describe('account manager', () => {
 
     await page.goto(new URL('/account', network.pds.url))
 
-    await page.assertTitle(`Compte utilisateur`)
+    await page.assertTitle('Mon compte Atmosphère')
 
     await page.clickOnAriaLabel(`Sélecteur de compte`)
     await page.clickOnText('Sélectionner un autre compte')
@@ -93,18 +99,13 @@ describe('account manager', () => {
 
     await page.goto(new URL('/account', network.pds.url))
 
-    await page.assertTitle(`Compte utilisateur`)
+    await page.assertTitle('Mon compte Atmosphère')
 
     await page.ensureTextVisibility('bob.test', 'span')
 
-    await page.ensureTextVisibility('alice.test', 'span').then(
-      () => {
-        throw new Error('Should not be visible')
-      },
-      (err) => {
-        expect(err).toBeInstanceOf(Error)
-      },
-    )
+    await expect(async () => {
+      await page.ensureTextVisibility('alice.test', 'span', 500)
+    }).rejects.toThrow('Waiting for selector')
   })
 
   it('allows changing the password', async () => {
@@ -112,9 +113,11 @@ describe('account manager', () => {
 
     await page.goto(new URL('/account', network.pds.url))
 
-    await page.assertTitle(`Compte utilisateur`)
+    await page.assertTitle('Mon compte Atmosphère')
+
+    await page.clickOnText('Compte utilisateur', 'a')
 
-    await page.clickOnText('Mot de passe', 'a')
+    await page.clickOnText('Mot de passe')
 
     using sendResetPasswordMock = jest
       .spyOn(network.pds.ctx.mailer, 'sendResetPassword')
@@ -122,7 +125,7 @@ describe('account manager', () => {
         // noop
       })
 
-    await page.clickOnText('Envoyer le code')
+    await page.clickOnText('Envoyer le code de vérification')
 
     await page.waitForNetworkIdle()
 
@@ -137,20 +140,23 @@ describe('account manager', () => {
     await page.typeInInput('code', params.token)
     await page.typeInInput('password', 'bob-new-pass')
 
-    await page.clickOnText('Soumettre')
+    await page.clickOnText('Valider')
 
-    await page.ensureTextVisibility(
-      'Réinitialisation du mot de passe réussie',
-      'div',
-    )
+    await page.ensureNotification('Réinitialisation du mot de passe réussie')
   })
 
-  it('allows validating the email address', async () => {
+  it('allows verifying the email address', async () => {
     await using page = await PageHelper.from(browser, { languages })
 
     await page.goto(new URL('/account', network.pds.url))
 
-    await page.clickOnText('Vérifier maintenant', 'button')
+    await page.assertTitle('Mon compte Atmosphère')
+
+    await page.clickOnText('Compte utilisateur', 'a')
+
+    await page.ensureTextVisibility('Votre adresse email doit être vérifiée.')
+
+    await page.clickOnText('Vérifier')
 
     using sendConfirmEmailMock = jest
       .spyOn(network.pds.ctx.mailer, 'sendConfirmEmail')
@@ -158,7 +164,7 @@ describe('account manager', () => {
         // noop
       })
 
-    await page.clickOnText('Envoyer le code', 'button')
+    await page.clickOnText('Envoyer le code de vérification', 'button')
 
     await page.waitForNetworkIdle()
 
@@ -171,9 +177,31 @@ describe('account manager', () => {
     })
 
     await page.typeInInput('code', params.token)
-    await page.clickOnText('Soumettre')
+    await page.clickOnText('Valider')
+
+    await page.ensureNotification('Adresse email vérifiée')
+  })
+
+  it('allows changing the username', async () => {
+    await using page = await PageHelper.from(browser, { languages })
+
+    await page.goto(new URL('/account', network.pds.url))
+
+    await page.assertTitle('Mon compte Atmosphère')
+
+    await page.clickOnText('Compte utilisateur', 'a')
+
+    await page.clickOnText("Nom d'utilisateur")
+
+    await page.clickOnText("Utiliser un nom d'utilisateur par défaut")
 
-    await page.ensureTextVisibility('Adresse email vérifiée', 'div')
+    await page.typeInInput('handle', 'bob-renamed')
+
+    await page.clickOnText('Valider')
+
+    await page.waitForNetworkIdle()
+
+    await page.ensureTextVisibility('bob-renamed.test', 'span')
   })
 
   it('allows changing the email address', async () => {
@@ -181,7 +209,11 @@ describe('account manager', () => {
 
     await page.goto(new URL('/account', network.pds.url))
 
-    await page.clickOnText('Email', 'a')
+    await page.assertTitle('Mon compte Atmosphère')
+
+    await page.clickOnText('Compte utilisateur', 'a')
+
+    await page.clickOnText('Adresse email')
 
     using sendUpdateEmailMock = jest
       .spyOn(network.pds.ctx.mailer, 'sendUpdateEmail')
@@ -189,13 +221,11 @@ describe('account manager', () => {
         // noop
       })
 
-    using sendConfirmEmailMock = jest
-      .spyOn(network.pds.ctx.mailer, 'sendConfirmEmail')
-      .mockImplementation(async () => {
-        // noop
-      })
-
-    await page.clickOnText('Envoyer le code', 'button')
+    const emailInput = await page.typeInInput(
+      'email',
+      'bob-new-email@example.com',
+    )
+    emailInput.press('Enter')
 
     await page.waitForNetworkIdle()
 
@@ -207,26 +237,147 @@ describe('account manager', () => {
       token: expect.any(String),
     })
 
-    await page.typeInInput('code', updateParams.token)
-    await page.typeInInput('email', 'bob-new-email@example.com')
-    await page.clickOnText('Soumettre')
+    const codeInput = await page.typeInInput('code', updateParams.token)
+    codeInput.press('Enter')
+
+    await page.ensureNotification("Modification de l'adresse email réussie")
+
+    // The email needs to be verified again
+    await page.ensureTextVisibility('Votre adresse email doit être vérifiée.')
+  })
+
+  it('allows signing out & signing back in', async () => {
+    await using page = await PageHelper.from(browser, { languages })
+
+    await page.goto(new URL('/account', network.pds.url))
+
+    await page.assertTitle('Mon compte Atmosphère')
+
+    await page.clickOnAriaLabel(`Sélecteur de compte`)
+    await page.clickOnText('Se déconnecter')
+
+    await page.assertTitle(`Se connecter`)
+    await page.clickOnText('Se connecter')
+
+    await page.clickOnText('Se souvenir de ce compte sur cet appareil', 'label')
+    await page.typeInInput('username', 'bob-new-email@example.com')
+    const input = await page.typeInInput('password', 'bob-new-pass')
+
+    input.press('Enter')
+
+    await page.ensureTextVisibility('bob-renamed.test', 'span')
+  })
+
+  it('does not ask for a token when changing a non-verified email', async () => {
+    await using page = await PageHelper.from(browser, { languages })
+
+    await page.goto(new URL('/account', network.pds.url))
+
+    await page.assertTitle('Mon compte Atmosphère')
+
+    await page.clickOnText('Compte utilisateur', 'a')
 
+    await page.clickOnText('Adresse email')
+
+    using sendUpdateEmailMock = jest
+      .spyOn(network.pds.ctx.mailer, 'sendUpdateEmail')
+      .mockImplementation(async () => {
+        // noop
+      })
+
+    const emailInput = await page.typeInInput(
+      'email',
+      'bob-new-email@example.com',
+    )
+    emailInput.press('Enter')
+
+    await page.waitForNetworkIdle()
+
+    expect(sendUpdateEmailMock).not.toHaveBeenCalled()
+
+    await page.clickOnText('Plus tard')
+
+    await page.ensureNotification("Modification de l'adresse email réussie")
+
+    // The email needs to be verified again
+    await page.ensureTextVisibility('Votre adresse email doit être vérifiée.')
+  })
+
+  it('rejects racial slurs when changing username', async () => {
+    await using page = await PageHelper.from(browser, { languages })
+
+    await page.goto(new URL('/account', network.pds.url))
+
+    await page.assertTitle('Mon compte Atmosphère')
+
+    await page.clickOnText('Compte utilisateur', 'a')
+
+    await page.clickOnText("Nom d'utilisateur")
+
+    await page.clickOnText("Utiliser un nom d'utilisateur par défaut")
+
+    // Try to change username to a racial slur
+    await page.typeInInput('handle', 'nigger')
+
+    await page.clickOnText('Valider')
+
+    await page.waitForNetworkIdle()
+
+    // Should display appropriate error message
     await page.ensureTextVisibility(
-      "Modification de l'adresse email réussie",
-      'div',
+      "Le nom d'utilisateur contient un langage inapproprié",
     )
 
-    expect(sendConfirmEmailMock).toHaveBeenCalledTimes(1)
+    // Username should not have changed
+    await page.clickOnText('Retour')
+    await page.ensureTextVisibility('bob-renamed.test', 'span')
+  })
 
-    const [confirmParams] = sendConfirmEmailMock.mock.lastCall!
-    expect(confirmParams).toEqual({
-      locale: 'fr',
-      token: expect.any(String),
-    })
+  it('rejects custom domain when not configured', async () => {
+    await using page = await PageHelper.from(browser, { languages })
+
+    await page.goto(new URL('/account', network.pds.url))
+
+    await page.assertTitle('Mon compte Atmosphère')
+
+    await page.clickOnText('Compte utilisateur', 'a')
+
+    await page.clickOnText("Nom d'utilisateur")
 
-    await page.typeInInput('code', confirmParams.token)
-    await page.clickOnText('Soumettre')
+    await page.clickOnText('Utiliser un nom de domaine que je possède')
+
+    // DNS is the default verification method
+    await page.ensureTextVisibility(
+      'Ajoutez le champ suivant à la configuration DNS de votre domaine.',
+    )
+    await page.ensureTextVisibility('_atproto.<votre-domaine>', 'code')
+    await page.ensureTextVisibility('TXT', 'code')
+
+    // Switch to HTTP verification method
+    await page.clickOnText('HTTP', 'span')
+
+    await page.ensureTextVisibility(
+      "Rendez un fichier texte avec le contenu ci-dessous disponible à l'URL suivante.",
+    )
+    await page.ensureTextVisibility(
+      'https://<votre-domaine>/.well-known/atproto-did',
+      'code',
+    )
+
+    // Try to use an unconfigured domain
+    await page.typeInInput('domain', 'notconfigured.com')
+
+    await page.clickOnText('Vérifier et enregistrer')
+
+    await page.waitForNetworkIdle()
+
+    // Should display appropriate error message
+    await page.ensureTextVisibility(
+      "Le nom d'utilisateur n'a pas pu être résolu",
+    )
 
-    await page.ensureTextVisibility('Adresse email vérifiée', 'div')
+    // Username should not have changed
+    await page.clickOnText('Retour')
+    await page.ensureTextVisibility('bob-renamed.test', 'span')
   })
 })

package/tests/oauth.test.ts

@@ -25,7 +25,7 @@ describe('oauth', () => {
       // For debugging:
       // headless: false,
       // devtools: true,
-      // slowMo: 250,
+      // slowMo: 25,
     })
 
     network = await TestNetworkNoAppView.create({
@@ -71,7 +71,7 @@ describe('oauth', () => {
 
     await page.navigationClick(`Sign up with ${new URL(network.pds.url).host}`)
 
-    await page.assertTitle("S'inscrire")
+    await page.assertTitle('Inscription')
 
     await page.typeInInput('handle', 'bob')
 
@@ -80,10 +80,10 @@ describe('oauth', () => {
     await page.typeInInput('email', 'bob@test.com')
     await page.typeInInput('password', 'bob-pass')
 
-    await page.clickOnText("S'inscrire")
+    await page.clickOnText('Inscription')
 
     await page.ensureTextVisibility(
-      `L'application demande un contrôle total sur votre identité, ce qui signifie qu'elle pourrait casser de façon permanente, ou même usurper, votre compte. N'authorisez l'accès qu'aux applications auxquelles vous faites vraiment confiance.`,
+      `L'application demande un contrôle total sur votre identité, ce qui signifie qu'elle pourrait casser de façon permanente, ou même usurper, votre compte. N'autorisez l'accès qu'aux applications auxquelles vous faites vraiment confiance.`,
     )
 
     // Make sure the new account is propagated to the PLC directory, allowing
@@ -112,7 +112,7 @@ describe('oauth', () => {
 
     await page.navigationClick(`Login with ${new URL(network.pds.url).host}`)
 
-    await page.assertTitle("S'identifier")
+    await page.assertTitle('Se connecter')
 
     // Cancel the OAuth flow:
     await page.navigationClick('Annuler')

package/tsconfig.build.json

@@ -3,7 +3,7 @@
   "compilerOptions": {
     "rootDir": "./src",
     "outDir": "./dist",
-    "noUnusedLocals": false
+    "noUnusedLocals": false,
   },
-  "include": ["./src"]
+  "include": ["./src"],
 }