storage: pass metadata during stream upload

Author: stephenplusplus

lib/common/util.js

@@ -304,7 +304,9 @@ function makeWritableStream(dup, options, onComplete) {
       // Write the metadata to the request.
       stream.write('--' + boundary + '\n');
       stream.write('Content-Type: application/json\n\n');
-      stream.write(JSON.stringify(options.metadata));
+      stream.write(JSON.stringify({
+        metadata: options.metadata
+      }));
       stream.write('\n\n');
       stream.write('--' + boundary + '\n');
       stream.write('Content-Type: ' + streamType + '\n\n');

regression/storage.js

@@ -376,7 +376,10 @@ describe('storage', function() {
 
       it('should write metadata', function(done) {
         var options = {
-          metadata: { contentType: 'image/png' },
+          metadata: {
+            contentType: 'image/png',
+            property: 'value'
+          },
           resumable: false
         };
 
@@ -386,6 +389,7 @@ describe('storage', function() {
           file.getMetadata(function(err, metadata) {
             assert.ifError(err);
             assert.equal(metadata.contentType, options.metadata.contentType);
+            assert.equal(metadata.metadata.property, options.metadata.property);
             file.delete(done);
           });
         });

test/common/util.js

@@ -210,7 +210,9 @@ describe('common/util', function() {
           assert(endFirstBoundaryIdx > startFirstBoundaryIdx);
           assert(endBoundaryIdx > -1);
 
-          assert(written.indexOf(JSON.stringify(metadata)) > -1);
+          assert(written.indexOf(JSON.stringify({
+            metadata: metadata
+          })) > -1);
 
           done();
         };