fix(createUpload): Setting ContentType

ContentType for createUpload was previously hardcoded to
applicaion/octet-stream
This fix uses the contentType parameter for uploading a file

Closes #104

Author: Samuel Jeffress

lib/create-space-api.js

@@ -585,13 +585,13 @@ export default function createSpaceApi ({
    * space.createUpload(uploadStream)
    */
   function createUpload (data) {
-    const { file } = data
+    const { file, contentType } = data
     if (!file) {
       return Promise.reject(new Error('Unable to locate a file to upload.'))
     }
     return httpUpload.post('uploads', file, {
       headers: {
-        'Content-Type': 'application/octet-stream'
+        'Content-Type': contentType || 'application/octet-stream'
       }
     })
     .then(uploadResponse => {

test/unit/create-space-api-test.js

@@ -374,6 +374,29 @@ test('API call createUpload', (t) => {
     file: '<svg><path fill="red" d="M50 50h150v50H50z"/></svg>'
   })
   .then(() => {
+    t.equals(httpUploadMock.post.args[0][2].headers['Content-Type'], 'image/svg')
+    t.equals(httpUploadMock.post.args[0][1], '<svg><path fill="red" d="M50 50h150v50H50z"/></svg>', 'uploads file to upload endpoint')
+    t.deepEqual(entitiesMock.upload.wrapUpload.args[0][1], mockedUpload, 'wrapUpload was called with correct raw upload object')
+  })
+})
+
+test('API call createUpload defaults the content type to octet-stream', (t) => {
+  const { api, httpUploadMock, entitiesMock } = setup(Promise.resolve({}))
+  const mockedUpload = {
+    sys: {
+      id: 'some_random_id'
+    }
+  }
+  httpUploadMock.post.returns(Promise.resolve({
+    data: mockedUpload
+  }))
+
+  return api.createUpload({ // no contentType set here
+    fileName: 'filename.svg',
+    file: '<svg><path fill="red" d="M50 50h150v50H50z"/></svg>'
+  })
+  .then(() => {
+    t.equals(httpUploadMock.post.args[0][2].headers['Content-Type'], 'application/octet-stream')
     t.equals(httpUploadMock.post.args[0][1], '<svg><path fill="red" d="M50 50h150v50H50z"/></svg>', 'uploads file to upload endpoint')
     t.deepEqual(entitiesMock.upload.wrapUpload.args[0][1], mockedUpload, 'wrapUpload was called with correct raw upload object')
   })