X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Ftests%2Fapi%2Fsearch%2Fsearch-index.ts;h=21473b6bf0a02e991996b7dadbd321204a490a36;hb=17ecdf61ce1d374cc8ba17601b93c9bda08112b2;hp=feb35411f5f51dc1dab6ec6a5bd199d965356cb2;hpb=4c7e60bc17ee5830399bac4aa273356903421b4c;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/tests/api/search/search-index.ts b/server/tests/api/search/search-index.ts index feb35411f..21473b6bf 100644 --- a/server/tests/api/search/search-index.ts +++ b/server/tests/api/search/search-index.ts @@ -1,13 +1,17 @@ /* eslint-disable @typescript-eslint/no-unused-expressions,@typescript-eslint/require-await */ -import 'mocha' -import * as chai from 'chai' -import { cleanupTests, createSingleServer, PeerTubeServer, SearchCommand, setAccessTokensToServers } from '@shared/extra-utils' -import { BooleanBothQuery, VideoPlaylistPrivacy, VideoPlaylistType, VideosSearchQuery } from '@shared/models' - -const expect = chai.expect - -describe('Test videos search', function () { +import { expect } from 'chai' +import { + BooleanBothQuery, + VideoChannelsSearchQuery, + VideoPlaylistPrivacy, + VideoPlaylistsSearchQuery, + VideoPlaylistType, + VideosSearchQuery +} from '@shared/models' +import { cleanupTests, createSingleServer, PeerTubeServer, SearchCommand, setAccessTokensToServers } from '@shared/server-commands' + +describe('Test index search', function () { const localVideoName = 'local video' + new Date().toISOString() let server: PeerTubeServer = null @@ -28,8 +32,6 @@ describe('Test videos search', function () { describe('Default search', async function () { it('Should make a local videos search by default', async function () { - this.timeout(10000) - await server.config.updateCustomSubConfig({ newConfig: { search: { @@ -53,7 +55,7 @@ describe('Test videos search', function () { expect(body.total).to.equal(1) expect(body.data[0].name).to.equal('root_channel') - expect(body.data[0].host).to.equal('localhost:' + server.port) + expect(body.data[0].host).to.equal(server.host) }) it('Should make an index videos search by default', async function () { @@ -77,31 +79,56 @@ describe('Test videos search', function () { const body = await command.searchChannels({ search: 'root' }) expect(body.total).to.be.greaterThan(2) }) + }) - it('Should make an index videos search if local search is disabled', async function () { - await server.config.updateCustomSubConfig({ - newConfig: { - search: { - searchIndex: { - enabled: true, - isDefaultSearch: false, - disableLocalSearch: true - } - } - } - }) + describe('Videos search', async function () { - const body = await command.searchVideos({ search: 'local video' }) - expect(body.total).to.be.greaterThan(2) - }) + async function check (search: VideosSearchQuery, exists = true) { + const body = await command.advancedVideoSearch({ search }) - it('Should make an index channels search if local search is disabled', async function () { - const body = await command.searchChannels({ search: 'root' }) - expect(body.total).to.be.greaterThan(2) - }) - }) + if (exists === false) { + expect(body.total).to.equal(0) + expect(body.data).to.have.lengthOf(0) + return + } - describe('Videos search', async function () { + expect(body.total).to.equal(1) + expect(body.data).to.have.lengthOf(1) + + const video = body.data[0] + + expect(video.name).to.equal('What is PeerTube?') + expect(video.category.label).to.equal('Science & Technology') + expect(video.licence.label).to.equal('Attribution - Share Alike') + expect(video.privacy.label).to.equal('Public') + expect(video.duration).to.equal(113) + expect(video.thumbnailUrl.startsWith('https://framatube.org/static/thumbnails')).to.be.true + + expect(video.account.host).to.equal('framatube.org') + expect(video.account.name).to.equal('framasoft') + expect(video.account.url).to.equal('https://framatube.org/accounts/framasoft') + // TODO: remove, deprecated in 4.2 + expect(video.account.avatar).to.exist + expect(video.account.avatars.length).to.equal(2, 'Account should have one avatar image') + + expect(video.channel.host).to.equal('framatube.org') + expect(video.channel.name).to.equal('joinpeertube') + expect(video.channel.url).to.equal('https://framatube.org/video-channels/joinpeertube') + // TODO: remove, deprecated in 4.2 + expect(video.channel.avatar).to.exist + expect(video.channel.avatars.length).to.equal(2, 'Channel should have one avatar image') + } + + const baseSearch: VideosSearchQuery = { + search: 'what is peertube', + start: 0, + count: 2, + categoryOneOf: [ 15 ], + licenceOneOf: [ 2 ], + tagsAllOf: [ 'framasoft', 'peertube' ], + startDate: '2018-10-01T10:50:46.396Z', + endDate: '2018-10-01T10:55:46.396Z' + } it('Should make a simple search and not have results', async function () { const body = await command.searchVideos({ search: 'djidane'.repeat(50) }) @@ -116,83 +143,84 @@ describe('Test videos search', function () { expect(body.total).to.be.greaterThan(1) }) - it('Should make a complex search', async function () { - - async function check (search: VideosSearchQuery, exists = true) { - const body = await command.advancedVideoSearch({ search }) - - if (exists === false) { - expect(body.total).to.equal(0) - expect(body.data).to.have.lengthOf(0) - return - } - - expect(body.total).to.equal(1) - expect(body.data).to.have.lengthOf(1) + it('Should make a simple search', async function () { + await check(baseSearch) + }) - const video = body.data[0] + it('Should search by start date', async function () { + const search = { ...baseSearch, startDate: '2018-10-01T10:54:46.396Z' } + await check(search, false) + }) - expect(video.name).to.equal('What is PeerTube?') - expect(video.category.label).to.equal('Science & Technology') - expect(video.licence.label).to.equal('Attribution - Share Alike') - expect(video.privacy.label).to.equal('Public') - expect(video.duration).to.equal(113) - expect(video.thumbnailUrl.startsWith('https://framatube.org/static/thumbnails')).to.be.true - - expect(video.account.host).to.equal('framatube.org') - expect(video.account.name).to.equal('framasoft') - expect(video.account.url).to.equal('https://framatube.org/accounts/framasoft') - expect(video.account.avatar).to.exist - - expect(video.channel.host).to.equal('framatube.org') - expect(video.channel.name).to.equal('bf54d359-cfad-4935-9d45-9d6be93f63e8') - expect(video.channel.url).to.equal('https://framatube.org/video-channels/bf54d359-cfad-4935-9d45-9d6be93f63e8') - expect(video.channel.avatar).to.exist - } + it('Should search by tags', async function () { + const search = { ...baseSearch, tagsAllOf: [ 'toto', 'framasoft' ] } + await check(search, false) + }) - const baseSearch: VideosSearchQuery = { - search: 'what is peertube', - start: 0, - count: 2, - categoryOneOf: [ 15 ], - licenceOneOf: [ 2 ], - tagsAllOf: [ 'framasoft', 'peertube' ], - startDate: '2018-10-01T10:50:46.396Z', - endDate: '2018-10-01T10:55:46.396Z' - } + it('Should search by duration', async function () { + const search = { ...baseSearch, durationMin: 2000 } + await check(search, false) + }) + it('Should search by nsfw attribute', async function () { { - await check(baseSearch) + const search = { ...baseSearch, nsfw: 'true' as BooleanBothQuery } + await check(search, false) } { - const search = { ...baseSearch, startDate: '2018-10-01T10:54:46.396Z' } - await check(search, false) + const search = { ...baseSearch, nsfw: 'false' as BooleanBothQuery } + await check(search, true) } { - const search = { ...baseSearch, tagsAllOf: [ 'toto', 'framasoft' ] } - await check(search, false) + const search = { ...baseSearch, nsfw: 'both' as BooleanBothQuery } + await check(search, true) } + }) + it('Should search by host', async function () { { - const search = { ...baseSearch, durationMin: 2000 } + const search = { ...baseSearch, host: 'example.com' } await check(search, false) } { - const search = { ...baseSearch, nsfw: 'true' as BooleanBothQuery } - await check(search, false) + const search = { ...baseSearch, host: 'framatube.org' } + await check(search, true) } + }) + + it('Should search by uuids', async function () { + const goodUUID = '9c9de5e8-0a1e-484a-b099-e80766180a6d' + const goodShortUUID = 'kkGMgK9ZtnKfYAgnEtQxbv' + const badUUID = 'c29c5b77-4a04-493d-96a9-2e9267e308f0' + const badShortUUID = 'rP5RgUeX9XwTSrspCdkDej' { - const search = { ...baseSearch, nsfw: 'false' as BooleanBothQuery } - await check(search, true) + const uuidsMatrix = [ + [ goodUUID ], + [ goodUUID, badShortUUID ], + [ badShortUUID, goodShortUUID ], + [ goodUUID, goodShortUUID ] + ] + + for (const uuids of uuidsMatrix) { + const search = { ...baseSearch, uuids } + await check(search, true) + } } { - const search = { ...baseSearch, nsfw: 'both' as BooleanBothQuery } - await check(search, true) + const uuidsMatrix = [ + [ badUUID ], + [ badShortUUID ] + ] + + for (const uuids of uuidsMatrix) { + const search = { ...baseSearch, uuids } + await check(search, false) + } } }) @@ -250,15 +278,14 @@ describe('Test videos search', function () { describe('Channels search', async function () { - it('Should make a simple search and not have results', async function () { - const body = await command.searchChannels({ search: 'a'.repeat(500) }) - - expect(body.total).to.equal(0) - expect(body.data).to.have.lengthOf(0) - }) + async function check (search: VideoChannelsSearchQuery, exists = true) { + const body = await command.advancedChannelSearch({ search }) - it('Should make a search and have results', async function () { - const body = await command.advancedChannelSearch({ search: { search: 'Framasoft', sort: 'createdAt' } }) + if (exists === false) { + expect(body.total).to.equal(0) + expect(body.data).to.have.lengthOf(0) + return + } expect(body.total).to.be.greaterThan(0) expect(body.data).to.have.length.greaterThan(0) @@ -266,13 +293,39 @@ describe('Test videos search', function () { const videoChannel = body.data[0] expect(videoChannel.url).to.equal('https://framatube.org/video-channels/bf54d359-cfad-4935-9d45-9d6be93f63e8') expect(videoChannel.host).to.equal('framatube.org') + // TODO: remove, deprecated in 4.2 expect(videoChannel.avatar).to.exist + expect(videoChannel.avatars.length).to.equal(2, 'Channel should have two avatar images') expect(videoChannel.displayName).to.exist expect(videoChannel.ownerAccount.url).to.equal('https://framatube.org/accounts/framasoft') expect(videoChannel.ownerAccount.name).to.equal('framasoft') expect(videoChannel.ownerAccount.host).to.equal('framatube.org') + // TODO: remove, deprecated in 4.2 expect(videoChannel.ownerAccount.avatar).to.exist + expect(videoChannel.ownerAccount.avatars.length).to.equal(2, 'Account should have two avatar images') + } + + it('Should make a simple search and not have results', async function () { + const body = await command.searchChannels({ search: 'a'.repeat(500) }) + + expect(body.total).to.equal(0) + expect(body.data).to.have.lengthOf(0) + }) + + it('Should make a search and have results', async function () { + await check({ search: 'Framasoft', sort: 'createdAt' }, true) + }) + + it('Should make host search and have appropriate results', async function () { + await check({ search: 'Framasoft videos', host: 'example.com' }, false) + await check({ search: 'Framasoft videos', host: 'framatube.org' }, true) + }) + + it('Should make handles search and have appropriate results', async function () { + await check({ handles: [ 'bf54d359-cfad-4935-9d45-9d6be93f63e8@framatube.org' ] }, true) + await check({ handles: [ 'jeanine', 'bf54d359-cfad-4935-9d45-9d6be93f63e8@framatube.org' ] }, true) + await check({ handles: [ 'jeanine', 'chocobozzz_channel2@peertube2.cpy.re' ] }, false) }) it('Should have a correct pagination', async function () { @@ -285,15 +338,14 @@ describe('Test videos search', function () { describe('Playlists search', async function () { - it('Should make a simple search and not have results', async function () { - const body = await command.searchPlaylists({ search: 'a'.repeat(500) }) - - expect(body.total).to.equal(0) - expect(body.data).to.have.lengthOf(0) - }) + async function check (search: VideoPlaylistsSearchQuery, exists = true) { + const body = await command.advancedPlaylistSearch({ search }) - it('Should make a search and have results', async function () { - const body = await command.advancedPlaylistSearch({ search: { search: 'E2E playlist', sort: '-match' } }) + if (exists === false) { + expect(body.total).to.equal(0) + expect(body.data).to.have.lengthOf(0) + return + } expect(body.total).to.be.greaterThan(0) expect(body.data).to.have.length.greaterThan(0) @@ -317,12 +369,65 @@ describe('Test videos search', function () { expect(videoPlaylist.ownerAccount.url).to.equal('https://peertube2.cpy.re/accounts/chocobozzz') expect(videoPlaylist.ownerAccount.name).to.equal('chocobozzz') expect(videoPlaylist.ownerAccount.host).to.equal('peertube2.cpy.re') + // TODO: remove, deprecated in 4.2 expect(videoPlaylist.ownerAccount.avatar).to.exist + expect(videoPlaylist.ownerAccount.avatars.length).to.equal(2, 'Account should have two avatar images') expect(videoPlaylist.videoChannel.url).to.equal('https://peertube2.cpy.re/video-channels/chocobozzz_channel') expect(videoPlaylist.videoChannel.name).to.equal('chocobozzz_channel') expect(videoPlaylist.videoChannel.host).to.equal('peertube2.cpy.re') + // TODO: remove, deprecated in 4.2 expect(videoPlaylist.videoChannel.avatar).to.exist + expect(videoPlaylist.videoChannel.avatars.length).to.equal(2, 'Channel should have two avatar images') + } + + it('Should make a simple search and not have results', async function () { + const body = await command.searchPlaylists({ search: 'a'.repeat(500) }) + + expect(body.total).to.equal(0) + expect(body.data).to.have.lengthOf(0) + }) + + it('Should make a search and have results', async function () { + await check({ search: 'E2E playlist', sort: '-match' }, true) + }) + + it('Should make host search and have appropriate results', async function () { + await check({ search: 'E2E playlist', host: 'example.com' }, false) + await check({ search: 'E2E playlist', host: 'peertube2.cpy.re', sort: '-match' }, true) + }) + + it('Should make a search by uuids and have appropriate results', async function () { + const goodUUID = '73804a40-da9a-40c2-b1eb-2c6d9eec8f0a' + const goodShortUUID = 'fgei1ws1oa6FCaJ2qZPG29' + const badUUID = 'c29c5b77-4a04-493d-96a9-2e9267e308f0' + const badShortUUID = 'rP5RgUeX9XwTSrspCdkDej' + + { + const uuidsMatrix = [ + [ goodUUID ], + [ goodUUID, badShortUUID ], + [ badShortUUID, goodShortUUID ], + [ goodUUID, goodShortUUID ] + ] + + for (const uuids of uuidsMatrix) { + const search = { search: 'E2E playlist', sort: '-match', uuids } + await check(search, true) + } + } + + { + const uuidsMatrix = [ + [ badUUID ], + [ badShortUUID ] + ] + + for (const uuids of uuidsMatrix) { + const search = { search: 'E2E playlist', sort: '-match', uuids } + await check(search, false) + } + } }) it('Should have a correct pagination', async function () {