fix: allow passing numbers as path template parameters (#756)

Author: alexander-fenster

packages/packages/google-gax/src/pathTemplate.ts

@@ -34,7 +34,7 @@ export interface Segment {
 }
 
 export interface Bindings {
-  [index: string]: string;
+  [index: string]: string | number;
 }
 
 export class PathTemplate {
@@ -129,7 +129,7 @@ export class PathTemplate {
           );
           throw new TypeError(msg);
         }
-        const tmp = new PathTemplate(bindings[segment.literal]);
+        const tmp = new PathTemplate(bindings[segment.literal].toString());
         Array.prototype.push.apply(out, tmp.segments);
         inABinding = true;
       } else if (segment.kind === extras.END_BINDING) {

…es/google-gax/test/unit/path_template.ts …ges/google-gax/test/unit/pathTemplate.tspackages/packages/google-gax/test/unit/path_template.ts renamed to packages/packages/google-gax/test/unit/pathTemplate.ts packages/packages/google-gax/test/unit/path_template.ts renamed to packages/packages/google-gax/test/unit/pathTemplate.ts

@@ -139,6 +139,18 @@ describe('PathTemplate', () => {
       const want = 'bar/1/2/foo/3';
       expect(template.render(params)).to.eql(want);
     });
+
+    it('should accept both strings and numbers as values', () => {
+      const template = new PathTemplate(
+        'projects/{project}/sessions/{session}'
+      );
+      const params = {
+        project: 'testProject',
+        session: 123,
+      };
+      const want = 'projects/testProject/sessions/123';
+      expect(template.render(params)).to.eql(want);
+    });
   });
 
   describe('method `inspect`', () => {