X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Ftests%2Fapi%2Fnotifications%2Fmoderation-notifications.ts;h=e7a5c47e953f87ddd10a1687bdbfe15e8745d27c;hb=d1c64fd970fc236b9ba01ac944659051e9e04a99;hp=eb3c29fe74532ce421e5a0ab11b880945a536783;hpb=29837f8885eb37fa300e4b80c90a6d03ab337084;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/tests/api/notifications/moderation-notifications.ts b/server/tests/api/notifications/moderation-notifications.ts index eb3c29fe7..e7a5c47e9 100644 --- a/server/tests/api/notifications/moderation-notifications.ts +++ b/server/tests/api/notifications/moderation-notifications.ts @@ -1,7 +1,5 @@ /* eslint-disable @typescript-eslint/no-unused-expressions,@typescript-eslint/require-await */ -import 'mocha' -import { buildUUID } from '@server/helpers/uuid' import { checkAbuseStateChange, checkAutoInstanceFollowing, @@ -13,22 +11,22 @@ import { checkNewInstanceFollower, checkNewVideoAbuseForModerators, checkNewVideoFromSubscription, - checkUserRegistered, checkVideoAutoBlacklistForModerators, checkVideoIsPublished, - cleanupTests, MockInstancesIndex, MockSmtpServer, - PeerTubeServer, - prepareNotificationsTest, - wait, - waitJobs -} from '@shared/extra-utils' -import { AbuseState, CustomConfig, UserNotification, VideoPrivacy } from '@shared/models' + prepareNotificationsTest +} from '@server/tests/shared' +import { wait } from '@shared/core-utils' +import { buildUUID } from '@shared/extra-utils' +import { AbuseState, CustomConfig, UserNotification, UserRole, VideoPrivacy } from '@shared/models' +import { cleanupTests, PeerTubeServer, waitJobs } from '@shared/server-commands' describe('Test moderation notifications', function () { let servers: PeerTubeServer[] = [] - let userAccessToken: string + let userToken1: string + let userToken2: string + let userNotifications: UserNotification[] = [] let adminNotifications: UserNotification[] = [] let adminNotificationsServer2: UserNotification[] = [] @@ -39,11 +37,13 @@ describe('Test moderation notifications', function () { const res = await prepareNotificationsTest(3) emails = res.emails - userAccessToken = res.userAccessToken + userToken1 = res.userAccessToken servers = res.servers userNotifications = res.userNotifications adminNotifications = res.adminNotifications adminNotificationsServer2 = res.adminNotificationsServer2 + + userToken2 = await servers[1].users.generateUserAndToken('user2', UserRole.USER) }) describe('Abuse for moderators notification', function () { @@ -58,60 +58,72 @@ describe('Test moderation notifications', function () { } }) - it('Should send a notification to moderators on local video abuse', async function () { - this.timeout(20000) + it('Should not send a notification to moderators on local abuse reported by an admin', async function () { + this.timeout(50000) const name = 'video for abuse ' + buildUUID() - const video = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) + const video = await servers[0].videos.upload({ token: userToken1, attributes: { name } }) await servers[0].abuses.report({ videoId: video.id, reason: 'super reason' }) + await waitJobs(servers) + await checkNewVideoAbuseForModerators({ ...baseParams, shortUUID: video.shortUUID, videoName: name, checkType: 'absence' }) + }) + + it('Should send a notification to moderators on local video abuse', async function () { + this.timeout(50000) + + const name = 'video for abuse ' + buildUUID() + const video = await servers[0].videos.upload({ token: userToken1, attributes: { name } }) + + await servers[0].abuses.report({ token: userToken1, videoId: video.id, reason: 'super reason' }) + await waitJobs(servers) await checkNewVideoAbuseForModerators({ ...baseParams, shortUUID: video.shortUUID, videoName: name, checkType: 'presence' }) }) it('Should send a notification to moderators on remote video abuse', async function () { - this.timeout(20000) + this.timeout(50000) const name = 'video for abuse ' + buildUUID() - const video = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) + const video = await servers[0].videos.upload({ token: userToken1, attributes: { name } }) await waitJobs(servers) const videoId = await servers[1].videos.getId({ uuid: video.uuid }) - await servers[1].abuses.report({ videoId, reason: 'super reason' }) + await servers[1].abuses.report({ token: userToken2, videoId, reason: 'super reason' }) await waitJobs(servers) await checkNewVideoAbuseForModerators({ ...baseParams, shortUUID: video.shortUUID, videoName: name, checkType: 'presence' }) }) it('Should send a notification to moderators on local comment abuse', async function () { - this.timeout(20000) + this.timeout(50000) const name = 'video for abuse ' + buildUUID() - const video = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) + const video = await servers[0].videos.upload({ token: userToken1, attributes: { name } }) const comment = await servers[0].comments.createThread({ - token: userAccessToken, + token: userToken1, videoId: video.id, text: 'comment abuse ' + buildUUID() }) await waitJobs(servers) - await servers[0].abuses.report({ commentId: comment.id, reason: 'super reason' }) + await servers[0].abuses.report({ token: userToken1, commentId: comment.id, reason: 'super reason' }) await waitJobs(servers) await checkNewCommentAbuseForModerators({ ...baseParams, shortUUID: video.shortUUID, videoName: name, checkType: 'presence' }) }) it('Should send a notification to moderators on remote comment abuse', async function () { - this.timeout(20000) + this.timeout(50000) const name = 'video for abuse ' + buildUUID() - const video = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) + const video = await servers[0].videos.upload({ token: userToken1, attributes: { name } }) await servers[0].comments.createThread({ - token: userAccessToken, + token: userToken1, videoId: video.id, text: 'comment abuse ' + buildUUID() }) @@ -120,27 +132,27 @@ describe('Test moderation notifications', function () { const { data } = await servers[1].comments.listThreads({ videoId: video.uuid }) const commentId = data[0].id - await servers[1].abuses.report({ commentId, reason: 'super reason' }) + await servers[1].abuses.report({ token: userToken2, commentId, reason: 'super reason' }) await waitJobs(servers) await checkNewCommentAbuseForModerators({ ...baseParams, shortUUID: video.shortUUID, videoName: name, checkType: 'presence' }) }) it('Should send a notification to moderators on local account abuse', async function () { - this.timeout(20000) + this.timeout(50000) const username = 'user' + new Date().getTime() const { account } = await servers[0].users.create({ username, password: 'donald' }) const accountId = account.id - await servers[0].abuses.report({ accountId, reason: 'super reason' }) + await servers[0].abuses.report({ token: userToken1, accountId, reason: 'super reason' }) await waitJobs(servers) await checkNewAccountAbuseForModerators({ ...baseParams, displayName: username, checkType: 'presence' }) }) it('Should send a notification to moderators on remote account abuse', async function () { - this.timeout(20000) + this.timeout(50000) const username = 'user' + new Date().getTime() const tmpToken = await servers[0].users.generateUserAndToken(username) @@ -149,7 +161,7 @@ describe('Test moderation notifications', function () { await waitJobs(servers) const account = await servers[1].accounts.get({ accountName: username + '@' + servers[0].host }) - await servers[1].abuses.report({ accountId: account.id, reason: 'super reason' }) + await servers[1].abuses.report({ token: userToken2, accountId: account.id, reason: 'super reason' }) await waitJobs(servers) await checkNewAccountAbuseForModerators({ ...baseParams, displayName: username, checkType: 'presence' }) @@ -165,18 +177,18 @@ describe('Test moderation notifications', function () { server: servers[0], emails, socketNotifications: userNotifications, - token: userAccessToken + token: userToken1 } const name = 'abuse ' + buildUUID() - const video = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) + const video = await servers[0].videos.upload({ token: userToken1, attributes: { name } }) - const body = await servers[0].abuses.report({ token: userAccessToken, videoId: video.id, reason: 'super reason' }) + const body = await servers[0].abuses.report({ token: userToken1, videoId: video.id, reason: 'super reason' }) abuseId = body.abuse.id }) it('Should send a notification to reporter if the abuse has been accepted', async function () { - this.timeout(10000) + this.timeout(30000) await servers[0].abuses.update({ abuseId, body: { state: AbuseState.ACCEPTED } }) await waitJobs(servers) @@ -185,7 +197,7 @@ describe('Test moderation notifications', function () { }) it('Should send a notification to reporter if the abuse has been rejected', async function () { - this.timeout(10000) + this.timeout(30000) await servers[0].abuses.update({ abuseId, body: { state: AbuseState.REJECTED } }) await waitJobs(servers) @@ -205,7 +217,7 @@ describe('Test moderation notifications', function () { server: servers[0], emails, socketNotifications: userNotifications, - token: userAccessToken + token: userToken1 } baseParamsAdmin = { @@ -216,21 +228,21 @@ describe('Test moderation notifications', function () { } const name = 'abuse ' + buildUUID() - const video = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) + const video = await servers[0].videos.upload({ token: userToken1, attributes: { name } }) { - const body = await servers[0].abuses.report({ token: userAccessToken, videoId: video.id, reason: 'super reason' }) + const body = await servers[0].abuses.report({ token: userToken1, videoId: video.id, reason: 'super reason' }) abuseId = body.abuse.id } { - const body = await servers[0].abuses.report({ token: userAccessToken, videoId: video.id, reason: 'super reason 2' }) + const body = await servers[0].abuses.report({ token: userToken1, videoId: video.id, reason: 'super reason 2' }) abuseId2 = body.abuse.id } }) it('Should send a notification to reporter on new message', async function () { - this.timeout(10000) + this.timeout(30000) const message = 'my super message to users' await servers[0].abuses.addMessage({ abuseId, message }) @@ -240,7 +252,7 @@ describe('Test moderation notifications', function () { }) it('Should not send a notification to the admin if sent by the admin', async function () { - this.timeout(10000) + this.timeout(30000) const message = 'my super message that should not be sent to the admin' await servers[0].abuses.addMessage({ abuseId, message }) @@ -251,10 +263,10 @@ describe('Test moderation notifications', function () { }) it('Should send a notification to moderators', async function () { - this.timeout(10000) + this.timeout(30000) const message = 'my super message to moderators' - await servers[0].abuses.addMessage({ token: userAccessToken, abuseId: abuseId2, message }) + await servers[0].abuses.addMessage({ token: userToken1, abuseId: abuseId2, message }) await waitJobs(servers) const toEmail = 'admin' + servers[0].internalServerNumber + '@example.com' @@ -262,10 +274,10 @@ describe('Test moderation notifications', function () { }) it('Should not send a notification to reporter if sent by the reporter', async function () { - this.timeout(10000) + this.timeout(30000) const message = 'my super message that should not be sent to reporter' - await servers[0].abuses.addMessage({ token: userAccessToken, abuseId: abuseId2, message }) + await servers[0].abuses.addMessage({ token: userToken1, abuseId: abuseId2, message }) await waitJobs(servers) const toEmail = 'user_1@example.com' @@ -281,15 +293,15 @@ describe('Test moderation notifications', function () { server: servers[0], emails, socketNotifications: userNotifications, - token: userAccessToken + token: userToken1 } }) it('Should send a notification to video owner on blacklist', async function () { - this.timeout(10000) + this.timeout(30000) const name = 'video for abuse ' + buildUUID() - const { uuid, shortUUID } = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) + const { uuid, shortUUID } = await servers[0].videos.upload({ token: userToken1, attributes: { name } }) await servers[0].blacklist.add({ videoId: uuid }) @@ -298,10 +310,10 @@ describe('Test moderation notifications', function () { }) it('Should send a notification to video owner on unblacklist', async function () { - this.timeout(10000) + this.timeout(30000) const name = 'video for abuse ' + buildUUID() - const { uuid, shortUUID } = await servers[0].videos.upload({ token: userAccessToken, attributes: { name } }) + const { uuid, shortUUID } = await servers[0].videos.upload({ token: userToken1, attributes: { name } }) await servers[0].blacklist.add({ videoId: uuid }) @@ -314,38 +326,12 @@ describe('Test moderation notifications', function () { }) }) - describe('New registration', function () { - let baseParams: CheckerBaseParams - - before(() => { - baseParams = { - server: servers[0], - emails, - socketNotifications: adminNotifications, - token: servers[0].accessToken - } - }) - - it('Should send a notification only to moderators when a user registers on the instance', async function () { - this.timeout(10000) - - await servers[0].users.register({ username: 'user_45' }) - - await waitJobs(servers) - - await checkUserRegistered({ ...baseParams, username: 'user_45', checkType: 'presence' }) - - const userOverride = { socketNotifications: userNotifications, token: userAccessToken, check: { web: true, mail: false } } - await checkUserRegistered({ ...baseParams, ...userOverride, username: 'user_45', checkType: 'absence' }) - }) - }) - describe('New instance follows', function () { const instanceIndexServer = new MockInstancesIndex() let config: any let baseParams: CheckerBaseParams - before(async () => { + before(async function () { baseParams = { server: servers[0], emails, @@ -360,7 +346,7 @@ describe('Test moderation notifications', function () { followings: { instance: { autoFollowIndex: { - indexUrl: `http://localhost:${port}/api/v1/instances/hosts`, + indexUrl: `http://127.0.0.1:${port}/api/v1/instances/hosts`, enabled: true } } @@ -369,16 +355,16 @@ describe('Test moderation notifications', function () { }) it('Should send a notification only to admin when there is a new instance follower', async function () { - this.timeout(20000) + this.timeout(60000) await servers[2].follows.follow({ hosts: [ servers[0].url ] }) await waitJobs(servers) - await checkNewInstanceFollower({ ...baseParams, followerHost: 'localhost:' + servers[2].port, checkType: 'presence' }) + await checkNewInstanceFollower({ ...baseParams, followerHost: servers[2].host, checkType: 'presence' }) - const userOverride = { socketNotifications: userNotifications, token: userAccessToken, check: { web: true, mail: false } } - await checkNewInstanceFollower({ ...baseParams, ...userOverride, followerHost: 'localhost:' + servers[2].port, checkType: 'absence' }) + const userOverride = { socketNotifications: userNotifications, token: userToken1, check: { web: true, mail: false } } + await checkNewInstanceFollower({ ...baseParams, ...userOverride, followerHost: servers[2].host, checkType: 'absence' }) }) it('Should send a notification on auto follow back', async function () { @@ -404,7 +390,7 @@ describe('Test moderation notifications', function () { const followingHost = servers[2].host await checkAutoInstanceFollowing({ ...baseParams, followerHost, followingHost, checkType: 'presence' }) - const userOverride = { socketNotifications: userNotifications, token: userAccessToken, check: { web: true, mail: false } } + const userOverride = { socketNotifications: userNotifications, token: userToken1, check: { web: true, mail: false } } await checkAutoInstanceFollowing({ ...baseParams, ...userOverride, followerHost, followingHost, checkType: 'absence' }) config.followings.instance.autoFollowBack.enabled = false @@ -441,7 +427,7 @@ describe('Test moderation notifications', function () { let videoName: string let currentCustomConfig: CustomConfig - before(async () => { + before(async function () { adminBaseParamsServer1 = { server: servers[0], @@ -461,7 +447,7 @@ describe('Test moderation notifications', function () { server: servers[0], emails, socketNotifications: userNotifications, - token: userAccessToken + token: userToken1 } currentCustomConfig = await servers[0].config.getCustomConfig() @@ -482,15 +468,15 @@ describe('Test moderation notifications', function () { autoBlacklistTestsCustomConfig.transcoding.enabled = true await servers[0].config.updateCustomConfig({ newCustomConfig: autoBlacklistTestsCustomConfig }) - await servers[0].subscriptions.add({ targetUri: 'user_1_channel@localhost:' + servers[0].port }) - await servers[1].subscriptions.add({ targetUri: 'user_1_channel@localhost:' + servers[0].port }) + await servers[0].subscriptions.add({ targetUri: 'user_1_channel@' + servers[0].host }) + await servers[1].subscriptions.add({ targetUri: 'user_1_channel@' + servers[0].host }) }) it('Should send notification to moderators on new video with auto-blacklist', async function () { - this.timeout(40000) + this.timeout(120000) videoName = 'video with auto-blacklist ' + buildUUID() - const video = await servers[0].videos.upload({ token: userAccessToken, attributes: { name: videoName } }) + const video = await servers[0].videos.upload({ token: userToken1, attributes: { name: videoName } }) shortUUID = video.shortUUID uuid = video.uuid @@ -499,10 +485,14 @@ describe('Test moderation notifications', function () { }) it('Should not send video publish notification if auto-blacklisted', async function () { + this.timeout(120000) + await checkVideoIsPublished({ ...userBaseParams, videoName, shortUUID, checkType: 'absence' }) }) it('Should not send a local user subscription notification if auto-blacklisted', async function () { + this.timeout(120000) + await checkNewVideoFromSubscription({ ...adminBaseParamsServer1, videoName, shortUUID, checkType: 'absence' }) }) @@ -511,7 +501,7 @@ describe('Test moderation notifications', function () { }) it('Should send video published and unblacklist after video unblacklisted', async function () { - this.timeout(40000) + this.timeout(120000) await servers[0].blacklist.remove({ videoId: uuid }) @@ -524,15 +514,19 @@ describe('Test moderation notifications', function () { }) it('Should send a local user subscription notification after removed from blacklist', async function () { + this.timeout(120000) + await checkNewVideoFromSubscription({ ...adminBaseParamsServer1, videoName, shortUUID, checkType: 'presence' }) }) it('Should send a remote user subscription notification after removed from blacklist', async function () { + this.timeout(120000) + await checkNewVideoFromSubscription({ ...adminBaseParamsServer2, videoName, shortUUID, checkType: 'presence' }) }) it('Should send unblacklist but not published/subscription notes after unblacklisted if scheduled update pending', async function () { - this.timeout(40000) + this.timeout(120000) const updateAt = new Date(new Date().getTime() + 1000000) @@ -547,7 +541,7 @@ describe('Test moderation notifications', function () { } } - const { shortUUID, uuid } = await servers[0].videos.upload({ token: userAccessToken, attributes }) + const { shortUUID, uuid } = await servers[0].videos.upload({ token: userToken1, attributes }) await servers[0].blacklist.remove({ videoId: uuid }) @@ -563,7 +557,7 @@ describe('Test moderation notifications', function () { }) it('Should not send publish/subscription notifications after scheduled update if video still auto-blacklisted', async function () { - this.timeout(40000) + this.timeout(120000) // In 2 seconds const updateAt = new Date(new Date().getTime() + 2000) @@ -579,7 +573,7 @@ describe('Test moderation notifications', function () { } } - const { shortUUID } = await servers[0].videos.upload({ token: userAccessToken, attributes }) + const { shortUUID } = await servers[0].videos.upload({ token: userToken1, attributes }) await wait(6000) await checkVideoIsPublished({ ...userBaseParams, videoName: name, shortUUID, checkType: 'absence' }) @@ -588,7 +582,7 @@ describe('Test moderation notifications', function () { }) it('Should not send a notification to moderators on new video without auto-blacklist', async function () { - this.timeout(60000) + this.timeout(120000) const name = 'video without auto-blacklist ' + buildUUID() @@ -602,8 +596,8 @@ describe('Test moderation notifications', function () { after(async () => { await servers[0].config.updateCustomConfig({ newCustomConfig: currentCustomConfig }) - await servers[0].subscriptions.remove({ uri: 'user_1_channel@localhost:' + servers[0].port }) - await servers[1].subscriptions.remove({ uri: 'user_1_channel@localhost:' + servers[0].port }) + await servers[0].subscriptions.remove({ uri: 'user_1_channel@' + servers[0].host }) + await servers[1].subscriptions.remove({ uri: 'user_1_channel@' + servers[0].host }) }) })