X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Ftests%2Fapi%2Fserver%2Femail.ts;h=cd8d703411c21de44bcbbfda5ecec35a91fb573e;hb=0fbc0dec59684527fad8772080a8e1f928c07ea1;hp=95b64a45959fd3a95333e15af4b0618619524318;hpb=2ad9dcda240ee843c5e4a5b98cc94f7b2aab2c89;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/tests/api/server/email.ts b/server/tests/api/server/email.ts index 95b64a459..cd8d70341 100644 --- a/server/tests/api/server/email.ts +++ b/server/tests/api/server/email.ts @@ -1,39 +1,26 @@ /* eslint-disable @typescript-eslint/no-unused-expressions,@typescript-eslint/require-await */ -import * as chai from 'chai' import 'mocha' -import { - addVideoToBlacklist, - askResetPassword, - askSendVerifyEmail, - blockUser, - cleanupTests, - createUser, - flushAndRunServer, - removeVideoFromBlacklist, - reportVideoAbuse, - resetPassword, - ServerInfo, - setAccessTokensToServers, - unblockUser, - uploadVideo, - userLogin, - verifyEmail -} from '../../../../shared/extra-utils' -import { MockSmtpServer } from '../../../../shared/extra-utils/miscs/email' -import { waitJobs } from '../../../../shared/extra-utils/server/jobs' +import * as chai from 'chai' +import { cleanupTests, createSingleServer, MockSmtpServer, PeerTubeServer, setAccessTokensToServers, waitJobs } from '@shared/extra-utils' +import { HttpStatusCode } from '@shared/models' const expect = chai.expect describe('Test emails', function () { - let server: ServerInfo + let server: PeerTubeServer let userId: number let userId2: number let userAccessToken: string - let videoUUID: string + + let videoShortUUID: string + let videoId: number + let videoUserUUID: string + let verificationString: string let verificationString2: string + const emails: object[] = [] const user = { username: 'user_1', @@ -52,30 +39,29 @@ describe('Test emails', function () { port: emailPort } } - server = await flushAndRunServer(1, overrideConfig) + server = await createSingleServer(1, overrideConfig) await setAccessTokensToServers([ server ]) { - const res = await createUser({ url: server.url, accessToken: server.accessToken, username: user.username, password: user.password }) - userId = res.body.user.id + const created = await server.users.create({ username: user.username, password: user.password }) + userId = created.id - userAccessToken = await userLogin(server, user) + userAccessToken = await server.login.getAccessToken(user) } { - const attributes = { - name: 'my super user video' - } - const res = await uploadVideo(server.url, userAccessToken, attributes) - videoUserUUID = res.body.video.uuid + const attributes = { name: 'my super user video' } + const { uuid } = await server.videos.upload({ token: userAccessToken, attributes }) + videoUserUUID = uuid } { const attributes = { name: 'my super name' } - const res = await uploadVideo(server.url, server.accessToken, attributes) - videoUUID = res.body.video.uuid + const { shortUUID, id } = await server.videos.upload({ attributes }) + videoShortUUID = shortUUID + videoId = id } }) @@ -84,14 +70,14 @@ describe('Test emails', function () { it('Should ask to reset the password', async function () { this.timeout(10000) - await askResetPassword(server.url, 'user_1@example.com') + await server.users.askResetPassword({ email: 'user_1@example.com' }) await waitJobs(server) expect(emails).to.have.lengthOf(1) 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,37 +96,47 @@ 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 server.users.resetPassword({ + userId, + verificationString: verificationString + 'b', + password: 'super_password2', + expectedStatus: HttpStatusCode.FORBIDDEN_403 + }) }) it('Should reset the password', async function () { - await resetPassword(server.url, userId, verificationString, 'super_password2') + await server.users.resetPassword({ userId, verificationString, password: 'super_password2' }) + }) + + it('Should not reset the password with the same verification string', async function () { + await server.users.resetPassword({ + userId, + verificationString, + password: 'super_password3', + expectedStatus: HttpStatusCode.FORBIDDEN_403 + }) }) it('Should login with this new password', async function () { user.password = 'super_password2' - await userLogin(server, user) + await server.login.getAccessToken(user) }) }) describe('When creating a user without password', function () { + it('Should send a create password email', async function () { this.timeout(10000) - await createUser({ - url: server.url, - accessToken: server.accessToken, - username: 'create_password', - password: '' - }) + await server.users.create({ username: 'create_password', password: '' }) await waitJobs(server) expect(emails).to.have.lengthOf(2) 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,38 +155,48 @@ 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 server.users.resetPassword({ + userId: userId2, + verificationString: verificationString2 + 'c', + password: 'newly_created_password', + expectedStatus: HttpStatusCode.FORBIDDEN_403 + }) }) it('Should reset the password', async function () { - await resetPassword(server.url, userId2, verificationString2, 'newly_created_password') + await server.users.resetPassword({ + userId: userId2, + verificationString: verificationString2, + password: 'newly_created_password' + }) }) it('Should login with this new password', async function () { - await userLogin(server, { + await server.login.getAccessToken({ username: 'create_password', password: 'newly_created_password' }) }) }) - 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.abuses.report({ token: userAccessToken, 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') - expect(email['text']).contains(videoUUID) + expect(email['text']).contains(videoShortUUID) }) }) @@ -200,32 +206,32 @@ describe('Test emails', function () { this.timeout(10000) const reason = 'my super bad reason' - await blockUser(server.url, userId, server.accessToken, 204, reason) + await server.users.banUser({ userId, 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 server.users.unbanUser({ userId }) 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') @@ -238,14 +244,14 @@ describe('Test emails', function () { this.timeout(10000) const reason = 'my super reason' - await addVideoToBlacklist(server.url, server.accessToken, videoUserUUID, reason) + await server.blacklist.add({ videoId: videoUserUUID, reason }) await waitJobs(server) expect(emails).to.have.lengthOf(6) 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') @@ -256,19 +262,24 @@ describe('Test emails', function () { it('Should send the notification email', async function () { this.timeout(10000) - await removeVideoFromBlacklist(server.url, server.accessToken, videoUserUUID) + await server.blacklist.remove({ videoId: videoUserUUID }) await waitJobs(server) expect(emails).to.have.lengthOf(7) 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 () { @@ -276,14 +287,14 @@ describe('Test emails', function () { it('Should ask to send the verification email', async function () { this.timeout(10000) - await askSendVerifyEmail(server.url, 'user_1@example.com') + await server.users.askSendVerifyEmail({ email: 'user_1@example.com' }) await waitJobs(server) expect(emails).to.have.lengthOf(8) 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,11 +313,16 @@ 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 server.users.verifyEmail({ + userId, + verificationString: verificationString + 'b', + isPendingEmail: false, + expectedStatus: HttpStatusCode.FORBIDDEN_403 + }) }) it('Should verify the email', async function () { - await verifyEmail(server.url, userId, verificationString) + await server.users.verifyEmail({ userId, verificationString }) }) })