Fix tests

diff --git a/client/src/standalone/videos/embed.ts b/client/src/standalone/videos/embed.ts
index da935ef4c97f2da49a500faa2f8f1dcb9181ef64..925367bb9bade9b5f3ab2e765a5f9fe5ef67ab14 100644 (file)
--- a/client/src/standalone/videos/embed.ts
+++ b/client/src/standalone/videos/embed.ts
@@ -7,7 +7,7 @@ import 'videojs-dock/dist/videojs-dock.es.js'
 import { VideoDetails } from '../../../../shared'
 
 function getVideoUrl (id: string) {
-  return window.location.origin + '/api/v1/videos/' + videoId
+  return window.location.origin + '/api/v1/videos/' + id
 }
 
 async function loadVideoInfo (videoId: string): Promise<VideoDetails> {

diff --git a/server/tests/api/fixtures/avatar-resized.png b/server/tests/api/fixtures/avatar-resized.png
index e05fc07ce896e0a096b2c58802cc2c9c9301f89a..9d84151f8e4ec30614b61a5ba36915a192644126 100644 (file)
Binary files a/server/tests/api/fixtures/avatar-resized.png and b/server/tests/api/fixtures/avatar-resized.png differ

diff --git a/server/tests/api/fixtures/avatar2-resized.png b/server/tests/api/fixtures/avatar2-resized.png
index a2e2613bfcb89e28f8d6f03deb7c1e921917a166..44149facbe39fc9e291e81bd9a52449f24e7be0a 100644 (file)
Binary files a/server/tests/api/fixtures/avatar2-resized.png and b/server/tests/api/fixtures/avatar2-resized.png differ

diff --git a/server/tests/api/users/users-multiple-servers.ts b/server/tests/api/users/users-multiple-servers.ts
index 0483b9c3de07c7a1ed1b0be66ce904a362efec3b..3c4eaff14976ac96a9e97872c8c69abb50d103ba 100644 (file)
--- a/server/tests/api/users/users-multiple-servers.ts
+++ b/server/tests/api/users/users-multiple-servers.ts
@@ -65,8 +65,7 @@ describe('Test users with multiple servers', function () {
     const res = await getMyUserInformation(servers[0].url, servers[0].accessToken)
     user = res.body
 
-    const test = await testImage(servers[0].url, 'avatar2-resized', user.account.avatar.path, '.png')
-    expect(test).to.equal(true)
+    await testImage(servers[0].url, 'avatar2-resized', user.account.avatar.path, '.png')
 
     await wait(5000)
   })
@@ -83,8 +82,7 @@ describe('Test users with multiple servers', function () {
       expect(rootServer1Get.name).to.equal('root')
       expect(rootServer1Get.host).to.equal('localhost:9001')
 
-      const test = await testImage(server.url, 'avatar2-resized', rootServer1Get.avatar.path, '.png')
-      expect(test).to.equal(true)
+      await testImage(server.url, 'avatar2-resized', rootServer1Get.avatar.path, '.png')
     }
   })
 

diff --git a/server/tests/api/users/users.ts b/server/tests/api/users/users.ts
index c23b5808980991d768d74527d3c462a80b68f8c5..ac167d4f97c20e404a3ed2898acf0f8e2c9d9c41 100644 (file)
--- a/server/tests/api/users/users.ts
+++ b/server/tests/api/users/users.ts
@@ -359,8 +359,7 @@ describe('Test users', function () {
     const res = await getMyUserInformation(server.url, accessTokenUser)
     const user = res.body
 
-    const test = await testImage(server.url, 'avatar-resized', user.account.avatar.path, '.png')
-    expect(test).to.equal(true)
+    await testImage(server.url, 'avatar-resized', user.account.avatar.path, '.png')
   })
 
   it('Should be able to update another user', async function () {

diff --git a/server/tests/api/videos/multiple-servers.ts b/server/tests/api/videos/multiple-servers.ts
index 3646fbb0febb417562e3c6531cc99c6721f84dd1..85d158d61aefe24c6755bbc3727be663e181a244 100644 (file)
--- a/server/tests/api/videos/multiple-servers.ts
+++ b/server/tests/api/videos/multiple-servers.ts
@@ -641,8 +641,7 @@ describe('Test multiple servers', function () {
         const res = await getVideo(server.url, videoUUID)
         const video = res.body
 
-        const test = await testImage(server.url, 'video_short1-preview.webm', video.previewPath)
-        expect(test).to.equal(true)
+        await testImage(server.url, 'video_short1-preview.webm', video.previewPath)
       }
     })
   })

diff --git a/server/tests/api/videos/single-server.ts b/server/tests/api/videos/single-server.ts
index 476a25479ebc2c69092516bfd4152f44437ebe3b..8f55075fb67d519904179f71008e6537f50ead74 100644 (file)
--- a/server/tests/api/videos/single-server.ts
+++ b/server/tests/api/videos/single-server.ts
@@ -341,9 +341,7 @@ describe('Test a single server', function () {
 
     for (const video of videos) {
       const videoName = video.name.replace(' name', '')
-      const test = await testImage(server.url, videoName, video.thumbnailPath)
-
-      expect(test).to.equal(true)
+      await testImage(server.url, videoName, video.thumbnailPath)
     }
   })
 

diff --git a/server/tests/api/videos/video-comments.ts b/server/tests/api/videos/video-comments.ts
index cf8166a9641232fc4baaebb8a52ea1b0c368aed6..f83d9508811dd55f20937abdf15bdc5eb49979f3 100644 (file)
--- a/server/tests/api/videos/video-comments.ts
+++ b/server/tests/api/videos/video-comments.ts
@@ -83,8 +83,7 @@ describe('Test video comments', function () {
     expect(comment.account.name).to.equal('root')
     expect(comment.account.host).to.equal('localhost:9001')
 
-    const test = await testImage(server.url, 'avatar-resized', comment.account.avatar.path, '.png')
-    expect(test).to.equal(true)
+    await testImage(server.url, 'avatar-resized', comment.account.avatar.path, '.png')
 
     expect(comment.totalReplies).to.equal(0)
     expect(dateIsValid(comment.createdAt as string)).to.be.true

diff --git a/server/tests/utils/miscs/miscs.ts b/server/tests/utils/miscs/miscs.ts
index 24cbf59ca254c6f429fa15c60f6d06eff31ca72f..53cb67baf7f54fcde78fc11b1aef4c68e9c3a694 100644 (file)
--- a/server/tests/utils/miscs/miscs.ts
+++ b/server/tests/utils/miscs/miscs.ts
@@ -1,10 +1,12 @@
 /* tslint:disable:no-unused-expression */
 
+import * as chai from 'chai'
 import { isAbsolute, join } from 'path'
 import * as request from 'supertest'
 import * as WebTorrent from 'webtorrent'
 import { readFileBufferPromise } from '../../../helpers/core-utils'
 
+const expect = chai.expect
 let webtorrent = new WebTorrent()
 
 function immutableAssign <T, U> (target: T, source: U) {
@@ -48,7 +50,8 @@ async function testImage (url: string, imageName: string, imagePath: string, ext
     const minLength = body.length - ((20 * body.length) / 100)
     const maxLength = body.length + ((20 * body.length) / 100)
 
-    return data.length > minLength && data.length < maxLength
+    expect(data.length).to.be.above(minLength)
+    expect(data.length).to.be.below(maxLength)
   } else {
     console.log('Do not test images. Enable it by setting NODE_TEST_IMAGE env variable.')
     return true

diff --git a/server/tests/utils/videos/videos.ts b/server/tests/utils/videos/videos.ts
index 8e58f1f0c7f4ff1f3b7c15a16fd703e38386252b..9a4af0b9f914c8b8040ae36ce78a116f3b50cf6c 100644 (file)
--- a/server/tests/utils/videos/videos.ts
+++ b/server/tests/utils/videos/videos.ts
@@ -445,13 +445,11 @@ async function completeVideoCheck (
     expect(file.size).to.be.above(minSize).and.below(maxSize)
 
     {
-      const test = await testImage(url, attributes.thumbnailfile || attributes.fixture, videoDetails.thumbnailPath)
-      expect(test).to.equal(true)
+      await testImage(url, attributes.thumbnailfile || attributes.fixture, videoDetails.thumbnailPath)
     }
 
     if (attributes.previewfile) {
-      const test = await testImage(url, attributes.previewfile, videoDetails.previewPath)
-      expect(test).to.equal(true)
+      await testImage(url, attributes.previewfile, videoDetails.previewPath)
     }
 
     const torrent = await webtorrentAdd(magnetUri, true)