X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Ftests%2Fapi%2Factivitypub%2Frefresher.ts;h=c717f1a307676ce0b0fecd90b43dc9d3aa24df2d;hb=cf21b2cbef61929177b9c09b5e017c3b7eb8535d;hp=921ee874c7e9893b2f955c6cd1f561bdc7a06746;hpb=1600235a2f4e30c5d4e7d4342d1c299845decc60;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/tests/api/activitypub/refresher.ts b/server/tests/api/activitypub/refresher.ts index 921ee874c..c717f1a30 100644 --- a/server/tests/api/activitypub/refresher.ts +++ b/server/tests/api/activitypub/refresher.ts @@ -1,4 +1,4 @@ -/* tslint:disable:no-unused-expression */ +/* eslint-disable @typescript-eslint/no-unused-expressions,@typescript-eslint/require-await */ import 'mocha' import { @@ -24,6 +24,7 @@ import { } from '../../../../shared/extra-utils' import { getAccount } from '../../../../shared/extra-utils/users/accounts' import { VideoPlaylistPrivacy } from '../../../../shared/models/videos' +import { HttpStatusCode } from '../../../../shared/core-utils/miscs/http-error-codes' describe('Test AP refresher', function () { let servers: ServerInfo[] = [] @@ -43,32 +44,32 @@ describe('Test AP refresher', function () { await setDefaultVideoChannel(servers) { - videoUUID1 = (await uploadVideoAndGetId({ server: servers[ 1 ], videoName: 'video1' })).uuid - videoUUID2 = (await uploadVideoAndGetId({ server: servers[ 1 ], videoName: 'video2' })).uuid - videoUUID3 = (await uploadVideoAndGetId({ server: servers[ 1 ], videoName: 'video3' })).uuid + videoUUID1 = (await uploadVideoAndGetId({ server: servers[1], videoName: 'video1' })).uuid + videoUUID2 = (await uploadVideoAndGetId({ server: servers[1], videoName: 'video2' })).uuid + videoUUID3 = (await uploadVideoAndGetId({ server: servers[1], videoName: 'video3' })).uuid } { - const a1 = await generateUserAccessToken(servers[ 1 ], 'user1') - await uploadVideo(servers[ 1 ].url, a1, { name: 'video4' }) + const a1 = await generateUserAccessToken(servers[1], 'user1') + await uploadVideo(servers[1].url, a1, { name: 'video4' }) - const a2 = await generateUserAccessToken(servers[ 1 ], 'user2') - await uploadVideo(servers[ 1 ].url, a2, { name: 'video5' }) + const a2 = await generateUserAccessToken(servers[1], 'user2') + await uploadVideo(servers[1].url, a2, { name: 'video5' }) } { - const playlistAttrs = { displayName: 'playlist1', privacy: VideoPlaylistPrivacy.PUBLIC, videoChannelId: servers[ 1 ].videoChannel.id } - const res = await createVideoPlaylist({ url: servers[ 1 ].url, token: servers[ 1 ].accessToken, playlistAttrs }) + const playlistAttrs = { displayName: 'playlist1', privacy: VideoPlaylistPrivacy.PUBLIC, videoChannelId: servers[1].videoChannel.id } + const res = await createVideoPlaylist({ url: servers[1].url, token: servers[1].accessToken, playlistAttrs }) playlistUUID1 = res.body.videoPlaylist.uuid } { - const playlistAttrs = { displayName: 'playlist2', privacy: VideoPlaylistPrivacy.PUBLIC, videoChannelId: servers[ 1 ].videoChannel.id } - const res = await createVideoPlaylist({ url: servers[ 1 ].url, token: servers[ 1 ].accessToken, playlistAttrs }) + const playlistAttrs = { displayName: 'playlist2', privacy: VideoPlaylistPrivacy.PUBLIC, videoChannelId: servers[1].videoChannel.id } + const res = await createVideoPlaylist({ url: servers[1].url, token: servers[1].accessToken, playlistAttrs }) playlistUUID2 = res.body.videoPlaylist.uuid } - await doubleFollow(servers[ 0 ], servers[ 1 ]) + await doubleFollow(servers[0], servers[1]) }) describe('Videos refresher', function () { @@ -79,38 +80,34 @@ describe('Test AP refresher', function () { await wait(10000) // Change UUID so the remote server returns a 404 - await setVideoField(servers[ 1 ].internalServerNumber, videoUUID1, 'uuid', '304afe4f-39f9-4d49-8ed7-ac57b86b174f') + await setVideoField(servers[1].internalServerNumber, videoUUID1, 'uuid', '304afe4f-39f9-4d49-8ed7-ac57b86b174f') - await getVideo(servers[ 0 ].url, videoUUID1) - await getVideo(servers[ 0 ].url, videoUUID2) + await getVideo(servers[0].url, videoUUID1) + await getVideo(servers[0].url, videoUUID2) await waitJobs(servers) - await getVideo(servers[ 0 ].url, videoUUID1, 404) - await getVideo(servers[ 0 ].url, videoUUID2, 200) + await getVideo(servers[0].url, videoUUID1, HttpStatusCode.NOT_FOUND_404) + await getVideo(servers[0].url, videoUUID2, HttpStatusCode.OK_200) }) it('Should not update a remote video if the remote instance is down', async function () { - this.timeout(60000) + this.timeout(70000) - killallServers([ servers[ 1 ] ]) + killallServers([ servers[1] ]) - await setVideoField(servers[ 1 ].internalServerNumber, videoUUID3, 'uuid', '304afe4f-39f9-4d49-8ed7-ac57b86b174e') + await setVideoField(servers[1].internalServerNumber, videoUUID3, 'uuid', '304afe4f-39f9-4d49-8ed7-ac57b86b174e') // Video will need a refresh await wait(10000) - await getVideo(servers[ 0 ].url, videoUUID3) + await getVideo(servers[0].url, videoUUID3) // The refresh should fail - await waitJobs([ servers[ 0 ] ]) - - await reRunServer(servers[ 1 ]) + await waitJobs([ servers[0] ]) - // Should not refresh the video, even if the last refresh failed (to avoir a loop on dead instances) - await getVideo(servers[ 0 ].url, videoUUID3) - await waitJobs(servers) + await reRunServer(servers[1]) - await getVideo(servers[ 0 ].url, videoUUID3, 200) + await getVideo(servers[0].url, videoUUID3, HttpStatusCode.OK_200) }) }) @@ -122,16 +119,16 @@ describe('Test AP refresher', function () { await wait(10000) // Change actor name so the remote server returns a 404 - const to = 'http://localhost:' + servers[ 1 ].port + '/accounts/user2' - await setActorField(servers[ 1 ].internalServerNumber, to, 'preferredUsername', 'toto') + const to = 'http://localhost:' + servers[1].port + '/accounts/user2' + await setActorField(servers[1].internalServerNumber, to, 'preferredUsername', 'toto') - await getAccount(servers[ 0 ].url, 'user1@localhost:' + servers[ 1 ].port) - await getAccount(servers[ 0 ].url, 'user2@localhost:' + servers[ 1 ].port) + await getAccount(servers[0].url, 'user1@localhost:' + servers[1].port) + await getAccount(servers[0].url, 'user2@localhost:' + servers[1].port) await waitJobs(servers) - await getAccount(servers[ 0 ].url, 'user1@localhost:' + servers[ 1 ].port, 200) - await getAccount(servers[ 0 ].url, 'user2@localhost:' + servers[ 1 ].port, 404) + await getAccount(servers[0].url, 'user1@localhost:' + servers[1].port, HttpStatusCode.OK_200) + await getAccount(servers[0].url, 'user2@localhost:' + servers[1].port, HttpStatusCode.NOT_FOUND_404) }) }) @@ -143,15 +140,15 @@ describe('Test AP refresher', function () { await wait(10000) // Change UUID so the remote server returns a 404 - await setPlaylistField(servers[ 1 ].internalServerNumber, playlistUUID2, 'uuid', '304afe4f-39f9-4d49-8ed7-ac57b86b178e') + await setPlaylistField(servers[1].internalServerNumber, playlistUUID2, 'uuid', '304afe4f-39f9-4d49-8ed7-ac57b86b178e') - await getVideoPlaylist(servers[ 0 ].url, playlistUUID1) - await getVideoPlaylist(servers[ 0 ].url, playlistUUID2) + await getVideoPlaylist(servers[0].url, playlistUUID1) + await getVideoPlaylist(servers[0].url, playlistUUID2) await waitJobs(servers) - await getVideoPlaylist(servers[ 0 ].url, playlistUUID1, 200) - await getVideoPlaylist(servers[ 0 ].url, playlistUUID2, 404) + await getVideoPlaylist(servers[0].url, playlistUUID1, HttpStatusCode.OK_200) + await getVideoPlaylist(servers[0].url, playlistUUID2, HttpStatusCode.NOT_FOUND_404) }) })