fix: remove trailing slashes from bucket name. (#266)

Fixes #170 

This cleans up the fragments of a gs:// URL, should the user provide one to the Bucket constructor.

Author: stephenplusplus

src/bucket.ts

@@ -302,7 +302,8 @@ class Bucket extends ServiceObject {
   constructor(storage, name, options?) {
     options = options || {};
 
-    name = name.replace(/^gs:\/\//, '');
+    // Allow for "gs://"-style input, and strip any trailing slashes.
+    name = name.replace(/^gs:\/\//, '').replace(/\/+$/, '');
 
     const methods = {
       /**

test/bucket.ts

@@ -177,6 +177,11 @@ describe('Bucket', () => {
       assert.strictEqual(bucket.name, 'bucket-name');
     });
 
+    it('should remove a trailing /', () => {
+      const bucket = new Bucket(STORAGE, 'bucket-name/');
+      assert.strictEqual(bucket.name, 'bucket-name');
+    });
+
     it('should localize the name', () => {
       assert.strictEqual(bucket.name, BUCKET_NAME);
     });