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

Fixes stephenh#160
ssilve1989 · Oct 26, 2020 · 13c48cd · 13c48cd
1 parent adfa489
commit 13c48cd
Showing 1 changed file with 2 additions and 7 deletions.
diff --git a/src/generate-nestjs.ts b/src/generate-nestjs.ts
@@ -113,13 +113,8 @@ 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));