Tar bort fakta-felles som bare ble brukt ca et sted (#6870)

* Tar bort fakta-felles som bare ble brukt ca et sted * Fikser test
navikt · Nov 27, 2024 · 5344d1f · 5344d1f
1 parent 471daa7
commit 5344d1f
Show file tree

Hide file tree

Showing 16 changed files with 32 additions and 142 deletions.
diff --git a/packages/fakta-arbeidsforhold/src/ArbeidsforholdFaktaIndex.tsx b/packages/fakta-arbeidsforhold/src/ArbeidsforholdFaktaIndex.tsx
@@ -1,10 +1,17 @@
-import React from 'react';
-import { createIntl, createIntlCache, RawIntlProvider } from 'react-intl';
+import { Aksjonspunkt, ArbeidsgiverOpplysningerPerId, Behandling, KodeverkMedNavn } from '@k9-sak-web/types';
 import ArbeidsforholdV2 from '@k9-sak-web/types/src/arbeidsforholdV2TsType';
-import { ArbeidsgiverOpplysningerPerId, Behandling, KodeverkMedNavn } from '@k9-sak-web/types';
-import StandardFaktaProps from '@k9-sak-web/fakta-felles/standardFaktaPropsTsType';
-import ArbeidsforholdInfoPanel from './components/ArbeidsforholdInfoPanel';
+import { createIntl, createIntlCache, RawIntlProvider } from 'react-intl';
 import messages from '../i18n/nb_NO.json';
+import ArbeidsforholdInfoPanel from './components/ArbeidsforholdInfoPanel';
+
+type StandardFaktaProps = Readonly<{
+  aksjonspunkter: Aksjonspunkt[];
+  readOnly: boolean;
+  submittable: boolean;
+  harApneAksjonspunkter: boolean;
+  alleMerknaderFraBeslutter: { [key: string]: { notAccepted?: boolean } };
+  submitCallback?: (aksjonspunktData: any) => Promise<any>;
+}>;
 
 const cache = createIntlCache();
 

diff --git a/packages/fakta-felles/i18n/nb_NO.json b/packages/fakta-felles/i18n/nb_NO.json
diff --git a/packages/fakta-felles/index.ts b/packages/fakta-felles/index.ts
diff --git a/packages/fakta-felles/package.json b/packages/fakta-felles/package.json
diff --git a/packages/fakta-felles/src/standardFaktaPropsTsType.ts b/packages/fakta-felles/src/standardFaktaPropsTsType.ts
diff --git a/packages/fakta-felles/standardFaktaPropsTsType.ts b/packages/fakta-felles/standardFaktaPropsTsType.ts
diff --git a/packages/fakta-medlemskap/package.json b/packages/fakta-medlemskap/package.json
@@ -11,7 +11,6 @@
     "@fpsak-frontend/prop-types": "1.0.0",
     "@fpsak-frontend/shared-components": "1.0.0",
     "@fpsak-frontend/utils": "1.0.0",
-    "@k9-sak-web/fakta-felles": "1.0.0",
     "@navikt/ft-form-hooks": "5.2.7",
     "@navikt/ft-plattform-komponenter": "5.1.3",
     "i18n-iso-countries": "7.13.0",

diff --git a/packages/fakta-verge/package.json b/packages/fakta-verge/package.json
@@ -10,7 +10,6 @@
     "@fpsak-frontend/prop-types": "1.0.0",
     "@fpsak-frontend/shared-components": "1.0.0",
     "@fpsak-frontend/utils": "1.0.0",
-    "@k9-sak-web/fakta-felles": "1.0.0",
     "prop-types": "15.8.1",
     "react": "18.3.1",
     "react-intl": "7.0.1",

diff --git a/...onents/FaktaBegrunnelseTextField.spec.tsx → ...onents/FaktaBegrunnelseTextField.spec.tsx b/...onents/FaktaBegrunnelseTextField.spec.tsx → ...onents/FaktaBegrunnelseTextField.spec.tsx
@@ -1,23 +1,18 @@
 import { renderWithIntlAndReduxForm } from '@fpsak-frontend/utils-test/test-utils';
 import { screen } from '@testing-library/react';
-import React from 'react';
-import messages from '../../i18n/nb_NO.json';
 import FaktaBegrunnelseTextField from './FaktaBegrunnelseTextField';
 
 describe('<FaktaBegrunnelseTextField>', () => {
   it('skal ikke vise tekstfelt når en ikke har lov til å løse aksjonspunkt', () => {
     renderWithIntlAndReduxForm(
       <FaktaBegrunnelseTextField isReadOnly={false} isSubmittable={false} hasBegrunnelse={false} />,
-      { messages },
     );
 
     expect(screen.queryByRole('textbox')).not.toBeInTheDocument();
   });
 
   it('skal vise tekstfelt når en har lov til å løse aksjonspunkt og en har gjort endringer', () => {
-    renderWithIntlAndReduxForm(<FaktaBegrunnelseTextField isReadOnly={false} isSubmittable hasBegrunnelse={false} />, {
-      messages,
-    });
+    renderWithIntlAndReduxForm(<FaktaBegrunnelseTextField isReadOnly={false} isSubmittable hasBegrunnelse={false} />);
 
     expect(screen.getByRole('textbox', { name: 'Begrunn endringene' })).toBeInTheDocument();
   });
@@ -29,17 +24,14 @@ describe('<FaktaBegrunnelseTextField>', () => {
   });
 
   it('skal vise standard-label når en ikke har valgt å vise vurderingstekst eller sende med tekstkode', () => {
-    renderWithIntlAndReduxForm(<FaktaBegrunnelseTextField isReadOnly={false} isSubmittable hasBegrunnelse={false} />, {
-      messages,
-    });
+    renderWithIntlAndReduxForm(<FaktaBegrunnelseTextField isReadOnly={false} isSubmittable hasBegrunnelse={false} />);
 
     expect(screen.getByRole('textbox', { name: 'Begrunn endringene' })).toBeInTheDocument();
   });
 
   it('skal vise label for vurdering når dette er markert av prop', () => {
     renderWithIntlAndReduxForm(
       <FaktaBegrunnelseTextField isReadOnly={false} isSubmittable hasBegrunnelse={false} hasVurderingText />,
-      { messages },
     );
 
     expect(screen.getByRole('textbox', { name: 'Vurdering' })).toBeInTheDocument();
@@ -48,7 +40,6 @@ describe('<FaktaBegrunnelseTextField>', () => {
   it('skal vise medsendt label', () => {
     renderWithIntlAndReduxForm(
       <FaktaBegrunnelseTextField isReadOnly={false} isSubmittable hasBegrunnelse={false} label="Test" />,
-      { messages },
     );
 
     expect(screen.getByRole('textbox', { name: 'Test' })).toBeInTheDocument();

diff --git a/.../components/FaktaBegrunnelseTextField.tsx → .../components/FaktaBegrunnelseTextField.tsx b/.../components/FaktaBegrunnelseTextField.tsx → .../components/FaktaBegrunnelseTextField.tsx
@@ -1,22 +1,9 @@
 import { TextAreaField } from '@fpsak-frontend/form';
 import { decodeHtmlEntity, hasValidText, maxLength, minLength, required } from '@fpsak-frontend/utils';
-import React from 'react';
-import { RawIntlProvider, createIntl, createIntlCache } from 'react-intl';
 
 import { Aksjonspunkt } from '@k9-sak-web/types';
-import messages from '../../i18n/nb_NO.json';
 import styles from './faktaBegrunnelseTextField.module.css';
 
-const cache = createIntlCache();
-
-const intl = createIntl(
-  {
-    locale: 'nb-NO',
-    messages,
-  },
-  cache,
-);
-
 const minLength3 = minLength(3);
 const maxLength1500 = maxLength(1500);
 
@@ -48,10 +35,10 @@ const FaktaBegrunnelseTextField = ({
   hasVurderingText = false,
   name = 'begrunnelse',
 }: OwnProps) => {
-  const code = hasVurderingText ? 'FaktaBegrunnelseTextField.Vurdering' : 'FaktaBegrunnelseTextField.BegrunnEndringene';
-  const textAreaLabel = label || { id: code };
+  const defaultLabel = hasVurderingText ? 'Vurdering' : 'Begrunn endringene';
+  const textAreaLabel = label || defaultLabel;
   return (
-    <RawIntlProvider value={intl}>
+    <>
       {(isSubmittable || hasBegrunnelse) && (
         <div className={styles.begrunnelseTextField}>
           <TextAreaField
@@ -63,7 +50,7 @@ const FaktaBegrunnelseTextField = ({
           />
         </div>
       )}
-    </RawIntlProvider>
+    </>
   );
 };
 

diff --git a/...src/components/FaktaSubmitButton.spec.tsx → ...src/components/FaktaSubmitButton.spec.tsx b/...src/components/FaktaSubmitButton.spec.tsx → ...src/components/FaktaSubmitButton.spec.tsx
@@ -1,12 +1,9 @@
-import { renderWithIntlAndReduxForm } from '@fpsak-frontend/utils-test/test-utils';
-import { screen } from '@testing-library/react';
-import React from 'react';
-import messages from '../../i18n/nb_NO.json';
+import { render, screen } from '@testing-library/react';
 import { FaktaSubmitButton } from './FaktaSubmitButton';
 
 describe('<FaktaSubmitButton>', () => {
   it('skal ikke vise knapp når readonly', () => {
-    renderWithIntlAndReduxForm(
+    render(
       <FaktaSubmitButton
         isReadOnly
         isSubmittable
@@ -17,14 +14,13 @@ describe('<FaktaSubmitButton>', () => {
         behandlingId={1}
         behandlingVersjon={2}
       />,
-      { messages },
     );
 
     expect(screen.queryByRole('button')).not.toBeInTheDocument();
   });
 
   it('skal vise knapp som trykkbar når en kan avklare aksjonspunkt og en har gjort endringer', () => {
-    renderWithIntlAndReduxForm(
+    render(
       <FaktaSubmitButton
         isReadOnly={false}
         isSubmittable
@@ -35,14 +31,13 @@ describe('<FaktaSubmitButton>', () => {
         behandlingId={1}
         behandlingVersjon={2}
       />,
-      { messages },
     );
 
     expect(screen.getByRole('button', { name: 'Bekreft og fortsett' })).not.toBeDisabled();
   });
 
   it('skal vise knapp som utgrået når en ikke kan avklare aksjonspunkt', () => {
-    renderWithIntlAndReduxForm(
+    render(
       <FaktaSubmitButton
         isReadOnly={false}
         isSubmittable={false}
@@ -53,14 +48,13 @@ describe('<FaktaSubmitButton>', () => {
         behandlingId={1}
         behandlingVersjon={2}
       />,
-      { messages },
     );
 
     expect(screen.getByRole('button', { name: 'Bekreft og fortsett' })).toBeDisabled();
   });
 
   it('skal vise knapp som utgrået når en har trykket på knapp', () => {
-    renderWithIntlAndReduxForm(
+    render(
       <FaktaSubmitButton
         isReadOnly={false}
         isSubmittable
@@ -71,14 +65,13 @@ describe('<FaktaSubmitButton>', () => {
         behandlingId={1}
         behandlingVersjon={2}
       />,
-      { messages },
     );
 
     expect(screen.getByRole('button', { name: 'Bekreft og fortsett' })).toBeDisabled();
   });
 
   it('skal vise knapp som utgrået når en ikke har gjort endringer og det er tomme obligatoriske felter', () => {
-    renderWithIntlAndReduxForm(
+    render(
       <FaktaSubmitButton
         isReadOnly={false}
         isSubmittable
@@ -89,14 +82,13 @@ describe('<FaktaSubmitButton>', () => {
         behandlingId={1}
         behandlingVersjon={2}
       />,
-      { messages },
     );
 
     expect(screen.getByRole('button', { name: 'Bekreft og fortsett' })).toBeDisabled();
   });
 
   it('skal vise knapp som trykkbar når en ikke har gjort endringer men alle obligatoriske felter er utfylte', () => {
-    renderWithIntlAndReduxForm(
+    render(
       <FaktaSubmitButton
         isReadOnly={false}
         isSubmittable
@@ -107,14 +99,13 @@ describe('<FaktaSubmitButton>', () => {
         behandlingId={1}
         behandlingVersjon={2}
       />,
-      { messages },
     );
 
     expect(screen.getByRole('button', { name: 'Bekreft og fortsett' })).not.toBeDisabled();
   });
 
   it('skal vise knapp som utgrået når en ikke har gjort endringer og aksjonspunktet er løst tidligere', () => {
-    renderWithIntlAndReduxForm(
+    render(
       <FaktaSubmitButton
         isReadOnly={false}
         isSubmittable
@@ -125,7 +116,6 @@ describe('<FaktaSubmitButton>', () => {
         behandlingId={1}
         behandlingVersjon={2}
       />,
-      { messages },
     );
 
     expect(screen.getByRole('button', { name: 'Bekreft og fortsett' })).toBeDisabled();

diff --git a/...lles/src/components/FaktaSubmitButton.tsx → ...erge/src/components/FaktaSubmitButton.tsx b/...lles/src/components/FaktaSubmitButton.tsx → ...erge/src/components/FaktaSubmitButton.tsx
@@ -1,22 +1,10 @@
 import React from 'react';
-import { FormattedMessage, RawIntlProvider, createIntl, createIntlCache } from 'react-intl';
 import { connect } from 'react-redux';
 
 import { hasBehandlingFormErrorsOfType, isBehandlingFormDirty, isBehandlingFormSubmitting } from '@fpsak-frontend/form';
 import { ariaCheck, isRequiredMessage } from '@fpsak-frontend/utils';
 
 import { Button } from '@navikt/ds-react';
-import messages from '../../i18n/nb_NO.json';
-
-const cache = createIntlCache();
-
-const intl = createIntl(
-  {
-    locale: 'nb-NO',
-    messages,
-  },
-  cache,
-);
 
 const isDisabled = (
   isDirty: boolean,
@@ -69,7 +57,7 @@ export const FaktaSubmitButton = ({
   onClick,
   dataId,
 }: Partial<PureOwnProps> & MappedOwnProps) => (
-  <RawIntlProvider value={intl}>
+  <div>
     {!isReadOnly && (
       <Button
         variant="primary"
@@ -81,10 +69,10 @@ export const FaktaSubmitButton = ({
         data-id={dataId}
       >
         {!!buttonText && buttonText}
-        {!buttonText && <FormattedMessage id="SubmitButton.ConfirmInformation" />}
+        {!buttonText && 'Bekreft og fortsett'}
       </Button>
     )}
-  </RawIntlProvider>
+  </div>
 );
 
 const mapStateToProps = (state: any, ownProps: PureOwnProps): MappedOwnProps => {

diff --git a/packages/fakta-verge/src/components/RegistrereVergeInfoPanel.jsx b/packages/fakta-verge/src/components/RegistrereVergeInfoPanel.jsx
@@ -1,16 +1,16 @@
 import { behandlingForm } from '@fpsak-frontend/form';
 import aksjonspunktCodes from '@fpsak-frontend/kodeverk/src/aksjonspunktCodes';
 import { AksjonspunktHelpText, VerticalSpacer } from '@fpsak-frontend/shared-components';
-import { FaktaBegrunnelseTextField, FaktaSubmitButton } from '@k9-sak-web/fakta-felles';
 import { BodyShort } from '@navikt/ds-react';
 import PropTypes from 'prop-types';
-import React from 'react';
 import { injectIntl } from 'react-intl';
 import { connect } from 'react-redux';
 import { createSelector } from 'reselect';
 
 import { decodeHtmlEntity } from '@fpsak-frontend/utils';
 import vergeAksjonspunkterPropType from '../propTypes/vergeAksjonspunkterPropType';
+import FaktaBegrunnelseTextField from './FaktaBegrunnelseTextField';
+import FaktaSubmitButton from './FaktaSubmitButton';
 
 /**
  * RegistrereVergeInfoPanel

diff --git a/...ents/faktaBegrunnelseTextField.module.css → ...ents/faktaBegrunnelseTextField.module.css b/...ents/faktaBegrunnelseTextField.module.css → ...ents/faktaBegrunnelseTextField.module.css
diff --git a/public/mockServiceWorker.js b/public/mockServiceWorker.js
@@ -128,7 +128,7 @@ async function handleRequest(event, requestId) {
   // Ensure MSW is active and ready to handle the message, otherwise
   // this message will pend indefinitely.
   if (client && activeClientIds.has(client.id)) {
-    ; (async function () {
+    ;(async function () {
       const responseClone = response.clone()
 
       sendToClient(