Merge pull request #997 from lbrooks/test-translator

cleanup: Add tests for translator

src/renderer/utils/translator.test.ts

@@ -0,0 +1,24 @@
+import { expect, it, describe } from "vitest";
+import getTranslator from "./translator";
+
+describe("translator", ()=> {
+  it("should return fr-FR for input fr-Fr", ()=>{
+    expect(getTranslator("fr-FR")).toBe("fr-FR");
+  });
+
+  it("should return fr-FR for input fr", ()=>{
+    expect(getTranslator("fr")).toBe("fr-FR");
+  });
+
+  it("should return fr-FR for input fr-???", ()=>{
+    expect(getTranslator("fr-???")).toBe("fr-FR");
+  });
+
+  it("should return default fallback", ()=>{
+    expect(getTranslator("abc")).toBe("en-US");
+  });
+
+  it("should return supplied fallback", ()=>{
+    expect(getTranslator("abc", "fr-FR")).toBe("fr-FR");
+  });
+});

src/renderer/utils/translator.ts

@@ -30,7 +30,7 @@ const translator: { [key: string]: string } = {
  * @param {string} localLanguage Local language
  * @param {string} defaultLanguage Default language
  */
-function getTranslator(localLanguage: string, defaultLanguage = "en-US") {
+function getTranslator(localLanguage: string, defaultLanguage: string = "en-US") {
   const language = localLanguage || defaultLanguage;
 
   if (translator[language] !== undefined) {