From f18a060a83d7053897173b2a24fb7984893131c7 Mon Sep 17 00:00:00 2001 From: Chocobozzz Date: Thu, 5 May 2022 13:47:51 +0200 Subject: Remove comments, rates and views from stats It prevent us to apply date filters --- server/models/view/local-video-viewer.ts | 14 +--- .../tests/api/views/video-views-overall-stats.ts | 79 ++++++---------------- 2 files changed, 21 insertions(+), 72 deletions(-) (limited to 'server') diff --git a/server/models/view/local-video-viewer.ts b/server/models/view/local-video-viewer.ts index b6ddcbb57..5928ba5f6 100644 --- a/server/models/view/local-video-viewer.ts +++ b/server/models/view/local-video-viewer.ts @@ -134,9 +134,6 @@ export class LocalVideoViewerModel extends Model(watchPeakQuery, options) - const commentsQuery = `SELECT COUNT(*) AS comments FROM "videoComment" WHERE "videoId" = :videoId` - const commentsPromise = LocalVideoViewerModel.sequelize.query(commentsQuery, options) - const countriesQuery = `SELECT country, COUNT(country) as viewers ` + `FROM "localVideoViewer" ` + `WHERE "videoId" = :videoId AND country IS NOT NULL ` + @@ -144,10 +141,9 @@ export class LocalVideoViewerModel extends Model(countriesQuery, options) - const [ rowsWatchTime, rowsWatchPeak, rowsComment, rowsCountries ] = await Promise.all([ + const [ rowsWatchTime, rowsWatchPeak, rowsCountries ] = await Promise.all([ watchTimePromise, watchPeakPromise, - commentsPromise, countriesPromise ]) @@ -166,14 +162,6 @@ export class LocalVideoViewerModel extends Model ({ isoCode: r.country, viewers: r.viewers diff --git a/server/tests/api/views/video-views-overall-stats.ts b/server/tests/api/views/video-views-overall-stats.ts index a70d6185e..72b072c96 100644 --- a/server/tests/api/views/video-views-overall-stats.ts +++ b/server/tests/api/views/video-views-overall-stats.ts @@ -17,57 +17,6 @@ describe('Test views overall stats', function () { servers = await prepareViewsServers() }) - describe('Test rates and comments of local videos on VOD', function () { - let vodVideoId: string - - before(async function () { - this.timeout(120000); - - ({ vodVideoId } = await prepareViewsVideos({ servers, live: false, vod: true })) - }) - - it('Should have the appropriate likes', async function () { - this.timeout(60000) - - await servers[0].videos.rate({ id: vodVideoId, rating: 'like' }) - await servers[1].videos.rate({ id: vodVideoId, rating: 'like' }) - - await waitJobs(servers) - - const stats = await servers[0].videoStats.getOverallStats({ videoId: vodVideoId }) - - expect(stats.likes).to.equal(2) - expect(stats.dislikes).to.equal(0) - }) - - it('Should have the appropriate dislikes', async function () { - this.timeout(60000) - - await servers[0].videos.rate({ id: vodVideoId, rating: 'dislike' }) - await servers[1].videos.rate({ id: vodVideoId, rating: 'dislike' }) - - await waitJobs(servers) - - const stats = await servers[0].videoStats.getOverallStats({ videoId: vodVideoId }) - - expect(stats.likes).to.equal(0) - expect(stats.dislikes).to.equal(2) - }) - - it('Should have the appropriate comments', async function () { - this.timeout(60000) - - await servers[0].comments.createThread({ videoId: vodVideoId, text: 'root' }) - await servers[0].comments.addReplyToLastThread({ text: 'reply' }) - await servers[1].comments.createThread({ videoId: vodVideoId, text: 'root' }) - - await waitJobs(servers) - - const stats = await servers[0].videoStats.getOverallStats({ videoId: vodVideoId }) - expect(stats.comments).to.equal(3) - }) - }) - describe('Test watch time stats of local videos on live and VOD', function () { let vodVideoId: string let liveVideoId: string @@ -82,8 +31,9 @@ describe('Test views overall stats', function () { it('Should display overall stats of a video with no viewers', async function () { for (const videoId of [ liveVideoId, vodVideoId ]) { const stats = await servers[0].videoStats.getOverallStats({ videoId }) + const video = await servers[0].videos.get({ id: videoId }) - expect(stats.views).to.equal(0) + expect(video.views).to.equal(0) expect(stats.averageWatchTime).to.equal(0) expect(stats.totalWatchTime).to.equal(0) } @@ -100,8 +50,9 @@ describe('Test views overall stats', function () { for (const videoId of [ liveVideoId, vodVideoId ]) { const stats = await servers[0].videoStats.getOverallStats({ videoId }) + const video = await servers[0].videos.get({ id: videoId }) - expect(stats.views).to.equal(0) + expect(video.views).to.equal(0) expect(stats.averageWatchTime).to.equal(1) expect(stats.totalWatchTime).to.equal(1) } @@ -118,14 +69,18 @@ describe('Test views overall stats', function () { { const stats = await servers[0].videoStats.getOverallStats({ videoId: vodVideoId }) - expect(stats.views).to.equal(1) + const video = await servers[0].videos.get({ id: vodVideoId }) + + expect(video.views).to.equal(1) expect(stats.averageWatchTime).to.equal(2) expect(stats.totalWatchTime).to.equal(4) } { const stats = await servers[0].videoStats.getOverallStats({ videoId: liveVideoId }) - expect(stats.views).to.equal(1) + const video = await servers[0].videos.get({ id: liveVideoId }) + + expect(video.views).to.equal(1) expect(stats.averageWatchTime).to.equal(21) expect(stats.totalWatchTime).to.equal(41) } @@ -143,16 +98,18 @@ describe('Test views overall stats', function () { { const stats = await servers[0].videoStats.getOverallStats({ videoId: vodVideoId }) + const video = await servers[0].videos.get({ id: vodVideoId }) - expect(stats.views).to.equal(1) + expect(video.views).to.equal(1) expect(stats.averageWatchTime).to.equal(2) expect(stats.totalWatchTime).to.equal(6) } { const stats = await servers[0].videoStats.getOverallStats({ videoId: liveVideoId }) + const video = await servers[0].videos.get({ id: liveVideoId }) - expect(stats.views).to.equal(1) + expect(video.views).to.equal(1) expect(stats.averageWatchTime).to.equal(14) expect(stats.totalWatchTime).to.equal(43) } @@ -167,14 +124,18 @@ describe('Test views overall stats', function () { { const stats = await servers[0].videoStats.getOverallStats({ videoId: vodVideoId }) - expect(stats.views).to.equal(2) + const video = await servers[0].videos.get({ id: vodVideoId }) + + expect(video.views).to.equal(2) expect(stats.averageWatchTime).to.equal(3) expect(stats.totalWatchTime).to.equal(11) } { const stats = await servers[0].videoStats.getOverallStats({ videoId: liveVideoId }) - expect(stats.views).to.equal(2) + const video = await servers[0].videos.get({ id: liveVideoId }) + + expect(video.views).to.equal(2) expect(stats.averageWatchTime).to.equal(22) expect(stats.totalWatchTime).to.equal(88) } -- cgit v1.2.3