Pass response status code back, trigger error failed if 500

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@navjobs/upload",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "HOC that displays a file upload on clicking children",
   "main": "dist/index.js",
   "scripts": {

src/request/register-listeners.js

@@ -11,11 +11,11 @@ export default ({ xhr, progress, resolve }) => {
     } catch (e) {
       response = xhr.response;
     }
-    resolve({ response });
+    resolve({ response, error: xhr.status === 500, status: xhr.status });
   });
 
   xhr.addEventListener('error', () => {
-    resolve({ error: true });
+    resolve({ error: true, status: xhr.status });
   });
 
   xhr.addEventListener('abort', () => {

src/uploader.js

@@ -22,21 +22,30 @@ export default class Uploader extends React.Component {
         files,
       },
       progress => this.setState({ progress })
-    ).then(({ response, error, abort }) => {
-      if (error) return this.setState({ failed: true });
+    ).then(({ response, error, abort, status }) => {
+      if (error) return this.setState({ failed: true, response, status });
       if (abort) return this.setState({ canceled: true });
 
-      if (onComplete) onComplete(response);
-      this.setState({ response, complete: true });
+      if (onComplete) onComplete(response, status);
+      this.setState({ response, status, complete: true });
     });
   }
 
   render() {
     let { children } = this.props;
-    let { progress, canceled, complete, failed, files, response } = this.state;
+    let {
+      progress,
+      canceled,
+      complete,
+      failed,
+      files,
+      response,
+      status,
+    } = this.state;
     return children({
       files,
       failed,
+      status,
       progress,
       canceled,
       complete,

tests/unit/uploader.js

@@ -4,6 +4,7 @@ import Uploader from '../../src/uploader';
 test('Triggers upload', async () => {
   let progressValue = 0;
   let completedValue = false;
+  let completedStatus = false;
 
   const output = mount(
     <Uploader
@@ -12,7 +13,10 @@ test('Triggers upload', async () => {
         method: 'POST',
       }}
       uploadOnSelection={true}
-      onComplete={response => completedValue = response}
+      onComplete={(response, status) => {
+        completedValue = response;
+        completedStatus = status;
+      }}
     >
       {({ onFiles, startUpload, progress, complete, canceled, failed }) => {
         if (progress) progressValue = progress;
@@ -41,11 +45,13 @@ test('Triggers upload', async () => {
   expect(progressValue).toEqual(50);
 
   expect(completedValue).toEqual({ finished: 'hell yes' });
+  expect(completedStatus).toEqual(200);
 });
 
 test('Triggers upload with headers and extra fields', async () => {
   let progressValue = 0;
   let wasCompleted = false;
+  let statusCode = false;
 
   const output = mount(
     <Uploader
@@ -62,8 +68,11 @@ test('Triggers upload with headers and extra fields', async () => {
       uploadOnSelection={true}
       onComplete={() => wasCompleted = true}
     >
-      {({ onFiles, startUpload, progress, complete, canceled, failed }) => {
+      {(
+        { onFiles, startUpload, progress, complete, canceled, failed, status }
+      ) => {
         if (progress) progressValue = progress;
+        if (status) statusCode = status;
         return (
           <UploadField onFiles={onFiles}>
             <div>
@@ -85,6 +94,7 @@ test('Triggers upload with headers and extra fields', async () => {
   expect(fields.test).toEqual('test value');
 
   expect(wasCompleted).toEqual(true);
+  expect(statusCode).toEqual(200);
 });
 
 test('doesnt upload unless upload button is clicked', async () => {
@@ -175,6 +185,7 @@ test('returns failed on bad request', async () => {
     >
       {({ onFiles, startUpload, progress, complete, canceled, failed }) => {
         if (failed) didFail = true;
+
         return (
           <div>
             <UploadField onFiles={onFiles}>

tests/xhr.js

@@ -10,6 +10,7 @@ let callbacks = [];
 global.headers = [];
 let blockSend;
 const mock = values => ({
+  status: 200,
   response: JSON.stringify({ finished: 'hell yes' }),
   addEventListener: (name, callback) => {
     callbacks.push({ name, callback });