Fix modular template to always generate class property syntax (#1398)

* Add test that generates content for modular (based on 13.2.7)

* format

* Fix modular template to always generate class property syntax

The modular template was incorrectly using object method syntax (`:` and
`,`) when route.namespace was present, introduced by PR #1326. This
caused TypeScript syntax errors in generated code.

The modular template should always generate class properties with arrow
functions (`=` and `;`), regardless of namespace presence.

This resolves the issue reported in #1366 where version 13.2.8 generated
invalid TypeScript code with modular templates.

Signed-off-by: Sora Morimoto <[email protected]>

---------

Signed-off-by: Sora Morimoto <[email protected]>
Co-authored-by: Sora Morimoto <[email protected]>

Author: k1rd3rf

.changeset/short-emus-fly.md

@@ -0,0 +1,15 @@
+---
+"swagger-typescript-api": patch
+---
+
+Fix modular template to always generate class property syntax.
+
+The modular template was incorrectly using object method syntax (`:` and
+`,`) when route.namespace was present, introduced by PR #1326. This
+caused TypeScript syntax errors in generated code.
+
+The modular template should always generate class properties with arrow
+functions (`=` and `;`), regardless of namespace presence.
+
+This resolves the issue reported in #1366 where version 13.2.8 generated
+invalid TypeScript code with modular templates.

templates/modular/procedure-call.ejs

@@ -90,7 +90,7 @@ const isValidIdentifier = (name) => /^[A-Za-z_$][A-Za-z0-9_$]*$/.test(name);
 <%~ routeDocs.lines %>
 
  */
-<% if (isValidIdentifier(route.routeName.usage)) { %><%~ route.routeName.usage %><%~ route.namespace ? ': ' : ' = ' %><% } else { %>"<%~ route.routeName.usage %>"<%~ route.namespace ? ': ' : ' = ' %><% } %>(<%~ wrapperArgs %>)<%~ config.toJS ? `: ${describeReturnType()}` : "" %> =>
+<% if (isValidIdentifier(route.routeName.usage)) { %><%~ route.routeName.usage %><% } else { %>"<%~ route.routeName.usage %>"<% } %> = (<%~ wrapperArgs %>)<%~ config.toJS ? `: ${describeReturnType()}` : "" %> =>
     <%~ config.singleHttpClient ? 'this.http.request' : 'this.request' %><<%~ type %>, <%~ errorType %>>({
         path: `<%~ path %>`,
         method: '<%~ _.upperCase(method) %>',
@@ -100,4 +100,4 @@ const isValidIdentifier = (name) => /^[A-Za-z_$][A-Za-z0-9_$]*$/.test(name);
         <%~ bodyContentKindTmpl ? `type: ${bodyContentKindTmpl},` : '' %>
         <%~ responseFormatTmpl ? `format: ${responseFormatTmpl},` : '' %>
         ...<%~ _.get(requestConfigParam, "name") %>,
-    })<%~ route.namespace ? ',' : '' %>
+    })