fixed test error #8

Author: Nathaniel Lin

.env

@@ -1,2 +1,4 @@
 NODE_ENV=DEV
 PORT=5000
+MONGO_URI="mongodb://localhost:27017/api_server_nodejs"
+SECRET="SuperS3cret_4277m"

.eslintrc.js

@@ -15,5 +15,6 @@ module.exports = {
     'no-param-reassign': 'off',
     'no-underscore-dangle': 'off',
     'consistent-return': 'off',
+    'no-console': 'off',
   },
 };

.nvmrc

@@ -0,0 +1 @@
+v14

jest.config.js

@@ -1,5 +1,9 @@
 module.exports = {
+  transform: {
+    '^.+\\.ts?$': 'ts-jest',
+  },
+  setupFiles: ['dotenv/config'],
   testEnvironment: 'node',
   coveragePathIgnorePatterns: ['node_modules'],
   coverageReporters: ['text', 'lcov', 'clover', 'html'],
-};
+};

package-lock.json

@@ -6,10 +6,10 @@
     "author": "AppSeed.us",
     "main": "src/index",
     "scripts": {
-        "start": "npm run build && node build/index.js",
+        "start": "yarn build && node build/index.js",
         "dev": "ts-node-dev src/index.ts",
         "test": "jest -i --colors --verbose --detectOpenHandles",
-        "lint": "eslint src",
+        "lint": "eslint src --ext .ts",
         "build": "tsc -p tsconfig.build.json",
         "typecheck": "tsc --noEmit"
     },
@@ -28,12 +28,13 @@
         "passport-jwt": "^4.0.0"
     },
     "devDependencies": {
+        "ts-jest": "^27.0.3",
+        "mongodb-memory-server": "^7.2.0",
         "@types/bcrypt-nodejs": "0.0.31",
         "@types/compression": "^1.7.1",
         "@types/cors": "^2.8.11",
         "@types/express": "^4.17.13",
         "@types/jest": "^26.0.24",
-        "@types/joi": "^17.2.3",
         "@types/node": "^16.0.1",
         "@types/passport-jwt": "^3.0.6",
         "@types/supertest": "^2.0.11",
@@ -48,4 +49,4 @@
         "ts-node-dev": "1.1.8",
         "typescript": "^4.3.5"
     }
-}
+}

package.json

@@ -8,12 +8,11 @@ import passport from 'passport';
 import { ExtractJwt, Strategy as JwtStrategy } from 'passport-jwt';
 
 import User from '../models/user';
-import config from './keys';
 
 export default (pass: passport.PassportStatic) => {
   const opts = {
     jwtFromRequest: ExtractJwt.fromAuthHeaderWithScheme('jwt'),
-    secretOrKey: config.secret,
+    secretOrKey: process.env.SECRET,
   };
 
   pass.use(

src/config/keys.ts

@@ -8,7 +8,6 @@ import express from 'express';
 import Joi from 'joi';
 import jwt from 'jsonwebtoken';
 
-import config from '../config/keys';
 import { checkToken } from '../config/safeRoutes';
 import ActiveSession from '../models/activeSession';
 import User from '../models/user';
@@ -87,7 +86,10 @@ router.post('/login', (req, res) => {
 
     bcrypt.compare(password, user.password, (_err2, isMatch) => {
       if (isMatch) {
-        const token = jwt.sign(user.toJSON(), config.secret, {
+        if (!process.env.SECRET) {
+          throw new Error('SECRET not provided');
+        }
+        const token = jwt.sign(user.toJSON(), process.env.SECRET, {
           expiresIn: 86400, // 1 week
         });
 

src/config/passport.ts

@@ -0,0 +1,43 @@
+import 'dotenv/config';
+
+import { MongoMemoryServer } from 'mongodb-memory-server';
+import mongoose from 'mongoose';
+
+const mongooseOptions: mongoose.ConnectionOptions = {
+  connectTimeoutMS: 30000,
+  keepAlive: true,
+  useNewUrlParser: true,
+  useFindAndModify: false,
+  useCreateIndex: true,
+  useUnifiedTopology: true,
+};
+
+export const prepareDB = async () => {
+  const server: MongoMemoryServer = await MongoMemoryServer.create();
+
+  const start = () => {
+    const mongoUri = server?.getUri();
+    mongoose.connect(mongoUri, mongooseOptions, (err) => {
+      if (err) console.error(err);
+    });
+  };
+
+  const stop = async () => {
+    await mongoose.disconnect();
+    return server.stop();
+  };
+
+  return {
+    start,
+    stop,
+  };
+};
+
+export default {
+  connect(): void {
+    mongoose
+      .connect(`${process.env.MONGO_URI}`, mongooseOptions)
+      .then(() => console.log('MongoDB connected'))
+      .catch((err: unknown) => console.error(err));
+  },
+};