refactor: add _buildApiFromDirectiveArguments()

Author: MH4GF

src/directive.ts

@@ -115,7 +115,7 @@ export function buildApi(config: FormattedDirectiveConfig, directives: ReadonlyA
     .map((directive) => {
       const directiveName = directive.name.value;
       const argsConfig = config[directiveName];
-      return buildApiFromDirectiveArguments(argsConfig, directive.arguments ?? []).join('');
+      return buildApiFromDirectiveArguments(argsConfig, directive.arguments ?? []);
     })
     .join('')
 }
@@ -148,7 +148,7 @@ export function buildApiForValibot(config: FormattedDirectiveConfig, directives:
     .map((directive) => {
       const directiveName = directive.name.value;
       const argsConfig = config[directiveName];
-      const apis = buildApiFromDirectiveArguments(argsConfig, directive.arguments ?? []);
+      const apis = _buildApiFromDirectiveArguments(argsConfig, directive.arguments ?? []);
       return apis.map(api => `v${api}`);
     }).flat()
 }
@@ -165,7 +165,11 @@ function buildApiSchema(validationSchema: string[] | undefined, argValue: ConstV
   return `.${schemaApi}(${schemaApiArgs.join(', ')})`;
 }
 
-function buildApiFromDirectiveArguments(config: FormattedDirectiveArguments, args: ReadonlyArray<ConstArgumentNode>): string[] {
+function buildApiFromDirectiveArguments(config: FormattedDirectiveArguments, args: ReadonlyArray<ConstArgumentNode>): string {
+  return _buildApiFromDirectiveArguments(config, args).join('');
+}
+
+function _buildApiFromDirectiveArguments(config: FormattedDirectiveArguments, args: ReadonlyArray<ConstArgumentNode>): string[] {
   return args
     .map((arg) => {
       const argName = arg.name.value;

tests/directive.spec.ts

@@ -353,7 +353,7 @@ describe('format directive config', () => {
         config: FormattedDirectiveArguments
         args: ReadonlyArray<ConstArgumentNode>
       }
-      want: string[]
+      want: string
     }[] = [
       {
         name: 'string',
@@ -365,7 +365,7 @@ describe('format directive config', () => {
             msg: `"hello"`,
           }),
         },
-        want: [`.required("hello")`],
+        want: `.required("hello")`,
       },
       {
         name: 'string with additional stuff',
@@ -377,7 +377,7 @@ describe('format directive config', () => {
             startWith: `"hello"`,
           }),
         },
-        want: [`.matched("^hello")`],
+        want: `.matched("^hello")`,
       },
       {
         name: 'number',
@@ -389,7 +389,7 @@ describe('format directive config', () => {
             minLength: `1`,
           }),
         },
-        want: [`.min(1)`],
+        want: `.min(1)`,
       },
       {
         name: 'boolean',
@@ -402,7 +402,7 @@ describe('format directive config', () => {
             enabled: `true`,
           }),
         },
-        want: [`.strict(true)`],
+        want: `.strict(true)`,
       },
       {
         name: 'list',
@@ -414,7 +414,7 @@ describe('format directive config', () => {
             minLength: `[1, "message"]`,
           }),
         },
-        want: [`.min(1, "message")`],
+        want: `.min(1, "message")`,
       },
       {
         name: 'object in list',
@@ -426,7 +426,7 @@ describe('format directive config', () => {
             matches: `["hello", {message:"message", excludeEmptyString:true}]`,
           }),
         },
-        want: [`.matches("hello", {"message":"message","excludeEmptyString":true})`],
+        want: `.matches("hello", {"message":"message","excludeEmptyString":true})`,
       },
       {
         name: 'two arguments but matched to first argument',
@@ -439,7 +439,7 @@ describe('format directive config', () => {
             msg2: `"world"`,
           }),
         },
-        want: [`.required("hello")`, ``],
+        want: `.required("hello")`,
       },
       {
         name: 'two arguments but matched to second argument',
@@ -452,7 +452,7 @@ describe('format directive config', () => {
             msg2: `"world"`,
           }),
         },
-        want: [``, `.required("world")`],
+        want: `.required("world")`,
       },
       {
         name: 'two arguments matched all',
@@ -466,7 +466,7 @@ describe('format directive config', () => {
             minLength: `1`,
           }),
         },
-        want: [`.required("message")`, `.min(1)`],
+        want: `.required("message").min(1)`,
       },
       {
         name: 'argument matches validation schema api',
@@ -480,7 +480,7 @@ describe('format directive config', () => {
             format: `"uri"`,
           }),
         },
-        want: [`.url()`],
+        want: `.url()`,
       },
       {
         name: 'argument matched argument but doesn\'t match api',
@@ -494,7 +494,7 @@ describe('format directive config', () => {
             format: `"uuid"`,
           }),
         },
-        want: [``],
+        want: ``,
       },
       {
         name: 'complex',
@@ -510,7 +510,7 @@ describe('format directive config', () => {
             format: `"uri"`,
           }),
         },
-        want: [`.required("message")`, `.url()`],
+        want: `.required("message").url()`,
       },
       {
         name: 'complex 2',
@@ -526,7 +526,7 @@ describe('format directive config', () => {
             format: `"uuid"`,
           }),
         },
-        want: [`.required("message")`, ``],
+        want: `.required("message")`,
       },
     ];
     for (const tc of cases) {