refactor(nestjs-client): Fix nest.js client signatures

Fixes stephenh#160
ssilve1989 · Oct 26, 2020 · db1e9b8 · db1e9b8
1 parent adfa489
commit db1e9b8
Showing 1 changed file with 3 additions and 8 deletions.
diff --git a/src/generate-nestjs.ts b/src/generate-nestjs.ts
@@ -113,14 +113,9 @@ export function generateNestjsServiceClient(
 
     requestFn = requestFn.addParameter('request', requestType(typeMap, methodDesc, options));
 
-    // Use metadata as last argument for interface only configuration
-    if (options.addGrpcMetadata) {
-      requestFn = requestFn.addParameter(options.addNestjsRestParameter ? 'metadata' : 'metadata?', 'Metadata@grpc');
-    }
-    if (options.addNestjsRestParameter) {
-      requestFn = requestFn.addParameter('...rest', 'any');
-    }
-
+    // Client Request should always accept an optional metadata argument
+    requestFn = requestFn.addParameter('metadata?', 'Metadata@grpc');
+
     // Return observable since nestjs client always returns an Observable
     requestFn = requestFn.returns(responseObservable(typeMap, methodDesc, options));