X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Ftests%2Fapi%2Fsearch%2Fsearch-index.ts;h=21473b6bf0a02e991996b7dadbd321204a490a36;hb=17ecdf61ce1d374cc8ba17601b93c9bda08112b2;hp=cc841a6ff03a7617a83855c839d28213221998c3;hpb=164c8d46cf5c948a28b4ac0e596fad9b83b2c229;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/tests/api/search/search-index.ts b/server/tests/api/search/search-index.ts index cc841a6ff..21473b6bf 100644 --- a/server/tests/api/search/search-index.ts +++ b/server/tests/api/search/search-index.ts @@ -1,8 +1,6 @@ /* 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 { expect } from 'chai' import { BooleanBothQuery, VideoChannelsSearchQuery, @@ -11,10 +9,9 @@ import { VideoPlaylistType, VideosSearchQuery } from '@shared/models' +import { cleanupTests, createSingleServer, PeerTubeServer, SearchCommand, setAccessTokensToServers } from '@shared/server-commands' -const expect = chai.expect - -describe('Test videos search', function () { +describe('Test index search', function () { const localVideoName = 'local video' + new Date().toISOString() let server: PeerTubeServer = null @@ -35,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: { @@ -60,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 () { @@ -84,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) }) @@ -123,70 +143,26 @@ 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) - - 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') - 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 - } - - 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' - } - - { - await check(baseSearch) - } + it('Should make a simple search', async function () { + await check(baseSearch) + }) - { - const search = { ...baseSearch, startDate: '2018-10-01T10:54:46.396Z' } - await check(search, false) - } + it('Should search by start date', async function () { + const search = { ...baseSearch, startDate: '2018-10-01T10:54:46.396Z' } + await check(search, false) + }) - { - const search = { ...baseSearch, tagsAllOf: [ 'toto', 'framasoft' ] } - await check(search, false) - } + it('Should search by tags', async function () { + const search = { ...baseSearch, tagsAllOf: [ 'toto', 'framasoft' ] } + await check(search, false) + }) - { - const search = { ...baseSearch, durationMin: 2000 } - await check(search, false) - } + it('Should search by duration', async function () { + const search = { ...baseSearch, durationMin: 2000 } + await check(search, false) + }) + it('Should search by nsfw attribute', async function () { { const search = { ...baseSearch, nsfw: 'true' as BooleanBothQuery } await check(search, false) @@ -201,7 +177,9 @@ describe('Test videos search', function () { const search = { ...baseSearch, nsfw: 'both' as BooleanBothQuery } await check(search, true) } + }) + it('Should search by host', async function () { { const search = { ...baseSearch, host: 'example.com' } await check(search, false) @@ -213,6 +191,39 @@ describe('Test videos search', function () { } }) + 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 uuidsMatrix = [ + [ goodUUID ], + [ goodUUID, badShortUUID ], + [ badShortUUID, goodShortUUID ], + [ goodUUID, goodShortUUID ] + ] + + for (const uuids of uuidsMatrix) { + const search = { ...baseSearch, uuids } + await check(search, true) + } + } + + { + const uuidsMatrix = [ + [ badUUID ], + [ badShortUUID ] + ] + + for (const uuids of uuidsMatrix) { + const search = { ...baseSearch, uuids } + await check(search, false) + } + } + }) + it('Should have a correct pagination', async function () { const search = { search: 'video', @@ -267,6 +278,34 @@ describe('Test videos search', function () { describe('Channels search', async function () { + async function check (search: VideoChannelsSearchQuery, exists = true) { + const body = await command.advancedChannelSearch({ search }) + + 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) + + 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) }) @@ -275,34 +314,18 @@ describe('Test videos search', function () { }) it('Should make a search and have results', async function () { + await check({ search: 'Framasoft', sort: 'createdAt' }, true) + }) - async function check (search: VideoChannelsSearchQuery, exists = true) { - const body = await command.advancedChannelSearch({ search }) - - 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) - - 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') - expect(videoChannel.avatar).to.exist - 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') - expect(videoChannel.ownerAccount.avatar).to.exist - } + 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) + }) - await check({ search: 'Framasoft', sort: 'createdAt' }, true) - await check({ search: 'Framasoft', host: 'example.com' }, false) - await check({ search: 'Framasoft', 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 () { @@ -315,57 +338,96 @@ 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) }) + async function check (search: VideoPlaylistsSearchQuery, exists = true) { + const body = await command.advancedPlaylistSearch({ search }) - expect(body.total).to.equal(0) - expect(body.data).to.have.lengthOf(0) - }) - - it('Should make a search and have results', async function () { + if (exists === false) { + expect(body.total).to.equal(0) + expect(body.data).to.have.lengthOf(0) + return + } - async function check (search: VideoPlaylistsSearchQuery, exists = true) { - const body = await command.advancedPlaylistSearch({ search }) + expect(body.total).to.be.greaterThan(0) + expect(body.data).to.have.length.greaterThan(0) - if (exists === false) { - expect(body.total).to.equal(0) - expect(body.data).to.have.lengthOf(0) - return - } + const videoPlaylist = body.data[0] - expect(body.total).to.be.greaterThan(0) - expect(body.data).to.have.length.greaterThan(0) + expect(videoPlaylist.url).to.equal('https://peertube2.cpy.re/videos/watch/playlist/73804a40-da9a-40c2-b1eb-2c6d9eec8f0a') + expect(videoPlaylist.thumbnailUrl).to.exist + expect(videoPlaylist.embedUrl).to.equal('https://peertube2.cpy.re/video-playlists/embed/73804a40-da9a-40c2-b1eb-2c6d9eec8f0a') - const videoPlaylist = body.data[0] + expect(videoPlaylist.type.id).to.equal(VideoPlaylistType.REGULAR) + expect(videoPlaylist.privacy.id).to.equal(VideoPlaylistPrivacy.PUBLIC) + expect(videoPlaylist.videosLength).to.exist - expect(videoPlaylist.url).to.equal('https://peertube2.cpy.re/videos/watch/playlist/73804a40-da9a-40c2-b1eb-2c6d9eec8f0a') - expect(videoPlaylist.thumbnailUrl).to.exist - expect(videoPlaylist.embedUrl).to.equal('https://peertube2.cpy.re/video-playlists/embed/73804a40-da9a-40c2-b1eb-2c6d9eec8f0a') + expect(videoPlaylist.createdAt).to.exist + expect(videoPlaylist.updatedAt).to.exist - expect(videoPlaylist.type.id).to.equal(VideoPlaylistType.REGULAR) - expect(videoPlaylist.privacy.id).to.equal(VideoPlaylistPrivacy.PUBLIC) - expect(videoPlaylist.videosLength).to.exist + expect(videoPlaylist.uuid).to.equal('73804a40-da9a-40c2-b1eb-2c6d9eec8f0a') + expect(videoPlaylist.displayName).to.exist - expect(videoPlaylist.createdAt).to.exist - expect(videoPlaylist.updatedAt).to.exist + 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.uuid).to.equal('73804a40-da9a-40c2-b1eb-2c6d9eec8f0a') - expect(videoPlaylist.displayName).to.exist + 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') + } - 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') - expect(videoPlaylist.ownerAccount.avatar).to.exist + it('Should make a simple search and not have results', async function () { + const body = await command.searchPlaylists({ search: 'a'.repeat(500) }) - 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') - expect(videoPlaylist.videoChannel.avatar).to.exist - } + 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' }, true) + 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 () {