chore: refactor translate() function and tests

Author: isaacdarcilla

src/index.ts

@@ -30,12 +30,6 @@ export const translate = (
   text: string,
   script: Script = Script.BAYBAYIN
 ): string => {
-  const scriptConverter = scriptConverters[script];
-
-  if (scriptConverter) {
-    return scriptConverter(text);
-  }
-
-  // Default to Baybayin if the script is not recognized
-  return toBaybayin(text);
+  const scriptConverter = scriptConverters[script] || toBaybayin;
+  return scriptConverter(text);
 };

test/index.spec.ts

@@ -1,49 +1,15 @@
 import { Script, translate } from '../src';
 
-describe('baybayin', () => {
-  describe('translate', () => {
-    it('should return a string containing the baybayin text', () => {
-      const text = 'a';
-
-      const result = translate(text);
-
-      expect(result).toMatch('\u1700');
-    });
-  });
-});
-
-describe('hanunoo', () => {
-  describe('translate', () => {
-    it('should return a string containing the hanunoo text', () => {
-      const text = 'a';
-
-      const result = translate(text, Script.HANUNOO);
-
-      expect(result).toMatch('\u1720');
-    });
-  });
-});
-
-describe('buhid', () => {
-  describe('translate', () => {
-    it('should return a string containing the buhid text', () => {
-      const text = 'nga';
-
-      const result = translate(text, Script.BUHID);
-
-      expect(result).toMatch('\u1745');
-    });
-  });
-});
-
-describe('tagbanwa', () => {
-  describe('translate', () => {
-    it('should return a string containing the tagbanwa text', () => {
-      const text = 'nga';
-
-      const result = translate(text, Script.TAGBANWA);
-
-      expect(result).toMatch('\u1765');
-    });
+const testCases = [
+  { script: Script.BAYBAYIN, text: 'a', expected: '\u1700' },
+  { script: Script.HANUNOO, text: 'a', expected: '\u1720' },
+  { script: Script.BUHID, text: 'nga', expected: '\u1745' },
+  { script: Script.TAGBANWA, text: 'nga', expected: '\u1765' },
+];
+
+testCases.forEach(({ script, text, expected }) => {
+  test(`translate using script '${script}' with text '${text}' should return '${expected}'`, () => {
+    const result = translate(text, script);
+    expect(result).toMatch(expected);
   });
 });