Add tests for thumbnails

server/tests/api/multiplePods.js

@@ -110,8 +110,13 @@ describe('Test multiple pods', function () {
                 expect(video.magnetUri).to.equal.magnetUri
               }
 
+              utils.testImage(server.url, 'video_short1.webm', video.thumbnail_path, function (err, test) {
+                if (err) throw err
+                expect(test).to.equal(true)
+
                 callback()
               })
+            })
           }, done)
         }
       )
@@ -160,8 +165,13 @@ describe('Test multiple pods', function () {
                 expect(video.magnetUri).to.equal.magnetUri
               }
 
+              utils.testImage(server.url, 'video_short2.webm', video.thumbnail_path, function (err, test) {
+                if (err) throw err
+                expect(test).to.equal(true)
+
                 callback()
               })
+            })
           }, done)
         }
       )
@@ -231,8 +241,18 @@ describe('Test multiple pods', function () {
                 expect(video2.magnetUri).to.equal.magnetUri
               }
 
+              utils.testImage(server.url, 'video_short3.webm', video1.thumbnail_path, function (err, test) {
+                if (err) throw err
+                expect(test).to.equal(true)
+
+                utils.testImage(server.url, 'video_short.webm', video2.thumbnail_path, function (err, test) {
+                  if (err) throw err
+                  expect(test).to.equal(true)
+
                   callback()
                 })
+              })
+            })
           }, done)
         }
       )

server/tests/api/singlePod.js

@@ -76,6 +76,10 @@ describe('Test a single pod', function () {
       expect(video.author).to.equal('root')
       expect(video.isLocal).to.be.true
 
+      utils.testImage(server.url, 'video_short.webm', video.thumbnail_path, function (err, test) {
+        if (err) throw err
+        expect(test).to.equal(true)
+
         video_id = video.id
 
         webtorrent.add(video.magnetUri, function (torrent) {
@@ -87,6 +91,7 @@ describe('Test a single pod', function () {
         })
       })
     })
+  })
 
   it('Should get the video', function (done) {
     // Yes, this could be long
@@ -103,6 +108,10 @@ describe('Test a single pod', function () {
       expect(video.author).to.equal('root')
       expect(video.isLocal).to.be.true
 
+      utils.testImage(server.url, 'video_short.webm', video.thumbnail_path, function (err, test) {
+        if (err) throw err
+        expect(test).to.equal(true)
+
         webtorrent.add(video.magnetUri, function (torrent) {
           expect(torrent.files).to.exist
           expect(torrent.files.length).to.equal(1)
@@ -112,6 +121,7 @@ describe('Test a single pod', function () {
         })
       })
     })
+  })
 
   it('Should search the video', function (done) {
     utils.searchVideo(server.url, 'my', function (err, res) {
@@ -127,9 +137,14 @@ describe('Test a single pod', function () {
       expect(video.author).to.equal('root')
       expect(video.isLocal).to.be.true
 
+      utils.testImage(server.url, 'video_short.webm', video.thumbnail_path, function (err, test) {
+        if (err) throw err
+        expect(test).to.equal(true)
+
         done()
       })
     })
+  })
 
   it('Should not find a search', function (done) {
     utils.searchVideo(server.url, 'hello', function (err, res) {
@@ -197,6 +212,24 @@ describe('Test a single pod', function () {
     })
   })
 
+  it('Should have the correct thumbnails', function (done) {
+    utils.getVideosList(server.url, function (err, res) {
+      const videos = res.body
+
+      async.each(videos, function (video, callback_each) {
+        if (err) throw err
+        const video_name = video.name.replace(' name', '')
+
+        utils.testImage(server.url, video_name, video.thumbnail_path, function (err, test) {
+          if (err) throw err
+
+          expect(test).to.equal(true)
+          callback_each()
+        })
+      }, done)
+    })
+  })
+
   after(function (done) {
     process.kill(-server.app.pid)
     process.kill(-webtorrent.app.pid)

server/tests/api/utils.js

@@ -3,6 +3,7 @@
 const child_process = require('child_process')
 const exec = child_process.exec
 const fork = child_process.fork
+const fs = require('fs')
 const pathUtils = require('path')
 const request = require('supertest')
 
@@ -19,6 +20,7 @@ const testUtils = {
   flushAndRunMultipleServers: flushAndRunMultipleServers,
   runServer: runServer,
   searchVideo: searchVideo,
+  testImage: testImage,
   uploadVideo: uploadVideo
 }
 
@@ -252,6 +254,21 @@ function searchVideo (url, search, end) {
     .end(end)
 }
 
+function testImage (url, video_name, image_path, callback) {
+  request(url)
+    .get(image_path)
+    .expect(200)
+    .end(function (err, res) {
+      if (err) return callback(err)
+
+      fs.readFile(pathUtils.join(__dirname, 'fixtures', video_name + '.jpg'), function (err, data) {
+        if (err) return callback(err)
+
+        callback(null, data.equals(res.body))
+      })
+    })
+}
+
 function uploadVideo (url, access_token, name, description, fixture, special_status, end) {
   if (!end) {
     end = special_status