fix: copy(): Use correct destination file name in URI (#389)

googleapis · Sep 13, 2018 · 02a37ad · 02a37ad
1 parent 6d05e94
commit 02a37ad
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/file.ts b/src/file.ts
@@ -530,7 +530,7 @@ class File extends ServiceObject {
         {
           method: 'POST',
           uri: `/rewriteTo/b/${destBucket.name}/o/${
-              encodeURIComponent(destName)}`,
+              encodeURIComponent(newFile.name)}`,
           qs: query,
           json: options,
           headers,

diff --git a/test/file.ts b/test/file.ts
@@ -453,9 +453,10 @@ describe('File', () => {
       }
 
       it('should allow a string', done => {
-        const newFileName = 'new-file-name.png';
+        const newFileName = '/new-file-name.png';
+        const newFile = new File(BUCKET, newFileName);
         const expectedPath =
-            `/rewriteTo/b/${file.bucket.name}/o/${newFileName}`;
+            `/rewriteTo/b/${file.bucket.name}/o/${newFile.name}`;
         assertPathEquals(file, expectedPath, done);
         file.copy(newFileName);
       });