X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Ftests%2Fapi%2Fserver%2Femail.ts;h=9141cc697f1c69be81ad5bd7280aaccb6ee6721c;hb=6a9b3151822461a85bbc72b7a50e12ffb69bec3a;hp=ae86b380fd3d8cd2af4fdc963b58609b2c134cc4;hpb=a24bd1ed41b43790bab6ba789580bb4e85f07d85;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/tests/api/server/email.ts b/server/tests/api/server/email.ts index ae86b380f..9141cc697 100644 --- a/server/tests/api/server/email.ts +++ b/server/tests/api/server/email.ts @@ -1,11 +1,16 @@ /* eslint-disable @typescript-eslint/no-unused-expressions,@typescript-eslint/require-await */ -import 'mocha' -import * as chai from 'chai' -import { cleanupTests, createSingleServer, MockSmtpServer, PeerTubeServer, setAccessTokensToServers, waitJobs } from '@shared/extra-utils' +import { expect } from 'chai' +import { MockSmtpServer } from '@server/tests/shared' import { HttpStatusCode } from '@shared/models' - -const expect = chai.expect +import { + cleanupTests, + ConfigCommand, + createSingleServer, + PeerTubeServer, + setAccessTokensToServers, + waitJobs +} from '@shared/server-commands' describe('Test emails', function () { let server: PeerTubeServer @@ -13,7 +18,7 @@ describe('Test emails', function () { let userId2: number let userAccessToken: string - let videoUUID: string + let videoShortUUID: string let videoId: number let videoUserUUID: string @@ -26,21 +31,15 @@ describe('Test emails', function () { username: 'user_1', password: 'super_password' } - let emailPort: number before(async function () { - this.timeout(50000) + this.timeout(120000) - emailPort = await MockSmtpServer.Instance.collectEmails(emails) + const emailPort = await MockSmtpServer.Instance.collectEmails(emails) + server = await createSingleServer(1, ConfigCommand.getEmailOverrideConfig(emailPort)) - const overrideConfig = { - smtp: { - hostname: 'localhost', - port: emailPort - } - } - server = await createSingleServer(1, overrideConfig) await setAccessTokensToServers([ server ]) + await server.config.enableSignup(true) { const created = await server.users.create({ username: user.username, password: user.password }) @@ -59,8 +58,8 @@ describe('Test emails', function () { const attributes = { name: 'my super name' } - const { uuid, id } = await server.videos.upload({ attributes }) - videoUUID = uuid + const { shortUUID, id } = await server.videos.upload({ attributes }) + videoShortUUID = shortUUID videoId = id } }) @@ -68,8 +67,6 @@ describe('Test emails', function () { describe('When resetting user password', function () { it('Should ask to reset the password', async function () { - this.timeout(10000) - await server.users.askResetPassword({ email: 'user_1@example.com' }) await waitJobs(server) @@ -78,7 +75,7 @@ describe('Test emails', function () { const email = emails[0] expect(email['from'][0]['name']).equal('PeerTube') - expect(email['from'][0]['address']).equal('test-admin@localhost') + expect(email['from'][0]['address']).equal('test-admin@127.0.0.1') expect(email['to'][0]['address']).equal('user_1@example.com') expect(email['subject']).contains('password') @@ -127,8 +124,6 @@ describe('Test emails', function () { describe('When creating a user without password', function () { it('Should send a create password email', async function () { - this.timeout(10000) - await server.users.create({ username: 'create_password', password: '' }) await waitJobs(server) @@ -137,7 +132,7 @@ describe('Test emails', function () { const email = emails[1] expect(email['from'][0]['name']).equal('PeerTube') - expect(email['from'][0]['address']).equal('test-admin@localhost') + expect(email['from'][0]['address']).equal('test-admin@127.0.0.1') expect(email['to'][0]['address']).equal('create_password@example.com') expect(email['subject']).contains('account') expect(email['subject']).contains('password') @@ -180,11 +175,10 @@ describe('Test emails', function () { }) describe('When creating an abuse', function () { - it('Should send the notification email', async function () { - this.timeout(10000) + it('Should send the notification email', async function () { const reason = 'my super bad reason' - await server.abuses.report({ videoId, reason }) + await server.abuses.report({ token: userAccessToken, videoId, reason }) await waitJobs(server) expect(emails).to.have.lengthOf(3) @@ -192,18 +186,16 @@ describe('Test emails', function () { const email = emails[2] expect(email['from'][0]['name']).equal('PeerTube') - expect(email['from'][0]['address']).equal('test-admin@localhost') + expect(email['from'][0]['address']).equal('test-admin@127.0.0.1') expect(email['to'][0]['address']).equal('admin' + server.internalServerNumber + '@example.com') expect(email['subject']).contains('abuse') - expect(email['text']).contains(videoUUID) + expect(email['text']).contains(videoShortUUID) }) }) describe('When blocking/unblocking user', function () { it('Should send the notification email when blocking a user', async function () { - this.timeout(10000) - const reason = 'my super bad reason' await server.users.banUser({ userId, reason }) @@ -213,7 +205,7 @@ describe('Test emails', function () { const email = emails[3] expect(email['from'][0]['name']).equal('PeerTube') - expect(email['from'][0]['address']).equal('test-admin@localhost') + expect(email['from'][0]['address']).equal('test-admin@127.0.0.1') expect(email['to'][0]['address']).equal('user_1@example.com') expect(email['subject']).contains(' blocked') expect(email['text']).contains(' blocked') @@ -221,8 +213,6 @@ describe('Test emails', function () { }) it('Should send the notification email when unblocking a user', async function () { - this.timeout(10000) - await server.users.unbanUser({ userId }) await waitJobs(server) @@ -231,7 +221,7 @@ describe('Test emails', function () { const email = emails[4] expect(email['from'][0]['name']).equal('PeerTube') - expect(email['from'][0]['address']).equal('test-admin@localhost') + expect(email['from'][0]['address']).equal('test-admin@127.0.0.1') expect(email['to'][0]['address']).equal('user_1@example.com') expect(email['subject']).contains(' unblocked') expect(email['text']).contains(' unblocked') @@ -240,8 +230,6 @@ describe('Test emails', function () { describe('When blacklisting a video', function () { it('Should send the notification email', async function () { - this.timeout(10000) - const reason = 'my super reason' await server.blacklist.add({ videoId: videoUserUUID, reason }) @@ -251,7 +239,7 @@ describe('Test emails', function () { const email = emails[5] expect(email['from'][0]['name']).equal('PeerTube') - expect(email['from'][0]['address']).equal('test-admin@localhost') + expect(email['from'][0]['address']).equal('test-admin@127.0.0.1') expect(email['to'][0]['address']).equal('user_1@example.com') expect(email['subject']).contains(' blacklisted') expect(email['text']).contains('my super user video') @@ -259,8 +247,6 @@ describe('Test emails', function () { }) it('Should send the notification email', async function () { - this.timeout(10000) - await server.blacklist.remove({ videoId: videoUserUUID }) await waitJobs(server) @@ -269,7 +255,7 @@ describe('Test emails', function () { const email = emails[6] expect(email['from'][0]['name']).equal('PeerTube') - expect(email['from'][0]['address']).equal('test-admin@localhost') + expect(email['from'][0]['address']).equal('test-admin@127.0.0.1') expect(email['to'][0]['address']).equal('user_1@example.com') expect(email['subject']).contains(' unblacklisted') expect(email['text']).contains('my super user video') @@ -284,8 +270,6 @@ describe('Test emails', function () { describe('When verifying a user email', function () { it('Should ask to send the verification email', async function () { - this.timeout(10000) - await server.users.askSendVerifyEmail({ email: 'user_1@example.com' }) await waitJobs(server) @@ -294,7 +278,7 @@ describe('Test emails', function () { const email = emails[7] expect(email['from'][0]['name']).equal('PeerTube') - expect(email['from'][0]['address']).equal('test-admin@localhost') + expect(email['from'][0]['address']).equal('test-admin@127.0.0.1') expect(email['to'][0]['address']).equal('user_1@example.com') expect(email['subject']).contains('Verify') @@ -325,6 +309,60 @@ describe('Test emails', function () { }) }) + describe('When verifying a registration email', function () { + let registrationId: number + let registrationIdEmail: number + + before(async function () { + const { id } = await server.registrations.requestRegistration({ + username: 'request_1', + email: 'request_1@example.com', + registrationReason: 'tt' + }) + registrationId = id + }) + + it('Should ask to send the verification email', async function () { + await server.registrations.askSendVerifyEmail({ email: 'request_1@example.com' }) + + await waitJobs(server) + expect(emails).to.have.lengthOf(9) + + const email = emails[8] + + expect(email['from'][0]['name']).equal('PeerTube') + expect(email['from'][0]['address']).equal('test-admin@127.0.0.1') + expect(email['to'][0]['address']).equal('request_1@example.com') + expect(email['subject']).contains('Verify') + + const verificationStringMatches = /verificationString=([a-z0-9]+)/.exec(email['text']) + expect(verificationStringMatches).not.to.be.null + + verificationString = verificationStringMatches[1] + expect(verificationString).to.not.be.undefined + expect(verificationString).to.have.length.above(2) + + const registrationIdMatches = /registrationId=([0-9]+)/.exec(email['text']) + expect(registrationIdMatches).not.to.be.null + + registrationIdEmail = parseInt(registrationIdMatches[1], 10) + + expect(registrationId).to.equal(registrationIdEmail) + }) + + it('Should not verify the email with an invalid verification string', async function () { + await server.registrations.verifyEmail({ + registrationId: registrationIdEmail, + verificationString: verificationString + 'b', + expectedStatus: HttpStatusCode.FORBIDDEN_403 + }) + }) + + it('Should verify the email', async function () { + await server.registrations.verifyEmail({ registrationId: registrationIdEmail, verificationString }) + }) + }) + after(async function () { MockSmtpServer.Instance.kill()