This PR completes the migration of extension requests to the /request API endpoint with a feature flag. Additionally, it adds the necessary controller and integration tests for handling extension requests uniformly through the /request API endpoint.
Dependent PRs
REFACTOR: request controller and model functions #2017 (PR Link): This PR refactors the request controller and model functions to support the migration of extension requests to the /request API endpoint.
Added middleware for extension request for /request route #2028 (PR Link): This PR adds middleware for handling extension requests specifically for the /request route.
Documentation Updated?
[ ] Yes
[x] No
Under Feature Flag
[x] Yes
[ ] No
Database Changes
[ ] Yes
[x] No
Breaking Changes
[ ] Yes
[x] No
Development Tested?
[x] Yes
[ ] No
Screenshots
Screenshots
### Successful create ER
### When the old ETA is not matched to the original task old ETA
### When task id is invalid
### When pending ER already exists
Test Coverage
Test Coverage
| File | Lines | Lines Covered | % Lines Covered | Details |
|---------------|----------|---------------|-----------------|------------------------------------------------------------|
| requests.ts | 85.07 | 67.74 | 100 | 44,109,113-114,124-130,133-139,149-150 |
##### Tests
/requests Extension
POST /requests
✔ should return 401(Unauthorized) if user is not logged in
✔ should create a new extension request (76ms)
✔ should create a new extension request by super user (98ms)
✔ should not create a new extension request by another user (68ms)
✔ should not create a new extension request if task is not exist (66ms)
✔ should not create a new extension request if assignee is not present (58ms)
✔ should not create a new extension request if old ETA does not match the task's ETA (50ms)
✔ should not create a new extension request if an extension request for this task already exists (151ms)
Issue Ticket Number
2019
Description
This PR completes the migration of extension requests to the
/requestAPI endpoint with a feature flag. Additionally, it adds the necessary controller and integration tests for handling extension requests uniformly through the/requestAPI endpoint.Dependent PRs
/requestAPI endpoint./requestroute.Documentation Updated?
Under Feature Flag
Database Changes
Breaking Changes
Development Tested?
Screenshots
Screenshots
### Successful create ER  ### When the old ETA is not matched to the original task old ETA  ### When task id is invalid  ### When pending ER already existsTest Coverage
Test Coverage
| File | Lines | Lines Covered | % Lines Covered | Details | |---------------|----------|---------------|-----------------|------------------------------------------------------------| | requests.ts | 85.07 | 67.74 | 100 | 44,109,113-114,124-130,133-139,149-150 | ##### Tests /requests Extension POST /requests ✔ should return 401(Unauthorized) if user is not logged in ✔ should create a new extension request (76ms) ✔ should create a new extension request by super user (98ms) ✔ should not create a new extension request by another user (68ms) ✔ should not create a new extension request if task is not exist (66ms) ✔ should not create a new extension request if assignee is not present (58ms) ✔ should not create a new extension request if old ETA does not match the task's ETA (50ms) ✔ should not create a new extension request if an extension request for this task already exists (151ms)Additional Notes