Chore: rydder i dependencies (#1675)

* flytt @types/-pakker til dev dependencies * slett prop-types som ikke trengs i package.json * slett react-modal som ikke brukes lenger * slett @testing-library/react-hooks * legg til @types/node for å unngå klaging fra ts-node * slett ubrukte dependencies * legg til avhengighet til @testing-library/dom som kreves av @testing-library/user-event * fix test som endrer på react state - må vente på at det blir gjort før man går videre * legg på act rundt test-innhold som endrer på state * legg til act foran alle test-kall som kan endre state * legg til act foran alle test-kall som kan endre state * bytt fra waitFor til act et sted for det er brukerevent * oppdater @testing-library/react og @testing-library/jest-dom * slå sammen act for aktiviteter som kan skje i parallell
navikt · Mar 5, 2024 · 22f72d8 · 22f72d8
1 parent 07d9ee3
commit 22f72d8
Show file tree

Hide file tree

Showing 15 changed files with 1,864 additions and 1,424 deletions.
diff --git a/package.json b/package.json
@@ -41,14 +41,6 @@
     "@navikt/landvelger": "^3.1.5",
     "@sentry/browser": "^7.104.0",
     "@sentry/webpack-plugin": "^2.14.2",
-    "@types/classnames": "^2.3.1",
-    "@types/deep-equal": "^1.0.1",
-    "@types/express": "^4.17.14",
-    "@types/react": "^18.2.39",
-    "@types/react-dom": "^18.0.9",
-    "@types/react-modal": "^3.13.1",
-    "@types/react-router-dom": "^5.3.3",
-    "@types/uuid": "^8.3.4",
     "axios": "^1.6.2",
     "babel-loader": "^9.1.2",
     "classnames": "^2.3.2",
@@ -59,11 +51,9 @@
     "dotenv": "^16.4.5",
     "express": "^4.18.2",
     "http-proxy-middleware": "^2.0.6",
-    "prop-types": "^15.8.1",
     "react": "^18.2.0",
     "react-collapse": "^5.1.1",
     "react-dom": "^18.2.0",
-    "react-modal": "3.16.x",
     "react-router-dom": "^6.14.1",
     "styled-components": "^6.1.1",
     "stylis": "^4.3.0",
@@ -78,18 +68,25 @@
     "@jest/types": "^29.6.0",
     "@peculiar/webcrypto": "^1.4.3",
     "@pmmmwh/react-refresh-webpack-plugin": "^0.5.10",
-    "@testing-library/jest-dom": "^6.1.5",
-    "@testing-library/react": "^14.1.0",
-    "@testing-library/react-hooks": "^8.0.1",
+    "@testing-library/dom": "^9.3.4",
+    "@testing-library/jest-dom": "^6.4.2",
+    "@testing-library/react": "^14.2.1",
     "@testing-library/user-event": "^14.5.1",
     "@types/case-sensitive-paths-webpack-plugin": "^2.1.6",
+    "@types/classnames": "^2.3.1",
+    "@types/deep-equal": "^1.0.1",
+    "@types/express": "^4.17.14",
     "@types/jest": "^29.5.2",
+    "@types/node": "^20.11.24",
+    "@types/react": "^18.2.39",
+    "@types/react-dom": "^18.0.9",
+    "@types/react-router-dom": "^5.3.3",
+    "@types/uuid": "^8.3.4",
     "@types/webpack-env": "^1.18.1",
     "@types/webpack-hot-middleware": "^2.25.6",
     "@typescript-eslint/eslint-plugin": "^7.1.0",
     "@typescript-eslint/parser": "^7.1.0",
     "babel-jest": "^29.6.0",
-    "babel-polyfill": "^6.26.0",
     "case-sensitive-paths-webpack-plugin": "^2.4.0",
     "compression-webpack-plugin": "^11.1.0",
     "css-loader": "^6.8.1",
@@ -116,9 +113,7 @@
     "style-loader": "^3.3.3",
     "terser-webpack-plugin": "^5.3.9",
     "ts-jest": "^29.1.1",
-    "ts-loader": "^9.4.4",
     "ts-node": "^10.9.1",
-    "tsconfig-paths-webpack-plugin": "^4.0.1",
     "typescript": "^5.3.3",
     "webpack": "^5.88.1",
     "webpack-cli": "^5.1.4",

diff --git a/src/frontend/komponenter/Fagsak/Fakta/FaktaContainer.test.tsx b/src/frontend/komponenter/Fagsak/Fakta/FaktaContainer.test.tsx
@@ -1,6 +1,6 @@
 import * as React from 'react';
 
-import { render, waitFor } from '@testing-library/react';
+import { act, render, waitFor } from '@testing-library/react';
 import { userEvent } from '@testing-library/user-event';
 import { mock } from 'jest-mock-extended';
 
@@ -141,44 +141,54 @@ describe('Tester: FaktaContainer', () => {
         expect(queryAllByText('Feltet må fylles ut')).toHaveLength(0);
         expect(getAllByRole('combobox')).toHaveLength(3);
 
-        await user.click(
-            getByRole('button', {
-                name: 'Bekreft og fortsett',
-            })
+        await act(() =>
+            user.click(
+                getByRole('button', {
+                    name: 'Bekreft og fortsett',
+                })
+            )
         );
         expect(queryAllByText('Feltet må fylles ut')).toHaveLength(4);
 
-        await user.selectOptions(getByTestId('perioder.0.årsak'), HendelseType.BOSATT_I_RIKET);
-        await user.selectOptions(getByTestId('perioder.1.årsak'), HendelseType.BOR_MED_SØKER);
-        await user.selectOptions(getByTestId('perioder.2.årsak'), HendelseType.BOSATT_I_RIKET);
-        await user.type(getByRole('textbox'), 'Begrunnelse');
+        act(() => {
+            user.selectOptions(getByTestId('perioder.0.årsak'), HendelseType.BOSATT_I_RIKET);
+            user.selectOptions(getByTestId('perioder.1.årsak'), HendelseType.BOR_MED_SØKER);
+            user.selectOptions(getByTestId('perioder.2.årsak'), HendelseType.BOSATT_I_RIKET);
+        });
+        await act(() => user.type(getByRole('textbox'), 'Begrunnelse'));
 
         expect(getAllByRole('combobox')).toHaveLength(6);
 
-        await user.click(
-            getByRole('button', {
-                name: 'Bekreft og fortsett',
-            })
+        await act(() =>
+            user.click(
+                getByRole('button', {
+                    name: 'Bekreft og fortsett',
+                })
+            )
         );
         expect(queryAllByText('Feltet må fylles ut')).toHaveLength(3);
 
-        await user.selectOptions(
-            getByTestId('perioder.0.underårsak'),
-            HendelseUndertype.BRUKER_BOR_IKKE_I_NORGE
-        );
-        await user.selectOptions(
-            getByTestId('perioder.1.underårsak'),
-            HendelseUndertype.BOR_IKKE_MED_BARN
-        );
-        await user.selectOptions(
-            getByTestId('perioder.2.underårsak'),
-            HendelseUndertype.BRUKER_FLYTTET_FRA_NORGE
-        );
+        act(() => {
+            user.selectOptions(
+                getByTestId('perioder.0.underårsak'),
+                HendelseUndertype.BRUKER_BOR_IKKE_I_NORGE
+            );
+            user.selectOptions(
+                getByTestId('perioder.1.underårsak'),
+                HendelseUndertype.BOR_IKKE_MED_BARN
+            );
+            user.selectOptions(
+                getByTestId('perioder.2.underårsak'),
+                HendelseUndertype.BRUKER_FLYTTET_FRA_NORGE
+            );
+        });
 
-        await user.click(
-            getByRole('button', {
-                name: 'Bekreft og fortsett',
-            })
+        await act(() =>
+            user.click(
+                getByRole('button', {
+                    name: 'Bekreft og fortsett',
+                })
+            )
         );
         expect(queryAllByText('Feltet må fylles ut')).toHaveLength(0);
     });
@@ -209,40 +219,54 @@ describe('Tester: FaktaContainer', () => {
         expect(queryAllByText('Feltet må fylles ut')).toHaveLength(0);
         expect(getAllByRole('combobox')).toHaveLength(3);
 
-        await user.click(
-            getByRole('button', {
-                name: 'Bekreft og fortsett',
-            })
+        await act(() =>
+            user.click(
+                getByRole('button', {
+                    name: 'Bekreft og fortsett',
+                })
+            )
         );
         expect(queryAllByText('Feltet må fylles ut')).toHaveLength(4);
 
-        await user.click(
-            getByRole('checkbox', {
-                name: 'Behandle alle perioder samlet',
-            })
+        await act(() =>
+            user.click(
+                getByRole('checkbox', {
+                    name: 'Behandle alle perioder samlet',
+                })
+            )
         );
 
-        await user.selectOptions(getByTestId('perioder.0.årsak'), HendelseType.BOSATT_I_RIKET);
-        await user.type(getByLabelText('Forklar årsaken(e) til feilutbetalingen'), 'Begrunnelse');
+        await act(() =>
+            user.selectOptions(getByTestId('perioder.0.årsak'), HendelseType.BOSATT_I_RIKET)
+        );
+        await act(() =>
+            user.type(getByLabelText('Forklar årsaken(e) til feilutbetalingen'), 'Begrunnelse')
+        );
 
         expect(getAllByRole('combobox')).toHaveLength(6);
 
-        await user.click(
-            getByRole('button', {
-                name: 'Bekreft og fortsett',
-            })
+        await act(() =>
+            user.click(
+                getByRole('button', {
+                    name: 'Bekreft og fortsett',
+                })
+            )
         );
         expect(queryAllByText('Feltet må fylles ut')).toHaveLength(3);
 
-        await user.selectOptions(
-            getByTestId('perioder.0.underårsak'),
-            HendelseUndertype.BRUKER_BOR_IKKE_I_NORGE
+        await act(() =>
+            user.selectOptions(
+                getByTestId('perioder.0.underårsak'),
+                HendelseUndertype.BRUKER_BOR_IKKE_I_NORGE
+            )
         );
 
-        await user.click(
-            getByRole('button', {
-                name: 'Bekreft og fortsett',
-            })
+        await act(() =>
+            user.click(
+                getByRole('button', {
+                    name: 'Bekreft og fortsett',
+                })
+            )
         );
         expect(queryAllByText('Feltet må fylles ut')).toHaveLength(0);
     });