X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Ftests%2Fapi%2Fusers%2Fusers-multiple-servers.ts;h=3e8b932c05cbdf3e34cd8f0a3d5cbbaeb68c08ba;hb=d0800f7661f13fabe7bb6f4aa0ea50764f106405;hp=bb458f7a501322ac7f05a924f7d39eba16a5b9a2;hpb=2422c46b27790d94fd29a7092170cee5a1b56008;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/tests/api/users/users-multiple-servers.ts b/server/tests/api/users/users-multiple-servers.ts index bb458f7a5..3e8b932c0 100644 --- a/server/tests/api/users/users-multiple-servers.ts +++ b/server/tests/api/users/users-multiple-servers.ts @@ -1,33 +1,45 @@ -/* tslint:disable:no-unused-expression */ +/* eslint-disable @typescript-eslint/no-unused-expressions,@typescript-eslint/require-await */ -import * as chai from 'chai' import 'mocha' -import { Account } from '../../../../shared/models/actors' +import * as chai from 'chai' +import { + checkActorFilesWereRemoved, + checkTmpIsEmpty, + checkVideoFilesWereRemoved, + saveVideoInServers, + testImage +} from '@server/tests/shared' +import { MyUser } from '@shared/models' import { - checkVideoFilesWereRemoved, createUser, doubleFollow, flushAndRunMultipleServers, removeUser, updateMyUser, userLogin, - wait -} from '../../utils' -import { flushTests, getMyUserInformation, killallServers, ServerInfo, testImage, updateMyAvatar, uploadVideo } from '../../utils/index' -import { checkActorFilesWereRemoved, getAccount, getAccountsList } from '../../utils/users/accounts' -import { setAccessTokensToServers } from '../../utils/users/login' + cleanupTests, + createMultipleServers, + doubleFollow, + PeerTubeServer, + setAccessTokensToServers, + setDefaultChannelAvatar, + waitJobs +} from '@shared/server-commands' const expect = chai.expect describe('Test users with multiple servers', function () { - let servers: ServerInfo[] = [] - let user - let userUUID - let userId - let videoUUID - let userAccessToken + let servers: PeerTubeServer[] = [] + + let user: MyUser + let userId: number + + let videoUUID: string + let userAccessToken: string + let userAvatarFilenames: string[] before(async function () { - this.timeout(120000) + this.timeout(120_000) - servers = await flushAndRunMultipleServers(3) + servers = await createMultipleServers(3) // Get the access tokens await setAccessTokensToServers(servers) + await setDefaultChannelAvatar(servers) // Server 1 and server 2 follow each other await doubleFollow(servers[0], servers[1]) @@ -37,115 +49,175 @@ describe('Test users with multiple servers', function () { await doubleFollow(servers[1], servers[2]) // The root user of server 1 is propagated to servers 2 and 3 - await uploadVideo(servers[0].url, servers[0].accessToken, {}) + await servers[0].videos.upload() - const user = { - username: 'user1', - password: 'password' + { + const username = 'user1' + const created = await servers[0].users.create({ username }) + userId = created.id + userAccessToken = await servers[0].login.getAccessToken(username) } - const resUser = await createUser(servers[0].url, servers[0].accessToken, user.username, user.password) - userUUID = resUser.body.user.uuid - userId = resUser.body.user.id - userAccessToken = await userLogin(servers[0], user) - const resVideo = await uploadVideo(servers[0].url, userAccessToken, {}) - videoUUID = resVideo.body.uuid + { + const { uuid } = await servers[0].videos.upload({ token: userAccessToken }) + videoUUID = uuid + + await waitJobs(servers) - await wait(5000) + await saveVideoInServers(servers, videoUUID) + } }) - it('Should be able to update my description', async function () { + it('Should be able to update my display name', async function () { this.timeout(10000) - await updateMyUser({ - url: servers[0].url, - accessToken: servers[0].accessToken, - description: 'my super description updated' - }) + await servers[0].users.updateMe({ displayName: 'my super display name' }) + + user = await servers[0].users.getMyInfo() + expect(user.account.displayName).to.equal('my super display name') + + await waitJobs(servers) + }) - const res = await getMyUserInformation(servers[0].url, servers[0].accessToken) - user = res.body + it('Should be able to update my description', async function () { + this.timeout(10_000) + + await servers[0].users.updateMe({ description: 'my super description updated' }) + + user = await servers[0].users.getMyInfo() + expect(user.account.displayName).to.equal('my super display name') expect(user.account.description).to.equal('my super description updated') - await wait(5000) + await waitJobs(servers) }) it('Should be able to update my avatar', async function () { - this.timeout(10000) + this.timeout(10_000) const fixture = 'avatar2.png' - await updateMyAvatar({ - url: servers[0].url, - accessToken: servers[0].accessToken, - fixture - }) + await servers[0].users.updateMyAvatar({ fixture }) + + user = await servers[0].users.getMyInfo() + userAvatarFilenames = user.account.avatars.map(({ path }) => path) + + for (const avatar of user.account.avatars) { + await testImage(servers[0].url, `avatar2-resized-${avatar.width}x${avatar.width}`, avatar.path, '.png') + } + + await waitJobs(servers) + }) + + it('Should have updated my profile on other servers too', async function () { + let createdAt: string | Date - const res = await getMyUserInformation(servers[0].url, servers[0].accessToken) - user = res.body + for (const server of servers) { + const body = await server.accounts.list({ sort: '-createdAt' }) + + const resList = body.data.find(a => a.name === 'root' && a.host === 'localhost:' + servers[0].port) + expect(resList).not.to.be.undefined + + const account = await server.accounts.get({ accountName: resList.name + '@' + resList.host }) - await testImage(servers[0].url, 'avatar2-resized', user.account.avatar.path, '.png') + if (!createdAt) createdAt = account.createdAt - await wait(5000) + expect(account.name).to.equal('root') + expect(account.host).to.equal('localhost:' + servers[0].port) + expect(account.displayName).to.equal('my super display name') + expect(account.description).to.equal('my super description updated') + expect(createdAt).to.equal(account.createdAt) + + if (server.serverNumber === 1) { + expect(account.userId).to.be.a('number') + } else { + expect(account.userId).to.be.undefined + } + + for (const avatar of account.avatars) { + await testImage(server.url, `avatar2-resized-${avatar.width}x${avatar.width}`, avatar.path, '.png') + } + } }) - it('Should have updated my avatar and my description on other servers too', async function () { + it('Should list account videos', async function () { for (const server of servers) { - const resAccounts = await getAccountsList(server.url, '-createdAt') + const { total, data } = await server.videos.listByAccount({ handle: 'user1@localhost:' + servers[0].port }) - const rootServer1List = resAccounts.body.data.find(a => a.name === 'root' && a.host === 'localhost:9001') as Account - expect(rootServer1List).not.to.be.undefined + expect(total).to.equal(1) + expect(data).to.be.an('array') + expect(data).to.have.lengthOf(1) + expect(data[0].uuid).to.equal(videoUUID) + } + }) + + it('Should search through account videos', async function () { + this.timeout(10_000) - const resAccount = await getAccount(server.url, rootServer1List.id) - const rootServer1Get = resAccount.body as Account - expect(rootServer1Get.name).to.equal('root') - expect(rootServer1Get.host).to.equal('localhost:9001') - expect(rootServer1Get.description).to.equal('my super description updated') + const created = await servers[0].videos.upload({ token: userAccessToken, attributes: { name: 'Kami no chikara' } }) - await testImage(server.url, 'avatar2-resized', rootServer1Get.avatar.path, '.png') + await waitJobs(servers) + + for (const server of servers) { + const { total, data } = await server.videos.listByAccount({ handle: 'user1@localhost:' + servers[0].port, search: 'Kami' }) + + expect(total).to.equal(1) + expect(data).to.be.an('array') + expect(data).to.have.lengthOf(1) + expect(data[0].uuid).to.equal(created.uuid) } }) it('Should remove the user', async function () { - this.timeout(10000) + this.timeout(10_000) for (const server of servers) { - const resAccounts = await getAccountsList(server.url, '-createdAt') + const body = await server.accounts.list({ sort: '-createdAt' }) + + const accountDeleted = body.data.find(a => a.name === 'user1' && a.host === 'localhost:' + servers[0].port) + expect(accountDeleted).not.to.be.undefined - const userServer1List = resAccounts.body.data.find(a => a.name === 'user1' && a.host === 'localhost:9001') as Account - expect(userServer1List).not.to.be.undefined + const { data } = await server.channels.list() + const videoChannelDeleted = data.find(a => a.displayName === 'Main user1 channel' && a.host === 'localhost:' + servers[0].port) + expect(videoChannelDeleted).not.to.be.undefined } - await removeUser(servers[0].url, userId, servers[0].accessToken) + await servers[0].users.remove({ userId }) - await wait(5000) + await waitJobs(servers) for (const server of servers) { - const resAccounts = await getAccountsList(server.url, '-createdAt') + const body = await server.accounts.list({ sort: '-createdAt' }) - const userServer1List = resAccounts.body.data.find(a => a.name === 'user1' && a.host === 'localhost:9001') as Account - expect(userServer1List).to.be.undefined + const accountDeleted = body.data.find(a => a.name === 'user1' && a.host === 'localhost:' + servers[0].port) + expect(accountDeleted).to.be.undefined + + const { data } = await server.channels.list() + const videoChannelDeleted = data.find(a => a.name === 'Main user1 channel' && a.host === 'localhost:' + servers[0].port) + expect(videoChannelDeleted).to.be.undefined } }) it('Should not have actor files', async () => { for (const server of servers) { - await checkActorFilesWereRemoved(userUUID, server.serverNumber) + for (const userAvatarFilename of userAvatarFilenames) { + await checkActorFilesWereRemoved(userAvatarFilename, server.internalServerNumber) + } } }) it('Should not have video files', async () => { for (const server of servers) { - await checkVideoFilesWereRemoved(videoUUID, server.serverNumber) + await checkVideoFilesWereRemoved({ server, video: server.store.videoDetails }) } }) - after(async function () { - killallServers(servers) - - // Keep the logs if the test failed - if (this[ 'ok' ]) { - await flushTests() + it('Should have an empty tmp directory', async function () { + for (const server of servers) { + await checkTmpIsEmpty(server) } }) + + after(async function () { + await cleanupTests(servers) + }) })