test(persona): MEMBER Login-only ESS HTTP is 200; payroll runs 403 - #910
Draft
jason931225 wants to merge 2 commits into
Draft
test(persona): MEMBER Login-only ESS HTTP is 200; payroll runs 403#910jason931225 wants to merge 2 commits into
jason931225 wants to merge 2 commits into
Conversation
Draft. Status lock only: inbox-docs?filter=payslip and attendance/me/exceptions 200 without PayrollRunRead or AttendanceExceptionManage; payroll/runs 403. Does not emit a payslip (see #900). New sqlx binary console-app --test persona_ess_http. executed-tests-baseline.json HOLD (#905). Base 2f99f9e. Independent review before merge.
Extend member_login_only_ess_is_200_and_payroll_runs_are_403: GET /payroll/payslips/me is 200 (self-scoped readiness, no won keys; unlinked empty page ok) and GET /attendance/me/week52 is 200 (Monday; unlinked status=not_available, projection omitted). Keep payroll/runs 403 and empty feature_grants. Does not emit a payslip (#900). executed-tests-baseline.json HOLD (#905). Independent review before merge.
This was referenced Aug 26, 2026
Collaborator
Author
|
Status after working the 16-PR queue:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Draft. Independent review required before merge.
Candidate SHA:
5f79af7766ef3b5220f5317a8f9d83ae189e7763Base:
2f99f9ea26b0813e8962c22a2e74f6376530e69b(origin/main)MEMBER JWT Login-only (empty
feature_grants: noPayrollRunRead, noAttendanceExceptionManage) on the assembled router:GET /api/v1/me/inbox-docs?filter=payslip→ 200 (empty or own docs), not 403GET /api/v1/attendance/me/exceptions→ 200, not 403 (date selector is list validation, not an authz grant)GET /api/v1/payroll/runs→ 403Status lock only. Does not duplicate #900 payslip emit. Does not edit
attendance_persona_api.rs,action_inbox_api.rs, orinbox/rest/tests/api.rs. Cargo auto-discoversbackend/app/tests/persona_ess_http.rs; Cargo.toml/lock untouched.Lease:
backend/app/tests/persona_ess_http.rsonly. Disjoint from identity/directory (#897/#908/#909), canonical adapters (#898), action-inbox (#899), inbox-payslip-filter (#900), attendance personas (#903), auth cookies (#904), FE/UI (#905), and attendance CONFIRM parse (#907).Tests
discovered 1, executed 1, passed 1.
HOLDs
docs/program/executed-tests-baseline.jsonnot updated (feat(ui): Leptos 0.9 SSR payroll vertical (stacked on GATE) #905 owns it). New#[sqlx::test]is a new binaryconsole-app --test persona_ess_http; the executed-tests ratchet may require--update/ CI wiring on that lane.merge-tree --write-tree (all exit 0)
origin/mainc9d8c558e390a25ef861f715152ca5cb5d30ddece5164a420c2b572cb3dccd2b44dbf184d8100a9bBlast radius / rollback
No merge.