diff options
author | Chocobozzz <me@florianbigard.com> | 2018-01-30 15:16:24 +0100 |
---|---|---|
committer | Chocobozzz <me@florianbigard.com> | 2018-01-30 15:16:24 +0100 |
commit | f076daa76a32074cba162459e38fa8c130ad42d0 (patch) | |
tree | ea8a92d6305afa158b7774b65e9e6098cc094b92 /server/tests | |
parent | ecb4e35f4e6c7304cb274593c13cb47fd5078b75 (diff) | |
download | PeerTube-f076daa76a32074cba162459e38fa8c130ad42d0.tar.gz PeerTube-f076daa76a32074cba162459e38fa8c130ad42d0.tar.zst PeerTube-f076daa76a32074cba162459e38fa8c130ad42d0.zip |
Add tests for emails
Diffstat (limited to 'server/tests')
-rw-r--r-- | server/tests/api/check-params/users.ts | 22 | ||||
-rw-r--r-- | server/tests/api/index-fast.ts | 1 | ||||
-rw-r--r-- | server/tests/api/server/email.ts | 94 | ||||
-rw-r--r-- | server/tests/utils/miscs/email.ts | 25 | ||||
-rw-r--r-- | server/tests/utils/users/users.ts | 26 |
5 files changed, 167 insertions, 1 deletions
diff --git a/server/tests/api/check-params/users.ts b/server/tests/api/check-params/users.ts index 28fefe79f..0fbc414c9 100644 --- a/server/tests/api/check-params/users.ts +++ b/server/tests/api/check-params/users.ts | |||
@@ -541,6 +541,28 @@ describe('Test users API validators', function () { | |||
541 | }) | 541 | }) |
542 | }) | 542 | }) |
543 | 543 | ||
544 | describe('When asking a password reset', function () { | ||
545 | const path = '/api/v1/users/ask-reset-password' | ||
546 | |||
547 | it('Should fail with a missing email', async function () { | ||
548 | const fields = {} | ||
549 | |||
550 | await makePostBodyRequest({ url: server.url, path, token: server.accessToken, fields }) | ||
551 | }) | ||
552 | |||
553 | it('Should fail with an invalid email', async function () { | ||
554 | const fields = { email: 'hello' } | ||
555 | |||
556 | await makePostBodyRequest({ url: server.url, path, token: server.accessToken, fields }) | ||
557 | }) | ||
558 | |||
559 | it('Should success with the correct params', async function () { | ||
560 | const fields = { email: 'admin@example.com' } | ||
561 | |||
562 | await makePostBodyRequest({ url: server.url, path, token: server.accessToken, fields, statusCodeExpected: 204 }) | ||
563 | }) | ||
564 | }) | ||
565 | |||
544 | after(async function () { | 566 | after(async function () { |
545 | killallServers([ server, serverWithRegistrationDisabled ]) | 567 | killallServers([ server, serverWithRegistrationDisabled ]) |
546 | 568 | ||
diff --git a/server/tests/api/index-fast.ts b/server/tests/api/index-fast.ts index e591d0fd2..9f52310dd 100644 --- a/server/tests/api/index-fast.ts +++ b/server/tests/api/index-fast.ts | |||
@@ -9,3 +9,4 @@ import './videos/video-blacklist-management' | |||
9 | import './videos/video-description' | 9 | import './videos/video-description' |
10 | import './videos/video-privacy' | 10 | import './videos/video-privacy' |
11 | import './videos/services' | 11 | import './videos/services' |
12 | import './server/email' | ||
diff --git a/server/tests/api/server/email.ts b/server/tests/api/server/email.ts new file mode 100644 index 000000000..8eb9c0fa4 --- /dev/null +++ b/server/tests/api/server/email.ts | |||
@@ -0,0 +1,94 @@ | |||
1 | /* tslint:disable:no-unused-expression */ | ||
2 | |||
3 | import * as chai from 'chai' | ||
4 | import 'mocha' | ||
5 | import { askResetPassword, createUser, resetPassword, runServer, userLogin, wait } from '../../utils' | ||
6 | import { flushTests, killallServers, ServerInfo, setAccessTokensToServers } from '../../utils/index' | ||
7 | import { mockSmtpServer } from '../../utils/miscs/email' | ||
8 | |||
9 | const expect = chai.expect | ||
10 | |||
11 | describe('Test emails', function () { | ||
12 | let server: ServerInfo | ||
13 | let userId: number | ||
14 | let verificationString: string | ||
15 | const emails: object[] = [] | ||
16 | const user = { | ||
17 | username: 'user_1', | ||
18 | password: 'super_password' | ||
19 | } | ||
20 | |||
21 | before(async function () { | ||
22 | this.timeout(30000) | ||
23 | |||
24 | await mockSmtpServer(emails) | ||
25 | |||
26 | await flushTests() | ||
27 | |||
28 | const overrideConfig = { | ||
29 | smtp: { | ||
30 | hostname: 'localhost' | ||
31 | } | ||
32 | } | ||
33 | server = await runServer(1, overrideConfig) | ||
34 | |||
35 | await wait(5000) | ||
36 | await setAccessTokensToServers([ server ]) | ||
37 | |||
38 | const res = await createUser(server.url, server.accessToken, user.username, user.password) | ||
39 | userId = res.body.user.id | ||
40 | }) | ||
41 | |||
42 | describe('When resetting user password', function () { | ||
43 | |||
44 | it('Should ask to reset the password', async function () { | ||
45 | this.timeout(10000) | ||
46 | |||
47 | await askResetPassword(server.url, 'user_1@example.com') | ||
48 | |||
49 | await wait(3000) | ||
50 | expect(emails).to.have.lengthOf(1) | ||
51 | |||
52 | const email = emails[0] | ||
53 | |||
54 | expect(email['from'][0]['address']).equal('test-admin@localhost') | ||
55 | expect(email['to'][0]['address']).equal('user_1@example.com') | ||
56 | expect(email['subject']).contains('password') | ||
57 | |||
58 | const verificationStringMatches = /verificationString=([a-z0-9]+)/.exec(email['text']) | ||
59 | expect(verificationStringMatches).not.to.be.null | ||
60 | |||
61 | verificationString = verificationStringMatches[1] | ||
62 | expect(verificationString).to.have.length.above(2) | ||
63 | |||
64 | const userIdMatches = /userId=([0-9]+)/.exec(email['text']) | ||
65 | expect(userIdMatches).not.to.be.null | ||
66 | |||
67 | userId = parseInt(userIdMatches[1], 10) | ||
68 | expect(verificationString).to.not.be.undefined | ||
69 | }) | ||
70 | |||
71 | it('Should not reset the password with an invalid verification string', async function () { | ||
72 | await resetPassword(server.url, userId, verificationString + 'b', 'super_password2', 403) | ||
73 | }) | ||
74 | |||
75 | it('Should reset the password', async function () { | ||
76 | await resetPassword(server.url, userId, verificationString, 'super_password2') | ||
77 | }) | ||
78 | |||
79 | it('Should login with this new password', async function () { | ||
80 | user.password = 'super_password2' | ||
81 | |||
82 | await userLogin(server, user) | ||
83 | }) | ||
84 | }) | ||
85 | |||
86 | after(async function () { | ||
87 | killallServers([ server ]) | ||
88 | |||
89 | // Keep the logs if the test failed | ||
90 | if (this['ok']) { | ||
91 | await flushTests() | ||
92 | } | ||
93 | }) | ||
94 | }) | ||
diff --git a/server/tests/utils/miscs/email.ts b/server/tests/utils/miscs/email.ts new file mode 100644 index 000000000..21accd09d --- /dev/null +++ b/server/tests/utils/miscs/email.ts | |||
@@ -0,0 +1,25 @@ | |||
1 | import * as MailDev from 'maildev' | ||
2 | |||
3 | function mockSmtpServer (emailsCollection: object[]) { | ||
4 | const maildev = new MailDev({ | ||
5 | ip: '127.0.0.1', | ||
6 | smtp: 1025, | ||
7 | disableWeb: true, | ||
8 | silent: true | ||
9 | }) | ||
10 | maildev.on('new', email => emailsCollection.push(email)) | ||
11 | |||
12 | return new Promise((res, rej) => { | ||
13 | maildev.listen(err => { | ||
14 | if (err) return rej(err) | ||
15 | |||
16 | return res() | ||
17 | }) | ||
18 | }) | ||
19 | } | ||
20 | |||
21 | // --------------------------------------------------------------------------- | ||
22 | |||
23 | export { | ||
24 | mockSmtpServer | ||
25 | } | ||
diff --git a/server/tests/utils/users/users.ts b/server/tests/utils/users/users.ts index 25351e2c7..9e33e6796 100644 --- a/server/tests/utils/users/users.ts +++ b/server/tests/utils/users/users.ts | |||
@@ -1,6 +1,6 @@ | |||
1 | import { isAbsolute, join } from 'path' | 1 | import { isAbsolute, join } from 'path' |
2 | import * as request from 'supertest' | 2 | import * as request from 'supertest' |
3 | import { makePostUploadRequest, makePutBodyRequest } from '../' | 3 | import { makePostBodyRequest, makePostUploadRequest, makePutBodyRequest } from '../' |
4 | 4 | ||
5 | import { UserRole } from '../../../../shared/index' | 5 | import { UserRole } from '../../../../shared/index' |
6 | 6 | ||
@@ -196,6 +196,28 @@ function updateUser (options: { | |||
196 | }) | 196 | }) |
197 | } | 197 | } |
198 | 198 | ||
199 | function askResetPassword (url: string, email: string) { | ||
200 | const path = '/api/v1/users/ask-reset-password' | ||
201 | |||
202 | return makePostBodyRequest({ | ||
203 | url, | ||
204 | path, | ||
205 | fields: { email }, | ||
206 | statusCodeExpected: 204 | ||
207 | }) | ||
208 | } | ||
209 | |||
210 | function resetPassword (url: string, userId: number, verificationString: string, password: string, statusCodeExpected = 204) { | ||
211 | const path = '/api/v1/users/' + userId + '/reset-password' | ||
212 | |||
213 | return makePostBodyRequest({ | ||
214 | url, | ||
215 | path, | ||
216 | fields: { password, verificationString }, | ||
217 | statusCodeExpected | ||
218 | }) | ||
219 | } | ||
220 | |||
199 | // --------------------------------------------------------------------------- | 221 | // --------------------------------------------------------------------------- |
200 | 222 | ||
201 | export { | 223 | export { |
@@ -210,5 +232,7 @@ export { | |||
210 | updateUser, | 232 | updateUser, |
211 | updateMyUser, | 233 | updateMyUser, |
212 | getUserInformation, | 234 | getUserInformation, |
235 | askResetPassword, | ||
236 | resetPassword, | ||
213 | updateMyAvatar | 237 | updateMyAvatar |
214 | } | 238 | } |