fix: allow passing numbers as path template parameters

googleapis · Mar 24, 2020 · 0d931e7 · 0d931e7
1 parent 836e81b
commit 0d931e7
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/pathTemplate.ts b/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) {

diff --git a/test/unit/path_template.ts → test/unit/pathTemplate.ts b/test/unit/path_template.ts → test/unit/pathTemplate.ts
@@ -139,6 +139,16 @@ 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`', () => {