Skip to content

Commit 9d6436c

Browse files
authored
fix: return 5 extension requests by default when size is not specified (#2378)
1 parent a7812ee commit 9d6436c

File tree

2 files changed

+19
-1
lines changed

2 files changed

+19
-1
lines changed

controllers/extensionRequests.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ const createTaskExtensionRequest = async (req, res) => {
105105
*/
106106
const fetchExtensionRequests = async (req, res) => {
107107
try {
108-
const { cursor, size, order } = req.query;
108+
const { cursor, size = 5, order } = req.query;
109109
const { status, taskId, assignee } = parseQueryParams(req._parsedUrl.search);
110110
const { transformedSize, transformedStatus } = transformQuery(size, status);
111111

test/integration/extensionRequests.test.js

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -281,6 +281,24 @@ describe("Extension Requests", function () {
281281
});
282282
});
283283

284+
it("should return 5 extension requests by default when size query is not provided", function (done) {
285+
chai
286+
.request(app)
287+
.get(`/extension-requests`)
288+
.set("cookie", `${cookieName}=${appOwnerjwt}`)
289+
.end((err, res) => {
290+
if (err) {
291+
return done(err);
292+
}
293+
expect(res).to.have.status(200);
294+
expect(res.body).to.be.a("object");
295+
expect(res.body.message).to.be.equal("Extension Requests returned successfully!");
296+
expect(res.body.allExtensionRequests).to.be.a("array");
297+
expect(res.body.allExtensionRequests).to.have.lengthOf(5);
298+
return done();
299+
});
300+
});
301+
284302
it("should return success response and all extension requests with query params", function (done) {
285303
chai
286304
.request(app)

0 commit comments

Comments
 (0)