Skip to content

Commit b1fa308

Browse files
committed
Added error handling to version requests
1 parent dbb8977 commit b1fa308

File tree

5 files changed

+45
-9
lines changed

5 files changed

+45
-9
lines changed

lib/versions/create.js

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,13 @@ exports.run = async function({ opts }) {
5151
auth: { user: key },
5252
};
5353

54-
return request
55-
.post(`${config.host}/api/v1/version`, options)
56-
.catch(err => Promise.reject(new Error(err)));
54+
return request.post(`${config.host}/api/v1/version`, options).catch(err => {
55+
let errorDesc;
56+
try {
57+
errorDesc = JSON.parse(err.error).description;
58+
} catch (e) {
59+
errorDesc = 'Failed to create a new version using your specified parameters.';
60+
}
61+
return Promise.reject(new Error(errorDesc));
62+
});
5763
};

lib/versions/delete.js

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,5 +31,13 @@ exports.run = async function({ opts }) {
3131
.delete(`${config.host}/api/v1/version/${version}`, {
3232
auth: { user: key },
3333
})
34-
.catch(err => Promise.reject(new Error(err)));
34+
.catch(err => {
35+
let errorDesc;
36+
try {
37+
errorDesc = JSON.parse(err.error).description;
38+
} catch (e) {
39+
errorDesc = 'Failed to delete target version.';
40+
}
41+
return Promise.reject(new Error(errorDesc));
42+
});
3543
};

lib/versions/index.js

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,5 +24,13 @@ exports.run = function({ opts }) {
2424
json: true,
2525
auth: { user: key },
2626
})
27-
.catch(err => Promise.reject(new Error(err)));
27+
.catch(err => {
28+
let errorDesc;
29+
try {
30+
errorDesc = JSON.parse(err.error).description;
31+
} catch (e) {
32+
errorDesc = 'Failed to get versions attached to the provided key.';
33+
}
34+
return Promise.reject(new Error(errorDesc));
35+
});
2836
};

lib/versions/update.js

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,13 @@ exports.run = async function({ opts }) {
5151
auth: { user: key },
5252
};
5353

54-
return request
55-
.put(`${config.host}/api/v1/version/${version}`, options)
56-
.catch(err => Promise.reject(new Error(err)));
54+
return request.put(`${config.host}/api/v1/version/${version}`, options).catch(err => {
55+
let errorDesc;
56+
try {
57+
errorDesc = JSON.parse(err.error).description;
58+
} catch (e) {
59+
errorDesc = 'Failed to update version using your specified parameters.';
60+
}
61+
return Promise.reject(new Error(errorDesc));
62+
});
5763
};

lib/versions/versionId.js

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,5 +32,13 @@ exports.run = function({ opts }) {
3232
json: true,
3333
auth: { user: key },
3434
})
35-
.catch(err => Promise.reject(new Error(err)));
35+
.catch(err => {
36+
let errorDesc;
37+
try {
38+
errorDesc = JSON.parse(err.error).description;
39+
} catch (e) {
40+
errorDesc = 'Failed to get specific version using provided identifier and key.';
41+
}
42+
return Promise.reject(new Error(errorDesc));
43+
});
3644
};

0 commit comments

Comments
 (0)