Skip to content

Commit eb67111

Browse files
fix-statuscode-tests
1 parent bb74cdd commit eb67111

File tree

2 files changed

+11
-6
lines changed

2 files changed

+11
-6
lines changed

controllers/oooRequests.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -205,6 +205,14 @@ export const acknowledgeOooRequest = async (
205205
timestamp: new Date().toISOString(),
206206
});
207207

208+
209+
if (error.statusCode === 409) {
210+
return res.boom.conflict(error.message);
211+
}
212+
if (error.statusCode === 400) {
213+
return res.boom.badRequest(error.message);
214+
}
215+
208216
next(new Error(ERROR_WHILE_ACKNOWLEDGING_REQUEST));
209217
}
210218
};

test/integration/requests.test.ts

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -425,8 +425,7 @@ describe("/requests OOO", function () {
425425
if (err) {
426426
return done(err);
427427
}
428-
// The system currently returns 500 for already approved requests due to error handling
429-
expect(res.statusCode).to.equal(500);
428+
expect(res.statusCode).to.equal(409);
430429
done();
431430
});
432431
});
@@ -441,8 +440,7 @@ describe("/requests OOO", function () {
441440
if (err) {
442441
return done(err);
443442
}
444-
// The system currently returns 500 for already rejected requests due to error handling
445-
expect(res.statusCode).to.equal(500);
443+
expect(res.statusCode).to.equal(409);
446444
done();
447445
});
448446
});
@@ -457,8 +455,7 @@ describe("/requests OOO", function () {
457455
if (err) {
458456
return done(err);
459457
}
460-
// The system currently returns 500 for invalid request types due to error handling
461-
expect(res.statusCode).to.equal(500);
458+
expect(res.statusCode).to.equal(400);
462459
done();
463460
});
464461
});

0 commit comments

Comments
 (0)