upgrade JSDOM to 15.2.1

- bump RAM allowance for prearchive scripts in Jenkins
department-of-veterans-affairs · May 19, 2020 · c72bb11 · c72bb11
1 parent eccb647
commit c72bb11
Show file tree

Hide file tree

Showing 7 changed files with 207 additions and 113 deletions.
diff --git a/jenkins/common.groovy b/jenkins/common.groovy
@@ -253,7 +253,7 @@ def buildAll(String ref, dockerContainer, Boolean contentOnlyBuild) {
 
 def prearchive(dockerContainer, envName) {
   dockerContainer.inside(DOCKER_ARGS) {
-    sh "cd /application && node script/prearchive.js --buildtype=${envName}"
+    sh "cd /application && node --max-old-space-size=4096 script/prearchive.js --buildtype=${envName}"
   }
 }
 

diff --git a/package.json b/package.json
@@ -181,7 +181,7 @@
     "isomorphic-fetch": "^2.2.1",
     "jest": "^23.6.0",
     "jest-image-snapshot": "^2.7.0",
-    "jsdom": "^11.1.0",
+    "jsdom": "^15.2.1",
     "json2csv": "^4.2.1",
     "jsonschema": "^1.1.1",
     "libxmljs": "^0.19.5",

diff --git a/src/applications/terms-and-conditions/tests/containers/MhvTermsAndConditions.unit.spec.jsx b/src/applications/terms-and-conditions/tests/containers/MhvTermsAndConditions.unit.spec.jsx
@@ -7,6 +7,8 @@ import sinon from 'sinon';
 import { MhvTermsAndConditions } from '../../containers/MhvTermsAndConditions';
 
 describe('<MhvTermsAndConditions>', () => {
+  let oldLocation;
+
   const props = {
     location: {
       pathname: '/health-care/medical-information-terms-conditions',
@@ -30,14 +32,20 @@ describe('<MhvTermsAndConditions>', () => {
   };
 
   const setup = () => {
-    global.window.location.replace = sinon.spy();
+    oldLocation = global.window.location;
+    delete global.window.location;
+    global.window.location = { replace: sinon.spy() };
     props.acceptTerms.reset();
     props.fetchLatestTerms.reset();
     props.fetchTermsAcceptance.reset();
   };
 
   beforeEach(setup);
 
+  afterEach(() => {
+    global.window.location = oldLocation;
+  });
+
   it('should show an error when there are errors', () => {
     const newProps = set('errors', { code: 404 }, props);
     const wrapper = shallow(<MhvTermsAndConditions {...newProps} />);

diff --git a/src/applications/vaos/tests/containers/VAOSApp.unit.spec.jsx b/src/applications/vaos/tests/containers/VAOSApp.unit.spec.jsx
@@ -136,14 +136,15 @@ describe('VAOS <VAOSApp>', () => {
       },
     };
 
-    const oldReplace = window.location.replace;
-    window.location.replace = sinon.spy();
+    const oldLocation = window.location;
+    delete window.location;
+    window.location = { replace: sinon.spy() };
     const tree = mount(<VAOSApp showApplication user={user} />);
 
     expect(window.location.replace.firstCall.args[0]).to.contain('verify');
     expect(tree.find('RequiredLoginView').exists()).to.be.true;
 
-    window.location.replace = oldReplace;
+    window.location.replace = oldLocation;
     tree.unmount();
   });
 });
diff --git a/src/platform/testing/unit/mocha-setup.js b/src/platform/testing/unit/mocha-setup.js
@@ -63,7 +63,18 @@ export default function setupJSDom() {
 
   win.dataLayer = [];
   win.scrollTo = () => {};
-  win.sessionStorage = {};
+  Object.defineProperty(win, 'sessionStorage', {
+    value: global.sessionStorage,
+    configurable: true,
+    enumerable: true,
+    writable: true,
+  });
+  Object.defineProperty(win, 'localStorage', {
+    value: global.localStorage,
+    configurable: true,
+    enumerable: true,
+    writable: true,
+  });
   win.requestAnimationFrame = func => func();
   win.matchMedia = () => ({
     matches: false,

diff --git a/src/platform/user/tests/authorization/containers/MHVApp.unit.spec.jsx b/src/platform/user/tests/authorization/containers/MHVApp.unit.spec.jsx
@@ -8,6 +8,8 @@ import backendServices from '../../../../user/profile/constants/backendServices'
 import { MHVApp } from '../../../authorization/containers/MHVApp';
 
 describe('<MHVApp>', () => {
+  let oldLocation;
+
   const props = {
     location: { pathname: '/health-care/prescriptions', query: {} },
     mhvAccount: {
@@ -28,7 +30,11 @@ describe('<MHVApp>', () => {
   };
 
   const setup = () => {
-    global.window.location.replace = sinon.spy();
+    oldLocation = global.window.location;
+    delete global.window.location;
+    global.window.location = {
+      replace: sinon.spy(),
+    };
     props.createMHVAccount.reset();
     props.fetchMHVAccount.reset();
     props.upgradeMHVAccount.reset();
@@ -52,6 +58,10 @@ describe('<MHVApp>', () => {
 
   beforeEach(setup);
 
+  afterEach(() => {
+    global.window.location = oldLocation;
+  });
+
   it('should show a loading indicator when fetching an account', () => {
     const newProps = set('mhvAccount.loading', true, props);
     const wrapper = shallow(<MHVApp {...newProps} />, { context });