From 41137192096590b171563bc3161ede6f5c1d15db Mon Sep 17 00:00:00 2001
From: John Livingston <git@john-livingston.fr>
Date: Thu, 8 Apr 2021 17:19:12 +0200
Subject: [PATCH] Tests that show the bug.

---
 .../peertube-plugin-test-unloading/lib.js     |  2 +
 .../peertube-plugin-test-unloading/main.js    | 14 +++
 .../package.json                              | 20 +++++
 server/tests/plugins/index.ts                 |  1 +
 server/tests/plugins/plugin-unloading.ts      | 89 +++++++++++++++++++
 5 files changed, 126 insertions(+)
 create mode 100644 server/tests/fixtures/peertube-plugin-test-unloading/lib.js
 create mode 100644 server/tests/fixtures/peertube-plugin-test-unloading/main.js
 create mode 100644 server/tests/fixtures/peertube-plugin-test-unloading/package.json
 create mode 100644 server/tests/plugins/plugin-unloading.ts

diff --git a/server/tests/fixtures/peertube-plugin-test-unloading/lib.js b/server/tests/fixtures/peertube-plugin-test-unloading/lib.js
new file mode 100644
index 000000000..f57e7cb01
--- /dev/null
+++ b/server/tests/fixtures/peertube-plugin-test-unloading/lib.js
@@ -0,0 +1,2 @@
+const d = new Date()
+exports.value = d.getTime()
diff --git a/server/tests/fixtures/peertube-plugin-test-unloading/main.js b/server/tests/fixtures/peertube-plugin-test-unloading/main.js
new file mode 100644
index 000000000..5c8457cef
--- /dev/null
+++ b/server/tests/fixtures/peertube-plugin-test-unloading/main.js
@@ -0,0 +1,14 @@
+const lib = require('./lib')
+
+async function register ({ getRouter }) {
+  const router = getRouter()
+  router.get('/get', (req, res) => res.json({ message: lib.value }))
+}
+
+async function unregister () {
+}
+
+module.exports = {
+  register,
+  unregister
+}
diff --git a/server/tests/fixtures/peertube-plugin-test-unloading/package.json b/server/tests/fixtures/peertube-plugin-test-unloading/package.json
new file mode 100644
index 000000000..7076d4b6f
--- /dev/null
+++ b/server/tests/fixtures/peertube-plugin-test-unloading/package.json
@@ -0,0 +1,20 @@
+{
+  "name": "peertube-plugin-test-unloading",
+  "version": "0.0.1",
+  "description": "Plugin test (modules unloading)",
+  "engine": {
+    "peertube": ">=1.3.0"
+  },
+  "keywords": [
+    "peertube",
+    "plugin"
+  ],
+  "homepage": "https://github.com/Chocobozzz/PeerTube",
+  "author": "Chocobozzz",
+  "bugs": "https://github.com/Chocobozzz/PeerTube/issues",
+  "library": "./main.js",
+  "staticDirs": {},
+  "css": [],
+  "clientScripts": [],
+  "translations": {}
+}
diff --git a/server/tests/plugins/index.ts b/server/tests/plugins/index.ts
index fd7116efd..4534120fd 100644
--- a/server/tests/plugins/index.ts
+++ b/server/tests/plugins/index.ts
@@ -7,5 +7,6 @@ import './plugin-helpers'
 import './plugin-router'
 import './plugin-storage'
 import './plugin-transcoding'
+import './plugin-unloading'
 import './translations'
 import './video-constants'
diff --git a/server/tests/plugins/plugin-unloading.ts b/server/tests/plugins/plugin-unloading.ts
new file mode 100644
index 000000000..74ca82e2f
--- /dev/null
+++ b/server/tests/plugins/plugin-unloading.ts
@@ -0,0 +1,89 @@
+/* eslint-disable @typescript-eslint/no-unused-expressions,@typescript-eslint/require-await */
+
+import 'mocha'
+import {
+  cleanupTests,
+  flushAndRunServer,
+  getPluginTestPath,
+  makeGetRequest,
+  installPlugin,
+  uninstallPlugin,
+  ServerInfo,
+  setAccessTokensToServers
+} from '../../../shared/extra-utils'
+import { HttpStatusCode } from '../../../shared/core-utils/miscs/http-error-codes'
+import { expect } from 'chai'
+
+describe('Test plugins module unloading', function () {
+  let server: ServerInfo = null
+  const requestPath = '/plugins/test-unloading/router/get'
+  let value: string = null
+
+  before(async function () {
+    this.timeout(30000)
+
+    server = await flushAndRunServer(1)
+    await setAccessTokensToServers([ server ])
+
+    await installPlugin({
+      url: server.url,
+      accessToken: server.accessToken,
+      path: getPluginTestPath('-unloading')
+    })
+  })
+
+  it('Should return a numeric value', async function () {
+    const res = await makeGetRequest({
+      url: server.url,
+      path: requestPath,
+      statusCodeExpected: HttpStatusCode.OK_200
+    })
+
+    expect(res.body.message).to.match(/^\d+$/)
+    value = res.body.message
+  })
+
+  it('Should return the same value the second time', async function () {
+    const res = await makeGetRequest({
+      url: server.url,
+      path: requestPath,
+      statusCodeExpected: HttpStatusCode.OK_200
+    })
+
+    expect(res.body.message).to.be.equal(value)
+  })
+
+  it('Should uninstall the plugin and free the route', async function () {
+    await uninstallPlugin({
+      url: server.url,
+      accessToken: server.accessToken,
+      npmName: 'peertube-plugin-test-unloading'
+    })
+
+    await makeGetRequest({
+      url: server.url,
+      path: requestPath,
+      statusCodeExpected: HttpStatusCode.NOT_FOUND_404
+    })
+  })
+
+  it('Should return a different numeric value', async function () {
+    await installPlugin({
+      url: server.url,
+      accessToken: server.accessToken,
+      path: getPluginTestPath('-unloading')
+    })
+    const res = await makeGetRequest({
+      url: server.url,
+      path: requestPath,
+      statusCodeExpected: HttpStatusCode.OK_200
+    })
+
+    expect(res.body.message).to.match(/^\d+$/)
+    expect(res.body.message).to.be.not.equal(value)
+  })
+
+  after(async function () {
+    await cleanupTests([ server ])
+  })
+})
-- 
2.41.0