fix: type of style property in requestEncodings (#110)

Himenon · Apr 4, 2023 · d709d5f · d709d5f
1 parent dfc07a9
commit d709d5f
Show file tree

Hide file tree

Showing 18 changed files with 38 additions and 37 deletions.
diff --git a/src/code-templates/_shared/ApiClientInterface.ts b/src/code-templates/_shared/ApiClientInterface.ts
@@ -128,9 +128,7 @@ const createEncodingInterface = (factory: TsGenerator.Factory.Type) => {
       factory.PropertySignature.create({
         name: "style",
         optional: true,
-        type: factory.TypeReferenceNode.create({
-          name: "string",
-        }),
+        type: factory.TypeNode.create({ type: "string", enum: ["form", "spaceDelimited", "pipeDelimited", "deepObject"] }),
       }),
       factory.PropertySignature.create({
         name: "explode",

diff --git a/src/code-templates/_shared/MethodBody/index.ts b/src/code-templates/_shared/MethodBody/index.ts
@@ -82,6 +82,9 @@ export const create = (factory: TsGenerator.Factory.Type, params: CodeGenerator.
           factory.VariableDeclaration.create({
             name: "requestEncodings",
             initializer: identifier,
+            type: factory.TypeReferenceNode.create({
+              name: "Record<string, Record<string, Encoding>>",
+            }),
           }),
         ],
       }),

diff --git a/test/__tests__/class/__snapshots__/argo-rollout-test.ts.snap b/test/__tests__/class/__snapshots__/argo-rollout-test.ts.snap
@@ -3829,7 +3829,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }

diff --git a/test/__tests__/class/__snapshots__/format.domain.ts.snap b/test/__tests__/class/__snapshots__/format.domain.ts.snap
@@ -46,7 +46,7 @@ export namespace ErrorResponse { }
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }

diff --git a/test/__tests__/class/__snapshots__/kubernetes-test.ts.snap b/test/__tests__/class/__snapshots__/kubernetes-test.ts.snap
@@ -41028,7 +41028,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }

diff --git a/test/__tests__/class/__snapshots__/multi-type.test.domain.ts.snap b/test/__tests__/class/__snapshots__/multi-type.test.domain.ts.snap
@@ -45,7 +45,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }

diff --git a/test/__tests__/class/__snapshots__/spit-code-test.ts.snap b/test/__tests__/class/__snapshots__/spit-code-test.ts.snap
@@ -126,7 +126,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }

diff --git a/test/__tests__/class/__snapshots__/template-only-test.ts.snap b/test/__tests__/class/__snapshots__/template-only-test.ts.snap
@@ -63,7 +63,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }
@@ -253,7 +253,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }
@@ -407,7 +407,7 @@ export namespace ErrorResponse { }
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }

diff --git a/test/__tests__/class/__snapshots__/typedef-with-template-test.ts.snap b/test/__tests__/class/__snapshots__/typedef-with-template-test.ts.snap
@@ -412,7 +412,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }
@@ -585,7 +585,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }
@@ -1034,7 +1034,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }
@@ -1211,7 +1211,7 @@ export namespace ErrorResponse { }
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }
@@ -1373,7 +1373,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }
@@ -1441,7 +1441,7 @@ export class Client<RequestOption> {
             "Content-Type": "application/x-www-form-urlencoded",
             Accept: "application/json"
         };
-        const requestEncodings = {
+        const requestEncodings: Record<string, Record<string, Encoding>> = {
   "application/x-www-form-urlencoded": {
     "color": {
       "style": "form",
@@ -1463,7 +1463,7 @@ export class Client<RequestOption> {
             "Content-Type": params.headers["Content-Type"],
             Accept: "application/json"
         };
-        const requestEncodings = {
+        const requestEncodings: Record<string, Record<string, Encoding>> = {
   "application/x-www-form-urlencoded": {
     "color": {
       "style": "form",
@@ -1599,7 +1599,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }

diff --git a/test/__tests__/class/__snapshots__/unknown-schema-domain-test.ts.snap b/test/__tests__/class/__snapshots__/unknown-schema-domain-test.ts.snap
@@ -44,7 +44,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }

diff --git a/test/__tests__/functional/__snapshots__/argo-rollout-test.ts.snap b/test/__tests__/functional/__snapshots__/argo-rollout-test.ts.snap
@@ -3829,7 +3829,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }

diff --git a/test/__tests__/functional/__snapshots__/format.domain.ts.snap b/test/__tests__/functional/__snapshots__/format.domain.ts.snap
@@ -46,7 +46,7 @@ export namespace ErrorResponse { }
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }

diff --git a/test/__tests__/functional/__snapshots__/kubernetes-test.ts.snap b/test/__tests__/functional/__snapshots__/kubernetes-test.ts.snap
@@ -41028,7 +41028,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }

diff --git a/test/__tests__/functional/__snapshots__/multi-type.test.domain.ts.snap b/test/__tests__/functional/__snapshots__/multi-type.test.domain.ts.snap
@@ -45,7 +45,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }

diff --git a/test/__tests__/functional/__snapshots__/spit-code-test.ts.snap b/test/__tests__/functional/__snapshots__/spit-code-test.ts.snap
@@ -126,7 +126,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }

diff --git a/test/__tests__/functional/__snapshots__/template-only-test.ts.snap b/test/__tests__/functional/__snapshots__/template-only-test.ts.snap
@@ -63,7 +63,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }
@@ -256,7 +256,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }
@@ -413,7 +413,7 @@ export namespace ErrorResponse { }
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }

diff --git a/test/__tests__/functional/__snapshots__/typedef-with-template-test.ts.snap b/test/__tests__/functional/__snapshots__/typedef-with-template-test.ts.snap
@@ -412,7 +412,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }
@@ -588,7 +588,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }
@@ -1040,7 +1040,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }
@@ -1220,7 +1220,7 @@ export namespace ErrorResponse { }
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }
@@ -1384,7 +1384,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }
@@ -1452,7 +1452,7 @@ export const createClient = <RequestOption>(apiClient: ApiClient<RequestOption>,
                 "Content-Type": "application/x-www-form-urlencoded",
                 Accept: "application/json"
             };
-            const requestEncodings = {
+            const requestEncodings: Record<string, Record<string, Encoding>> = {
   "application/x-www-form-urlencoded": {
     "color": {
       "style": "form",
@@ -1474,7 +1474,7 @@ export const createClient = <RequestOption>(apiClient: ApiClient<RequestOption>,
                 "Content-Type": params.headers["Content-Type"],
                 Accept: "application/json"
             };
-            const requestEncodings = {
+            const requestEncodings: Record<string, Record<string, Encoding>> = {
   "application/x-www-form-urlencoded": {
     "color": {
       "style": "form",
@@ -1613,7 +1613,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }

diff --git a/test/__tests__/functional/__snapshots__/unknown-schema-domain-test.ts.snap b/test/__tests__/functional/__snapshots__/unknown-schema-domain-test.ts.snap
@@ -44,7 +44,7 @@ export namespace ErrorResponse {
 export interface Encoding {
     contentType?: string;
     headers?: Record<string, any>;
-    style?: string;
+    style?: "form" | "spaceDelimited" | "pipeDelimited" | "deepObject";
     explode?: boolean;
     allowReserved?: boolean;
 }