storage: strip leading slash in File() constructor (#2269)

googleapis · May 1, 2017 · e107247 · e107247
1 parent a0341b2
commit e107247
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/packages/storage/src/file.js b/packages/storage/src/file.js
@@ -103,7 +103,7 @@ function File(bucket, name, options) {
 
   Object.defineProperty(this, 'name', {
     enumerable: true,
-    value: name
+    value: name.replace(/^\/+/, '') // Remove leading slashes.
   });
 
   var generation = parseInt(options.generation, 10);

diff --git a/packages/storage/test/file.js b/packages/storage/test/file.js
@@ -161,6 +161,11 @@ describe('File', function() {
       assert.strictEqual(file.storage, BUCKET.storage);
     });
 
+    it('should strip a single leading slash', function() {
+      var file = new File(BUCKET, '/name');
+      assert.strictEqual(file.name, 'name');
+    });
+
     it('should accept specifying a generation', function() {
       var file = new File(BUCKET, 'name', { generation: 2 });
       assert.equal(file.generation, 2);