add test and fix issue with partially filled translations

Author: magnurh-cx

packages/bygger/src/translations/utils/editFormTranslationsUtils.test.ts

@@ -4,68 +4,68 @@ import {
   FormsApiGlobalTranslation,
   MockedComponentObjectForTest,
 } from '@navikt/skjemadigitalisering-shared-domain';
-import { generateAndPopulateTranslationsForForm } from './editFormTranslationsUtils';
+import { generateAndPopulateTranslationsForForm, generateUnsavedGlobalTranslations } from './editFormTranslationsUtils';
 
 const { createFormsApiFormObject, createPanelObject, createDummyTextfield, createDummySelectComponent } =
   MockedComponentObjectForTest;
 
 describe('editFormTranslationsUtils', () => {
-  describe('generateAndPopulateTranslationsForForm', () => {
-    let form: Form;
+  let form: Form;
 
-    beforeEach(() => {
-      form = createFormsApiFormObject(
-        [
-          createPanelObject('Side 1', [
-            createDummyTextfield('keyBasedTranslation'),
-            createDummyTextfield('Tekst uten oversettelse'),
-            createDummyTextfield('Tekst med oversettelse'),
-            createDummyTextfield('Tekst med globalt overskrevet oversettelse'),
-            createDummyTextfield('Tekst uten oversettelse, med global oversettelse'),
-            createDummySelectComponent('Hjemmelaget landvelger', [{ value: 'Norge', label: 'Norge' }]),
-          ]),
-        ],
-        'Form title',
-      );
-    });
+  beforeEach(() => {
+    form = createFormsApiFormObject(
+      [
+        createPanelObject('Side 1', [
+          createDummyTextfield('keyBasedTranslation'),
+          createDummyTextfield('Tekst uten oversettelse'),
+          createDummyTextfield('Tekst med oversettelse'),
+          createDummyTextfield('Tekst med globalt overskrevet oversettelse'),
+          createDummyTextfield('Tekst uten oversettelse, med global oversettelse'),
+          createDummySelectComponent('Hjemmelaget landvelger', [{ value: 'Norge', label: 'Norge' }]),
+        ]),
+      ],
+      'Form title',
+    );
+  });
 
-    const storedTranslations: Record<string, FormsApiFormTranslation> = {
-      keyBasedTranslation: {
-        key: 'keyBasedTranslation',
-        nb: 'Nøkkelbasert oversettelse',
-        en: 'Key based translation',
-      },
-      'Tekst med oversettelse': {
-        key: 'Tekst med oversettelse',
-        nb: 'Tekst med oversettelse',
-        en: 'Text with translation',
-      },
-      'Tekst med globalt overskrevet oversettelse': {
-        key: 'Tekst med globalt overskrevet oversettelse',
-        nb: 'Tekst med globalt overskrevet oversettelse',
-        globalTranslationId: 1,
-      },
-    };
+  const storedTranslations: Record<string, FormsApiFormTranslation> = {
+    keyBasedTranslation: {
+      key: 'keyBasedTranslation',
+      nb: 'Nøkkelbasert oversettelse',
+      en: 'Key based translation',
+    },
+    'Tekst med oversettelse': {
+      key: 'Tekst med oversettelse',
+      nb: 'Tekst med oversettelse',
+      en: 'Text with translation',
+    },
+    'Tekst med globalt overskrevet oversettelse': {
+      key: 'Tekst med globalt overskrevet oversettelse',
+      nb: 'Tekst med globalt overskrevet oversettelse',
+      globalTranslationId: 1,
+    },
+  };
 
-    const globalTranslations: Record<string, FormsApiGlobalTranslation> = {
-      'Tekst med globalt overskrevet oversettelse': {
-        id: 1,
-        key: 'Tekst med globalt overskrevet oversettelse',
-        nb: 'Tekst med globalt overskrevet oversettelse',
-        nn: 'Global Tekst NN',
-        en: 'Global Text EN',
-        tag: 'statiske-tekster',
-      },
-      'Tekst uten oversettelse, med global oversettelse': {
-        id: 2,
-        key: 'Tekst uten oversettelse, med global oversettelse',
-        nb: 'Tekst uten oversettelse, med global oversettelse',
-        nn: 'Tekst uten oversettelse, med global oversettelse NN',
-        en: 'Key based global translation EN',
-        tag: 'statiske-tekster',
-      },
-    };
+  const globalTranslations: Record<string, FormsApiGlobalTranslation> = {
+    'Tekst med globalt overskrevet oversettelse': {
+      id: 1,
+      key: 'Tekst med globalt overskrevet oversettelse',
+      nb: 'Tekst med globalt overskrevet oversettelse',
+      nn: 'Global Tekst NN',
+      en: 'Global Text EN',
+      tag: 'statiske-tekster',
+    },
+    'Tekst uten oversettelse, med global oversettelse': {
+      id: 2,
+      key: 'Tekst uten oversettelse, med global oversettelse',
+      nb: 'Tekst uten oversettelse, med global oversettelse',
+      nn: 'Tekst uten oversettelse, med global oversettelse NN',
+      en: 'Key based global translation EN',
+      tag: 'statiske-tekster',
+    },
+  };
 
+  describe('generateAndPopulateTranslationsForForm', () => {
     it('should populate translations from stored translations', () => {
       const result = generateAndPopulateTranslationsForForm(form, storedTranslations, globalTranslations);
       expect(result).toEqual([
@@ -112,4 +112,18 @@ describe('editFormTranslationsUtils', () => {
       ]);
     });
   });
+
+  describe('generateUnsavedGlobalTranslations', () => {
+    it('should return global translations for texts without stored translations', () => {
+      expect(generateUnsavedGlobalTranslations(form, storedTranslations, globalTranslations)).toEqual([
+        {
+          en: 'Key based global translation EN',
+          globalTranslationId: 2,
+          key: 'Tekst uten oversettelse, med global oversettelse',
+          nb: 'Tekst uten oversettelse, med global oversettelse',
+          nn: 'Tekst uten oversettelse, med global oversettelse NN',
+        },
+      ]);
+    });
+  });
 });

packages/bygger/src/translations/utils/editFormTranslationsUtils.ts

@@ -44,7 +44,7 @@ const generateUnsavedGlobalTranslations = (
   globalTranslations: Record<string, FormsApiGlobalTranslation>,
 ) => {
   return generateAndPopulateTranslationsForForm(form, {}, globalTranslations).filter(
-    (translation) => !storedTranslations[translation.nb ?? translation.key],
+    (translation) => translation.globalTranslationId && !storedTranslations[translation.nb ?? translation.key],
   );
 };