X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Ftests%2Fapi%2Fvideos%2Fsingle-server.ts;h=0ae405950b54025e949eed4c412fa8b0f101106c;hb=fa58a19819cb821a5c5a26802efd492461be9c04;hp=92d42eb80aef41f22b0de5f0c4d15cc94057957d;hpb=7f2cfe3a792856f7de6f1d13688aa3d06ec1bf70;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/tests/api/videos/single-server.ts b/server/tests/api/videos/single-server.ts index 92d42eb80..0ae405950 100644 --- a/server/tests/api/videos/single-server.ts +++ b/server/tests/api/videos/single-server.ts @@ -1,4 +1,4 @@ -/* tslint:disable:no-unused-expression */ +/* eslint-disable @typescript-eslint/no-unused-expressions,@typescript-eslint/require-await */ import * as chai from 'chai' import { keyBy } from 'lodash' @@ -6,8 +6,9 @@ import 'mocha' import { VideoPrivacy } from '../../../../shared/models/videos' import { checkVideoFilesWereRemoved, + cleanupTests, completeVideoCheck, - flushTests, + flushAndRunServer, getVideo, getVideoCategories, getVideoLanguages, @@ -17,10 +18,8 @@ import { getVideosListPagination, getVideosListSort, getVideosWithFilters, - killallServers, rateVideo, removeVideo, - runServer, ServerInfo, setAccessTokensToServers, testImage, @@ -28,17 +27,18 @@ import { uploadVideo, viewVideo, wait -} from '../../utils' +} from '../../../../shared/extra-utils' const expect = chai.expect describe('Test a single server', function () { let server: ServerInfo = null let videoId = -1 + let videoId2 = -1 let videoUUID = '' let videosListBase: any[] = null - const getCheckAttributes = { + const getCheckAttributes = () => ({ name: 'my super name', category: 2, licence: 6, @@ -48,7 +48,7 @@ describe('Test a single server', function () { support: 'my super support text', account: { name: 'root', - host: 'localhost:9001' + host: 'localhost:' + server.port }, isLocal: true, duration: 5, @@ -69,9 +69,9 @@ describe('Test a single server', function () { size: 218910 } ] - } + }) - const updateCheckAttributes = { + const updateCheckAttributes = () => ({ name: 'my super video updated', category: 4, licence: 2, @@ -81,7 +81,7 @@ describe('Test a single server', function () { support: 'my super support text updated', account: { name: 'root', - host: 'localhost:9001' + host: 'localhost:' + server.port }, isLocal: true, tags: [ 'tagup1', 'tagup2' ], @@ -102,14 +102,12 @@ describe('Test a single server', function () { size: 292677 } ] - } + }) before(async function () { this.timeout(30000) - await flushTests() - - server = await runServer(1) + server = await flushAndRunServer(1) await setAccessTokensToServers([ server ]) }) @@ -120,7 +118,7 @@ describe('Test a single server', function () { const categories = res.body expect(Object.keys(categories)).to.have.length.above(10) - expect(categories[11]).to.equal('News') + expect(categories[11]).to.equal('News & Politics') }) it('Should list video licences', async function () { @@ -185,7 +183,7 @@ describe('Test a single server', function () { expect(res.body.data.length).to.equal(1) const video = res.body.data[0] - await completeVideoCheck(server.url, video, getCheckAttributes) + await completeVideoCheck(server.url, video, getCheckAttributes()) }) it('Should get the video by UUID', async function () { @@ -194,7 +192,7 @@ describe('Test a single server', function () { const res = await getVideo(server.url, videoUUID) const video = res.body - await completeVideoCheck(server.url, video, getCheckAttributes) + await completeVideoCheck(server.url, video, getCheckAttributes()) }) it('Should have the views updated', async function () { @@ -240,12 +238,11 @@ describe('Test a single server', function () { it('Should upload 6 videos', async function () { this.timeout(25000) - const videos = [ + const videos = new Set([ 'video_short.mp4', 'video_short.ogv', 'video_short.webm', 'video_short1.webm', 'video_short2.webm', 'video_short3.webm' - ] + ]) - const tasks: Promise[] = [] for (const video of videos) { const videoAttributes = { name: video + ' name', @@ -258,11 +255,8 @@ describe('Test a single server', function () { fixture: video } - const p = uploadVideo(server.url, server.accessToken, videoAttributes) - tasks.push(p) + await uploadVideo(server.url, server.accessToken, videoAttributes) } - - await Promise.all(tasks) }) it('Should have the correct durations', async function () { @@ -325,6 +319,15 @@ describe('Test a single server', function () { expect(videos[0].name).to.equal(videosListBase[5].name) }) + it('Should not have the total field', async function () { + const res = await getVideosListPagination(server.url, 5, 6, 'name', true) + + const videos = res.body.data + expect(res.body.total).to.not.exist + expect(videos.length).to.equal(1) + expect(videos[0].name).to.equal(videosListBase[5].name) + }) + it('Should list and sort by name in descending order', async function () { const res = await getVideosListSort(server.url, '-name') @@ -339,6 +342,7 @@ describe('Test a single server', function () { expect(videos[5].name).to.equal('video_short1.webm name') videoId = videos[3].uuid + videoId2 = videos[5].uuid }) it('Should list and sort by trending in descending order', async function () { @@ -379,7 +383,7 @@ describe('Test a single server', function () { const res = await getVideo(server.url, videoId) const video = res.body - await completeVideoCheck(server.url, video, updateCheckAttributes) + await completeVideoCheck(server.url, video, updateCheckAttributes()) }) it('Should update only the tags of a video', async function () { @@ -391,7 +395,7 @@ describe('Test a single server', function () { const res = await getVideo(server.url, videoId) const video = res.body - await completeVideoCheck(server.url, video, Object.assign(updateCheckAttributes, attributes)) + await completeVideoCheck(server.url, video, Object.assign(updateCheckAttributes(), attributes)) }) it('Should update only the description of a video', async function () { @@ -403,7 +407,8 @@ describe('Test a single server', function () { const res = await getVideo(server.url, videoId) const video = res.body - await completeVideoCheck(server.url, video, Object.assign(updateCheckAttributes, attributes)) + const expectedAttributes = Object.assign(updateCheckAttributes(), { tags: [ 'supertag', 'tag1', 'tag2' ] }, attributes) + await completeVideoCheck(server.url, video, expectedAttributes) }) it('Should like a video', async function () { @@ -426,12 +431,44 @@ describe('Test a single server', function () { expect(video.dislikes).to.equal(1) }) - after(async function () { - killallServers([ server ]) + it('Should sort by originallyPublishedAt', async function () { + { + + { + const now = new Date() + const attributes = { originallyPublishedAt: now.toISOString() } + await updateVideo(server.url, server.accessToken, videoId, attributes) + + const res = await getVideosListSort(server.url, '-originallyPublishedAt') + const names = res.body.data.map(v => v.name) + + expect(names[0]).to.equal('my super video updated') + expect(names[1]).to.equal('video_short2.webm name') + expect(names[2]).to.equal('video_short1.webm name') + expect(names[3]).to.equal('video_short.webm name') + expect(names[4]).to.equal('video_short.ogv name') + expect(names[5]).to.equal('video_short.mp4 name') + } - // Keep the logs if the test failed - if (this['ok']) { - await flushTests() + { + const now = new Date() + const attributes = { originallyPublishedAt: now.toISOString() } + await updateVideo(server.url, server.accessToken, videoId2, attributes) + + const res = await getVideosListSort(server.url, '-originallyPublishedAt') + const names = res.body.data.map(v => v.name) + + expect(names[0]).to.equal('video_short1.webm name') + expect(names[1]).to.equal('my super video updated') + expect(names[2]).to.equal('video_short2.webm name') + expect(names[3]).to.equal('video_short.webm name') + expect(names[4]).to.equal('video_short.ogv name') + expect(names[5]).to.equal('video_short.mp4 name') + } } }) + + after(async function () { + await cleanupTests([ server ]) + }) })