Merge pull request #882 from ral-facilities/dependabot/npm_and_yarn/r…

…eact-scripts-5.0.0

Bump react-scripts from 4.0.3 to 5.0.0

package.json

@@ -2,7 +2,6 @@
   "name": "scigateway",
   "version": "0.1.0",
   "private": true,
-  "proxy": "http://localhost:8000/",
   "dependencies": {
     "@material-ui/core": "^4.11.3",
     "@material-ui/icons": "^4.11.2",
@@ -29,14 +28,13 @@
     "react-redux-toastr": "^7.6.5",
     "react-router": "^5.0.1",
     "react-router-dom": "^5.2.0",
-    "react-scripts": "4.0.3",
+    "react-scripts": "5.0.0",
     "redux": "^4.1.2",
     "redux-logger": "^3.0.6",
     "redux-thunk": "^2.3.0",
     "single-spa": "^5.9.1",
     "typeface-roboto": "1.1.13",
-    "typescript": "4.2.2",
-    "webpack": "4.44.2"
+    "typescript": "4.2.2"
   },
   "scripts": {
     "lint:js": "eslint --max-warnings=0 --ext=tsx --ext=ts --ext=js --ext=jsx --fix ./src",
@@ -88,7 +86,6 @@
     "not op_mini all"
   ],
   "devDependencies": {
-    "@babel/core": "^7.13.1",
     "@types/classnames": "^2.2.11",
     "@types/enzyme": "^3.10.8",
     "@types/history": "^4.7.2",
@@ -105,28 +102,21 @@
     "@types/redux-logger": "^3.0.8",
     "@types/redux-mock-store": "^1.0.2",
     "@types/single-spa-react": "^3.0.1",
-    "@typescript-eslint/eslint-plugin": "^4.17.0",
-    "@typescript-eslint/parser": "^4.17.0",
+    "@typescript-eslint/eslint-plugin": "^5.5.0",
+    "@typescript-eslint/parser": "^5.5.0",
     "axios-mock-adapter": "^1.19.0",
     "babel-eslint": "10.1.0",
-    "babel-loader": "8.1.0",
-    "body-parser": "^1.19.0",
     "concurrently": "^6.4.0",
     "cross-env": "^7.0.3",
     "cypress": "^6.6.0",
     "enzyme": "^3.11.0",
     "enzyme-adapter-react-16": "^1.15.6",
     "enzyme-to-json": "^3.6.2",
-    "eslint": "^7.2.0",
+    "eslint": "^8.0.0",
     "eslint-config-prettier": "^8.3.0",
-    "eslint-config-react-app": "^6.0.0",
+    "eslint-config-react-app": "^7.0.0",
     "eslint-plugin-cypress": "^2.11.2",
-    "eslint-plugin-flowtype": "^5.10.0",
-    "eslint-plugin-import": "^2.22.0",
-    "eslint-plugin-jsx-a11y": "^6.5.1",
     "eslint-plugin-prettier": "^3.4.1",
-    "eslint-plugin-react": "^7.28.0",
-    "eslint-plugin-react-hooks": "^4.0.0",
     "husky": "^7.0.4",
     "lint-staged": "^12.1.2",
     "prettier": "^2.2.1",
@@ -135,4 +125,4 @@
     "start-server-and-test": "^1.12.0",
     "wait-on": "^5.2.1"
   }
-}
+}

src/App.test.tsx

@@ -19,6 +19,7 @@ describe('App', () => {
 
   afterEach(() => {
     mount.cleanUp();
+    jest.useRealTimers();
   });
 
   it('renders without crashing', () => {
@@ -28,7 +29,9 @@ describe('App', () => {
   });
 
   it('loadMaintenanceState dispatched when maintenance changes', async () => {
-    jest.useFakeTimers();
+    // this test only works with old jest fake timers
+    // when they remove legacy timers refactor this test to use real timers
+    jest.useFakeTimers('legacy');
     (axios.get as jest.Mock).mockImplementation(() =>
       Promise.resolve({
         data: {

src/App.tsx

@@ -39,8 +39,7 @@ const middleware = [
   autoLoginMiddleware,
 ];
 if (process.env.NODE_ENV === `development`) {
-  /* eslint-disable-next-line @typescript-eslint/no-explicit-any */
-  const logger = (createLogger as any)({ collapsed: true });
+  const logger = createLogger({ collapsed: true });
   middleware.push(logger);
   // const {whyDidYouUpdate} = require('why-did-you-update');
   // whyDidYouUpdate(React);