Bumped to version v5.2.1

[github/Chocobozzz/PeerTube.git] / client / e2e / src / suites-local / signup.e2e-spec.ts

diff --git a/client/e2e/src/suites-local/signup.e2e-spec.ts b/client/e2e/src/suites-local/signup.e2e-spec.ts
index b6f7ad1a788bd272ca8ea5b77d783d1104f6832f..98dc469c9c1fbd57eb1591d687df76c927bd719a 100644 (file)
--- a/client/e2e/src/suites-local/signup.e2e-spec.ts
+++ b/client/e2e/src/suites-local/signup.e2e-spec.ts
@@ -2,7 +2,16 @@ import { AdminConfigPage } from '../po/admin-config.po'
 import { AdminRegistrationPage } from '../po/admin-registration.po'
 import { LoginPage } from '../po/login.po'
 import { SignupPage } from '../po/signup.po'
-import { browserSleep, getVerificationLink, go, findEmailTo, isMobileDevice, MockSMTPServer, waitServerUp } from '../utils'
+import {
+  browserSleep,
+  findEmailTo,
+  getScreenshotPath,
+  getVerificationLink,
+  go,
+  isMobileDevice,
+  MockSMTPServer,
+  waitServerUp
+} from '../utils'
 
 function checkEndMessage (options: {
   message: string
@@ -155,7 +164,7 @@ describe('Signup', () => {
           afterEmailVerification: false
         })
 
-        await browser.saveScreenshot('./screenshots/direct-without-email.png')
+        await browser.saveScreenshot(getScreenshotPath('direct-without-email.png'))
 
         await loginPage.logout()
       })
@@ -203,7 +212,7 @@ describe('Signup', () => {
           afterEmailVerification: false
         })
 
-        await browser.saveScreenshot('./screenshots/request-without-email.png')
+        await browser.saveScreenshot(getScreenshotPath('request-without-email.png'))
       })
 
       it('Should display a message when trying to login with this account', async function () {
@@ -234,10 +243,12 @@ describe('Signup', () => {
     let emailPort: number
 
     before(async () => {
+      const key = browser.options.baseUrl + '-emailPort'
       // FIXME: typings are wrong, get returns a promise
-      emailPort = await browser.sharedStore.get('emailPort') as unknown as number
+      // FIXME: use * because the key is not properly escaped by the shared store when using get(key)
+      emailPort = (await (browser.sharedStore.get('*') as unknown as Promise<number>))[key]
 
-      MockSMTPServer.Instance.collectEmails(emailPort, emails)
+      await MockSMTPServer.Instance.collectEmails(emailPort, emails)
     })
 
     describe('Direct registration', function () {
@@ -283,7 +294,7 @@ describe('Signup', () => {
           afterEmailVerification: false
         })
 
-        await browser.saveScreenshot('./screenshots/direct-with-email.png')
+        await browser.saveScreenshot(getScreenshotPath('direct-with-email.png'))
       })
 
       it('Should validate the email', async function () {
@@ -304,7 +315,7 @@ describe('Signup', () => {
           afterEmailVerification: true
         })
 
-        await browser.saveScreenshot('./screenshots/direct-after-email.png')
+        await browser.saveScreenshot(getScreenshotPath('direct-after-email.png'))
       })
     })
 
@@ -355,7 +366,7 @@ describe('Signup', () => {
           afterEmailVerification: false
         })
 
-        await browser.saveScreenshot('./screenshots/request-with-email.png')
+        await browser.saveScreenshot(getScreenshotPath('request-with-email.png'))
       })
 
       it('Should display a message when trying to login with this account', async function () {
@@ -391,11 +402,11 @@ describe('Signup', () => {
           afterEmailVerification: true
         })
 
-        await browser.saveScreenshot('./screenshots/request-after-email.png')
+        await browser.saveScreenshot(getScreenshotPath('request-after-email.png'))
       })
     })
 
-    before(() => {
+    after(() => {
       MockSMTPServer.Instance.kill()
     })
   })