X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Ftests%2Fapi%2Fusers%2Fusers.ts;h=0a66bd1ce5f89f20c497bac4be0e4e52bf7bc998;hb=8491293b02ed2ec53eb0fa128161ea0b08d3def9;hp=95b1bb62603357324217039ab2306bddb00ed7e7;hpb=8c559fad1e1c4c2ab7f1388c73200aa4c6256d74;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/tests/api/users/users.ts b/server/tests/api/users/users.ts index 95b1bb626..0a66bd1ce 100644 --- a/server/tests/api/users/users.ts +++ b/server/tests/api/users/users.ts @@ -1,9 +1,10 @@ -/* tslint:disable:no-unused-expression */ +/* eslint-disable @typescript-eslint/no-unused-expressions,@typescript-eslint/require-await */ import * as chai from 'chai' import 'mocha' -import { User, UserRole, Video } from '../../../../shared/index' +import { MyUser, User, UserRole, Video, VideoAbuseState, VideoAbuseUpdate, VideoPlaylistType } from '../../../../shared/index' import { + addVideoCommentThread, blockUser, cleanupTests, createUser, @@ -11,33 +12,41 @@ import { flushAndRunServer, getAccountRatings, getBlacklistedVideosList, + getCustomConfig, getMyUserInformation, getMyUserVideoQuotaUsed, getMyUserVideoRating, getUserInformation, getUsersList, getUsersListPaginationAndSort, + getVideoAbusesList, getVideoChannel, - getVideosList, installPlugin, + getVideosList, + installPlugin, login, makePutBodyRequest, rateVideo, registerUserWithChannel, removeUser, removeVideo, + reportVideoAbuse, ServerInfo, testImage, unblockUser, + updateCustomSubConfig, updateMyAvatar, updateMyUser, updateUser, + updateVideoAbuse, uploadVideo, - userLogin + userLogin, + waitJobs } from '../../../../shared/extra-utils' import { follow } from '../../../../shared/extra-utils/server/follows' -import { setAccessTokensToServers } from '../../../../shared/extra-utils/users/login' +import { logout, serverLogin, setAccessTokensToServers } from '../../../../shared/extra-utils/users/login' import { getMyVideos } from '../../../../shared/extra-utils/videos/videos' import { UserAdminFlag } from '../../../../shared/models/users/user-flag.model' +import { CustomConfig } from '@shared/models/server' const expect = chai.expect @@ -54,7 +63,14 @@ describe('Test users', function () { before(async function () { this.timeout(30000) - server = await flushAndRunServer(1) + + server = await flushAndRunServer(1, { + rates_limit: { + login: { + max: 30 + } + } + }) await setAccessTokensToServers([ server ]) @@ -121,13 +137,13 @@ describe('Test users', function () { it('Should be able to login with an insensitive username', async function () { const user = { username: 'RoOt', password: server.user.password } - const res = await login(server.url, server.client, user, 200) + await login(server.url, server.client, user, 200) const user2 = { username: 'rOoT', password: server.user.password } - const res2 = await login(server.url, server.client, user2, 200) + await login(server.url, server.client, user2, 200) const user3 = { username: 'ROOt', password: server.user.password } - const res3 = await login(server.url, server.client, user3, 200) + await login(server.url, server.client, user3, 200) }) }) @@ -137,7 +153,7 @@ describe('Test users', function () { const videoAttributes = {} await uploadVideo(server.url, accessToken, videoAttributes) const res = await getVideosList(server.url) - const video = res.body.data[ 0 ] + const video = res.body.data[0] expect(video.account.name).to.equal('root') videoId = video.id @@ -167,8 +183,8 @@ describe('Test users', function () { const ratings = res.body expect(ratings.total).to.equal(1) - expect(ratings.data[ 0 ].video.id).to.equal(videoId) - expect(ratings.data[ 0 ].rating).to.equal('like') + expect(ratings.data[0].video.id).to.equal(videoId) + expect(ratings.data[0].rating).to.equal('like') }) it('Should retrieve ratings list by rating type', async function () { @@ -199,13 +215,17 @@ describe('Test users', function () { }) describe('Logout', function () { - it('Should logout (revoke token)') - - it('Should not be able to get the user information') + it('Should logout (revoke token)', async function () { + await logout(server.url, server.accessToken) + }) - it('Should not be able to upload a video') + it('Should not be able to get the user information', async function () { + await getMyUserInformation(server.url, server.accessToken, 401) + }) - it('Should not be able to remove a video') + it('Should not be able to upload a video', async function () { + await uploadVideo(server.url, server.accessToken, { name: 'video' }, 401) + }) it('Should not be able to rate a video', async function () { const path = '/api/v1/videos/' @@ -223,13 +243,17 @@ describe('Test users', function () { await makePutBodyRequest(options) }) - it('Should be able to login again') + it('Should be able to login again', async function () { + server.accessToken = await serverLogin(server) + }) it('Should have an expired access token') it('Should refresh the token') - it('Should be able to upload a video again') + it('Should be able to get my user information again', async function () { + await getMyUserInformation(server.url, server.accessToken) + }) }) describe('Creating a user', function () { @@ -241,7 +265,7 @@ describe('Test users', function () { username: user.username, password: user.password, videoQuota: 2 * 1024 * 1024, - adminFlags: UserAdminFlag.BY_PASS_VIDEO_AUTO_BLACKLIST + adminFlags: UserAdminFlag.BYPASS_VIDEO_AUTO_BLACKLIST }) }) @@ -251,9 +275,9 @@ describe('Test users', function () { it('Should be able to get user information', async function () { const res1 = await getMyUserInformation(server.url, accessTokenUser) - const userMe: User = res1.body + const userMe: MyUser = res1.body - const res2 = await getUserInformation(server.url, server.accessToken, userMe.id) + const res2 = await getUserInformation(server.url, server.accessToken, userMe.id, true) const userGet: User = res2.body for (const user of [ userMe, userGet ]) { @@ -268,14 +292,27 @@ describe('Test users', function () { } expect(userMe.adminFlags).to.be.undefined - expect(userGet.adminFlags).to.equal(UserAdminFlag.BY_PASS_VIDEO_AUTO_BLACKLIST) + expect(userGet.adminFlags).to.equal(UserAdminFlag.BYPASS_VIDEO_AUTO_BLACKLIST) + + expect(userMe.specialPlaylists).to.have.lengthOf(1) + expect(userMe.specialPlaylists[0].type).to.equal(VideoPlaylistType.WATCH_LATER) + + // Check stats are included with withStats + expect(userGet.videosCount).to.be.a('number') + expect(userGet.videosCount).to.equal(0) + expect(userGet.videoCommentsCount).to.be.a('number') + expect(userGet.videoCommentsCount).to.equal(0) + expect(userGet.videoAbusesCount).to.be.a('number') + expect(userGet.videoAbusesCount).to.equal(0) + expect(userGet.videoAbusesAcceptedCount).to.be.a('number') + expect(userGet.videoAbusesAcceptedCount).to.equal(0) }) }) describe('My videos & quotas', function () { it('Should be able to upload a video with this user', async function () { - this.timeout(5000) + this.timeout(10000) const videoAttributes = { name: 'super user video', @@ -304,11 +341,59 @@ describe('Test users', function () { const videos = res.body.data expect(videos).to.have.lengthOf(1) - const video: Video = videos[ 0 ] + const video: Video = videos[0] expect(video.name).to.equal('super user video') expect(video.thumbnailPath).to.not.be.null expect(video.previewPath).to.not.be.null }) + + it('Should be able to search in my videos', async function () { + { + const res = await getMyVideos(server.url, accessTokenUser, 0, 5, '-createdAt', 'user video') + expect(res.body.total).to.equal(1) + + const videos = res.body.data + expect(videos).to.have.lengthOf(1) + } + + { + const res = await getMyVideos(server.url, accessTokenUser, 0, 5, '-createdAt', 'toto') + expect(res.body.total).to.equal(0) + + const videos = res.body.data + expect(videos).to.have.lengthOf(0) + } + }) + + it('Should disable webtorrent, enable HLS, and update my quota', async function () { + this.timeout(60000) + + { + const res = await getCustomConfig(server.url, server.accessToken) + const config = res.body as CustomConfig + config.transcoding.webtorrent.enabled = false + config.transcoding.hls.enabled = true + config.transcoding.enabled = true + await updateCustomSubConfig(server.url, server.accessToken, config) + } + + { + const videoAttributes = { + name: 'super user video 2', + fixture: 'video_short.webm' + } + await uploadVideo(server.url, accessTokenUser, videoAttributes) + + await waitJobs([ server ]) + } + + { + const res = await getMyUserVideoQuotaUsed(server.url, accessTokenUser) + const data = res.body + + expect(data.videoQuotaUsed).to.be.greaterThan(220000) + } + }) }) describe('Users listing', function () { @@ -323,16 +408,19 @@ describe('Test users', function () { expect(users).to.be.an('array') expect(users.length).to.equal(2) - const user = users[ 0 ] + const user = users[0] expect(user.username).to.equal('user_1') expect(user.email).to.equal('user_1@example.com') expect(user.nsfwPolicy).to.equal('display') - const rootUser = users[ 1 ] + const rootUser = users[1] expect(rootUser.username).to.equal('root') expect(rootUser.email).to.equal('admin' + server.internalServerNumber + '@example.com') expect(user.nsfwPolicy).to.equal('display') + expect(rootUser.lastLoginDate).to.exist + expect(user.lastLoginDate).to.exist + userId = user.id }) @@ -346,7 +434,7 @@ describe('Test users', function () { expect(total).to.equal(2) expect(users.length).to.equal(1) - const user = users[ 0 ] + const user = users[0] expect(user.username).to.equal('root') expect(user.email).to.equal('admin' + server.internalServerNumber + '@example.com') expect(user.roleLabel).to.equal('Administrator') @@ -362,7 +450,7 @@ describe('Test users', function () { expect(total).to.equal(2) expect(users.length).to.equal(1) - const user = users[ 0 ] + const user = users[0] expect(user.username).to.equal('user_1') expect(user.email).to.equal('user_1@example.com') expect(user.nsfwPolicy).to.equal('display') @@ -377,7 +465,7 @@ describe('Test users', function () { expect(total).to.equal(2) expect(users.length).to.equal(1) - const user = users[ 0 ] + const user = users[0] expect(user.username).to.equal('user_1') expect(user.email).to.equal('user_1@example.com') expect(user.nsfwPolicy).to.equal('display') @@ -392,13 +480,13 @@ describe('Test users', function () { expect(total).to.equal(2) expect(users.length).to.equal(2) - expect(users[ 0 ].username).to.equal('root') - expect(users[ 0 ].email).to.equal('admin' + server.internalServerNumber + '@example.com') - expect(users[ 0 ].nsfwPolicy).to.equal('display') + expect(users[0].username).to.equal('root') + expect(users[0].email).to.equal('admin' + server.internalServerNumber + '@example.com') + expect(users[0].nsfwPolicy).to.equal('display') - expect(users[ 1 ].username).to.equal('user_1') - expect(users[ 1 ].email).to.equal('user_1@example.com') - expect(users[ 1 ].nsfwPolicy).to.equal('display') + expect(users[1].username).to.equal('user_1') + expect(users[1].email).to.equal('user_1@example.com') + expect(users[1].nsfwPolicy).to.equal('display') }) it('Should search user by username', async function () { @@ -408,7 +496,7 @@ describe('Test users', function () { expect(res.body.total).to.equal(1) expect(users.length).to.equal(1) - expect(users[ 0 ].username).to.equal('root') + expect(users[0].username).to.equal('root') }) it('Should search user by email', async function () { @@ -419,8 +507,8 @@ describe('Test users', function () { expect(res.body.total).to.equal(1) expect(users.length).to.equal(1) - expect(users[ 0 ].username).to.equal('user_1') - expect(users[ 0 ].email).to.equal('user_1@example.com') + expect(users[0].username).to.equal('user_1') + expect(users[0].email).to.equal('user_1@example.com') } { @@ -430,8 +518,8 @@ describe('Test users', function () { expect(res.body.total).to.equal(2) expect(users.length).to.equal(2) - expect(users[ 0 ].username).to.equal('root') - expect(users[ 1 ].username).to.equal('user_1') + expect(users[0].username).to.equal('root') + expect(users[1].username).to.equal('user_1') } }) }) @@ -481,6 +569,19 @@ describe('Test users', function () { expect(user.autoPlayVideo).to.be.false }) + it('Should be able to change the autoPlayNextVideo attribute', async function () { + await updateMyUser({ + url: server.url, + accessToken: accessTokenUser, + autoPlayNextVideo: true + }) + + const res = await getMyUserInformation(server.url, accessTokenUser) + const user = res.body + + expect(user.autoPlayNextVideo).to.be.true + }) + it('Should be able to change the email attribute', async function () { await updateMyUser({ url: server.url, @@ -588,7 +689,6 @@ describe('Test users', function () { }) describe('Updating another user', function () { - it('Should be able to update another user', async function () { await updateUser({ url: server.url, @@ -657,12 +757,14 @@ describe('Test users', function () { expect(res.body.total).to.equal(1) - const video = res.body.data[ 0 ] + const video = res.body.data[0] expect(video.account.name).to.equal('root') }) }) describe('Registering a new user', function () { + let user15AccessToken + it('Should register a new user', async function () { const user = { displayName: 'super user 15', username: 'user_15', password: 'my super password' } const channel = { name: 'my_user_15_channel', displayName: 'my channel rocks' } @@ -676,18 +778,18 @@ describe('Test users', function () { password: 'my super password' } - accessToken = await userLogin(server, user15) + user15AccessToken = await userLogin(server, user15) }) it('Should have the correct display name', async function () { - const res = await getMyUserInformation(server.url, accessToken) + const res = await getMyUserInformation(server.url, user15AccessToken) const user: User = res.body expect(user.account.displayName).to.equal('super user 15') }) it('Should have the correct video quota', async function () { - const res = await getMyUserInformation(server.url, accessToken) + const res = await getMyUserInformation(server.url, user15AccessToken) const user = res.body expect(user.videoQuota).to.equal(5 * 1024 * 1024) @@ -705,7 +807,7 @@ describe('Test users', function () { expect(res.body.data.find(u => u.username === 'user_15')).to.not.be.undefined } - await deleteMe(server.url, accessToken) + await deleteMe(server.url, user15AccessToken) { const res = await getUsersList(server.url, server.accessToken) @@ -715,30 +817,133 @@ describe('Test users', function () { }) describe('User blocking', function () { - it('Should block and unblock a user', async function () { - const user16 = { - username: 'user_16', - password: 'my super password' - } + let user16Id + let user16AccessToken + const user16 = { + username: 'user_16', + password: 'my super password' + } + + it('Should block a user', async function () { const resUser = await createUser({ url: server.url, accessToken: server.accessToken, username: user16.username, password: user16.password }) - const user16Id = resUser.body.user.id + user16Id = resUser.body.user.id - accessToken = await userLogin(server, user16) + user16AccessToken = await userLogin(server, user16) - await getMyUserInformation(server.url, accessToken, 200) + await getMyUserInformation(server.url, user16AccessToken, 200) await blockUser(server.url, user16Id, server.accessToken) - await getMyUserInformation(server.url, accessToken, 401) + await getMyUserInformation(server.url, user16AccessToken, 401) await userLogin(server, user16, 400) + }) + it('Should search user by banned status', async function () { + { + const res = await getUsersListPaginationAndSort(server.url, server.accessToken, 0, 2, 'createdAt', undefined, true) + const users = res.body.data as User[] + + expect(res.body.total).to.equal(1) + expect(users.length).to.equal(1) + + expect(users[0].username).to.equal(user16.username) + } + + { + const res = await getUsersListPaginationAndSort(server.url, server.accessToken, 0, 2, 'createdAt', undefined, false) + const users = res.body.data as User[] + + expect(res.body.total).to.equal(1) + expect(users.length).to.equal(1) + + expect(users[0].username).to.not.equal(user16.username) + } + }) + + it('Should unblock a user', async function () { await unblockUser(server.url, user16Id, server.accessToken) - accessToken = await userLogin(server, user16) - await getMyUserInformation(server.url, accessToken, 200) + user16AccessToken = await userLogin(server, user16) + await getMyUserInformation(server.url, user16AccessToken, 200) + }) + }) + + describe('User stats', function () { + let user17Id + let user17AccessToken + + it('Should report correct initial statistics about a user', async function () { + const user17 = { + username: 'user_17', + password: 'my super password' + } + const resUser = await createUser({ + url: server.url, + accessToken: server.accessToken, + username: user17.username, + password: user17.password + }) + + user17Id = resUser.body.user.id + user17AccessToken = await userLogin(server, user17) + + const res = await getUserInformation(server.url, server.accessToken, user17Id, true) + const user: User = res.body + + expect(user.videosCount).to.equal(0) + expect(user.videoCommentsCount).to.equal(0) + expect(user.videoAbusesCount).to.equal(0) + expect(user.videoAbusesCreatedCount).to.equal(0) + expect(user.videoAbusesAcceptedCount).to.equal(0) + }) + + it('Should report correct videos count', async function () { + const videoAttributes = { + name: 'video to test user stats' + } + await uploadVideo(server.url, user17AccessToken, videoAttributes) + const res1 = await getVideosList(server.url) + videoId = res1.body.data.find(video => video.name === videoAttributes.name).id + + const res2 = await getUserInformation(server.url, server.accessToken, user17Id, true) + const user: User = res2.body + + expect(user.videosCount).to.equal(1) + }) + + it('Should report correct video comments for user', async function () { + const text = 'super comment' + await addVideoCommentThread(server.url, user17AccessToken, videoId, text) + + const res = await getUserInformation(server.url, server.accessToken, user17Id, true) + const user: User = res.body + + expect(user.videoCommentsCount).to.equal(1) + }) + + it('Should report correct video abuses counts', async function () { + const reason = 'my super bad reason' + await reportVideoAbuse(server.url, user17AccessToken, videoId, reason) + + const res1 = await getVideoAbusesList({ url: server.url, token: server.accessToken }) + const abuseId = res1.body.data[0].id + + const res2 = await getUserInformation(server.url, server.accessToken, user17Id, true) + const user2: User = res2.body + + expect(user2.videoAbusesCount).to.equal(1) // number of incriminations + expect(user2.videoAbusesCreatedCount).to.equal(1) // number of reports created + + const body: VideoAbuseUpdate = { state: VideoAbuseState.ACCEPTED } + await updateVideoAbuse(server.url, server.accessToken, videoId, abuseId, body) + + const res3 = await getUserInformation(server.url, server.accessToken, user17Id, true) + const user3: User = res3.body + + expect(user3.videoAbusesAcceptedCount).to.equal(1) // number of reports created accepted }) })