X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;ds=sidebyside;f=server%2Ftests%2Fapi%2Fserver%2Femail.ts;h=85844ae9df7c3c7fd2af2c5f76056027a9122af1;hb=0c1a77e9ccf915184c431145a8b326d4ce271b46;hp=95b64a45959fd3a95333e15af4b0618619524318;hpb=c2777c1dfe688c8fab1ef2fed50e360100fa9198;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/tests/api/server/email.ts b/server/tests/api/server/email.ts index 95b64a459..85844ae9d 100644 --- a/server/tests/api/server/email.ts +++ b/server/tests/api/server/email.ts @@ -1,7 +1,8 @@ /* eslint-disable @typescript-eslint/no-unused-expressions,@typescript-eslint/require-await */ -import * as chai from 'chai' import 'mocha' +import * as chai from 'chai' +import { HttpStatusCode } from '../../../../shared/core-utils/miscs/http-error-codes' import { addVideoToBlacklist, askResetPassword, @@ -11,7 +12,6 @@ import { createUser, flushAndRunServer, removeVideoFromBlacklist, - reportVideoAbuse, resetPassword, ServerInfo, setAccessTokensToServers, @@ -20,7 +20,7 @@ import { userLogin, verifyEmail } from '../../../../shared/extra-utils' -import { MockSmtpServer } from '../../../../shared/extra-utils/miscs/email' +import { MockSmtpServer } from '../../../../shared/extra-utils/mock-servers/mock-email' import { waitJobs } from '../../../../shared/extra-utils/server/jobs' const expect = chai.expect @@ -30,10 +30,15 @@ describe('Test emails', function () { let userId: number let userId2: number let userAccessToken: string + let videoUUID: string + let videoId: number + let videoUserUUID: string + let verificationString: string let verificationString2: string + const emails: object[] = [] const user = { username: 'user_1', @@ -76,6 +81,7 @@ describe('Test emails', function () { } const res = await uploadVideo(server.url, server.accessToken, attributes) videoUUID = res.body.video.uuid + videoId = res.body.video.id } }) @@ -91,7 +97,7 @@ describe('Test emails', function () { const email = emails[0] - expect(email['from'][0]['name']).equal('localhost:' + server.port) + expect(email['from'][0]['name']).equal('PeerTube') expect(email['from'][0]['address']).equal('test-admin@localhost') expect(email['to'][0]['address']).equal('user_1@example.com') expect(email['subject']).contains('password') @@ -110,13 +116,17 @@ describe('Test emails', function () { }) it('Should not reset the password with an invalid verification string', async function () { - await resetPassword(server.url, userId, verificationString + 'b', 'super_password2', 403) + await resetPassword(server.url, userId, verificationString + 'b', 'super_password2', HttpStatusCode.FORBIDDEN_403) }) it('Should reset the password', async function () { await resetPassword(server.url, userId, verificationString, 'super_password2') }) + it('Should not reset the password with the same verification string', async function () { + await resetPassword(server.url, userId, verificationString, 'super_password3', HttpStatusCode.FORBIDDEN_403) + }) + it('Should login with this new password', async function () { user.password = 'super_password2' @@ -140,7 +150,7 @@ describe('Test emails', function () { const email = emails[1] - expect(email['from'][0]['name']).equal('localhost:' + server.port) + expect(email['from'][0]['name']).equal('PeerTube') expect(email['from'][0]['address']).equal('test-admin@localhost') expect(email['to'][0]['address']).equal('create_password@example.com') expect(email['subject']).contains('account') @@ -159,7 +169,7 @@ describe('Test emails', function () { }) it('Should not reset the password with an invalid verification string', async function () { - await resetPassword(server.url, userId2, verificationString2 + 'c', 'newly_created_password', 403) + await resetPassword(server.url, userId2, verificationString2 + 'c', 'newly_created_password', HttpStatusCode.FORBIDDEN_403) }) it('Should reset the password', async function () { @@ -174,19 +184,19 @@ describe('Test emails', function () { }) }) - describe('When creating a video abuse', function () { + describe('When creating an abuse', function () { it('Should send the notification email', async function () { this.timeout(10000) const reason = 'my super bad reason' - await reportVideoAbuse(server.url, server.accessToken, videoUUID, reason) + await server.abusesCommand.report({ videoId, reason }) await waitJobs(server) expect(emails).to.have.lengthOf(3) const email = emails[2] - expect(email['from'][0]['name']).equal('localhost:' + server.port) + expect(email['from'][0]['name']).equal('PeerTube') expect(email['from'][0]['address']).equal('test-admin@localhost') expect(email['to'][0]['address']).equal('admin' + server.internalServerNumber + '@example.com') expect(email['subject']).contains('abuse') @@ -200,32 +210,32 @@ describe('Test emails', function () { this.timeout(10000) const reason = 'my super bad reason' - await blockUser(server.url, userId, server.accessToken, 204, reason) + await blockUser(server.url, userId, server.accessToken, HttpStatusCode.NO_CONTENT_204, reason) await waitJobs(server) expect(emails).to.have.lengthOf(4) const email = emails[3] - expect(email['from'][0]['name']).equal('localhost:' + server.port) + expect(email['from'][0]['name']).equal('PeerTube') expect(email['from'][0]['address']).equal('test-admin@localhost') expect(email['to'][0]['address']).equal('user_1@example.com') expect(email['subject']).contains(' blocked') expect(email['text']).contains(' blocked') - expect(email['text']).contains(reason) + expect(email['text']).contains('bad reason') }) it('Should send the notification email when unblocking a user', async function () { this.timeout(10000) - await unblockUser(server.url, userId, server.accessToken, 204) + await unblockUser(server.url, userId, server.accessToken, HttpStatusCode.NO_CONTENT_204) await waitJobs(server) expect(emails).to.have.lengthOf(5) const email = emails[4] - expect(email['from'][0]['name']).equal('localhost:' + server.port) + expect(email['from'][0]['name']).equal('PeerTube') expect(email['from'][0]['address']).equal('test-admin@localhost') expect(email['to'][0]['address']).equal('user_1@example.com') expect(email['subject']).contains(' unblocked') @@ -245,7 +255,7 @@ describe('Test emails', function () { const email = emails[5] - expect(email['from'][0]['name']).equal('localhost:' + server.port) + expect(email['from'][0]['name']).equal('PeerTube') expect(email['from'][0]['address']).equal('test-admin@localhost') expect(email['to'][0]['address']).equal('user_1@example.com') expect(email['subject']).contains(' blacklisted') @@ -263,12 +273,17 @@ describe('Test emails', function () { const email = emails[6] - expect(email['from'][0]['name']).equal('localhost:' + server.port) + expect(email['from'][0]['name']).equal('PeerTube') expect(email['from'][0]['address']).equal('test-admin@localhost') expect(email['to'][0]['address']).equal('user_1@example.com') expect(email['subject']).contains(' unblacklisted') expect(email['text']).contains('my super user video') }) + + it('Should have the manage preferences link in the email', async function () { + const email = emails[6] + expect(email['text']).to.contain('Manage your notification preferences') + }) }) describe('When verifying a user email', function () { @@ -283,7 +298,7 @@ describe('Test emails', function () { const email = emails[7] - expect(email['from'][0]['name']).equal('localhost:' + server.port) + expect(email['from'][0]['name']).equal('PeerTube') expect(email['from'][0]['address']).equal('test-admin@localhost') expect(email['to'][0]['address']).equal('user_1@example.com') expect(email['subject']).contains('Verify') @@ -302,7 +317,7 @@ describe('Test emails', function () { }) it('Should not verify the email with an invalid verification string', async function () { - await verifyEmail(server.url, userId, verificationString + 'b', false, 403) + await verifyEmail(server.url, userId, verificationString + 'b', false, HttpStatusCode.FORBIDDEN_403) }) it('Should verify the email', async function () {