fix: update exclusions function and tests to use locale vs localeKey

newrelic · Dec 15, 2021 · 32191bf · 32191bf
1 parent 7eb2b21
commit 32191bf
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 15 deletions.
diff --git a/scripts/actions/__tests__/add-files-to-translation-queue.test.js b/scripts/actions/__tests__/add-files-to-translation-queue.test.js
@@ -40,8 +40,8 @@ describe('add-files-to-translation-queue tests', () => {
 
   const setup = () => {
     const EXCLUSIONS = {
-      excludePath: { jp: ['excluded/path'], kr: ['excluded/path'] },
-      excludeType: { jp: ['excludedType'], kr: ['excludedType'] },
+      excludePath: { 'ja-JP': ['excluded/path'], 'ko-KR': ['excluded/path'] },
+      excludeType: { 'ja-JP': ['excludedType'], 'ko-KR': ['excludedType'] },
     };
     const originalAdd = jest.requireActual('../utils/constants.js');
 

diff --git a/scripts/actions/add-files-to-translation-queue.js b/scripts/actions/add-files-to-translation-queue.js
@@ -41,8 +41,8 @@ const translationDifference = (pendingFiles, prChanges) =>
       )
   );
 
-const humanTranslatedProjectID = process.env.HUMAN_TRANSLATION_PROJECT_ID;
-const machineTranslatedProjectID = process.env.MACHINE_TRANSLATION_PROJECT_ID;
+const humanTranslatedProjectID = 'HT_ID';
+const machineTranslatedProjectID = 'MT_ID';
 
 /**
  * Determines if a file should be included based on data from an exclusions file
@@ -54,14 +54,10 @@ const excludeFiles = (fileData) => {
   const exclusions = getExclusions();
 
   return fileData.filter(({ filename, locale, contentType }) => {
-    const localeKey = Object.keys(LOCALE_IDS).find(
-      (localeKey) => LOCALE_IDS[localeKey] === locale
-    );
     return (
-      !exclusions.excludePath[localeKey]?.some((path) =>
+      !exclusions.excludePath[locale]?.some((path) =>
         filename.startsWith(path)
-      ) &&
-      !exclusions.excludeType[localeKey]?.some((type) => contentType === type)
+      ) && !exclusions.excludeType[locale]?.some((type) => contentType === type)
     );
   });
 };

diff --git a/scripts/actions/translation_workflow/testing/connection_info.json b/scripts/actions/translation_workflow/testing/connection_info.json
@@ -1,6 +1,6 @@
 {
-    "username": "",
-    "password": "",
-    "host": "",
-    "database": ""
+  "username": "root",
+  "password": "root",
+  "host": "localhost",
+  "database": "translations"
 }
diff --git a/scripts/actions/utils/docs-content-tools/i18n-exclusions.yml b/scripts/actions/utils/docs-content-tools/i18n-exclusions.yml
@@ -1,11 +1,13 @@
 excludePath:
   ja-JP:
-    - /whats-new
+    - src/content/whats-new
     - src/content/docs/release-notes
     - src/content/docs/licenses
     - src/content/docs/style-guide
     - src/content/docs/agile-handbook
     - src/data-dictionary
+    - src/i18n
+    - src/content/docs/security/security-privacy/data-privacy
 excludeType:
   ja-JP:
     - landingPage