-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
area:webControllers, JSP/JSTL, view models, and UI wiring.Controllers, JSP/JSTL, view models, and UI wiring.post-mvpUse for all issues/PRs that do not belong to the MVP release. Will be implement post-MVP.Use for all issues/PRs that do not belong to the MVP release. Will be implement post-MVP.priority:P2-normalNormal priority; plan within the current milestone.Normal priority; plan within the current milestone.status:needs-testsMissing or insufficient unit/integration tests; add tests before merge.Missing or insufficient unit/integration tests; add tests before merge.status:triageNewly filed or uncategorized. Needs initial review, labeling, and priority assignment.Newly filed or uncategorized. Needs initial review, labeling, and priority assignment.
Description
Layer
Web / Controller
Scope
- Problem / context
- Ensure the Drill flow pages render and redirects occur as expected, including unauthenticated redirects.
- Proposed solution
- WebMvc tests:
- GET /drill/next when due exists → redirect to /drill/{id}
- GET /drill/{id} renders solve JSP (session user present)
- POST /drill/{id}/submit uses stub outcome, sets flash, redirects to /drill/next
- Unauthenticated attempts to Drill routes → redirected to /logIn by filter
- WebMvc tests:
- In scope
- WebMvc tests with view resolver and session attrs
- Out of scope
- Full integration tests
Acceptance criteria
- Tests cover the above scenarios and pass
Related issues/PRs
Metadata
Metadata
Assignees
Labels
area:webControllers, JSP/JSTL, view models, and UI wiring.Controllers, JSP/JSTL, view models, and UI wiring.post-mvpUse for all issues/PRs that do not belong to the MVP release. Will be implement post-MVP.Use for all issues/PRs that do not belong to the MVP release. Will be implement post-MVP.priority:P2-normalNormal priority; plan within the current milestone.Normal priority; plan within the current milestone.status:needs-testsMissing or insufficient unit/integration tests; add tests before merge.Missing or insufficient unit/integration tests; add tests before merge.status:triageNewly filed or uncategorized. Needs initial review, labeling, and priority assignment.Newly filed or uncategorized. Needs initial review, labeling, and priority assignment.
Projects
Status
Todo