chore(tests): [RO-26598] Improve test coverage

.editorconfig

@@ -1,12 +1,14 @@
 root = true
 
 [*]
-indent_style = space
-indent_size = 2
-end_of_line = lf
 charset = utf-8
-trim_trailing_whitespace = true
+end_of_line = lf
+indent_size = 2
+indent_style = space
 insert_final_newline = true
+max_line_length = 120
+quote_type = single
+trim_trailing_whitespace = true
 
 [node_modules/**.js]
 codepaint = false

.vscode/settings.json

@@ -0,0 +1,3 @@
+{
+  "prettier.enable": false,
+}

jest.config.js

@@ -6,14 +6,14 @@ module.exports = {
   coveragePathIgnorePatterns: ['/node_modules/'],
   coverageThreshold: {
     global: {
-      branches: 62,
-      functions: 84,
-      lines: 89,
-      statements: 89
-    }
+      branches: 88,
+      functions: 98,
+      lines: 98,
+      statements: 98,
+    },
   },
   maxWorkers: 1,
-  moduleFileExtensions: ['js','json'],
+  moduleFileExtensions: ['js', 'json'],
   rootDir: '.',
   testEnvironment: 'node',
   testMatch: ['**/*.spec.js'],

package.json

@@ -65,6 +65,7 @@
   "devDependencies": {
     "@types/jest": "29.5.12",
     "colors": "1.4.0",
+    "ejs": "3.1.9",
     "eslint": "8.42.0",
     "eslint-plugin-node": "11.1.0",
     "jest": "29.7.0",

src/middleware/ErrorMiddleware.js

@@ -1,14 +1,8 @@
 const _some = require('lodash.some');
 const _assignIn = require('lodash.assignin');
 
-module.exports = function (
-  SpurErrors,
-  Logger,
-  HtmlErrorRender,
-  BaseMiddleware
-) {
+module.exports = function (SpurErrors, Logger, HtmlErrorRender, BaseMiddleware) {
   class ErrorMiddleware extends BaseMiddleware {
-
     configure(app) {
       super.configure(app);
 
@@ -34,29 +28,28 @@ module.exports = function (
       res.format({
         text: () => this.sendTextResponse(err, req, res),
         html: () => this.sendHtmlResponse(err, req, res),
-        json: () => this.sendJsonResponse(err, req, res)
+        json: () => this.sendJsonResponse(err, req, res),
       });
 
       next();
     }
 
     logErrorStack(err) {
-      const statusCode = err.statusCode || 0;
+      const { statusCode } = err;
       const checkStatus = (status) => status === statusCode;
 
       if (!_some(this.EXCLUDE_STATUSCODE_FROM_LOGS, checkStatus)) {
-        Logger.error(err, '\n', err.stack, '\n', (err.data || ''));
+        Logger.error(err, '\n', err.stack, '\n', err.data);
       }
     }
 
     appendRequestData(err, req) {
-      if (err.data == null) {
-        err.data = {};
-      }
-
-      err.data = _assignIn(err.data, {
-        url: req.url
-      });
+      err.data = _assignIn(
+        { ...err.data },
+        {
+          url: req.url,
+        },
+      );
     }
 
     sendTextResponse(err, req, res) {

src/middleware/PromiseMiddleware.js

@@ -1,10 +1,5 @@
-module.exports = function (
-  Promise,
-  BaseMiddleware
-) {
-
+module.exports = function (Promise, BaseMiddleware) {
   class PromiseMiddleware extends BaseMiddleware {
-
     configure(app) {
       super.configure(app);
 
@@ -14,8 +9,8 @@ module.exports = function (
           .catch(this.req.next);
       };
 
-      this.app.response.renderAsync = function (view, properties = {}) {
-        return Promise.props(properties)
+      this.app.response.renderAsync = function (view, properties) {
+        return Promise.props(properties ?? {})
           .then((props) => this.render(view, props))
           .catch(this.req.next);
       };
@@ -42,7 +37,7 @@ module.exports = function (
               },
               json: () => {
                 this.json(results);
-              }
+              },
             });
           })
           .catch(this.req.next);

src/webserver/BaseWebServer.js

@@ -7,22 +7,15 @@ module.exports = function (
   ErrorMiddleware,
   config,
   ControllerRegistration,
-  WinstonRequestLoggingMiddleware
+  WinstonRequestLoggingMiddleware,
 ) {
   class BaseWebServer {
-
     constructor() {
       this.app = express();
     }
 
     getPort() {
-      let port = config.Port;
-
-      if (this.server) {
-        port = this.server.address().port || port;
-      }
-
-      return port;
+      return this.server?.address()?.port ?? config.Port;
     }
 
     registerDefaultMiddleware() {
@@ -88,19 +81,20 @@ module.exports = function (
     }
 
     getCloseAsync() {
-      if (this.server && this.server.closeAsync) {
+      if (this.server?.closeAsync) {
         return this.server.closeAsync();
       }
 
       return Promise.resolve();
     }
 
     startedMessage() {
+      const port = this.getPort();
       if (this.cluster) {
-        return `Worker ${this.cluster.worker.id} started on port ${this.getPort()}`;
+        return `Worker ${this.cluster.worker.id} started on port ${port}`;
       }
 
-      return `Express app started on port ${this.getPort()}`;
+      return `Express app started on port ${port}`;
     }
 
     logSectionHeader(message) {

test/fixtures/controllers/PromiseMiddlewareTestController.js

@@ -1,9 +1,11 @@
 module.exports = function (BaseController, Promise) {
-
   class PromiseMiddlewareTestController extends BaseController {
-
     configure(app) {
       super.configure(app);
+
+      app.set('view engine', 'ejs');
+      app.set('views', `${__dirname}/views`);
+
       app.get('/promise-middleware-test--jsonasync', this.getJsonAsync);
       app.get('/promise-middleware-test--renderasync', this.getRenderAsync);
       app.get('/promise-middleware-test--sendAsync', this.getSendAsync);
@@ -16,7 +18,15 @@ module.exports = function (BaseController, Promise) {
     }
 
     getRenderAsync(req, res) {
-      res.renderAsync('renderView', Promise.resolve({ microapp: 'renderAsync success' }));
+      const { headers } = req;
+      const view = headers['x-view'];
+      const viewPropsHeader = headers['x-view-props'];
+      const viewProps = viewPropsHeader !== 'undefined' ? JSON.parse(viewPropsHeader) : undefined;
+      if (viewProps) {
+        res.renderAsync(view, Promise.resolve(viewProps));
+      } else {
+        res.renderAsync(view);
+      }
     }
 
     getSendAsync(req, res) {
@@ -28,7 +38,7 @@ module.exports = function (BaseController, Promise) {
     }
 
     getFormatAsync(req, res) {
-      res.formatAsync(Promise.resolve('formatAsync success'));
+      res.formatAsync('innerHTML', Promise.resolve('formatAsync success'));
     }
   }
 

test/fixtures/controllers/views/home.ejs

@@ -0,0 +1,9 @@
+<!DOCTYPE html>
+<html lang="en">
+<head>
+    <title>Testing</title>
+</head>
+<body>
+  No render view-props provided
+</body>
+</html>

test/fixtures/controllers/views/microapp.ejs

@@ -0,0 +1,9 @@
+<!DOCTYPE html>
+<html lang="en">
+<head>
+    <title>Testing</title>
+</head>
+<body>
+  <%= microapp %>
+</body>
+</html>

test/unit/middleware/NoCacheMiddleware.spec.js

@@ -1,5 +1,4 @@
 describe('NoCacheMiddleware', function () {
-
   beforeEach(() => {
     return injector().inject((NoCacheMiddleware) => {
       this.NoCacheMiddleware = NoCacheMiddleware;
@@ -9,4 +8,21 @@ describe('NoCacheMiddleware', function () {
   it('should exist', () => {
     expect(this.NoCacheMiddleware).toBeDefined();
   });
+
+  it('should set no-cache headers', () => {
+    const response = {
+      headers: { Expires: 5184000 },
+      header: (name, value) => {
+        response.headers[name] = value;
+      },
+    };
+
+    this.NoCacheMiddleware({}, response, () => {});
+
+    expect(response.headers).toStrictEqual({
+      'Cache-Control': 'no-cache, no-store, must-revalidate',
+      Pragma: 'no-cache',
+      Expires: 0,
+    });
+  });
 });