Skip to content

Commit 2db248c

Browse files
AnujChhikaravikasosmiumAchintya-Chatterjee
authored
fix: handle id query parameter in Get Requests API (#2367)
Co-authored-by: Vikas Singh <[email protected]> Co-authored-by: Achintya Chatterjee <[email protected]>
1 parent d98bfa1 commit 2db248c

File tree

2 files changed

+18
-0
lines changed

2 files changed

+18
-0
lines changed

controllers/requests.ts

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,13 @@ export const getRequestsController = async (req: any, res: any) => {
6060
return res.status(204).send();
6161
}
6262

63+
if (query.id) {
64+
return res.status(200).json({
65+
message: REQUEST_FETCHED_SUCCESSFULLY,
66+
data: requests,
67+
});
68+
}
69+
6370
const { allRequests, next, prev, page } = requests;
6471
if (allRequests.length === 0) {
6572
return res.status(204).send();

test/integration/requests.test.ts

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -273,6 +273,17 @@ describe("/requests OOO", function () {
273273
});
274274
});
275275

276+
it("should return the request by Id query", function (done) {
277+
chai
278+
.request(app)
279+
.get(`/requests?id=${oooRequestId}`)
280+
.end(function (err, res) {
281+
expect(res).to.have.status(200);
282+
expect(res.body.data.id === oooRequestId);
283+
done();
284+
});
285+
});
286+
276287
it("should return all requests by specific user", function (done) {
277288
chai
278289
.request(app)

0 commit comments

Comments
 (0)