All tests file imports changed from commonjs to es-module

tests/helpers/createCacheEntry.js

@@ -1,7 +1,5 @@
-const redisCache = require("../../src/cache/redis");
+import redisCache from "../../src/cache/redis.js";
 
-function createCacheEntry(key, value) {
+export default function createCacheEntry(key, value) {
   return redisCache.set(key, value);
 }
-
-module.exports = createCacheEntry;

tests/helpers/createToken.js

@@ -1,10 +1,8 @@
-const User = require("../../src/user/user");
+import User from "../../src/user/user.js";
-const Token = require("../../src/user/token");
+import Token from "../../src/user/token.js";
 
-function createToken(username, secret) {
+export default function createToken(username, secret) {
   const user = new User(username);
   const token = new Token(user);
   return token.toString(secret);
 }
-
-module.exports = createToken;

tests/helpers/createUser.js

@@ -1,11 +1,9 @@
-const User = require("../../src/user/user");
+import User from "../../src/user/user.js";
-const UserSecurity = require("../../src/user/userSecurity");
+import UserSecurity from "../../src/user/userSecurity.js";
 
-function createUser(username, password) {
+export default function createUser(username, password) {
   const userSecurity = new UserSecurity();
   const user = new User(username);
 
   return userSecurity.createNewUser(user, password);
 }
-
-module.exports = createUser;

tests/helpers/resetDatabase.js

@@ -10,13 +10,11 @@
 //   }
 // }
 
-const establishedDatabase = require("../../src/database/database");
+import establishedDatabase from "../../src/database/database.js";
 // const establishedDatabase = new EstablishedDatabase();
 
-function resetDatabase() {
+export default function resetDatabase() {
   return Promise.resolve()
     .then(() => establishedDatabase.tearDown())
     .then(() => establishedDatabase.setUp());
 }
-
-module.exports = resetDatabase;

tests/helpers/tmdbMock2.js

@@ -12,4 +12,4 @@ const tmdbMock = () => ({
   }
 });
 
-module.exports = tmdbMock;
+export default tmdbMock;

tests/system/asADeveloperIWantTheServerToRegister.js

@@ -1,9 +1,9 @@
-const assert = require("assert");
+import assert from "assert";
-const chai = require("chai");
+import chai from "chai";
-const chaiHttp = require("chai-http");
+import chaiHttp from "chai-http";
 
-const server = require("../../src/webserver/server");
+import server from "../../src/webserver/server.js";
-const resetDatabase = require("../helpers/resetDatabase");
+import resetDatabase from "../helpers/resetDatabase.js";
 
 chai.use(chaiHttp);
 

tests/system/asADeveloperIWantTheServerToStart.js.disabled

@@ -1,6 +1,6 @@
 /* eslint-disable no-return-assign */
-const net = require("net");
+import net from "net";
-const server = require("../../src/webserver/server");
+import server from "../../src/webserver/server.js";
 
 describe("As a developer I want the server to start", () => {
   after(() => {

tests/system/asADeveloperIWantToLogin.js

@@ -1,10 +1,10 @@
-const assert = require("assert");
+import assert from "assert";
-const chai = require("chai");
+import chai from "chai";
-const chaiHttp = require("chai-http");
+import chaiHttp from "chai-http";
 
-const server = require("../../src/webserver/server");
+import server from "../../src/webserver/server.js";
-const createUser = require("../helpers/createUser");
+import createUser from "../helpers/createUser.js";
-const resetDatabase = require("../helpers/resetDatabase");
+import resetDatabase from "../helpers/resetDatabase.js";
 
 chai.use(chaiHttp);
 

tests/system/asAUserIWantAForbiddenErrorIfTheTokenIsMalformed.js

@@ -1,9 +1,9 @@
-const assert = require("assert");
+import assert from "assert";
-const chai = require("chai");
+import chai from "chai";
-const chaiHttp = require("chai-http");
+import chaiHttp from "chai-http";
 
-const server = require("../../src/webserver/server");
+import server from "../../src/webserver/server.js";
-const resetDatabase = require("../helpers/resetDatabase");
+import resetDatabase from "../helpers/resetDatabase.js";
 
 chai.use(chaiHttp);
 

tests/system/asAUserIWantToGetErrorWhenRegisteringExistingUsername.js

@@ -1,10 +1,10 @@
-const assert = require("assert");
+import assert from "assert";
-const chai = require("chai");
+import chai from "chai";
-const chaiHttp = require("chai-http");
+import chaiHttp from "chai-http";
 
-const server = require("../../src/webserver/server");
+import server from "../../src/webserver/server.js";
-const createUser = require("../helpers/createUser");
+import createUser from "../helpers/createUser.js";
-const resetDatabase = require("../helpers/resetDatabase");
+import resetDatabase from "../helpers/resetDatabase.js";
 
 chai.use(chaiHttp);
 

tests/system/asAUserIWantToGetPopularMovies.js

@@ -1,17 +1,22 @@
-const assert = require("assert");
+import assert from "assert";
-const chai = require("chai");
+import chai from "chai";
-const chaiHttp = require("chai-http");
+import chaiHttp from "chai-http";
 
-const server = require("../../src/webserver/server");
+import server from "../../src/webserver/server.js";
-const resetDatabase = require("../helpers/resetDatabase");
+import resetDatabase from "../helpers/resetDatabase.js";
-const createCacheEntry = require("../helpers/createCacheEntry");
+import createCacheEntry from "../helpers/createCacheEntry.js";
-const popularMoviesSuccess = require("../fixtures/popular-movies-success-response.json");
+const popularMoviesSuccess = await import(
+  "../fixtures/popular-movies-success-response.json",
+  {
+    assert: { type: "json" }
+  }
+);
 
 chai.use(chaiHttp);
 
 describe("As a user I want to get popular movies", () => {
   beforeEach(() => resetDatabase());
-  beforeEach(() => createCacheEntry("tmdb/pm:1", popularMoviesSuccess));
+  beforeEach(() => createCacheEntry("tmdb/pm:1", popularMoviesSuccess.default));
 
   it("should return 200 with the information", done => {
     chai

tests/system/asAUserIWantToGetPopularShows.js

@@ -1,17 +1,22 @@
-const assert = require("assert");
+import assert from "assert";
-const chai = require("chai");
+import chai from "chai";
-const chaiHttp = require("chai-http");
+import chaiHttp from "chai-http";
 
-const server = require("../../src/webserver/server");
+import server from "../../src/webserver/server.js";
-const resetDatabase = require("../helpers/resetDatabase");
+import resetDatabase from "../helpers/resetDatabase.js";
-const createCacheEntry = require("../helpers/createCacheEntry");
+import createCacheEntry from "../helpers/createCacheEntry.js";
-const popularShowsSuccess = require("../fixtures/popular-show-success-response.json");
+const popularShowsSuccess = await import(
+  "../fixtures/popular-show-success-response.json",
+  {
+    assert: { type: "json" }
+  }
+);
 
 chai.use(chaiHttp);
 
 describe("As a user I want to get popular shows", () => {
   beforeEach(() => resetDatabase());
-  beforeEach(() => createCacheEntry("tmdb/pt:1", popularShowsSuccess));
+  beforeEach(() => createCacheEntry("tmdb/pt:1", popularShowsSuccess.default));
 
   it("should return 200 with the information", done => {
     chai

tests/system/asAUserIWantToRequestAMovie.js

@@ -1,20 +1,28 @@
-const assert = require("assert");
+import assert from "assert";
-const chai = require("chai");
+import chai from "chai";
-const chaiHttp = require("chai-http");
+import chaiHttp from "chai-http";
 
-const server = require("../../src/webserver/server");
+import server from "../../src/webserver/server.js";
-const createUser = require("../helpers/createUser");
+import createUser from "../helpers/createUser.js";
-const createToken = require("../helpers/createToken");
+import createToken from "../helpers/createToken.js";
-const resetDatabase = require("../helpers/resetDatabase");
+import resetDatabase from "../helpers/resetDatabase.js";
-const createCacheEntry = require("../helpers/createCacheEntry");
+import createCacheEntry from "../helpers/createCacheEntry.js";
-const infoMovieSuccess = require("../fixtures/blade_runner_2049-info-success-response.json");
+
+const infoMovieSuccess = await import(
+  "../fixtures/blade_runner_2049-info-success-response.json",
+  {
+    assert: { type: "json" }
+  }
+);
 
 chai.use(chaiHttp);
 
 describe("As a user I want to request a movie", () => {
   beforeEach(() => resetDatabase());
   beforeEach(() => createUser("test_user", "test@gmail.com", "password"));
-  beforeEach(() => createCacheEntry("mi:335984:false", infoMovieSuccess));
+  beforeEach(() =>
+    createCacheEntry("mi:335984:false", infoMovieSuccess.default)
+  );
 
   it("should return 200 when item is requested", () => {
     chai

tests/system/asAnAnonymousUserIWantToSearchForAMovie.js

@@ -1,18 +1,26 @@
-const assert = require("assert");
+import assert from "assert";
-const chai = require("chai");
+import chai from "chai";
-const chaiHttp = require("chai-http");
+import chaiHttp from "chai-http";
 
-const server = require("../../src/webserver/server");
+import server from "../../src/webserver/server.js";
-const resetDatabase = require("../helpers/resetDatabase");
+import resetDatabase from "../helpers/resetDatabase.js";
-const createCacheEntry = require("../helpers/createCacheEntry");
+import createCacheEntry from "../helpers/createCacheEntry.js";
-const interstellarQuerySuccess = require("../fixtures/interstellar-query-movie-success-response.json");
+const interstellarQuerySuccess = await import(
+  "../fixtures/interstellar-query-movie-success-response.json",
+  {
+    assert: { type: "json" }
+  }
+);
 
 chai.use(chaiHttp);
 
 describe("As an anonymous user I want to search for a movie", () => {
   beforeEach(() => resetDatabase());
   beforeEach(() =>
-    createCacheEntry("tmdb/mos:1:interstellar:false", interstellarQuerySuccess)
+    createCacheEntry(
+      "tmdb/mos:1:interstellar:false",
+      interstellarQuerySuccess.default
+    )
   );
 
   it("should return 200 with the search results even if user is not logged in", done => {

tests/unit/config/testConfig.js

@@ -1,78 +1,77 @@
-const assert = require("assert");
+import assert from "assert";
-const Config = require("../../../src/config/configuration");
+import Configuration from "../../../src/config/configuration.js";
+
+const configuration = Configuration.getInstance();
+
+let backedUpEnvironmentVariables;
+let backedUpConfigFields;
 
 describe("Config", () => {
   beforeEach(() => {
-    this.backedUpEnvironmentVariables = { ...process.env };
+    backedUpEnvironmentVariables = { ...process.env };
-    this.backedUpConfigFields = { ...Config.getInstance().fields };
+    backedUpConfigFields = { ...configuration.fields };
   });
 
   afterEach(() => {
-    process.env = this.backedUpEnvironmentVariables;
+    process.env = backedUpEnvironmentVariables;
-    Config.getInstance().fields = this.backedUpConfigFields;
+    configuration.fields = backedUpConfigFields;
   });
 
   it("should retrieve section and option from config file", () => {
-    Config.getInstance().fields = { webserver: { port: 1337 } };
+    configuration.fields = { webserver: { port: 1337 } };
-    assert.equal(Config.getInstance().get("webserver", "port"), 1337);
+    assert.equal(configuration.get("webserver", "port"), 1337);
   });
 
   it("should resolve to environment variables if option is filtered with env", () => {
-    Config.getInstance().fields = {
+    configuration.fields = {
       webserver: { port: "env|SEASONED_WEBSERVER_PORT" }
     };
     process.env.SEASONED_WEBSERVER_PORT = "1338";
-    assert.equal(Config.getInstance().get("webserver", "port"), 1338);
+    assert.equal(configuration.get("webserver", "port"), 1338);
   });
 
   it("raises an exception if the environment variable does not exist", () => {
-    Config.getInstance().fields = { webserver: { port: "env|DOES_NOT_EXIST" } };
+    configuration.fields = { webserver: { port: "env|DOES_NOT_EXIST" } };
     process.env.SEASONED_WEBSERVER_PORT = "1338";
-    assert.throws(() => Config.getInstance().get("webserver", "port"), /empty/);
+    assert.throws(() => configuration.get("webserver", "port"), /empty/);
   });
 
   it("raises an exception if the environment variable is empty", () => {
-    Config.getInstance().fields = {
+    configuration.fields = {
       webserver: { port: "env|SEASONED_WEBSERVER_PORT" }
     };
     process.env.SEASONED_WEBSERVER_PORT = "";
-    assert.throws(() => Config.getInstance().get("webserver", "port"), /empty/);
+    assert.throws(() => configuration.get("webserver", "port"), /empty/);
   });
 
   it("raises an exception if the section does not exist in the file", () => {
-    Config.getInstance().fields = { webserver: { port: "1338" } };
+    configuration.fields = { webserver: { port: "1338" } };
-    assert.throws(
+    assert.throws(() => configuration.get("woops", "port"), /does not exist/);
-      () => Config.getInstance().get("woops", "port"),
-      /does not exist/
-    );
   });
 
   it("raises an exception if the option does not exist in the file", () => {
-    Config.getInstance().fields = { webserver: { port: "1338" } };
+    configuration.fields = { webserver: { port: "1338" } };
     assert.throws(
-      () => Config.getInstance().get("webserver", "woops"),
+      () => configuration.get("webserver", "woops"),
       /does not exist/
     );
   });
 
   it("returns an array if field is an array", () => {
-    Config.getInstance().fields = { bouncer: { whitelist: [1, 2, 3] } };
+    configuration.fields = { bouncer: { whitelist: [1, 2, 3] } };
-    assert.deepEqual(
+    assert.deepEqual(configuration.get("bouncer", "whitelist"), [1, 2, 3]);
-      Config.getInstance().get("bouncer", "whitelist"),
-      [1, 2, 3]
-    );
   });
 
   it("decodes field as base64 if base64| is before the variable", () => {
-    Config.getInstance().fields = { webserver: { port: "base64|MTMzOA==" } };
+    configuration.fields = { webserver: { port: "base64|MTMzOA==" } };
-    assert.equal(Config.getInstance().get("webserver", "port"), 1338);
+    assert.equal(configuration.get("webserver", "port"), 1338);
   });
 
   it("decodes environment variable as base64 if BASE64= is before the variable", () => {
-    Config.getInstance().fields = {
+    configuration.fields = {
       webserver: { port: "env|base64|SEASONED_WEBSERVER_PORT" }
     };
     process.env.SEASONED_WEBSERVER_PORT = "MTMzOA==";
-    assert.equal(Config.getInstance().get("webserver", "port"), 1338);
+    assert.equal(configuration.get("webserver", "port"), 1338);
   });
 });

tests/unit/config/testField.js

@@ -1,5 +1,5 @@
-const assert = require("assert");
+import assert from "assert";
-const Field = require("../../../src/config/field");
+import Field from "../../../src/config/field.js";
 
 describe("Field", () => {
   it("should return an array if it is an array", () => {

tests/unit/config/testFilters.js

@@ -1,5 +1,5 @@
-const assert = require("assert");
+import assert from "assert";
-const Filters = require("../../../src/config/filters");
+import Filters from "../../../src/config/filters.js";
 
 describe("Filters", () => {
   it("should extract base64 as filter if it is at start of string followed by pipe", () => {

tests/unit/tmdb/testConvertTmdbToMovie.js

@@ -1,38 +1,37 @@
-const assert = require("assert");
+import assert from "assert";
-// const convertTmdbToMovie = require('src/tmdb/convertTmdbToMovie');
+// import convertTmdbToMovie = require('src/tmdb/convertTmdbToMovie');
-const { Movie } = require("../../../src/tmdb/types");
+import { Movie } from "../../../src/tmdb/types.js";
-const bladeRunnerQuerySuccess = require("../../fixtures/blade_runner_2049-info-success-response.json");
+const bladeRunnerQuerySuccess = await import(
+  "../../fixtures/blade_runner_2049-info-success-response.json",
+  {
+    assert: { type: "json" }
+  }
+);
+
+let bladeRunnerTmdbMovie;
 
 describe("Convert tmdb movieInfo to movie", () => {
   beforeEach(() => {
-    [this.bladeRunnerTmdbMovie] = bladeRunnerQuerySuccess;
+    bladeRunnerTmdbMovie = bladeRunnerQuerySuccess.default[0];
   });
 
   it("should translate the tmdb release date to movie year", () => {
-    const bladeRunner = Movie.convertFromTmdbResponse(
+    const bladeRunner = Movie.convertFromTmdbResponse(bladeRunnerTmdbMovie);
-      this.bladeRunnerTmdbMovie
-    );
     assert.strictEqual(bladeRunner.year, 2017);
   });
 
   it("should translate the tmdb release date to instance of Date", () => {
-    const bladeRunner = Movie.convertFromTmdbResponse(
+    const bladeRunner = Movie.convertFromTmdbResponse(bladeRunnerTmdbMovie);
-      this.bladeRunnerTmdbMovie
-    );
     assert(bladeRunner.releaseDate instanceof Date);
   });
 
   it("should translate the tmdb title to title", () => {
-    const bladeRunner = Movie.convertFromTmdbResponse(
+    const bladeRunner = Movie.convertFromTmdbResponse(bladeRunnerTmdbMovie);
-      this.bladeRunnerTmdbMovie
-    );
     assert.equal(bladeRunner.title, "Blade Runner 2049");
   });
 
   it("should translate the tmdb vote_average to rating", () => {
-    const bladeRunner = Movie.convertFromTmdbResponse(
+    const bladeRunner = Movie.convertFromTmdbResponse(bladeRunnerTmdbMovie);
-      this.bladeRunnerTmdbMovie
-    );
     assert.equal(bladeRunner.rating, 7.3);
   });
 });

tests/unit/tmdb/testTmdb.disabled

@@ -1,13 +1,13 @@
-const assert = require('assert');
+import assert 'assert';
-// const Movie = require('src/movie/movie');
+// import require('src/movie/movie.js';
-const TMDB = require('src/tmdb/tmdb');
+import TMDB 'src/tmdb/tmdb.js';
-const Cache = require('src/tmdb/cache');
+import Cache 'src/tmdb/cache.js';
-const SqliteDatabase = require('src/database/sqliteDatabase');
+import SqliteDatabase 'src/database/sqliteDatabase.js';
-const tmdbMock = require('test/helpers/tmdbMock');
+import tmdbMock 'test/helpers/tmdbMock.js';
 
-const emptyQuerySuccess = require('tests/fixtures/empty-query-success-response.json');
+import emptyQuerySuccess 'tests/fixtures/empty-query-success-response.json';
-const interstellarQuerySuccess = require('tests/fixtures/arrival-info-success-response.json');
+import interstellarQuerySuccess 'tests/fixture/arrival-info-success-response.json');
-const popularMovieSuccessResponse = require('tests/fixtures/popular-movies-success-response.json');
+import popularMovieSuccessResponse 'tests/fixture/popular-movies-success-response.json');
 
 describe('TMDB', function test() {
   beforeEach(() => {