fix: don't modify passed in options (googleapis#1895)

* fix: don't modify passed in options

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* added test

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

Author: 2 people

src/file.ts

@@ -1804,7 +1804,7 @@ class File extends ServiceObject<File> {
    */
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   createWriteStream(options: CreateWriteStreamOptions = {}): Writable {
-    options = Object.assign({metadata: {}}, options);
+    options = extend(true, {metadata: {}}, options);
 
     if (options.contentType) {
       options.metadata.contentType = options.contentType;
@@ -3695,7 +3695,8 @@ class File extends ServiceObject<File> {
     dup: Duplexify,
     options: CreateResumableUploadOptions
   ): void {
-    options = Object.assign(
+    options = extend(
+      true,
       {
         metadata: {},
       },
@@ -3762,7 +3763,8 @@ class File extends ServiceObject<File> {
    * @private
    */
   startSimpleUpload_(dup: Duplexify, options?: CreateWriteStreamOptions): void {
-    options = Object.assign(
+    options = extend(
+      true,
       {
         metadata: {},
       },

test/file.ts

@@ -1993,7 +1993,6 @@ describe('File', () => {
       const writable = file.createWriteStream(options);
 
       file.startSimpleUpload_ = (stream: {}, options_: {}) => {
-        assert.deepStrictEqual(options_, options);
         done();
       };
 
@@ -2009,7 +2008,6 @@ describe('File', () => {
       const writable = file.createWriteStream(options);
 
       file.startResumableUpload_ = (stream: {}, options_: {}) => {
-        assert.deepStrictEqual(options_, options);
         done();
       };
 
@@ -2023,7 +2021,6 @@ describe('File', () => {
 
       // eslint-disable-next-line @typescript-eslint/no-explicit-any
       file.startResumableUpload_ = (stream: {}, options: any) => {
-        assert.deepStrictEqual(options.metadata, METADATA);
         done();
       };
 
@@ -2064,6 +2061,19 @@ describe('File', () => {
       writable.write('data');
     });
 
+    it('should not overwrite passed in options', done => {
+      const emptyObject = {};
+      const writable = file.createWriteStream(emptyObject);
+      // eslint-disable-next-line @typescript-eslint/no-explicit-any
+      file.startResumableUpload_ = (stream: {}, options: any) => {
+        assert.strictEqual(options.metadata.contentType, 'image/png');
+        assert.deepStrictEqual(emptyObject, {});
+        done();
+      };
+
+      writable.write('data');
+    });
+
     it('should not set a contentType if mime lookup failed', done => {
       const file = new File('file-without-ext');
       const writable = file.createWriteStream();
@@ -4622,7 +4632,7 @@ describe('File', () => {
             assert.strictEqual(opts.file, file.name);
             assert.strictEqual(opts.generation, file.generation);
             assert.strictEqual(opts.key, file.encryptionKey);
-            assert.strictEqual(opts.metadata, options.metadata);
+            assert.deepStrictEqual(opts.metadata, options.metadata);
             assert.strictEqual(opts.offset, options.offset);
             assert.strictEqual(opts.predefinedAcl, options.predefinedAcl);
             assert.strictEqual(opts.private, options.private);
@@ -4785,7 +4795,7 @@ describe('File', () => {
 
       // eslint-disable-next-line @typescript-eslint/no-explicit-any
       makeWritableStreamOverride = (stream: {}, options_: any) => {
-        assert.strictEqual(options_.metadata, options.metadata);
+        assert.deepStrictEqual(options_.metadata, options.metadata);
         assert.deepStrictEqual(options_.request, {
           qs: {
             name: file.name,