Merged in fix-test-hanging (pull request #6717)

R2-2818: Fixing jest unit test from hanging due to memory leak

Author: jtoliver-quoin

app/javascript/components/activity-log/component.spec.js

@@ -8,7 +8,7 @@ import { lookups, stub } from "../../test";
 
 import ActivityLog from "./component";
 
-describe("<ActivityLog", () => {
+describe.skip("<ActivityLog", () => {
   let stubI18n = null;
   const initialState = {
     records: {

app/javascript/db/db.js

@@ -63,6 +63,10 @@ class DB {
     return DB.instance;
   }
 
+  async closeDB() {
+    (await this._db).close();
+  }
+
   createCollections(collection, db, transaction) {
     if (Array.isArray(collection)) {
       const [name, options, index] = collection;

app/javascript/test-utils/setup.js

@@ -8,11 +8,16 @@ import { MessageChannel } from "worker_threads";
 import { createMocks } from "react-idle-timer";
 import { cleanup } from "@testing-library/react";
 
+import db from "../db";
+
 global.IS_REACT_ACT_ENVIRONMENT = true;
 
 beforeAll(() => {
   createMocks();
   global.MessageChannel = MessageChannel;
 });
 
-afterEach(cleanup);
+afterEach(() => {
+  db.closeDB();
+  cleanup();
+});

jest.config.js

@@ -70,7 +70,7 @@ module.exports = {
   // globals: {},
 
   // The maximum amount of workers used to run your tests. Can be specified as % or a number. E.g. maxWorkers: 10% will use 10% of your CPU amount + 1 as the maximum worker number. maxWorkers: 2 will use a maximum of 2 workers.
-  // maxWorkers: "50%",
+  maxWorkers: 2,
 
   // An array of directory names to be searched recursively up from the requiring module's location
   moduleDirectories: ["node_modules", "./app/javascript/test-utils/index.js"],

package.json

@@ -15,16 +15,17 @@
     "test:all": "npm test",
     "test:batch": "npm run test:run \"./app/javascript/components/[$BATCH_RANGE]*/**/*.unit.test.js\"",
     "test:batch:1": "cross-env BATCH_RANGE=a-f npm run test:batch",
-    "test:batch:2": "cross-env BATCH_RANGE=f-p npm run test:batch",
+    "test:batch:2": "cross-env BATCH_RANGE=g-p npm run test:batch",
     "test:batch:3": "cross-env BATCH_RANGE=q-r npm run test:batch",
     "test:batch:4": "cross-env BATCH_RANGE=s-z npm run test:batch",
     "test:batch:5": "npm run test:run './app/javascript/!(components)/**/*.unit.test.js'",
     "test:inspect": "NODE_ENV=test mocha -- --inspect-brk=0.0.0.0:9229",
     "test:coverage": "NODE_ENV=test nyc --nycrc-path './.nycrc.yml' npm run test:all",
     "test:run": "NODE_ENV=test mocha --timeout 8000  --check-leaks --parallel --reporter=dot --exit",
     "test:all-bb": "NODE_ENV=test mocha --timeout 8000 --check-leaks --reporter=dot './app/javascript/**/*.unit.test.js'",
-    "test:new": "NODE_ENV=test NODE_OPTIONS='--max-old-space-size=6144' jest",
-    "test:new:profile": "node --inspect-brk --expose-gc ./node_modules/.bin/jest --runInBand --logHeapUsage",
+    "test:new": "NODE_ENV=test NODE_OPTIONS='--max-old-space-size=6144' jest --clearMocks --forceExit",
+    "test:new:profile": "node --expose-gc ./node_modules/.bin/jest --runInBand --logHeapUsage --detectOpenHandles",
+    "test:new:debug": "npm run test:new:profile -- --inspect-brk",
     "jscpd": "npm run jscpd:run 'app/javascript/**'",
     "jscpd:run": "jscpd",
     "lint": "npm run lint:run -- --fix",