]> git.immae.eu Git - github/Chocobozzz/PeerTube.git/blobdiff - server/tests/api/live/live.ts
Fix audio sync after saving replay
[github/Chocobozzz/PeerTube.git] / server / tests / api / live / live.ts
index f351e96509630ef89404db76d9879c7a200af3a9..c795f201afe0560f60dca143d810eeff85941a6c 100644 (file)
@@ -2,8 +2,12 @@
 
 import 'mocha'
 import * as chai from 'chai'
-import { LiveVideo, LiveVideoCreate, User, VideoDetails, VideoPrivacy } from '@shared/models'
+import { getLiveNotificationSocket } from '@shared/extra-utils/socket/socket-io'
+import { LiveVideo, LiveVideoCreate, User, Video, VideoDetails, VideoPrivacy, VideoState, VideoStreamingPlaylistType } from '@shared/models'
 import {
+  addVideoToBlacklist,
+  checkLiveCleanup,
+  checkResolutionsInMasterPlaylist,
   cleanupTests,
   createLive,
   createUser,
@@ -12,27 +16,29 @@ import {
   getLive,
   getMyUserInformation,
   getVideo,
+  getVideoIdFromUUID,
   getVideosList,
   makeRawRequest,
   removeVideo,
+  sendRTMPStream,
+  sendRTMPStreamInVideo,
   ServerInfo,
   setAccessTokensToServers,
   setDefaultVideoChannel,
+  stopFfmpeg,
   testFfmpegStreamError,
   testImage,
   updateCustomSubConfig,
   updateLive,
-  updateUser,
   userLogin,
-  wait,
-  waitJobs
+  waitJobs,
+  waitUntilLiveStarts
 } from '../../../../shared/extra-utils'
 
 const expect = chai.expect
 
 describe('Test live', function () {
   let servers: ServerInfo[] = []
-  let liveVideoUUID: string
   let userId: number
   let userAccessToken: string
   let userChannelId: number
@@ -49,7 +55,10 @@ describe('Test live', function () {
     await updateCustomSubConfig(servers[0].url, servers[0].accessToken, {
       live: {
         enabled: true,
-        allowReplay: true
+        allowReplay: true,
+        transcoding: {
+          enabled: false
+        }
       }
     })
 
@@ -74,6 +83,7 @@ describe('Test live', function () {
   })
 
   describe('Live creation, update and delete', function () {
+    let liveVideoUUID: string
 
     it('Should create a live with the appropriate parameters', async function () {
       this.timeout(20000)
@@ -220,246 +230,298 @@ describe('Test live', function () {
     })
   })
 
-  describe('Test live constraints', function () {
+  describe('Stream checks', function () {
+    let liveVideo: LiveVideo & VideoDetails
+    let rtmpUrl: string
 
-    async function createLiveWrapper (saveReplay: boolean) {
+    before(function () {
+      rtmpUrl = 'rtmp://' + servers[0].hostname + ':1936'
+    })
+
+    async function createLiveWrapper () {
       const liveAttributes = {
         name: 'user live',
-        channelId: userChannelId,
+        channelId: servers[0].videoChannel.id,
         privacy: VideoPrivacy.PUBLIC,
-        saveReplay
+        saveReplay: false
       }
 
-      const res = await createLive(servers[0].url, userAccessToken, liveAttributes)
-      return res.body.video.uuid as string
+      const res = await createLive(servers[0].url, servers[0].accessToken, liveAttributes)
+      const uuid = res.body.video.uuid
+
+      const resLive = await getLive(servers[0].url, servers[0].accessToken, uuid)
+      const resVideo = await getVideo(servers[0].url, uuid)
+
+      return Object.assign(resVideo.body, resLive.body) as LiveVideo & VideoDetails
     }
 
-    before(async function () {
-      await updateCustomSubConfig(servers[0].url, servers[0].accessToken, {
-        live: {
-          enabled: true,
-          allowReplay: true
-        }
-      })
+    it('Should not allow a stream without the appropriate path', async function () {
+      this.timeout(30000)
 
-      await updateUser({
-        url: servers[0].url,
-        userId,
-        accessToken: servers[0].accessToken,
-        videoQuota: 1,
-        videoQuotaDaily: -1
-      })
+      liveVideo = await createLiveWrapper()
+
+      const command = sendRTMPStream(rtmpUrl + '/bad-live', liveVideo.streamKey)
+      await testFfmpegStreamError(command, true)
     })
 
-    it('Should not have size limit if save replay is disabled', async function () {
+    it('Should not allow a stream without the appropriate stream key', async function () {
       this.timeout(30000)
 
-      const userVideoLiveoId = await createLiveWrapper(false)
-      await testFfmpegStreamError(servers[0].url, userAccessToken, userVideoLiveoId, false)
+      const command = sendRTMPStream(rtmpUrl + '/live', 'bad-stream-key')
+      await testFfmpegStreamError(command, true)
     })
 
-    it('Should have size limit depending on user global quota if save replay is enabled', async function () {
+    it('Should succeed with the correct params', async function () {
       this.timeout(30000)
 
-      const userVideoLiveoId = await createLiveWrapper(true)
-      await testFfmpegStreamError(servers[0].url, userAccessToken, userVideoLiveoId, true)
+      const command = sendRTMPStream(rtmpUrl + '/live', liveVideo.streamKey)
+      await testFfmpegStreamError(command, false)
+    })
 
-      await waitJobs(servers)
+    it('Should not allow a stream on a live that was blacklisted', async function () {
+      this.timeout(30000)
 
-      for (const server of servers) {
-        const res = await getVideo(server.url, userVideoLiveoId)
+      liveVideo = await createLiveWrapper()
 
-        const video: VideoDetails = res.body
-        expect(video.isLive).to.be.false
-        expect(video.duration).to.be.greaterThan(0)
-      }
+      await addVideoToBlacklist(servers[0].url, servers[0].accessToken, liveVideo.uuid)
 
-      // TODO: check stream correctly saved + cleaned
+      const command = sendRTMPStream(rtmpUrl + '/live', liveVideo.streamKey)
+      await testFfmpegStreamError(command, true)
     })
 
-    it('Should have size limit depending on user daily quota if save replay is enabled', async function () {
+    it('Should not allow a stream on a live that was deleted', async function () {
       this.timeout(30000)
 
-      await updateUser({
-        url: servers[0].url,
-        userId,
-        accessToken: servers[0].accessToken,
-        videoQuota: -1,
-        videoQuotaDaily: 1
-      })
+      liveVideo = await createLiveWrapper()
 
-      const userVideoLiveoId = await createLiveWrapper(true)
-      await testFfmpegStreamError(servers[0].url, userAccessToken, userVideoLiveoId, true)
+      await removeVideo(servers[0].url, servers[0].accessToken, liveVideo.uuid)
 
-      // TODO: check stream correctly saved + cleaned
+      const command = sendRTMPStream(rtmpUrl + '/live', liveVideo.streamKey)
+      await testFfmpegStreamError(command, true)
     })
+  })
 
-    it('Should succeed without quota limit', async function () {
-      this.timeout(30000)
+  describe('Live transcoding', function () {
+    let liveVideoId: string
 
-      // Wait for user quota memoize cache invalidation
-      await wait(5000)
+    async function createLiveWrapper (saveReplay: boolean) {
+      const liveAttributes = {
+        name: 'live video',
+        channelId: servers[0].videoChannel.id,
+        privacy: VideoPrivacy.PUBLIC,
+        saveReplay
+      }
 
-      await updateUser({
-        url: servers[0].url,
-        userId,
-        accessToken: servers[0].accessToken,
-        videoQuota: 10 * 1000 * 1000,
-        videoQuotaDaily: -1
-      })
+      const res = await createLive(servers[0].url, servers[0].accessToken, liveAttributes)
+      return res.body.video.uuid
+    }
 
-      const userVideoLiveoId = await createLiveWrapper(true)
-      await testFfmpegStreamError(servers[0].url, userAccessToken, userVideoLiveoId, false)
-    })
+    async function testVideoResolutions (liveVideoId: string, resolutions: number[]) {
+      for (const server of servers) {
+        const resList = await getVideosList(server.url)
+        const videos: Video[] = resList.body.data
 
-    it('Should have max duration limit', async function () {
-      this.timeout(30000)
+        expect(videos.find(v => v.uuid === liveVideoId)).to.exist
 
-      await updateCustomSubConfig(servers[0].url, servers[0].accessToken, {
+        const resVideo = await getVideo(server.url, liveVideoId)
+        const video: VideoDetails = resVideo.body
+
+        expect(video.streamingPlaylists).to.have.lengthOf(1)
+
+        const hlsPlaylist = video.streamingPlaylists.find(s => s.type === VideoStreamingPlaylistType.HLS)
+        expect(hlsPlaylist).to.exist
+
+        // Only finite files are displayed
+        expect(hlsPlaylist.files).to.have.lengthOf(0)
+
+        await checkResolutionsInMasterPlaylist(hlsPlaylist.playlistUrl, resolutions)
+      }
+    }
+
+    function updateConf (resolutions: number[]) {
+      return updateCustomSubConfig(servers[0].url, servers[0].accessToken, {
         live: {
           enabled: true,
           allowReplay: true,
-          maxDuration: 1
+          maxDuration: null,
+          transcoding: {
+            enabled: true,
+            resolutions: {
+              '240p': resolutions.includes(240),
+              '360p': resolutions.includes(360),
+              '480p': resolutions.includes(480),
+              '720p': resolutions.includes(720),
+              '1080p': resolutions.includes(1080),
+              '2160p': resolutions.includes(2160)
+            }
+          }
         }
       })
+    }
 
-      const userVideoLiveoId = await createLiveWrapper(true)
-      await testFfmpegStreamError(servers[0].url, userAccessToken, userVideoLiveoId, true)
-
-      // TODO: check stream correctly saved + cleaned
+    before(async function () {
+      await updateConf([])
     })
-  })
 
-  describe('With save replay disabled', function () {
+    it('Should enable transcoding without additional resolutions', async function () {
+      this.timeout(30000)
 
-    it('Should correctly create and federate the "waiting for stream" live', async function () {
+      liveVideoId = await createLiveWrapper(false)
 
-    })
+      const command = await sendRTMPStreamInVideo(servers[0].url, servers[0].accessToken, liveVideoId)
+      await waitUntilLiveStarts(servers[0].url, servers[0].accessToken, liveVideoId)
+      await waitJobs(servers)
 
-    it('Should correctly have updated the live and federated it when streaming in the live', async function () {
+      await testVideoResolutions(liveVideoId, [ 720 ])
 
+      await stopFfmpeg(command)
     })
 
-    it('Should correctly delete the video and the live after the stream ended', async function () {
-      // Wait 10 seconds
-      // get video 404
-      // get video federation 404
-
-      // check cleanup
-    })
+    it('Should enable transcoding with some resolutions', async function () {
+      this.timeout(30000)
 
-    it('Should correctly terminate the stream on blacklist and delete the live', async function () {
-      // Wait 10 seconds
-      // get video 404
-      // get video federation 404
+      const resolutions = [ 240, 480 ]
+      await updateConf(resolutions)
+      liveVideoId = await createLiveWrapper(false)
 
-      // check cleanup
-    })
+      const command = await sendRTMPStreamInVideo(servers[0].url, servers[0].accessToken, liveVideoId)
+      await waitUntilLiveStarts(servers[0].url, servers[0].accessToken, liveVideoId)
+      await waitJobs(servers)
 
-    it('Should correctly terminate the stream on delete and delete the video', async function () {
-      // Wait 10 seconds
-      // get video 404
-      // get video federation 404
+      await testVideoResolutions(liveVideoId, resolutions)
 
-      // check cleanup
+      await stopFfmpeg(command)
     })
-  })
 
-  describe('With save replay enabled', function () {
+    it('Should enable transcoding with some resolutions and correctly save them', async function () {
+      this.timeout(60000)
 
-    it('Should correctly create and federate the "waiting for stream" live', async function () {
+      const resolutions = [ 240, 360, 720 ]
+      await updateConf(resolutions)
+      liveVideoId = await createLiveWrapper(true)
 
-    })
+      const command = await sendRTMPStreamInVideo(servers[0].url, servers[0].accessToken, liveVideoId)
+      await waitUntilLiveStarts(servers[0].url, servers[0].accessToken, liveVideoId)
+      await waitJobs(servers)
 
-    it('Should correctly have updated the live and federated it when streaming in the live', async function () {
+      await testVideoResolutions(liveVideoId, resolutions)
 
-    })
+      await stopFfmpeg(command)
 
-    it('Should correctly have saved the live and federated it after the streaming', async function () {
+      await waitJobs(servers)
 
-    })
+      for (const server of servers) {
+        const resVideo = await getVideo(server.url, liveVideoId)
+        const video: VideoDetails = resVideo.body
 
-    it('Should update the saved live and correctly federate the updated attributes', async function () {
+        expect(video.duration).to.be.greaterThan(1)
+        expect(video.files).to.have.lengthOf(0)
 
-    })
+        const hlsPlaylist = video.streamingPlaylists.find(s => s.type === VideoStreamingPlaylistType.HLS)
 
-    it('Should have cleaned up the live files', async function () {
+        expect(hlsPlaylist.files).to.have.lengthOf(resolutions.length)
 
-    })
+        for (const resolution of resolutions) {
+          const file = hlsPlaylist.files.find(f => f.resolution.id === resolution)
 
-    it('Should correctly terminate the stream on blacklist and blacklist the saved replay video', async function () {
-      // Wait 10 seconds
-      // get video -> blacklisted
-      // get video federation -> blacklisted
+          expect(file).to.exist
+          expect(file.fps).to.equal(25)
+          expect(file.size).to.be.greaterThan(1)
 
-      // check cleanup live files quand meme
+          await makeRawRequest(file.torrentUrl, 200)
+          await makeRawRequest(file.fileUrl, 200)
+        }
+      }
     })
 
-    it('Should correctly terminate the stream on delete and delete the video', async function () {
-      // Wait 10 seconds
-      // get video 404
-      // get video federation 404
+    it('Should correctly have cleaned up the live files', async function () {
+      this.timeout(30000)
 
-      // check cleanup
+      await checkLiveCleanup(servers[0], liveVideoId, [ 240, 360, 720 ])
     })
   })
 
-  describe('Stream checks', function () {
+  describe('Live socket messages', function () {
 
-    it('Should not allow a stream without the appropriate path', async function () {
+    async function createLiveWrapper () {
+      const liveAttributes = {
+        name: 'live video',
+        channelId: servers[0].videoChannel.id,
+        privacy: VideoPrivacy.PUBLIC
+      }
 
-    })
+      const res = await createLive(servers[0].url, servers[0].accessToken, liveAttributes)
+      return res.body.video.uuid
+    }
 
-    it('Should not allow a stream without the appropriate stream key', async function () {
+    it('Should correctly send a message when the live starts and ends', async function () {
+      this.timeout(60000)
 
-    })
+      const localStateChanges: VideoState[] = []
+      const remoteStateChanges: VideoState[] = []
 
-    it('Should not allow a stream on a live that was blacklisted', async function () {
+      const liveVideoUUID = await createLiveWrapper()
+      await waitJobs(servers)
 
-    })
+      {
+        const videoId = await getVideoIdFromUUID(servers[0].url, liveVideoUUID)
 
-    it('Should not allow a stream on a live that was deleted', async function () {
+        const localSocket = getLiveNotificationSocket(servers[0].url)
+        localSocket.on('state-change', data => localStateChanges.push(data.state))
+        localSocket.emit('subscribe', { videoId })
+      }
 
-    })
-  })
+      {
+        const videoId = await getVideoIdFromUUID(servers[1].url, liveVideoUUID)
 
-  describe('Live transcoding', function () {
+        const remoteSocket = getLiveNotificationSocket(servers[1].url)
+        remoteSocket.on('state-change', data => remoteStateChanges.push(data.state))
+        remoteSocket.emit('subscribe', { videoId })
+      }
 
-    it('Should enable transcoding without additional resolutions', async function () {
-      // enable
-      // stream
-      // wait federation + test
+      const command = await sendRTMPStreamInVideo(servers[0].url, servers[0].accessToken, liveVideoUUID)
+      await waitUntilLiveStarts(servers[0].url, servers[0].accessToken, liveVideoUUID)
+      await waitJobs(servers)
 
-    })
+      for (const stateChanges of [ localStateChanges, remoteStateChanges ]) {
+        expect(stateChanges).to.have.lengthOf(1)
+        expect(stateChanges[0]).to.equal(VideoState.PUBLISHED)
+      }
 
-    it('Should enable transcoding with some resolutions', async function () {
-      // enable
-      // stream
-      // wait federation + test
-    })
+      await stopFfmpeg(command)
+      await waitJobs(servers)
 
-    it('Should enable transcoding with some resolutions and correctly save them', async function () {
-      // enable
-      // stream
-      // end stream
-      // wait federation + test
+      for (const stateChanges of [ localStateChanges, remoteStateChanges ]) {
+        expect(stateChanges).to.have.lengthOf(2)
+        expect(stateChanges[1]).to.equal(VideoState.LIVE_ENDED)
+      }
     })
 
-    it('Should correctly have cleaned up the live files', async function () {
-      // check files
-    })
-  })
+    it('Should not receive a notification after unsubscribe', async function () {
+      this.timeout(60000)
 
-  describe('Live socket messages', function () {
+      const stateChanges: VideoState[] = []
 
-    it('Should correctly send a message when the live starts', async function () {
-      // local
-      // federation
-    })
+      const liveVideoUUID = await createLiveWrapper()
+      await waitJobs(servers)
+
+      const videoId = await getVideoIdFromUUID(servers[0].url, liveVideoUUID)
+
+      const socket = getLiveNotificationSocket(servers[0].url)
+      socket.on('state-change', data => stateChanges.push(data.state))
+      socket.emit('subscribe', { videoId })
+
+      const command = await sendRTMPStreamInVideo(servers[0].url, servers[0].accessToken, liveVideoUUID)
+      await waitUntilLiveStarts(servers[0].url, servers[0].accessToken, liveVideoUUID)
+      await waitJobs(servers)
+
+      expect(stateChanges).to.have.lengthOf(1)
+      socket.emit('unsubscribe', { videoId })
+
+      await stopFfmpeg(command)
+      await waitJobs(servers)
 
-    it('Should correctly send a message when the live ends', async function () {
-      // local
-      // federation
+      expect(stateChanges).to.have.lengthOf(1)
     })
   })