diff options
Diffstat (limited to 'server/tests/api/notifications')
-rw-r--r-- | server/tests/api/notifications/admin-notifications.ts | 39 | ||||
-rw-r--r-- | server/tests/api/notifications/comments-notifications.ts | 173 | ||||
-rw-r--r-- | server/tests/api/notifications/moderation-notifications.ts | 218 | ||||
-rw-r--r-- | server/tests/api/notifications/notifications-api.ts | 130 | ||||
-rw-r--r-- | server/tests/api/notifications/user-notifications.ts | 115 |
5 files changed, 289 insertions, 386 deletions
diff --git a/server/tests/api/notifications/admin-notifications.ts b/server/tests/api/notifications/admin-notifications.ts index cfe0bd2bb..b36ba11a9 100644 --- a/server/tests/api/notifications/admin-notifications.ts +++ b/server/tests/api/notifications/admin-notifications.ts | |||
@@ -2,21 +2,21 @@ | |||
2 | 2 | ||
3 | import 'mocha' | 3 | import 'mocha' |
4 | import { expect } from 'chai' | 4 | import { expect } from 'chai' |
5 | import { MockJoinPeerTubeVersions } from '@shared/extra-utils/mock-servers/joinpeertube-versions' | ||
6 | import { PluginType } from '@shared/models' | ||
7 | import { cleanupTests, installPlugin, setPluginLatestVersion, setPluginVersion, wait } from '../../../../shared/extra-utils' | ||
8 | import { ServerInfo } from '../../../../shared/extra-utils/index' | ||
9 | import { MockSmtpServer } from '../../../../shared/extra-utils/miscs/email' | ||
10 | import { | 5 | import { |
11 | CheckerBaseParams, | 6 | CheckerBaseParams, |
12 | checkNewPeerTubeVersion, | 7 | checkNewPeerTubeVersion, |
13 | checkNewPluginVersion, | 8 | checkNewPluginVersion, |
14 | prepareNotificationsTest | 9 | cleanupTests, |
15 | } from '../../../../shared/extra-utils/users/user-notifications' | 10 | MockJoinPeerTubeVersions, |
16 | import { UserNotification, UserNotificationType } from '../../../../shared/models/users' | 11 | MockSmtpServer, |
12 | PeerTubeServer, | ||
13 | prepareNotificationsTest, | ||
14 | wait | ||
15 | } from '@shared/extra-utils' | ||
16 | import { PluginType, UserNotification, UserNotificationType } from '@shared/models' | ||
17 | 17 | ||
18 | describe('Test admin notifications', function () { | 18 | describe('Test admin notifications', function () { |
19 | let server: ServerInfo | 19 | let server: PeerTubeServer |
20 | let userNotifications: UserNotification[] = [] | 20 | let userNotifications: UserNotification[] = [] |
21 | let adminNotifications: UserNotification[] = [] | 21 | let adminNotifications: UserNotification[] = [] |
22 | let emails: object[] = [] | 22 | let emails: object[] = [] |
@@ -58,17 +58,8 @@ describe('Test admin notifications', function () { | |||
58 | token: server.accessToken | 58 | token: server.accessToken |
59 | } | 59 | } |
60 | 60 | ||
61 | await installPlugin({ | 61 | await server.plugins.install({ npmName: 'peertube-plugin-hello-world' }) |
62 | url: server.url, | 62 | await server.plugins.install({ npmName: 'peertube-theme-background-red' }) |
63 | accessToken: server.accessToken, | ||
64 | npmName: 'peertube-plugin-hello-world' | ||
65 | }) | ||
66 | |||
67 | await installPlugin({ | ||
68 | url: server.url, | ||
69 | accessToken: server.accessToken, | ||
70 | npmName: 'peertube-theme-background-red' | ||
71 | }) | ||
72 | }) | 63 | }) |
73 | 64 | ||
74 | describe('Latest PeerTube version notification', function () { | 65 | describe('Latest PeerTube version notification', function () { |
@@ -127,8 +118,8 @@ describe('Test admin notifications', function () { | |||
127 | it('Should send a notification to admins on new plugin version', async function () { | 118 | it('Should send a notification to admins on new plugin version', async function () { |
128 | this.timeout(30000) | 119 | this.timeout(30000) |
129 | 120 | ||
130 | await setPluginVersion(server.internalServerNumber, 'hello-world', '0.0.1') | 121 | await server.sql.setPluginVersion('hello-world', '0.0.1') |
131 | await setPluginLatestVersion(server.internalServerNumber, 'hello-world', '0.0.1') | 122 | await server.sql.setPluginLatestVersion('hello-world', '0.0.1') |
132 | await wait(6000) | 123 | await wait(6000) |
133 | 124 | ||
134 | await checkNewPluginVersion(baseParams, PluginType.PLUGIN, 'hello-world', 'presence') | 125 | await checkNewPluginVersion(baseParams, PluginType.PLUGIN, 'hello-world', 'presence') |
@@ -149,8 +140,8 @@ describe('Test admin notifications', function () { | |||
149 | it('Should send a new notification after a new plugin release', async function () { | 140 | it('Should send a new notification after a new plugin release', async function () { |
150 | this.timeout(30000) | 141 | this.timeout(30000) |
151 | 142 | ||
152 | await setPluginVersion(server.internalServerNumber, 'hello-world', '0.0.1') | 143 | await server.sql.setPluginVersion('hello-world', '0.0.1') |
153 | await setPluginLatestVersion(server.internalServerNumber, 'hello-world', '0.0.1') | 144 | await server.sql.setPluginLatestVersion('hello-world', '0.0.1') |
154 | await wait(6000) | 145 | await wait(6000) |
155 | 146 | ||
156 | expect(adminNotifications.filter(n => n.type === UserNotificationType.NEW_PEERTUBE_VERSION)).to.have.lengthOf(2) | 147 | expect(adminNotifications.filter(n => n.type === UserNotificationType.NEW_PEERTUBE_VERSION)).to.have.lengthOf(2) |
diff --git a/server/tests/api/notifications/comments-notifications.ts b/server/tests/api/notifications/comments-notifications.ts index d2badf237..cbb46e510 100644 --- a/server/tests/api/notifications/comments-notifications.ts +++ b/server/tests/api/notifications/comments-notifications.ts | |||
@@ -3,30 +3,22 @@ | |||
3 | import 'mocha' | 3 | import 'mocha' |
4 | import * as chai from 'chai' | 4 | import * as chai from 'chai' |
5 | import { | 5 | import { |
6 | addAccountToAccountBlocklist, | ||
7 | addVideoCommentReply, | ||
8 | addVideoCommentThread, | ||
9 | checkCommentMention, | 6 | checkCommentMention, |
10 | CheckerBaseParams, | 7 | CheckerBaseParams, |
11 | checkNewCommentOnMyVideo, | 8 | checkNewCommentOnMyVideo, |
12 | cleanupTests, | 9 | cleanupTests, |
13 | getVideoCommentThreads, | ||
14 | getVideoThreadComments, | ||
15 | MockSmtpServer, | 10 | MockSmtpServer, |
11 | PeerTubeServer, | ||
16 | prepareNotificationsTest, | 12 | prepareNotificationsTest, |
17 | removeAccountFromAccountBlocklist, | ||
18 | ServerInfo, | ||
19 | updateMyUser, | ||
20 | uploadVideo, | ||
21 | waitJobs | 13 | waitJobs |
22 | } from '@shared/extra-utils' | 14 | } from '@shared/extra-utils' |
23 | import { UserNotification, VideoCommentThreadTree } from '@shared/models' | 15 | import { UserNotification } from '@shared/models' |
24 | 16 | ||
25 | const expect = chai.expect | 17 | const expect = chai.expect |
26 | 18 | ||
27 | describe('Test comments notifications', function () { | 19 | describe('Test comments notifications', function () { |
28 | let servers: ServerInfo[] = [] | 20 | let servers: PeerTubeServer[] = [] |
29 | let userAccessToken: string | 21 | let userToken: string |
30 | let userNotifications: UserNotification[] = [] | 22 | let userNotifications: UserNotification[] = [] |
31 | let emails: object[] = [] | 23 | let emails: object[] = [] |
32 | 24 | ||
@@ -40,7 +32,7 @@ describe('Test comments notifications', function () { | |||
40 | 32 | ||
41 | const res = await prepareNotificationsTest(2) | 33 | const res = await prepareNotificationsTest(2) |
42 | emails = res.emails | 34 | emails = res.emails |
43 | userAccessToken = res.userAccessToken | 35 | userToken = res.userAccessToken |
44 | servers = res.servers | 36 | servers = res.servers |
45 | userNotifications = res.userNotifications | 37 | userNotifications = res.userNotifications |
46 | }) | 38 | }) |
@@ -53,18 +45,17 @@ describe('Test comments notifications', function () { | |||
53 | server: servers[0], | 45 | server: servers[0], |
54 | emails, | 46 | emails, |
55 | socketNotifications: userNotifications, | 47 | socketNotifications: userNotifications, |
56 | token: userAccessToken | 48 | token: userToken |
57 | } | 49 | } |
58 | }) | 50 | }) |
59 | 51 | ||
60 | it('Should not send a new comment notification after a comment on another video', async function () { | 52 | it('Should not send a new comment notification after a comment on another video', async function () { |
61 | this.timeout(20000) | 53 | this.timeout(20000) |
62 | 54 | ||
63 | const resVideo = await uploadVideo(servers[0].url, servers[0].accessToken, { name: 'super video' }) | 55 | const { uuid } = await servers[0].videos.upload({ attributes: { name: 'super video' } }) |
64 | const uuid = resVideo.body.video.uuid | ||
65 | 56 | ||
66 | const resComment = await addVideoCommentThread(servers[0].url, servers[0].accessToken, uuid, 'comment') | 57 | const created = await servers[0].comments.createThread({ videoId: uuid, text: 'comment' }) |
67 | const commentId = resComment.body.comment.id | 58 | const commentId = created.id |
68 | 59 | ||
69 | await waitJobs(servers) | 60 | await waitJobs(servers) |
70 | await checkNewCommentOnMyVideo(baseParams, uuid, commentId, commentId, 'absence') | 61 | await checkNewCommentOnMyVideo(baseParams, uuid, commentId, commentId, 'absence') |
@@ -73,11 +64,10 @@ describe('Test comments notifications', function () { | |||
73 | it('Should not send a new comment notification if I comment my own video', async function () { | 64 | it('Should not send a new comment notification if I comment my own video', async function () { |
74 | this.timeout(20000) | 65 | this.timeout(20000) |
75 | 66 | ||
76 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name: 'super video' }) | 67 | const { uuid } = await servers[0].videos.upload({ token: userToken, attributes: { name: 'super video' } }) |
77 | const uuid = resVideo.body.video.uuid | ||
78 | 68 | ||
79 | const resComment = await addVideoCommentThread(servers[0].url, userAccessToken, uuid, 'comment') | 69 | const created = await servers[0].comments.createThread({ token: userToken, videoId: uuid, text: 'comment' }) |
80 | const commentId = resComment.body.comment.id | 70 | const commentId = created.id |
81 | 71 | ||
82 | await waitJobs(servers) | 72 | await waitJobs(servers) |
83 | await checkNewCommentOnMyVideo(baseParams, uuid, commentId, commentId, 'absence') | 73 | await checkNewCommentOnMyVideo(baseParams, uuid, commentId, commentId, 'absence') |
@@ -86,28 +76,26 @@ describe('Test comments notifications', function () { | |||
86 | it('Should not send a new comment notification if the account is muted', async function () { | 76 | it('Should not send a new comment notification if the account is muted', async function () { |
87 | this.timeout(20000) | 77 | this.timeout(20000) |
88 | 78 | ||
89 | await addAccountToAccountBlocklist(servers[0].url, userAccessToken, 'root') | 79 | await servers[0].blocklist.addToMyBlocklist({ token: userToken, account: 'root' }) |
90 | 80 | ||
91 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name: 'super video' }) | 81 | const { uuid } = await servers[0].videos.upload({ token: userToken, attributes: { name: 'super video' } }) |
92 | const uuid = resVideo.body.video.uuid | ||
93 | 82 | ||
94 | const resComment = await addVideoCommentThread(servers[0].url, servers[0].accessToken, uuid, 'comment') | 83 | const created = await servers[0].comments.createThread({ videoId: uuid, text: 'comment' }) |
95 | const commentId = resComment.body.comment.id | 84 | const commentId = created.id |
96 | 85 | ||
97 | await waitJobs(servers) | 86 | await waitJobs(servers) |
98 | await checkNewCommentOnMyVideo(baseParams, uuid, commentId, commentId, 'absence') | 87 | await checkNewCommentOnMyVideo(baseParams, uuid, commentId, commentId, 'absence') |
99 | 88 | ||
100 | await removeAccountFromAccountBlocklist(servers[0].url, userAccessToken, 'root') | 89 | await servers[0].blocklist.removeFromMyBlocklist({ token: userToken, account: 'root' }) |
101 | }) | 90 | }) |
102 | 91 | ||
103 | it('Should send a new comment notification after a local comment on my video', async function () { | 92 | it('Should send a new comment notification after a local comment on my video', async function () { |
104 | this.timeout(20000) | 93 | this.timeout(20000) |
105 | 94 | ||
106 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name: 'super video' }) | 95 | const { uuid } = await servers[0].videos.upload({ token: userToken, attributes: { name: 'super video' } }) |
107 | const uuid = resVideo.body.video.uuid | ||
108 | 96 | ||
109 | const resComment = await addVideoCommentThread(servers[0].url, servers[0].accessToken, uuid, 'comment') | 97 | const created = await servers[0].comments.createThread({ videoId: uuid, text: 'comment' }) |
110 | const commentId = resComment.body.comment.id | 98 | const commentId = created.id |
111 | 99 | ||
112 | await waitJobs(servers) | 100 | await waitJobs(servers) |
113 | await checkNewCommentOnMyVideo(baseParams, uuid, commentId, commentId, 'presence') | 101 | await checkNewCommentOnMyVideo(baseParams, uuid, commentId, commentId, 'presence') |
@@ -116,33 +104,29 @@ describe('Test comments notifications', function () { | |||
116 | it('Should send a new comment notification after a remote comment on my video', async function () { | 104 | it('Should send a new comment notification after a remote comment on my video', async function () { |
117 | this.timeout(20000) | 105 | this.timeout(20000) |
118 | 106 | ||
119 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name: 'super video' }) | 107 | const { uuid } = await servers[0].videos.upload({ token: userToken, attributes: { name: 'super video' } }) |
120 | const uuid = resVideo.body.video.uuid | ||
121 | 108 | ||
122 | await waitJobs(servers) | 109 | await waitJobs(servers) |
123 | 110 | ||
124 | await addVideoCommentThread(servers[1].url, servers[1].accessToken, uuid, 'comment') | 111 | await servers[1].comments.createThread({ videoId: uuid, text: 'comment' }) |
125 | 112 | ||
126 | await waitJobs(servers) | 113 | await waitJobs(servers) |
127 | 114 | ||
128 | const resComment = await getVideoCommentThreads(servers[0].url, uuid, 0, 5) | 115 | const { data } = await servers[0].comments.listThreads({ videoId: uuid }) |
129 | expect(resComment.body.data).to.have.lengthOf(1) | 116 | expect(data).to.have.lengthOf(1) |
130 | const commentId = resComment.body.data[0].id | ||
131 | 117 | ||
118 | const commentId = data[0].id | ||
132 | await checkNewCommentOnMyVideo(baseParams, uuid, commentId, commentId, 'presence') | 119 | await checkNewCommentOnMyVideo(baseParams, uuid, commentId, commentId, 'presence') |
133 | }) | 120 | }) |
134 | 121 | ||
135 | it('Should send a new comment notification after a local reply on my video', async function () { | 122 | it('Should send a new comment notification after a local reply on my video', async function () { |
136 | this.timeout(20000) | 123 | this.timeout(20000) |
137 | 124 | ||
138 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name: 'super video' }) | 125 | const { uuid } = await servers[0].videos.upload({ token: userToken, attributes: { name: 'super video' } }) |
139 | const uuid = resVideo.body.video.uuid | ||
140 | 126 | ||
141 | const resThread = await addVideoCommentThread(servers[0].url, servers[0].accessToken, uuid, 'comment') | 127 | const { id: threadId } = await servers[0].comments.createThread({ videoId: uuid, text: 'comment' }) |
142 | const threadId = resThread.body.comment.id | ||
143 | 128 | ||
144 | const resComment = await addVideoCommentReply(servers[0].url, servers[0].accessToken, uuid, threadId, 'reply') | 129 | const { id: commentId } = await servers[0].comments.addReply({ videoId: uuid, toCommentId: threadId, text: 'reply' }) |
145 | const commentId = resComment.body.comment.id | ||
146 | 130 | ||
147 | await waitJobs(servers) | 131 | await waitJobs(servers) |
148 | await checkNewCommentOnMyVideo(baseParams, uuid, commentId, threadId, 'presence') | 132 | await checkNewCommentOnMyVideo(baseParams, uuid, commentId, threadId, 'presence') |
@@ -151,24 +135,22 @@ describe('Test comments notifications', function () { | |||
151 | it('Should send a new comment notification after a remote reply on my video', async function () { | 135 | it('Should send a new comment notification after a remote reply on my video', async function () { |
152 | this.timeout(20000) | 136 | this.timeout(20000) |
153 | 137 | ||
154 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name: 'super video' }) | 138 | const { uuid } = await servers[0].videos.upload({ token: userToken, attributes: { name: 'super video' } }) |
155 | const uuid = resVideo.body.video.uuid | ||
156 | await waitJobs(servers) | 139 | await waitJobs(servers) |
157 | 140 | ||
158 | { | 141 | { |
159 | const resThread = await addVideoCommentThread(servers[1].url, servers[1].accessToken, uuid, 'comment') | 142 | const created = await servers[1].comments.createThread({ videoId: uuid, text: 'comment' }) |
160 | const threadId = resThread.body.comment.id | 143 | const threadId = created.id |
161 | await addVideoCommentReply(servers[1].url, servers[1].accessToken, uuid, threadId, 'reply') | 144 | await servers[1].comments.addReply({ videoId: uuid, toCommentId: threadId, text: 'reply' }) |
162 | } | 145 | } |
163 | 146 | ||
164 | await waitJobs(servers) | 147 | await waitJobs(servers) |
165 | 148 | ||
166 | const resThread = await getVideoCommentThreads(servers[0].url, uuid, 0, 5) | 149 | const { data } = await servers[0].comments.listThreads({ videoId: uuid }) |
167 | expect(resThread.body.data).to.have.lengthOf(1) | 150 | expect(data).to.have.lengthOf(1) |
168 | const threadId = resThread.body.data[0].id | ||
169 | 151 | ||
170 | const resComments = await getVideoThreadComments(servers[0].url, uuid, threadId) | 152 | const threadId = data[0].id |
171 | const tree = resComments.body as VideoCommentThreadTree | 153 | const tree = await servers[0].comments.getThread({ videoId: uuid, threadId }) |
172 | 154 | ||
173 | expect(tree.children).to.have.lengthOf(1) | 155 | expect(tree.children).to.have.lengthOf(1) |
174 | const commentId = tree.children[0].comment.id | 156 | const commentId = tree.children[0].comment.id |
@@ -179,10 +161,9 @@ describe('Test comments notifications', function () { | |||
179 | it('Should convert markdown in comment to html', async function () { | 161 | it('Should convert markdown in comment to html', async function () { |
180 | this.timeout(20000) | 162 | this.timeout(20000) |
181 | 163 | ||
182 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name: 'cool video' }) | 164 | const { uuid } = await servers[0].videos.upload({ token: userToken, attributes: { name: 'cool video' } }) |
183 | const uuid = resVideo.body.video.uuid | ||
184 | 165 | ||
185 | await addVideoCommentThread(servers[0].url, servers[0].accessToken, uuid, commentText) | 166 | await servers[0].comments.createThread({ videoId: uuid, text: commentText }) |
186 | 167 | ||
187 | await waitJobs(servers) | 168 | await waitJobs(servers) |
188 | 169 | ||
@@ -199,30 +180,19 @@ describe('Test comments notifications', function () { | |||
199 | server: servers[0], | 180 | server: servers[0], |
200 | emails, | 181 | emails, |
201 | socketNotifications: userNotifications, | 182 | socketNotifications: userNotifications, |
202 | token: userAccessToken | 183 | token: userToken |
203 | } | 184 | } |
204 | 185 | ||
205 | await updateMyUser({ | 186 | await servers[0].users.updateMe({ displayName: 'super root name' }) |
206 | url: servers[0].url, | 187 | await servers[1].users.updateMe({ displayName: 'super root 2 name' }) |
207 | accessToken: servers[0].accessToken, | ||
208 | displayName: 'super root name' | ||
209 | }) | ||
210 | |||
211 | await updateMyUser({ | ||
212 | url: servers[1].url, | ||
213 | accessToken: servers[1].accessToken, | ||
214 | displayName: 'super root 2 name' | ||
215 | }) | ||
216 | }) | 188 | }) |
217 | 189 | ||
218 | it('Should not send a new mention comment notification if I mention the video owner', async function () { | 190 | it('Should not send a new mention comment notification if I mention the video owner', async function () { |
219 | this.timeout(10000) | 191 | this.timeout(10000) |
220 | 192 | ||
221 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name: 'super video' }) | 193 | const { uuid } = await servers[0].videos.upload({ token: userToken, attributes: { name: 'super video' } }) |
222 | const uuid = resVideo.body.video.uuid | ||
223 | 194 | ||
224 | const resComment = await addVideoCommentThread(servers[0].url, servers[0].accessToken, uuid, '@user_1 hello') | 195 | const { id: commentId } = await servers[0].comments.createThread({ videoId: uuid, text: '@user_1 hello' }) |
225 | const commentId = resComment.body.comment.id | ||
226 | 196 | ||
227 | await waitJobs(servers) | 197 | await waitJobs(servers) |
228 | await checkCommentMention(baseParams, uuid, commentId, commentId, 'super root name', 'absence') | 198 | await checkCommentMention(baseParams, uuid, commentId, commentId, 'super root name', 'absence') |
@@ -231,11 +201,9 @@ describe('Test comments notifications', function () { | |||
231 | it('Should not send a new mention comment notification if I mention myself', async function () { | 201 | it('Should not send a new mention comment notification if I mention myself', async function () { |
232 | this.timeout(10000) | 202 | this.timeout(10000) |
233 | 203 | ||
234 | const resVideo = await uploadVideo(servers[0].url, servers[0].accessToken, { name: 'super video' }) | 204 | const { uuid } = await servers[0].videos.upload({ attributes: { name: 'super video' } }) |
235 | const uuid = resVideo.body.video.uuid | ||
236 | 205 | ||
237 | const resComment = await addVideoCommentThread(servers[0].url, userAccessToken, uuid, '@user_1 hello') | 206 | const { id: commentId } = await servers[0].comments.createThread({ token: userToken, videoId: uuid, text: '@user_1 hello' }) |
238 | const commentId = resComment.body.comment.id | ||
239 | 207 | ||
240 | await waitJobs(servers) | 208 | await waitJobs(servers) |
241 | await checkCommentMention(baseParams, uuid, commentId, commentId, 'super root name', 'absence') | 209 | await checkCommentMention(baseParams, uuid, commentId, commentId, 'super root name', 'absence') |
@@ -244,29 +212,25 @@ describe('Test comments notifications', function () { | |||
244 | it('Should not send a new mention notification if the account is muted', async function () { | 212 | it('Should not send a new mention notification if the account is muted', async function () { |
245 | this.timeout(10000) | 213 | this.timeout(10000) |
246 | 214 | ||
247 | await addAccountToAccountBlocklist(servers[0].url, userAccessToken, 'root') | 215 | await servers[0].blocklist.addToMyBlocklist({ token: userToken, account: 'root' }) |
248 | 216 | ||
249 | const resVideo = await uploadVideo(servers[0].url, servers[0].accessToken, { name: 'super video' }) | 217 | const { uuid } = await servers[0].videos.upload({ attributes: { name: 'super video' } }) |
250 | const uuid = resVideo.body.video.uuid | ||
251 | 218 | ||
252 | const resComment = await addVideoCommentThread(servers[0].url, servers[0].accessToken, uuid, '@user_1 hello') | 219 | const { id: commentId } = await servers[0].comments.createThread({ videoId: uuid, text: '@user_1 hello' }) |
253 | const commentId = resComment.body.comment.id | ||
254 | 220 | ||
255 | await waitJobs(servers) | 221 | await waitJobs(servers) |
256 | await checkCommentMention(baseParams, uuid, commentId, commentId, 'super root name', 'absence') | 222 | await checkCommentMention(baseParams, uuid, commentId, commentId, 'super root name', 'absence') |
257 | 223 | ||
258 | await removeAccountFromAccountBlocklist(servers[0].url, userAccessToken, 'root') | 224 | await servers[0].blocklist.removeFromMyBlocklist({ token: userToken, account: 'root' }) |
259 | }) | 225 | }) |
260 | 226 | ||
261 | it('Should not send a new mention notification if the remote account mention a local account', async function () { | 227 | it('Should not send a new mention notification if the remote account mention a local account', async function () { |
262 | this.timeout(20000) | 228 | this.timeout(20000) |
263 | 229 | ||
264 | const resVideo = await uploadVideo(servers[0].url, servers[0].accessToken, { name: 'super video' }) | 230 | const { uuid } = await servers[0].videos.upload({ attributes: { name: 'super video' } }) |
265 | const uuid = resVideo.body.video.uuid | ||
266 | 231 | ||
267 | await waitJobs(servers) | 232 | await waitJobs(servers) |
268 | const resThread = await addVideoCommentThread(servers[1].url, servers[1].accessToken, uuid, '@user_1 hello') | 233 | const { id: threadId } = await servers[1].comments.createThread({ videoId: uuid, text: '@user_1 hello' }) |
269 | const threadId = resThread.body.comment.id | ||
270 | 234 | ||
271 | await waitJobs(servers) | 235 | await waitJobs(servers) |
272 | await checkCommentMention(baseParams, uuid, threadId, threadId, 'super root 2 name', 'absence') | 236 | await checkCommentMention(baseParams, uuid, threadId, threadId, 'super root 2 name', 'absence') |
@@ -275,17 +239,14 @@ describe('Test comments notifications', function () { | |||
275 | it('Should send a new mention notification after local comments', async function () { | 239 | it('Should send a new mention notification after local comments', async function () { |
276 | this.timeout(10000) | 240 | this.timeout(10000) |
277 | 241 | ||
278 | const resVideo = await uploadVideo(servers[0].url, servers[0].accessToken, { name: 'super video' }) | 242 | const { uuid } = await servers[0].videos.upload({ attributes: { name: 'super video' } }) |
279 | const uuid = resVideo.body.video.uuid | ||
280 | 243 | ||
281 | const resThread = await addVideoCommentThread(servers[0].url, servers[0].accessToken, uuid, '@user_1 hello 1') | 244 | const { id: threadId } = await servers[0].comments.createThread({ videoId: uuid, text: '@user_1 hellotext: 1' }) |
282 | const threadId = resThread.body.comment.id | ||
283 | 245 | ||
284 | await waitJobs(servers) | 246 | await waitJobs(servers) |
285 | await checkCommentMention(baseParams, uuid, threadId, threadId, 'super root name', 'presence') | 247 | await checkCommentMention(baseParams, uuid, threadId, threadId, 'super root name', 'presence') |
286 | 248 | ||
287 | const resComment = await addVideoCommentReply(servers[0].url, servers[0].accessToken, uuid, threadId, 'hello 2 @user_1') | 249 | const { id: commentId } = await servers[0].comments.addReply({ videoId: uuid, toCommentId: threadId, text: 'hello 2 @user_1' }) |
288 | const commentId = resComment.body.comment.id | ||
289 | 250 | ||
290 | await waitJobs(servers) | 251 | await waitJobs(servers) |
291 | await checkCommentMention(baseParams, uuid, commentId, threadId, 'super root name', 'presence') | 252 | await checkCommentMention(baseParams, uuid, commentId, threadId, 'super root name', 'presence') |
@@ -294,29 +255,27 @@ describe('Test comments notifications', function () { | |||
294 | it('Should send a new mention notification after remote comments', async function () { | 255 | it('Should send a new mention notification after remote comments', async function () { |
295 | this.timeout(20000) | 256 | this.timeout(20000) |
296 | 257 | ||
297 | const resVideo = await uploadVideo(servers[0].url, servers[0].accessToken, { name: 'super video' }) | 258 | const { uuid } = await servers[0].videos.upload({ attributes: { name: 'super video' } }) |
298 | const uuid = resVideo.body.video.uuid | ||
299 | 259 | ||
300 | await waitJobs(servers) | 260 | await waitJobs(servers) |
301 | 261 | ||
302 | const text1 = `hello @user_1@localhost:${servers[0].port} 1` | 262 | const text1 = `hello @user_1@localhost:${servers[0].port} 1` |
303 | const resThread = await addVideoCommentThread(servers[1].url, servers[1].accessToken, uuid, text1) | 263 | const { id: server2ThreadId } = await servers[1].comments.createThread({ videoId: uuid, text: text1 }) |
304 | const server2ThreadId = resThread.body.comment.id | ||
305 | 264 | ||
306 | await waitJobs(servers) | 265 | await waitJobs(servers) |
307 | 266 | ||
308 | const resThread2 = await getVideoCommentThreads(servers[0].url, uuid, 0, 5) | 267 | const { data } = await servers[0].comments.listThreads({ videoId: uuid }) |
309 | expect(resThread2.body.data).to.have.lengthOf(1) | 268 | expect(data).to.have.lengthOf(1) |
310 | const server1ThreadId = resThread2.body.data[0].id | 269 | |
270 | const server1ThreadId = data[0].id | ||
311 | await checkCommentMention(baseParams, uuid, server1ThreadId, server1ThreadId, 'super root 2 name', 'presence') | 271 | await checkCommentMention(baseParams, uuid, server1ThreadId, server1ThreadId, 'super root 2 name', 'presence') |
312 | 272 | ||
313 | const text2 = `@user_1@localhost:${servers[0].port} hello 2 @root@localhost:${servers[0].port}` | 273 | const text2 = `@user_1@localhost:${servers[0].port} hello 2 @root@localhost:${servers[0].port}` |
314 | await addVideoCommentReply(servers[1].url, servers[1].accessToken, uuid, server2ThreadId, text2) | 274 | await servers[1].comments.addReply({ videoId: uuid, toCommentId: server2ThreadId, text: text2 }) |
315 | 275 | ||
316 | await waitJobs(servers) | 276 | await waitJobs(servers) |
317 | 277 | ||
318 | const resComments = await getVideoThreadComments(servers[0].url, uuid, server1ThreadId) | 278 | const tree = await servers[0].comments.getThread({ videoId: uuid, threadId: server1ThreadId }) |
319 | const tree = resComments.body as VideoCommentThreadTree | ||
320 | 279 | ||
321 | expect(tree.children).to.have.lengthOf(1) | 280 | expect(tree.children).to.have.lengthOf(1) |
322 | const commentId = tree.children[0].comment.id | 281 | const commentId = tree.children[0].comment.id |
@@ -327,13 +286,11 @@ describe('Test comments notifications', function () { | |||
327 | it('Should convert markdown in comment to html', async function () { | 286 | it('Should convert markdown in comment to html', async function () { |
328 | this.timeout(10000) | 287 | this.timeout(10000) |
329 | 288 | ||
330 | const resVideo = await uploadVideo(servers[0].url, servers[0].accessToken, { name: 'super video' }) | 289 | const { uuid } = await servers[0].videos.upload({ attributes: { name: 'super video' } }) |
331 | const uuid = resVideo.body.video.uuid | ||
332 | 290 | ||
333 | const resThread = await addVideoCommentThread(servers[0].url, servers[0].accessToken, uuid, '@user_1 hello 1') | 291 | const { id: threadId } = await servers[0].comments.createThread({ videoId: uuid, text: '@user_1 hello 1' }) |
334 | const threadId = resThread.body.comment.id | ||
335 | 292 | ||
336 | await addVideoCommentReply(servers[0].url, servers[0].accessToken, uuid, threadId, '@user_1 ' + commentText) | 293 | await servers[0].comments.addReply({ videoId: uuid, toCommentId: threadId, text: '@user_1 ' + commentText }) |
337 | 294 | ||
338 | await waitJobs(servers) | 295 | await waitJobs(servers) |
339 | 296 | ||
diff --git a/server/tests/api/notifications/moderation-notifications.ts b/server/tests/api/notifications/moderation-notifications.ts index 3425480ae..6f74709b3 100644 --- a/server/tests/api/notifications/moderation-notifications.ts +++ b/server/tests/api/notifications/moderation-notifications.ts | |||
@@ -2,33 +2,6 @@ | |||
2 | 2 | ||
3 | import 'mocha' | 3 | import 'mocha' |
4 | import { buildUUID } from '@server/helpers/uuid' | 4 | import { buildUUID } from '@server/helpers/uuid' |
5 | import { AbuseState } from '@shared/models' | ||
6 | import { | ||
7 | addAbuseMessage, | ||
8 | addVideoCommentThread, | ||
9 | addVideoToBlacklist, | ||
10 | cleanupTests, | ||
11 | createUser, | ||
12 | follow, | ||
13 | generateUserAccessToken, | ||
14 | getAccount, | ||
15 | getCustomConfig, | ||
16 | getVideoCommentThreads, | ||
17 | getVideoIdFromUUID, | ||
18 | immutableAssign, | ||
19 | MockInstancesIndex, | ||
20 | registerUser, | ||
21 | removeVideoFromBlacklist, | ||
22 | reportAbuse, | ||
23 | unfollow, | ||
24 | updateAbuse, | ||
25 | updateCustomConfig, | ||
26 | updateCustomSubConfig, | ||
27 | wait | ||
28 | } from '../../../../shared/extra-utils' | ||
29 | import { ServerInfo, uploadVideo } from '../../../../shared/extra-utils/index' | ||
30 | import { MockSmtpServer } from '../../../../shared/extra-utils/miscs/email' | ||
31 | import { waitJobs } from '../../../../shared/extra-utils/server/jobs' | ||
32 | import { | 5 | import { |
33 | checkAbuseStateChange, | 6 | checkAbuseStateChange, |
34 | checkAutoInstanceFollowing, | 7 | checkAutoInstanceFollowing, |
@@ -43,15 +16,18 @@ import { | |||
43 | checkUserRegistered, | 16 | checkUserRegistered, |
44 | checkVideoAutoBlacklistForModerators, | 17 | checkVideoAutoBlacklistForModerators, |
45 | checkVideoIsPublished, | 18 | checkVideoIsPublished, |
46 | prepareNotificationsTest | 19 | cleanupTests, |
47 | } from '../../../../shared/extra-utils/users/user-notifications' | 20 | MockInstancesIndex, |
48 | import { addUserSubscription, removeUserSubscription } from '../../../../shared/extra-utils/users/user-subscriptions' | 21 | MockSmtpServer, |
49 | import { CustomConfig } from '../../../../shared/models/server' | 22 | PeerTubeServer, |
50 | import { UserNotification } from '../../../../shared/models/users' | 23 | prepareNotificationsTest, |
51 | import { VideoPrivacy } from '../../../../shared/models/videos' | 24 | wait, |
25 | waitJobs | ||
26 | } from '@shared/extra-utils' | ||
27 | import { AbuseState, CustomConfig, UserNotification, VideoPrivacy } from '@shared/models' | ||
52 | 28 | ||
53 | describe('Test moderation notifications', function () { | 29 | describe('Test moderation notifications', function () { |
54 | let servers: ServerInfo[] = [] | 30 | let servers: PeerTubeServer[] = [] |
55 | let userAccessToken: string | 31 | let userAccessToken: string |
56 | let userNotifications: UserNotification[] = [] | 32 | let userNotifications: UserNotification[] = [] |
57 | let adminNotifications: UserNotification[] = [] | 33 | let adminNotifications: UserNotification[] = [] |
@@ -86,10 +62,9 @@ describe('Test moderation notifications', function () { | |||
86 | this.timeout(20000) | 62 | this.timeout(20000) |
87 | 63 | ||
88 | const name = 'video for abuse ' + buildUUID() | 64 | const name = 'video for abuse ' + buildUUID() |
89 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name }) | 65 | const video = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) |
90 | const video = resVideo.body.video | ||
91 | 66 | ||
92 | await reportAbuse({ url: servers[0].url, token: servers[0].accessToken, videoId: video.id, reason: 'super reason' }) | 67 | await servers[0].abuses.report({ videoId: video.id, reason: 'super reason' }) |
93 | 68 | ||
94 | await waitJobs(servers) | 69 | await waitJobs(servers) |
95 | await checkNewVideoAbuseForModerators(baseParams, video.uuid, name, 'presence') | 70 | await checkNewVideoAbuseForModerators(baseParams, video.uuid, name, 'presence') |
@@ -99,13 +74,12 @@ describe('Test moderation notifications', function () { | |||
99 | this.timeout(20000) | 74 | this.timeout(20000) |
100 | 75 | ||
101 | const name = 'video for abuse ' + buildUUID() | 76 | const name = 'video for abuse ' + buildUUID() |
102 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name }) | 77 | const video = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) |
103 | const video = resVideo.body.video | ||
104 | 78 | ||
105 | await waitJobs(servers) | 79 | await waitJobs(servers) |
106 | 80 | ||
107 | const videoId = await getVideoIdFromUUID(servers[1].url, video.uuid) | 81 | const videoId = await servers[1].videos.getId({ uuid: video.uuid }) |
108 | await reportAbuse({ url: servers[1].url, token: servers[1].accessToken, videoId, reason: 'super reason' }) | 82 | await servers[1].abuses.report({ videoId, reason: 'super reason' }) |
109 | 83 | ||
110 | await waitJobs(servers) | 84 | await waitJobs(servers) |
111 | await checkNewVideoAbuseForModerators(baseParams, video.uuid, name, 'presence') | 85 | await checkNewVideoAbuseForModerators(baseParams, video.uuid, name, 'presence') |
@@ -115,14 +89,16 @@ describe('Test moderation notifications', function () { | |||
115 | this.timeout(20000) | 89 | this.timeout(20000) |
116 | 90 | ||
117 | const name = 'video for abuse ' + buildUUID() | 91 | const name = 'video for abuse ' + buildUUID() |
118 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name }) | 92 | const video = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) |
119 | const video = resVideo.body.video | 93 | const comment = await servers[0].comments.createThread({ |
120 | const resComment = await addVideoCommentThread(servers[0].url, userAccessToken, video.id, 'comment abuse ' + buildUUID()) | 94 | token: userAccessToken, |
121 | const comment = resComment.body.comment | 95 | videoId: video.id, |
96 | text: 'comment abuse ' + buildUUID() | ||
97 | }) | ||
122 | 98 | ||
123 | await waitJobs(servers) | 99 | await waitJobs(servers) |
124 | 100 | ||
125 | await reportAbuse({ url: servers[0].url, token: servers[0].accessToken, commentId: comment.id, reason: 'super reason' }) | 101 | await servers[0].abuses.report({ commentId: comment.id, reason: 'super reason' }) |
126 | 102 | ||
127 | await waitJobs(servers) | 103 | await waitJobs(servers) |
128 | await checkNewCommentAbuseForModerators(baseParams, video.uuid, name, 'presence') | 104 | await checkNewCommentAbuseForModerators(baseParams, video.uuid, name, 'presence') |
@@ -132,15 +108,19 @@ describe('Test moderation notifications', function () { | |||
132 | this.timeout(20000) | 108 | this.timeout(20000) |
133 | 109 | ||
134 | const name = 'video for abuse ' + buildUUID() | 110 | const name = 'video for abuse ' + buildUUID() |
135 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name }) | 111 | const video = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) |
136 | const video = resVideo.body.video | 112 | |
137 | await addVideoCommentThread(servers[0].url, userAccessToken, video.id, 'comment abuse ' + buildUUID()) | 113 | await servers[0].comments.createThread({ |
114 | token: userAccessToken, | ||
115 | videoId: video.id, | ||
116 | text: 'comment abuse ' + buildUUID() | ||
117 | }) | ||
138 | 118 | ||
139 | await waitJobs(servers) | 119 | await waitJobs(servers) |
140 | 120 | ||
141 | const resComments = await getVideoCommentThreads(servers[1].url, video.uuid, 0, 5) | 121 | const { data } = await servers[1].comments.listThreads({ videoId: video.uuid }) |
142 | const commentId = resComments.body.data[0].id | 122 | const commentId = data[0].id |
143 | await reportAbuse({ url: servers[1].url, token: servers[1].accessToken, commentId, reason: 'super reason' }) | 123 | await servers[1].abuses.report({ commentId, reason: 'super reason' }) |
144 | 124 | ||
145 | await waitJobs(servers) | 125 | await waitJobs(servers) |
146 | await checkNewCommentAbuseForModerators(baseParams, video.uuid, name, 'presence') | 126 | await checkNewCommentAbuseForModerators(baseParams, video.uuid, name, 'presence') |
@@ -150,10 +130,10 @@ describe('Test moderation notifications', function () { | |||
150 | this.timeout(20000) | 130 | this.timeout(20000) |
151 | 131 | ||
152 | const username = 'user' + new Date().getTime() | 132 | const username = 'user' + new Date().getTime() |
153 | const resUser = await createUser({ url: servers[0].url, accessToken: servers[0].accessToken, username, password: 'donald' }) | 133 | const { account } = await servers[0].users.create({ username, password: 'donald' }) |
154 | const accountId = resUser.body.user.account.id | 134 | const accountId = account.id |
155 | 135 | ||
156 | await reportAbuse({ url: servers[0].url, token: servers[0].accessToken, accountId, reason: 'super reason' }) | 136 | await servers[0].abuses.report({ accountId, reason: 'super reason' }) |
157 | 137 | ||
158 | await waitJobs(servers) | 138 | await waitJobs(servers) |
159 | await checkNewAccountAbuseForModerators(baseParams, username, 'presence') | 139 | await checkNewAccountAbuseForModerators(baseParams, username, 'presence') |
@@ -163,13 +143,13 @@ describe('Test moderation notifications', function () { | |||
163 | this.timeout(20000) | 143 | this.timeout(20000) |
164 | 144 | ||
165 | const username = 'user' + new Date().getTime() | 145 | const username = 'user' + new Date().getTime() |
166 | const tmpToken = await generateUserAccessToken(servers[0], username) | 146 | const tmpToken = await servers[0].users.generateUserAndToken(username) |
167 | await uploadVideo(servers[0].url, tmpToken, { name: 'super video' }) | 147 | await servers[0].videos.upload({ token: tmpToken, attributes: { name: 'super video' } }) |
168 | 148 | ||
169 | await waitJobs(servers) | 149 | await waitJobs(servers) |
170 | 150 | ||
171 | const resAccount = await getAccount(servers[1].url, username + '@' + servers[0].host) | 151 | const account = await servers[1].accounts.get({ accountName: username + '@' + servers[0].host }) |
172 | await reportAbuse({ url: servers[1].url, token: servers[1].accessToken, accountId: resAccount.body.id, reason: 'super reason' }) | 152 | await servers[1].abuses.report({ accountId: account.id, reason: 'super reason' }) |
173 | 153 | ||
174 | await waitJobs(servers) | 154 | await waitJobs(servers) |
175 | await checkNewAccountAbuseForModerators(baseParams, username, 'presence') | 155 | await checkNewAccountAbuseForModerators(baseParams, username, 'presence') |
@@ -189,17 +169,16 @@ describe('Test moderation notifications', function () { | |||
189 | } | 169 | } |
190 | 170 | ||
191 | const name = 'abuse ' + buildUUID() | 171 | const name = 'abuse ' + buildUUID() |
192 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name }) | 172 | const video = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) |
193 | const video = resVideo.body.video | ||
194 | 173 | ||
195 | const res = await reportAbuse({ url: servers[0].url, token: userAccessToken, videoId: video.id, reason: 'super reason' }) | 174 | const body = await servers[0].abuses.report({ token: userAccessToken, videoId: video.id, reason: 'super reason' }) |
196 | abuseId = res.body.abuse.id | 175 | abuseId = body.abuse.id |
197 | }) | 176 | }) |
198 | 177 | ||
199 | it('Should send a notification to reporter if the abuse has been accepted', async function () { | 178 | it('Should send a notification to reporter if the abuse has been accepted', async function () { |
200 | this.timeout(10000) | 179 | this.timeout(10000) |
201 | 180 | ||
202 | await updateAbuse(servers[0].url, servers[0].accessToken, abuseId, { state: AbuseState.ACCEPTED }) | 181 | await servers[0].abuses.update({ abuseId, body: { state: AbuseState.ACCEPTED } }) |
203 | await waitJobs(servers) | 182 | await waitJobs(servers) |
204 | 183 | ||
205 | await checkAbuseStateChange(baseParams, abuseId, AbuseState.ACCEPTED, 'presence') | 184 | await checkAbuseStateChange(baseParams, abuseId, AbuseState.ACCEPTED, 'presence') |
@@ -208,7 +187,7 @@ describe('Test moderation notifications', function () { | |||
208 | it('Should send a notification to reporter if the abuse has been rejected', async function () { | 187 | it('Should send a notification to reporter if the abuse has been rejected', async function () { |
209 | this.timeout(10000) | 188 | this.timeout(10000) |
210 | 189 | ||
211 | await updateAbuse(servers[0].url, servers[0].accessToken, abuseId, { state: AbuseState.REJECTED }) | 190 | await servers[0].abuses.update({ abuseId, body: { state: AbuseState.REJECTED } }) |
212 | await waitJobs(servers) | 191 | await waitJobs(servers) |
213 | 192 | ||
214 | await checkAbuseStateChange(baseParams, abuseId, AbuseState.REJECTED, 'presence') | 193 | await checkAbuseStateChange(baseParams, abuseId, AbuseState.REJECTED, 'presence') |
@@ -237,17 +216,16 @@ describe('Test moderation notifications', function () { | |||
237 | } | 216 | } |
238 | 217 | ||
239 | const name = 'abuse ' + buildUUID() | 218 | const name = 'abuse ' + buildUUID() |
240 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name }) | 219 | const video = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) |
241 | const video = resVideo.body.video | ||
242 | 220 | ||
243 | { | 221 | { |
244 | const res = await reportAbuse({ url: servers[0].url, token: userAccessToken, videoId: video.id, reason: 'super reason' }) | 222 | const body = await servers[0].abuses.report({ token: userAccessToken, videoId: video.id, reason: 'super reason' }) |
245 | abuseId = res.body.abuse.id | 223 | abuseId = body.abuse.id |
246 | } | 224 | } |
247 | 225 | ||
248 | { | 226 | { |
249 | const res = await reportAbuse({ url: servers[0].url, token: userAccessToken, videoId: video.id, reason: 'super reason 2' }) | 227 | const body = await servers[0].abuses.report({ token: userAccessToken, videoId: video.id, reason: 'super reason 2' }) |
250 | abuseId2 = res.body.abuse.id | 228 | abuseId2 = body.abuse.id |
251 | } | 229 | } |
252 | }) | 230 | }) |
253 | 231 | ||
@@ -255,7 +233,7 @@ describe('Test moderation notifications', function () { | |||
255 | this.timeout(10000) | 233 | this.timeout(10000) |
256 | 234 | ||
257 | const message = 'my super message to users' | 235 | const message = 'my super message to users' |
258 | await addAbuseMessage(servers[0].url, servers[0].accessToken, abuseId, message) | 236 | await servers[0].abuses.addMessage({ abuseId, message }) |
259 | await waitJobs(servers) | 237 | await waitJobs(servers) |
260 | 238 | ||
261 | await checkNewAbuseMessage(baseParamsUser, abuseId, message, 'user_1@example.com', 'presence') | 239 | await checkNewAbuseMessage(baseParamsUser, abuseId, message, 'user_1@example.com', 'presence') |
@@ -265,7 +243,7 @@ describe('Test moderation notifications', function () { | |||
265 | this.timeout(10000) | 243 | this.timeout(10000) |
266 | 244 | ||
267 | const message = 'my super message that should not be sent to the admin' | 245 | const message = 'my super message that should not be sent to the admin' |
268 | await addAbuseMessage(servers[0].url, servers[0].accessToken, abuseId, message) | 246 | await servers[0].abuses.addMessage({ abuseId, message }) |
269 | await waitJobs(servers) | 247 | await waitJobs(servers) |
270 | 248 | ||
271 | await checkNewAbuseMessage(baseParamsAdmin, abuseId, message, 'admin' + servers[0].internalServerNumber + '@example.com', 'absence') | 249 | await checkNewAbuseMessage(baseParamsAdmin, abuseId, message, 'admin' + servers[0].internalServerNumber + '@example.com', 'absence') |
@@ -275,7 +253,7 @@ describe('Test moderation notifications', function () { | |||
275 | this.timeout(10000) | 253 | this.timeout(10000) |
276 | 254 | ||
277 | const message = 'my super message to moderators' | 255 | const message = 'my super message to moderators' |
278 | await addAbuseMessage(servers[0].url, userAccessToken, abuseId2, message) | 256 | await servers[0].abuses.addMessage({ token: userAccessToken, abuseId: abuseId2, message }) |
279 | await waitJobs(servers) | 257 | await waitJobs(servers) |
280 | 258 | ||
281 | await checkNewAbuseMessage(baseParamsAdmin, abuseId2, message, 'admin' + servers[0].internalServerNumber + '@example.com', 'presence') | 259 | await checkNewAbuseMessage(baseParamsAdmin, abuseId2, message, 'admin' + servers[0].internalServerNumber + '@example.com', 'presence') |
@@ -285,7 +263,7 @@ describe('Test moderation notifications', function () { | |||
285 | this.timeout(10000) | 263 | this.timeout(10000) |
286 | 264 | ||
287 | const message = 'my super message that should not be sent to reporter' | 265 | const message = 'my super message that should not be sent to reporter' |
288 | await addAbuseMessage(servers[0].url, userAccessToken, abuseId2, message) | 266 | await servers[0].abuses.addMessage({ token: userAccessToken, abuseId: abuseId2, message }) |
289 | await waitJobs(servers) | 267 | await waitJobs(servers) |
290 | 268 | ||
291 | await checkNewAbuseMessage(baseParamsUser, abuseId2, message, 'user_1@example.com', 'absence') | 269 | await checkNewAbuseMessage(baseParamsUser, abuseId2, message, 'user_1@example.com', 'absence') |
@@ -308,10 +286,9 @@ describe('Test moderation notifications', function () { | |||
308 | this.timeout(10000) | 286 | this.timeout(10000) |
309 | 287 | ||
310 | const name = 'video for abuse ' + buildUUID() | 288 | const name = 'video for abuse ' + buildUUID() |
311 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name }) | 289 | const { uuid } = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) |
312 | const uuid = resVideo.body.video.uuid | ||
313 | 290 | ||
314 | await addVideoToBlacklist(servers[0].url, servers[0].accessToken, uuid) | 291 | await servers[0].blacklist.add({ videoId: uuid }) |
315 | 292 | ||
316 | await waitJobs(servers) | 293 | await waitJobs(servers) |
317 | await checkNewBlacklistOnMyVideo(baseParams, uuid, name, 'blacklist') | 294 | await checkNewBlacklistOnMyVideo(baseParams, uuid, name, 'blacklist') |
@@ -321,13 +298,12 @@ describe('Test moderation notifications', function () { | |||
321 | this.timeout(10000) | 298 | this.timeout(10000) |
322 | 299 | ||
323 | const name = 'video for abuse ' + buildUUID() | 300 | const name = 'video for abuse ' + buildUUID() |
324 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name }) | 301 | const { uuid } = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) |
325 | const uuid = resVideo.body.video.uuid | ||
326 | 302 | ||
327 | await addVideoToBlacklist(servers[0].url, servers[0].accessToken, uuid) | 303 | await servers[0].blacklist.add({ videoId: uuid }) |
328 | 304 | ||
329 | await waitJobs(servers) | 305 | await waitJobs(servers) |
330 | await removeVideoFromBlacklist(servers[0].url, servers[0].accessToken, uuid) | 306 | await servers[0].blacklist.remove({ videoId: uuid }) |
331 | await waitJobs(servers) | 307 | await waitJobs(servers) |
332 | 308 | ||
333 | await wait(500) | 309 | await wait(500) |
@@ -350,14 +326,14 @@ describe('Test moderation notifications', function () { | |||
350 | it('Should send a notification only to moderators when a user registers on the instance', async function () { | 326 | it('Should send a notification only to moderators when a user registers on the instance', async function () { |
351 | this.timeout(10000) | 327 | this.timeout(10000) |
352 | 328 | ||
353 | await registerUser(servers[0].url, 'user_45', 'password') | 329 | await servers[0].users.register({ username: 'user_45' }) |
354 | 330 | ||
355 | await waitJobs(servers) | 331 | await waitJobs(servers) |
356 | 332 | ||
357 | await checkUserRegistered(baseParams, 'user_45', 'presence') | 333 | await checkUserRegistered(baseParams, 'user_45', 'presence') |
358 | 334 | ||
359 | const userOverride = { socketNotifications: userNotifications, token: userAccessToken, check: { web: true, mail: false } } | 335 | const userOverride = { socketNotifications: userNotifications, token: userAccessToken, check: { web: true, mail: false } } |
360 | await checkUserRegistered(immutableAssign(baseParams, userOverride), 'user_45', 'absence') | 336 | await checkUserRegistered({ ...baseParams, ...userOverride }, 'user_45', 'absence') |
361 | }) | 337 | }) |
362 | }) | 338 | }) |
363 | 339 | ||
@@ -392,20 +368,20 @@ describe('Test moderation notifications', function () { | |||
392 | it('Should send a notification only to admin when there is a new instance follower', async function () { | 368 | it('Should send a notification only to admin when there is a new instance follower', async function () { |
393 | this.timeout(20000) | 369 | this.timeout(20000) |
394 | 370 | ||
395 | await follow(servers[2].url, [ servers[0].url ], servers[2].accessToken) | 371 | await servers[2].follows.follow({ hosts: [ servers[0].url ] }) |
396 | 372 | ||
397 | await waitJobs(servers) | 373 | await waitJobs(servers) |
398 | 374 | ||
399 | await checkNewInstanceFollower(baseParams, 'localhost:' + servers[2].port, 'presence') | 375 | await checkNewInstanceFollower(baseParams, 'localhost:' + servers[2].port, 'presence') |
400 | 376 | ||
401 | const userOverride = { socketNotifications: userNotifications, token: userAccessToken, check: { web: true, mail: false } } | 377 | const userOverride = { socketNotifications: userNotifications, token: userAccessToken, check: { web: true, mail: false } } |
402 | await checkNewInstanceFollower(immutableAssign(baseParams, userOverride), 'localhost:' + servers[2].port, 'absence') | 378 | await checkNewInstanceFollower({ ...baseParams, ...userOverride }, 'localhost:' + servers[2].port, 'absence') |
403 | }) | 379 | }) |
404 | 380 | ||
405 | it('Should send a notification on auto follow back', async function () { | 381 | it('Should send a notification on auto follow back', async function () { |
406 | this.timeout(40000) | 382 | this.timeout(40000) |
407 | 383 | ||
408 | await unfollow(servers[2].url, servers[2].accessToken, servers[0]) | 384 | await servers[2].follows.unfollow({ target: servers[0] }) |
409 | await waitJobs(servers) | 385 | await waitJobs(servers) |
410 | 386 | ||
411 | const config = { | 387 | const config = { |
@@ -415,9 +391,9 @@ describe('Test moderation notifications', function () { | |||
415 | } | 391 | } |
416 | } | 392 | } |
417 | } | 393 | } |
418 | await updateCustomSubConfig(servers[0].url, servers[0].accessToken, config) | 394 | await servers[0].config.updateCustomSubConfig({ newConfig: config }) |
419 | 395 | ||
420 | await follow(servers[2].url, [ servers[0].url ], servers[2].accessToken) | 396 | await servers[2].follows.follow({ hosts: [ servers[0].url ] }) |
421 | 397 | ||
422 | await waitJobs(servers) | 398 | await waitJobs(servers) |
423 | 399 | ||
@@ -426,19 +402,19 @@ describe('Test moderation notifications', function () { | |||
426 | await checkAutoInstanceFollowing(baseParams, followerHost, followingHost, 'presence') | 402 | await checkAutoInstanceFollowing(baseParams, followerHost, followingHost, 'presence') |
427 | 403 | ||
428 | const userOverride = { socketNotifications: userNotifications, token: userAccessToken, check: { web: true, mail: false } } | 404 | const userOverride = { socketNotifications: userNotifications, token: userAccessToken, check: { web: true, mail: false } } |
429 | await checkAutoInstanceFollowing(immutableAssign(baseParams, userOverride), followerHost, followingHost, 'absence') | 405 | await checkAutoInstanceFollowing({ ...baseParams, ...userOverride }, followerHost, followingHost, 'absence') |
430 | 406 | ||
431 | config.followings.instance.autoFollowBack.enabled = false | 407 | config.followings.instance.autoFollowBack.enabled = false |
432 | await updateCustomSubConfig(servers[0].url, servers[0].accessToken, config) | 408 | await servers[0].config.updateCustomSubConfig({ newConfig: config }) |
433 | await unfollow(servers[0].url, servers[0].accessToken, servers[2]) | 409 | await servers[0].follows.unfollow({ target: servers[2] }) |
434 | await unfollow(servers[2].url, servers[2].accessToken, servers[0]) | 410 | await servers[2].follows.unfollow({ target: servers[0] }) |
435 | }) | 411 | }) |
436 | 412 | ||
437 | it('Should send a notification on auto instances index follow', async function () { | 413 | it('Should send a notification on auto instances index follow', async function () { |
438 | this.timeout(30000) | 414 | this.timeout(30000) |
439 | await unfollow(servers[0].url, servers[0].accessToken, servers[1]) | 415 | await servers[0].follows.unfollow({ target: servers[1] }) |
440 | 416 | ||
441 | await updateCustomSubConfig(servers[0].url, servers[0].accessToken, config) | 417 | await servers[0].config.updateCustomSubConfig({ newConfig: config }) |
442 | 418 | ||
443 | await wait(5000) | 419 | await wait(5000) |
444 | await waitJobs(servers) | 420 | await waitJobs(servers) |
@@ -448,8 +424,8 @@ describe('Test moderation notifications', function () { | |||
448 | await checkAutoInstanceFollowing(baseParams, followerHost, followingHost, 'presence') | 424 | await checkAutoInstanceFollowing(baseParams, followerHost, followingHost, 'presence') |
449 | 425 | ||
450 | config.followings.instance.autoFollowIndex.enabled = false | 426 | config.followings.instance.autoFollowIndex.enabled = false |
451 | await updateCustomSubConfig(servers[0].url, servers[0].accessToken, config) | 427 | await servers[0].config.updateCustomSubConfig({ newConfig: config }) |
452 | await unfollow(servers[0].url, servers[0].accessToken, servers[1]) | 428 | await servers[0].follows.unfollow({ target: servers[1] }) |
453 | }) | 429 | }) |
454 | }) | 430 | }) |
455 | 431 | ||
@@ -484,9 +460,11 @@ describe('Test moderation notifications', function () { | |||
484 | token: userAccessToken | 460 | token: userAccessToken |
485 | } | 461 | } |
486 | 462 | ||
487 | const resCustomConfig = await getCustomConfig(servers[0].url, servers[0].accessToken) | 463 | currentCustomConfig = await servers[0].config.getCustomConfig() |
488 | currentCustomConfig = resCustomConfig.body | 464 | |
489 | const autoBlacklistTestsCustomConfig = immutableAssign(currentCustomConfig, { | 465 | const autoBlacklistTestsCustomConfig = { |
466 | ...currentCustomConfig, | ||
467 | |||
490 | autoBlacklist: { | 468 | autoBlacklist: { |
491 | videos: { | 469 | videos: { |
492 | ofUsers: { | 470 | ofUsers: { |
@@ -494,13 +472,14 @@ describe('Test moderation notifications', function () { | |||
494 | } | 472 | } |
495 | } | 473 | } |
496 | } | 474 | } |
497 | }) | 475 | } |
476 | |||
498 | // enable transcoding otherwise own publish notification after transcoding not expected | 477 | // enable transcoding otherwise own publish notification after transcoding not expected |
499 | autoBlacklistTestsCustomConfig.transcoding.enabled = true | 478 | autoBlacklistTestsCustomConfig.transcoding.enabled = true |
500 | await updateCustomConfig(servers[0].url, servers[0].accessToken, autoBlacklistTestsCustomConfig) | 479 | await servers[0].config.updateCustomConfig({ newCustomConfig: autoBlacklistTestsCustomConfig }) |
501 | 480 | ||
502 | await addUserSubscription(servers[0].url, servers[0].accessToken, 'user_1_channel@localhost:' + servers[0].port) | 481 | await servers[0].subscriptions.add({ targetUri: 'user_1_channel@localhost:' + servers[0].port }) |
503 | await addUserSubscription(servers[1].url, servers[1].accessToken, 'user_1_channel@localhost:' + servers[0].port) | 482 | await servers[1].subscriptions.add({ targetUri: 'user_1_channel@localhost:' + servers[0].port }) |
504 | 483 | ||
505 | }) | 484 | }) |
506 | 485 | ||
@@ -508,8 +487,8 @@ describe('Test moderation notifications', function () { | |||
508 | this.timeout(40000) | 487 | this.timeout(40000) |
509 | 488 | ||
510 | videoName = 'video with auto-blacklist ' + buildUUID() | 489 | videoName = 'video with auto-blacklist ' + buildUUID() |
511 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, { name: videoName }) | 490 | const { uuid } = await servers[0].videos.upload({ token: userAccessToken, attributes: { name: videoName } }) |
512 | videoUUID = resVideo.body.video.uuid | 491 | videoUUID = uuid |
513 | 492 | ||
514 | await waitJobs(servers) | 493 | await waitJobs(servers) |
515 | await checkVideoAutoBlacklistForModerators(adminBaseParamsServer1, videoUUID, videoName, 'presence') | 494 | await checkVideoAutoBlacklistForModerators(adminBaseParamsServer1, videoUUID, videoName, 'presence') |
@@ -530,7 +509,7 @@ describe('Test moderation notifications', function () { | |||
530 | it('Should send video published and unblacklist after video unblacklisted', async function () { | 509 | it('Should send video published and unblacklist after video unblacklisted', async function () { |
531 | this.timeout(40000) | 510 | this.timeout(40000) |
532 | 511 | ||
533 | await removeVideoFromBlacklist(servers[0].url, servers[0].accessToken, videoUUID) | 512 | await servers[0].blacklist.remove({ videoId: videoUUID }) |
534 | 513 | ||
535 | await waitJobs(servers) | 514 | await waitJobs(servers) |
536 | 515 | ||
@@ -555,19 +534,18 @@ describe('Test moderation notifications', function () { | |||
555 | 534 | ||
556 | const name = 'video with auto-blacklist and future schedule ' + buildUUID() | 535 | const name = 'video with auto-blacklist and future schedule ' + buildUUID() |
557 | 536 | ||
558 | const data = { | 537 | const attributes = { |
559 | name, | 538 | name, |
560 | privacy: VideoPrivacy.PRIVATE, | 539 | privacy: VideoPrivacy.PRIVATE, |
561 | scheduleUpdate: { | 540 | scheduleUpdate: { |
562 | updateAt: updateAt.toISOString(), | 541 | updateAt: updateAt.toISOString(), |
563 | privacy: VideoPrivacy.PUBLIC | 542 | privacy: VideoPrivacy.PUBLIC as VideoPrivacy.PUBLIC |
564 | } | 543 | } |
565 | } | 544 | } |
566 | 545 | ||
567 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, data) | 546 | const { uuid } = await servers[0].videos.upload({ token: userAccessToken, attributes }) |
568 | const uuid = resVideo.body.video.uuid | ||
569 | 547 | ||
570 | await removeVideoFromBlacklist(servers[0].url, servers[0].accessToken, uuid) | 548 | await servers[0].blacklist.remove({ videoId: uuid }) |
571 | 549 | ||
572 | await waitJobs(servers) | 550 | await waitJobs(servers) |
573 | await checkNewBlacklistOnMyVideo(userBaseParams, uuid, name, 'unblacklist') | 551 | await checkNewBlacklistOnMyVideo(userBaseParams, uuid, name, 'unblacklist') |
@@ -588,17 +566,16 @@ describe('Test moderation notifications', function () { | |||
588 | 566 | ||
589 | const name = 'video with schedule done and still auto-blacklisted ' + buildUUID() | 567 | const name = 'video with schedule done and still auto-blacklisted ' + buildUUID() |
590 | 568 | ||
591 | const data = { | 569 | const attributes = { |
592 | name, | 570 | name, |
593 | privacy: VideoPrivacy.PRIVATE, | 571 | privacy: VideoPrivacy.PRIVATE, |
594 | scheduleUpdate: { | 572 | scheduleUpdate: { |
595 | updateAt: updateAt.toISOString(), | 573 | updateAt: updateAt.toISOString(), |
596 | privacy: VideoPrivacy.PUBLIC | 574 | privacy: VideoPrivacy.PUBLIC as VideoPrivacy.PUBLIC |
597 | } | 575 | } |
598 | } | 576 | } |
599 | 577 | ||
600 | const resVideo = await uploadVideo(servers[0].url, userAccessToken, data) | 578 | const { uuid } = await servers[0].videos.upload({ token: userAccessToken, attributes }) |
601 | const uuid = resVideo.body.video.uuid | ||
602 | 579 | ||
603 | await wait(6000) | 580 | await wait(6000) |
604 | await checkVideoIsPublished(userBaseParams, name, uuid, 'absence') | 581 | await checkVideoIsPublished(userBaseParams, name, uuid, 'absence') |
@@ -612,18 +589,17 @@ describe('Test moderation notifications', function () { | |||
612 | const name = 'video without auto-blacklist ' + buildUUID() | 589 | const name = 'video without auto-blacklist ' + buildUUID() |
613 | 590 | ||
614 | // admin with blacklist right will not be auto-blacklisted | 591 | // admin with blacklist right will not be auto-blacklisted |
615 | const resVideo = await uploadVideo(servers[0].url, servers[0].accessToken, { name }) | 592 | const { uuid } = await servers[0].videos.upload({ attributes: { name } }) |
616 | const uuid = resVideo.body.video.uuid | ||
617 | 593 | ||
618 | await waitJobs(servers) | 594 | await waitJobs(servers) |
619 | await checkVideoAutoBlacklistForModerators(adminBaseParamsServer1, uuid, name, 'absence') | 595 | await checkVideoAutoBlacklistForModerators(adminBaseParamsServer1, uuid, name, 'absence') |
620 | }) | 596 | }) |
621 | 597 | ||
622 | after(async () => { | 598 | after(async () => { |
623 | await updateCustomConfig(servers[0].url, servers[0].accessToken, currentCustomConfig) | 599 | await servers[0].config.updateCustomConfig({ newCustomConfig: currentCustomConfig }) |
624 | 600 | ||
625 | await removeUserSubscription(servers[0].url, servers[0].accessToken, 'user_1_channel@localhost:' + servers[0].port) | 601 | await servers[0].subscriptions.remove({ uri: 'user_1_channel@localhost:' + servers[0].port }) |
626 | await removeUserSubscription(servers[1].url, servers[1].accessToken, 'user_1_channel@localhost:' + servers[0].port) | 602 | await servers[1].subscriptions.remove({ uri: 'user_1_channel@localhost:' + servers[0].port }) |
627 | }) | 603 | }) |
628 | }) | 604 | }) |
629 | 605 | ||
diff --git a/server/tests/api/notifications/notifications-api.ts b/server/tests/api/notifications/notifications-api.ts index b81995449..fa4b53db6 100644 --- a/server/tests/api/notifications/notifications-api.ts +++ b/server/tests/api/notifications/notifications-api.ts | |||
@@ -2,28 +2,24 @@ | |||
2 | 2 | ||
3 | import 'mocha' | 3 | import 'mocha' |
4 | import * as chai from 'chai' | 4 | import * as chai from 'chai' |
5 | import { addUserSubscription } from '@shared/extra-utils/users/user-subscriptions' | ||
6 | import { cleanupTests, getMyUserInformation, immutableAssign, uploadRandomVideo, waitJobs } from '../../../../shared/extra-utils' | ||
7 | import { ServerInfo } from '../../../../shared/extra-utils/index' | ||
8 | import { MockSmtpServer } from '../../../../shared/extra-utils/miscs/email' | ||
9 | import { | 5 | import { |
10 | CheckerBaseParams, | 6 | CheckerBaseParams, |
11 | checkNewVideoFromSubscription, | 7 | checkNewVideoFromSubscription, |
8 | cleanupTests, | ||
12 | getAllNotificationsSettings, | 9 | getAllNotificationsSettings, |
13 | getUserNotifications, | 10 | MockSmtpServer, |
14 | markAsReadAllNotifications, | 11 | PeerTubeServer, |
15 | markAsReadNotifications, | ||
16 | prepareNotificationsTest, | 12 | prepareNotificationsTest, |
17 | updateMyNotificationSettings | 13 | waitJobs |
18 | } from '../../../../shared/extra-utils/users/user-notifications' | 14 | } from '@shared/extra-utils' |
19 | import { User, UserNotification, UserNotificationSettingValue } from '../../../../shared/models/users' | 15 | import { UserNotification, UserNotificationSettingValue } from '@shared/models' |
20 | 16 | ||
21 | const expect = chai.expect | 17 | const expect = chai.expect |
22 | 18 | ||
23 | describe('Test notifications API', function () { | 19 | describe('Test notifications API', function () { |
24 | let server: ServerInfo | 20 | let server: PeerTubeServer |
25 | let userNotifications: UserNotification[] = [] | 21 | let userNotifications: UserNotification[] = [] |
26 | let userAccessToken: string | 22 | let userToken: string |
27 | let emails: object[] = [] | 23 | let emails: object[] = [] |
28 | 24 | ||
29 | before(async function () { | 25 | before(async function () { |
@@ -31,14 +27,14 @@ describe('Test notifications API', function () { | |||
31 | 27 | ||
32 | const res = await prepareNotificationsTest(1) | 28 | const res = await prepareNotificationsTest(1) |
33 | emails = res.emails | 29 | emails = res.emails |
34 | userAccessToken = res.userAccessToken | 30 | userToken = res.userAccessToken |
35 | userNotifications = res.userNotifications | 31 | userNotifications = res.userNotifications |
36 | server = res.servers[0] | 32 | server = res.servers[0] |
37 | 33 | ||
38 | await addUserSubscription(server.url, userAccessToken, 'root_channel@localhost:' + server.port) | 34 | await server.subscriptions.add({ token: userToken, targetUri: 'root_channel@localhost:' + server.port }) |
39 | 35 | ||
40 | for (let i = 0; i < 10; i++) { | 36 | for (let i = 0; i < 10; i++) { |
41 | await uploadRandomVideo(server, false) | 37 | await server.videos.randomUpload({ wait: false }) |
42 | } | 38 | } |
43 | 39 | ||
44 | await waitJobs([ server ]) | 40 | await waitJobs([ server ]) |
@@ -47,49 +43,46 @@ describe('Test notifications API', function () { | |||
47 | describe('Mark as read', function () { | 43 | describe('Mark as read', function () { |
48 | 44 | ||
49 | it('Should mark as read some notifications', async function () { | 45 | it('Should mark as read some notifications', async function () { |
50 | const res = await getUserNotifications(server.url, userAccessToken, 2, 3) | 46 | const { data } = await server.notifications.list({ token: userToken, start: 2, count: 3 }) |
51 | const ids = res.body.data.map(n => n.id) | 47 | const ids = data.map(n => n.id) |
52 | 48 | ||
53 | await markAsReadNotifications(server.url, userAccessToken, ids) | 49 | await server.notifications.markAsRead({ token: userToken, ids }) |
54 | }) | 50 | }) |
55 | 51 | ||
56 | it('Should have the notifications marked as read', async function () { | 52 | it('Should have the notifications marked as read', async function () { |
57 | const res = await getUserNotifications(server.url, userAccessToken, 0, 10) | 53 | const { data } = await server.notifications.list({ token: userToken, start: 0, count: 10 }) |
58 | 54 | ||
59 | const notifications = res.body.data as UserNotification[] | 55 | expect(data[0].read).to.be.false |
60 | expect(notifications[0].read).to.be.false | 56 | expect(data[1].read).to.be.false |
61 | expect(notifications[1].read).to.be.false | 57 | expect(data[2].read).to.be.true |
62 | expect(notifications[2].read).to.be.true | 58 | expect(data[3].read).to.be.true |
63 | expect(notifications[3].read).to.be.true | 59 | expect(data[4].read).to.be.true |
64 | expect(notifications[4].read).to.be.true | 60 | expect(data[5].read).to.be.false |
65 | expect(notifications[5].read).to.be.false | ||
66 | }) | 61 | }) |
67 | 62 | ||
68 | it('Should only list read notifications', async function () { | 63 | it('Should only list read notifications', async function () { |
69 | const res = await getUserNotifications(server.url, userAccessToken, 0, 10, false) | 64 | const { data } = await server.notifications.list({ token: userToken, start: 0, count: 10, unread: false }) |
70 | 65 | ||
71 | const notifications = res.body.data as UserNotification[] | 66 | for (const notification of data) { |
72 | for (const notification of notifications) { | ||
73 | expect(notification.read).to.be.true | 67 | expect(notification.read).to.be.true |
74 | } | 68 | } |
75 | }) | 69 | }) |
76 | 70 | ||
77 | it('Should only list unread notifications', async function () { | 71 | it('Should only list unread notifications', async function () { |
78 | const res = await getUserNotifications(server.url, userAccessToken, 0, 10, true) | 72 | const { data } = await server.notifications.list({ token: userToken, start: 0, count: 10, unread: true }) |
79 | 73 | ||
80 | const notifications = res.body.data as UserNotification[] | 74 | for (const notification of data) { |
81 | for (const notification of notifications) { | ||
82 | expect(notification.read).to.be.false | 75 | expect(notification.read).to.be.false |
83 | } | 76 | } |
84 | }) | 77 | }) |
85 | 78 | ||
86 | it('Should mark as read all notifications', async function () { | 79 | it('Should mark as read all notifications', async function () { |
87 | await markAsReadAllNotifications(server.url, userAccessToken) | 80 | await server.notifications.markAsReadAll({ token: userToken }) |
88 | 81 | ||
89 | const res = await getUserNotifications(server.url, userAccessToken, 0, 10, true) | 82 | const body = await server.notifications.list({ token: userToken, start: 0, count: 10, unread: true }) |
90 | 83 | ||
91 | expect(res.body.total).to.equal(0) | 84 | expect(body.total).to.equal(0) |
92 | expect(res.body.data).to.have.lengthOf(0) | 85 | expect(body.data).to.have.lengthOf(0) |
93 | }) | 86 | }) |
94 | }) | 87 | }) |
95 | 88 | ||
@@ -101,97 +94,100 @@ describe('Test notifications API', function () { | |||
101 | server: server, | 94 | server: server, |
102 | emails, | 95 | emails, |
103 | socketNotifications: userNotifications, | 96 | socketNotifications: userNotifications, |
104 | token: userAccessToken | 97 | token: userToken |
105 | } | 98 | } |
106 | }) | 99 | }) |
107 | 100 | ||
108 | it('Should not have notifications', async function () { | 101 | it('Should not have notifications', async function () { |
109 | this.timeout(20000) | 102 | this.timeout(20000) |
110 | 103 | ||
111 | await updateMyNotificationSettings(server.url, userAccessToken, immutableAssign(getAllNotificationsSettings(), { | 104 | await server.notifications.updateMySettings({ |
112 | newVideoFromSubscription: UserNotificationSettingValue.NONE | 105 | token: userToken, |
113 | })) | 106 | settings: { ...getAllNotificationsSettings(), newVideoFromSubscription: UserNotificationSettingValue.NONE } |
107 | }) | ||
114 | 108 | ||
115 | { | 109 | { |
116 | const res = await getMyUserInformation(server.url, userAccessToken) | 110 | const info = await server.users.getMyInfo({ token: userToken }) |
117 | const info = res.body as User | ||
118 | expect(info.notificationSettings.newVideoFromSubscription).to.equal(UserNotificationSettingValue.NONE) | 111 | expect(info.notificationSettings.newVideoFromSubscription).to.equal(UserNotificationSettingValue.NONE) |
119 | } | 112 | } |
120 | 113 | ||
121 | const { name, uuid } = await uploadRandomVideo(server) | 114 | const { name, uuid } = await server.videos.randomUpload() |
122 | 115 | ||
123 | const check = { web: true, mail: true } | 116 | const check = { web: true, mail: true } |
124 | await checkNewVideoFromSubscription(immutableAssign(baseParams, { check }), name, uuid, 'absence') | 117 | await checkNewVideoFromSubscription({ ...baseParams, check }, name, uuid, 'absence') |
125 | }) | 118 | }) |
126 | 119 | ||
127 | it('Should only have web notifications', async function () { | 120 | it('Should only have web notifications', async function () { |
128 | this.timeout(20000) | 121 | this.timeout(20000) |
129 | 122 | ||
130 | await updateMyNotificationSettings(server.url, userAccessToken, immutableAssign(getAllNotificationsSettings(), { | 123 | await server.notifications.updateMySettings({ |
131 | newVideoFromSubscription: UserNotificationSettingValue.WEB | 124 | token: userToken, |
132 | })) | 125 | settings: { ...getAllNotificationsSettings(), newVideoFromSubscription: UserNotificationSettingValue.WEB } |
126 | }) | ||
133 | 127 | ||
134 | { | 128 | { |
135 | const res = await getMyUserInformation(server.url, userAccessToken) | 129 | const info = await server.users.getMyInfo({ token: userToken }) |
136 | const info = res.body as User | ||
137 | expect(info.notificationSettings.newVideoFromSubscription).to.equal(UserNotificationSettingValue.WEB) | 130 | expect(info.notificationSettings.newVideoFromSubscription).to.equal(UserNotificationSettingValue.WEB) |
138 | } | 131 | } |
139 | 132 | ||
140 | const { name, uuid } = await uploadRandomVideo(server) | 133 | const { name, uuid } = await server.videos.randomUpload() |
141 | 134 | ||
142 | { | 135 | { |
143 | const check = { mail: true, web: false } | 136 | const check = { mail: true, web: false } |
144 | await checkNewVideoFromSubscription(immutableAssign(baseParams, { check }), name, uuid, 'absence') | 137 | await checkNewVideoFromSubscription({ ...baseParams, check }, name, uuid, 'absence') |
145 | } | 138 | } |
146 | 139 | ||
147 | { | 140 | { |
148 | const check = { mail: false, web: true } | 141 | const check = { mail: false, web: true } |
149 | await checkNewVideoFromSubscription(immutableAssign(baseParams, { check }), name, uuid, 'presence') | 142 | await checkNewVideoFromSubscription({ ...baseParams, check }, name, uuid, 'presence') |
150 | } | 143 | } |
151 | }) | 144 | }) |
152 | 145 | ||
153 | it('Should only have mail notifications', async function () { | 146 | it('Should only have mail notifications', async function () { |
154 | this.timeout(20000) | 147 | this.timeout(20000) |
155 | 148 | ||
156 | await updateMyNotificationSettings(server.url, userAccessToken, immutableAssign(getAllNotificationsSettings(), { | 149 | await server.notifications.updateMySettings({ |
157 | newVideoFromSubscription: UserNotificationSettingValue.EMAIL | 150 | token: userToken, |
158 | })) | 151 | settings: { ...getAllNotificationsSettings(), newVideoFromSubscription: UserNotificationSettingValue.EMAIL } |
152 | }) | ||
159 | 153 | ||
160 | { | 154 | { |
161 | const res = await getMyUserInformation(server.url, userAccessToken) | 155 | const info = await server.users.getMyInfo({ token: userToken }) |
162 | const info = res.body as User | ||
163 | expect(info.notificationSettings.newVideoFromSubscription).to.equal(UserNotificationSettingValue.EMAIL) | 156 | expect(info.notificationSettings.newVideoFromSubscription).to.equal(UserNotificationSettingValue.EMAIL) |
164 | } | 157 | } |
165 | 158 | ||
166 | const { name, uuid } = await uploadRandomVideo(server) | 159 | const { name, uuid } = await server.videos.randomUpload() |
167 | 160 | ||
168 | { | 161 | { |
169 | const check = { mail: false, web: true } | 162 | const check = { mail: false, web: true } |
170 | await checkNewVideoFromSubscription(immutableAssign(baseParams, { check }), name, uuid, 'absence') | 163 | await checkNewVideoFromSubscription({ ...baseParams, check }, name, uuid, 'absence') |
171 | } | 164 | } |
172 | 165 | ||
173 | { | 166 | { |
174 | const check = { mail: true, web: false } | 167 | const check = { mail: true, web: false } |
175 | await checkNewVideoFromSubscription(immutableAssign(baseParams, { check }), name, uuid, 'presence') | 168 | await checkNewVideoFromSubscription({ ...baseParams, check }, name, uuid, 'presence') |
176 | } | 169 | } |
177 | }) | 170 | }) |
178 | 171 | ||
179 | it('Should have email and web notifications', async function () { | 172 | it('Should have email and web notifications', async function () { |
180 | this.timeout(20000) | 173 | this.timeout(20000) |
181 | 174 | ||
182 | await updateMyNotificationSettings(server.url, userAccessToken, immutableAssign(getAllNotificationsSettings(), { | 175 | await server.notifications.updateMySettings({ |
183 | newVideoFromSubscription: UserNotificationSettingValue.WEB | UserNotificationSettingValue.EMAIL | 176 | token: userToken, |
184 | })) | 177 | settings: { |
178 | ...getAllNotificationsSettings(), | ||
179 | newVideoFromSubscription: UserNotificationSettingValue.WEB | UserNotificationSettingValue.EMAIL | ||
180 | } | ||
181 | }) | ||
185 | 182 | ||
186 | { | 183 | { |
187 | const res = await getMyUserInformation(server.url, userAccessToken) | 184 | const info = await server.users.getMyInfo({ token: userToken }) |
188 | const info = res.body as User | ||
189 | expect(info.notificationSettings.newVideoFromSubscription).to.equal( | 185 | expect(info.notificationSettings.newVideoFromSubscription).to.equal( |
190 | UserNotificationSettingValue.WEB | UserNotificationSettingValue.EMAIL | 186 | UserNotificationSettingValue.WEB | UserNotificationSettingValue.EMAIL |
191 | ) | 187 | ) |
192 | } | 188 | } |
193 | 189 | ||
194 | const { name, uuid } = await uploadRandomVideo(server) | 190 | const { name, uuid } = await server.videos.randomUpload() |
195 | 191 | ||
196 | await checkNewVideoFromSubscription(baseParams, name, uuid, 'presence') | 192 | await checkNewVideoFromSubscription(baseParams, name, uuid, 'presence') |
197 | }) | 193 | }) |
diff --git a/server/tests/api/notifications/user-notifications.ts b/server/tests/api/notifications/user-notifications.ts index e981c1718..ca592d466 100644 --- a/server/tests/api/notifications/user-notifications.ts +++ b/server/tests/api/notifications/user-notifications.ts | |||
@@ -4,34 +4,26 @@ import 'mocha' | |||
4 | import * as chai from 'chai' | 4 | import * as chai from 'chai' |
5 | import { buildUUID } from '@server/helpers/uuid' | 5 | import { buildUUID } from '@server/helpers/uuid' |
6 | import { | 6 | import { |
7 | cleanupTests, | ||
8 | updateMyUser, | ||
9 | updateVideo, | ||
10 | updateVideoChannel, | ||
11 | uploadRandomVideoOnServers, | ||
12 | wait | ||
13 | } from '../../../../shared/extra-utils' | ||
14 | import { ServerInfo } from '../../../../shared/extra-utils/index' | ||
15 | import { MockSmtpServer } from '../../../../shared/extra-utils/miscs/email' | ||
16 | import { waitJobs } from '../../../../shared/extra-utils/server/jobs' | ||
17 | import { | ||
18 | CheckerBaseParams, | 7 | CheckerBaseParams, |
19 | checkMyVideoImportIsFinished, | 8 | checkMyVideoImportIsFinished, |
20 | checkNewActorFollow, | 9 | checkNewActorFollow, |
21 | checkNewVideoFromSubscription, | 10 | checkNewVideoFromSubscription, |
22 | checkVideoIsPublished, | 11 | checkVideoIsPublished, |
23 | getLastNotification, | 12 | cleanupTests, |
24 | prepareNotificationsTest | 13 | FIXTURE_URLS, |
25 | } from '../../../../shared/extra-utils/users/user-notifications' | 14 | MockSmtpServer, |
26 | import { addUserSubscription, removeUserSubscription } from '../../../../shared/extra-utils/users/user-subscriptions' | 15 | PeerTubeServer, |
27 | import { getBadVideoUrl, getGoodVideoUrl, importVideo } from '../../../../shared/extra-utils/videos/video-imports' | 16 | prepareNotificationsTest, |
28 | import { UserNotification, UserNotificationType } from '../../../../shared/models/users' | 17 | uploadRandomVideoOnServers, |
29 | import { VideoPrivacy } from '../../../../shared/models/videos' | 18 | wait, |
19 | waitJobs | ||
20 | } from '@shared/extra-utils' | ||
21 | import { UserNotification, UserNotificationType, VideoPrivacy } from '@shared/models' | ||
30 | 22 | ||
31 | const expect = chai.expect | 23 | const expect = chai.expect |
32 | 24 | ||
33 | describe('Test user notifications', function () { | 25 | describe('Test user notifications', function () { |
34 | let servers: ServerInfo[] = [] | 26 | let servers: PeerTubeServer[] = [] |
35 | let userAccessToken: string | 27 | let userAccessToken: string |
36 | let userNotifications: UserNotification[] = [] | 28 | let userNotifications: UserNotification[] = [] |
37 | let adminNotifications: UserNotification[] = [] | 29 | let adminNotifications: UserNotification[] = [] |
@@ -69,7 +61,7 @@ describe('Test user notifications', function () { | |||
69 | 61 | ||
70 | await uploadRandomVideoOnServers(servers, 1) | 62 | await uploadRandomVideoOnServers(servers, 1) |
71 | 63 | ||
72 | const notification = await getLastNotification(servers[0].url, userAccessToken) | 64 | const notification = await servers[0].notifications.getLastest({ token: userAccessToken }) |
73 | expect(notification).to.be.undefined | 65 | expect(notification).to.be.undefined |
74 | 66 | ||
75 | expect(emails).to.have.lengthOf(0) | 67 | expect(emails).to.have.lengthOf(0) |
@@ -79,7 +71,7 @@ describe('Test user notifications', function () { | |||
79 | it('Should send a new video notification if the user follows the local video publisher', async function () { | 71 | it('Should send a new video notification if the user follows the local video publisher', async function () { |
80 | this.timeout(15000) | 72 | this.timeout(15000) |
81 | 73 | ||
82 | await addUserSubscription(servers[0].url, userAccessToken, 'root_channel@localhost:' + servers[0].port) | 74 | await servers[0].subscriptions.add({ token: userAccessToken, targetUri: 'root_channel@localhost:' + servers[0].port }) |
83 | await waitJobs(servers) | 75 | await waitJobs(servers) |
84 | 76 | ||
85 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 1) | 77 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 1) |
@@ -89,7 +81,7 @@ describe('Test user notifications', function () { | |||
89 | it('Should send a new video notification from a remote account', async function () { | 81 | it('Should send a new video notification from a remote account', async function () { |
90 | this.timeout(150000) // Server 2 has transcoding enabled | 82 | this.timeout(150000) // Server 2 has transcoding enabled |
91 | 83 | ||
92 | await addUserSubscription(servers[0].url, userAccessToken, 'root_channel@localhost:' + servers[1].port) | 84 | await servers[0].subscriptions.add({ token: userAccessToken, targetUri: 'root_channel@localhost:' + servers[1].port }) |
93 | await waitJobs(servers) | 85 | await waitJobs(servers) |
94 | 86 | ||
95 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 2) | 87 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 2) |
@@ -106,7 +98,7 @@ describe('Test user notifications', function () { | |||
106 | privacy: VideoPrivacy.PRIVATE, | 98 | privacy: VideoPrivacy.PRIVATE, |
107 | scheduleUpdate: { | 99 | scheduleUpdate: { |
108 | updateAt: updateAt.toISOString(), | 100 | updateAt: updateAt.toISOString(), |
109 | privacy: VideoPrivacy.PUBLIC | 101 | privacy: VideoPrivacy.PUBLIC as VideoPrivacy.PUBLIC |
110 | } | 102 | } |
111 | } | 103 | } |
112 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 1, data) | 104 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 1, data) |
@@ -125,7 +117,7 @@ describe('Test user notifications', function () { | |||
125 | privacy: VideoPrivacy.PRIVATE, | 117 | privacy: VideoPrivacy.PRIVATE, |
126 | scheduleUpdate: { | 118 | scheduleUpdate: { |
127 | updateAt: updateAt.toISOString(), | 119 | updateAt: updateAt.toISOString(), |
128 | privacy: VideoPrivacy.PUBLIC | 120 | privacy: VideoPrivacy.PUBLIC as VideoPrivacy.PUBLIC |
129 | } | 121 | } |
130 | } | 122 | } |
131 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 2, data) | 123 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 2, data) |
@@ -144,7 +136,7 @@ describe('Test user notifications', function () { | |||
144 | privacy: VideoPrivacy.PRIVATE, | 136 | privacy: VideoPrivacy.PRIVATE, |
145 | scheduleUpdate: { | 137 | scheduleUpdate: { |
146 | updateAt: updateAt.toISOString(), | 138 | updateAt: updateAt.toISOString(), |
147 | privacy: VideoPrivacy.PUBLIC | 139 | privacy: VideoPrivacy.PUBLIC as VideoPrivacy.PUBLIC |
148 | } | 140 | } |
149 | } | 141 | } |
150 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 1, data) | 142 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 1, data) |
@@ -161,7 +153,7 @@ describe('Test user notifications', function () { | |||
161 | 153 | ||
162 | await checkNewVideoFromSubscription(baseParams, name, uuid, 'absence') | 154 | await checkNewVideoFromSubscription(baseParams, name, uuid, 'absence') |
163 | 155 | ||
164 | await updateVideo(servers[0].url, servers[0].accessToken, uuid, { privacy: VideoPrivacy.PUBLIC }) | 156 | await servers[0].videos.update({ id: uuid, attributes: { privacy: VideoPrivacy.PUBLIC } }) |
165 | 157 | ||
166 | await waitJobs(servers) | 158 | await waitJobs(servers) |
167 | await checkNewVideoFromSubscription(baseParams, name, uuid, 'presence') | 159 | await checkNewVideoFromSubscription(baseParams, name, uuid, 'presence') |
@@ -175,7 +167,7 @@ describe('Test user notifications', function () { | |||
175 | 167 | ||
176 | await checkNewVideoFromSubscription(baseParams, name, uuid, 'absence') | 168 | await checkNewVideoFromSubscription(baseParams, name, uuid, 'absence') |
177 | 169 | ||
178 | await updateVideo(servers[1].url, servers[1].accessToken, uuid, { privacy: VideoPrivacy.PUBLIC }) | 170 | await servers[1].videos.update({ id: uuid, attributes: { privacy: VideoPrivacy.PUBLIC } }) |
179 | 171 | ||
180 | await waitJobs(servers) | 172 | await waitJobs(servers) |
181 | await checkNewVideoFromSubscription(baseParams, name, uuid, 'presence') | 173 | await checkNewVideoFromSubscription(baseParams, name, uuid, 'presence') |
@@ -187,7 +179,7 @@ describe('Test user notifications', function () { | |||
187 | const data = { privacy: VideoPrivacy.PRIVATE } | 179 | const data = { privacy: VideoPrivacy.PRIVATE } |
188 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 1, data) | 180 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 1, data) |
189 | 181 | ||
190 | await updateVideo(servers[0].url, servers[0].accessToken, uuid, { privacy: VideoPrivacy.UNLISTED }) | 182 | await servers[0].videos.update({ id: uuid, attributes: { privacy: VideoPrivacy.UNLISTED } }) |
191 | 183 | ||
192 | await checkNewVideoFromSubscription(baseParams, name, uuid, 'absence') | 184 | await checkNewVideoFromSubscription(baseParams, name, uuid, 'absence') |
193 | }) | 185 | }) |
@@ -198,7 +190,7 @@ describe('Test user notifications', function () { | |||
198 | const data = { privacy: VideoPrivacy.PRIVATE } | 190 | const data = { privacy: VideoPrivacy.PRIVATE } |
199 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 2, data) | 191 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 2, data) |
200 | 192 | ||
201 | await updateVideo(servers[1].url, servers[1].accessToken, uuid, { privacy: VideoPrivacy.UNLISTED }) | 193 | await servers[1].videos.update({ id: uuid, attributes: { privacy: VideoPrivacy.UNLISTED } }) |
202 | 194 | ||
203 | await waitJobs(servers) | 195 | await waitJobs(servers) |
204 | await checkNewVideoFromSubscription(baseParams, name, uuid, 'absence') | 196 | await checkNewVideoFromSubscription(baseParams, name, uuid, 'absence') |
@@ -213,14 +205,13 @@ describe('Test user notifications', function () { | |||
213 | name, | 205 | name, |
214 | channelId, | 206 | channelId, |
215 | privacy: VideoPrivacy.PUBLIC, | 207 | privacy: VideoPrivacy.PUBLIC, |
216 | targetUrl: getGoodVideoUrl() | 208 | targetUrl: FIXTURE_URLS.goodVideo |
217 | } | 209 | } |
218 | const res = await importVideo(servers[0].url, servers[0].accessToken, attributes) | 210 | const { video } = await servers[0].imports.importVideo({ attributes }) |
219 | const uuid = res.body.video.uuid | ||
220 | 211 | ||
221 | await waitJobs(servers) | 212 | await waitJobs(servers) |
222 | 213 | ||
223 | await checkNewVideoFromSubscription(baseParams, name, uuid, 'presence') | 214 | await checkNewVideoFromSubscription(baseParams, name, video.uuid, 'presence') |
224 | }) | 215 | }) |
225 | }) | 216 | }) |
226 | 217 | ||
@@ -251,7 +242,7 @@ describe('Test user notifications', function () { | |||
251 | await uploadRandomVideoOnServers(servers, 2, { waitTranscoding: false }) | 242 | await uploadRandomVideoOnServers(servers, 2, { waitTranscoding: false }) |
252 | await waitJobs(servers) | 243 | await waitJobs(servers) |
253 | 244 | ||
254 | const notification = await getLastNotification(servers[0].url, userAccessToken) | 245 | const notification = await servers[0].notifications.getLastest({ token: userAccessToken }) |
255 | if (notification) { | 246 | if (notification) { |
256 | expect(notification.type).to.not.equal(UserNotificationType.MY_VIDEO_PUBLISHED) | 247 | expect(notification.type).to.not.equal(UserNotificationType.MY_VIDEO_PUBLISHED) |
257 | } | 248 | } |
@@ -284,14 +275,13 @@ describe('Test user notifications', function () { | |||
284 | name, | 275 | name, |
285 | channelId, | 276 | channelId, |
286 | privacy: VideoPrivacy.PUBLIC, | 277 | privacy: VideoPrivacy.PUBLIC, |
287 | targetUrl: getGoodVideoUrl(), | 278 | targetUrl: FIXTURE_URLS.goodVideo, |
288 | waitTranscoding: true | 279 | waitTranscoding: true |
289 | } | 280 | } |
290 | const res = await importVideo(servers[1].url, servers[1].accessToken, attributes) | 281 | const { video } = await servers[1].imports.importVideo({ attributes }) |
291 | const uuid = res.body.video.uuid | ||
292 | 282 | ||
293 | await waitJobs(servers) | 283 | await waitJobs(servers) |
294 | await checkVideoIsPublished(baseParams, name, uuid, 'presence') | 284 | await checkVideoIsPublished(baseParams, name, video.uuid, 'presence') |
295 | }) | 285 | }) |
296 | 286 | ||
297 | it('Should send a notification when the scheduled update has been proceeded', async function () { | 287 | it('Should send a notification when the scheduled update has been proceeded', async function () { |
@@ -304,7 +294,7 @@ describe('Test user notifications', function () { | |||
304 | privacy: VideoPrivacy.PRIVATE, | 294 | privacy: VideoPrivacy.PRIVATE, |
305 | scheduleUpdate: { | 295 | scheduleUpdate: { |
306 | updateAt: updateAt.toISOString(), | 296 | updateAt: updateAt.toISOString(), |
307 | privacy: VideoPrivacy.PUBLIC | 297 | privacy: VideoPrivacy.PUBLIC as VideoPrivacy.PUBLIC |
308 | } | 298 | } |
309 | } | 299 | } |
310 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 2, data) | 300 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 2, data) |
@@ -322,7 +312,7 @@ describe('Test user notifications', function () { | |||
322 | privacy: VideoPrivacy.PRIVATE, | 312 | privacy: VideoPrivacy.PRIVATE, |
323 | scheduleUpdate: { | 313 | scheduleUpdate: { |
324 | updateAt: updateAt.toISOString(), | 314 | updateAt: updateAt.toISOString(), |
325 | privacy: VideoPrivacy.PUBLIC | 315 | privacy: VideoPrivacy.PUBLIC as VideoPrivacy.PUBLIC |
326 | } | 316 | } |
327 | } | 317 | } |
328 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 2, data) | 318 | const { name, uuid } = await uploadRandomVideoOnServers(servers, 2, data) |
@@ -353,13 +343,12 @@ describe('Test user notifications', function () { | |||
353 | name, | 343 | name, |
354 | channelId, | 344 | channelId, |
355 | privacy: VideoPrivacy.PRIVATE, | 345 | privacy: VideoPrivacy.PRIVATE, |
356 | targetUrl: getBadVideoUrl() | 346 | targetUrl: FIXTURE_URLS.badVideo |
357 | } | 347 | } |
358 | const res = await importVideo(servers[0].url, servers[0].accessToken, attributes) | 348 | const { video } = await servers[0].imports.importVideo({ attributes }) |
359 | const uuid = res.body.video.uuid | ||
360 | 349 | ||
361 | await waitJobs(servers) | 350 | await waitJobs(servers) |
362 | await checkMyVideoImportIsFinished(baseParams, name, uuid, getBadVideoUrl(), false, 'presence') | 351 | await checkMyVideoImportIsFinished(baseParams, name, video.uuid, FIXTURE_URLS.badVideo, false, 'presence') |
363 | }) | 352 | }) |
364 | 353 | ||
365 | it('Should send a notification when the video import succeeded', async function () { | 354 | it('Should send a notification when the video import succeeded', async function () { |
@@ -371,13 +360,12 @@ describe('Test user notifications', function () { | |||
371 | name, | 360 | name, |
372 | channelId, | 361 | channelId, |
373 | privacy: VideoPrivacy.PRIVATE, | 362 | privacy: VideoPrivacy.PRIVATE, |
374 | targetUrl: getGoodVideoUrl() | 363 | targetUrl: FIXTURE_URLS.goodVideo |
375 | } | 364 | } |
376 | const res = await importVideo(servers[0].url, servers[0].accessToken, attributes) | 365 | const { video } = await servers[0].imports.importVideo({ attributes }) |
377 | const uuid = res.body.video.uuid | ||
378 | 366 | ||
379 | await waitJobs(servers) | 367 | await waitJobs(servers) |
380 | await checkMyVideoImportIsFinished(baseParams, name, uuid, getGoodVideoUrl(), true, 'presence') | 368 | await checkMyVideoImportIsFinished(baseParams, name, video.uuid, FIXTURE_URLS.goodVideo, true, 'presence') |
381 | }) | 369 | }) |
382 | }) | 370 | }) |
383 | 371 | ||
@@ -394,47 +382,42 @@ describe('Test user notifications', function () { | |||
394 | token: userAccessToken | 382 | token: userAccessToken |
395 | } | 383 | } |
396 | 384 | ||
397 | await updateMyUser({ | 385 | await servers[0].users.updateMe({ displayName: 'super root name' }) |
398 | url: servers[0].url, | ||
399 | accessToken: servers[0].accessToken, | ||
400 | displayName: 'super root name' | ||
401 | }) | ||
402 | 386 | ||
403 | await updateMyUser({ | 387 | await servers[0].users.updateMe({ |
404 | url: servers[0].url, | 388 | token: userAccessToken, |
405 | accessToken: userAccessToken, | ||
406 | displayName: myUserName | 389 | displayName: myUserName |
407 | }) | 390 | }) |
408 | 391 | ||
409 | await updateMyUser({ | 392 | await servers[1].users.updateMe({ displayName: 'super root 2 name' }) |
410 | url: servers[1].url, | ||
411 | accessToken: servers[1].accessToken, | ||
412 | displayName: 'super root 2 name' | ||
413 | }) | ||
414 | 393 | ||
415 | await updateVideoChannel(servers[0].url, userAccessToken, 'user_1_channel', { displayName: myChannelName }) | 394 | await servers[0].channels.update({ |
395 | token: userAccessToken, | ||
396 | channelName: 'user_1_channel', | ||
397 | attributes: { displayName: myChannelName } | ||
398 | }) | ||
416 | }) | 399 | }) |
417 | 400 | ||
418 | it('Should notify when a local channel is following one of our channel', async function () { | 401 | it('Should notify when a local channel is following one of our channel', async function () { |
419 | this.timeout(50000) | 402 | this.timeout(50000) |
420 | 403 | ||
421 | await addUserSubscription(servers[0].url, servers[0].accessToken, 'user_1_channel@localhost:' + servers[0].port) | 404 | await servers[0].subscriptions.add({ targetUri: 'user_1_channel@localhost:' + servers[0].port }) |
422 | await waitJobs(servers) | 405 | await waitJobs(servers) |
423 | 406 | ||
424 | await checkNewActorFollow(baseParams, 'channel', 'root', 'super root name', myChannelName, 'presence') | 407 | await checkNewActorFollow(baseParams, 'channel', 'root', 'super root name', myChannelName, 'presence') |
425 | 408 | ||
426 | await removeUserSubscription(servers[0].url, servers[0].accessToken, 'user_1_channel@localhost:' + servers[0].port) | 409 | await servers[0].subscriptions.remove({ uri: 'user_1_channel@localhost:' + servers[0].port }) |
427 | }) | 410 | }) |
428 | 411 | ||
429 | it('Should notify when a remote channel is following one of our channel', async function () { | 412 | it('Should notify when a remote channel is following one of our channel', async function () { |
430 | this.timeout(50000) | 413 | this.timeout(50000) |
431 | 414 | ||
432 | await addUserSubscription(servers[1].url, servers[1].accessToken, 'user_1_channel@localhost:' + servers[0].port) | 415 | await servers[1].subscriptions.add({ targetUri: 'user_1_channel@localhost:' + servers[0].port }) |
433 | await waitJobs(servers) | 416 | await waitJobs(servers) |
434 | 417 | ||
435 | await checkNewActorFollow(baseParams, 'channel', 'root', 'super root 2 name', myChannelName, 'presence') | 418 | await checkNewActorFollow(baseParams, 'channel', 'root', 'super root 2 name', myChannelName, 'presence') |
436 | 419 | ||
437 | await removeUserSubscription(servers[1].url, servers[1].accessToken, 'user_1_channel@localhost:' + servers[0].port) | 420 | await servers[1].subscriptions.remove({ uri: 'user_1_channel@localhost:' + servers[0].port }) |
438 | }) | 421 | }) |
439 | 422 | ||
440 | // PeerTube does not support accout -> account follows | 423 | // PeerTube does not support accout -> account follows |