Skip to content

test(progress): cover revoked delegate retake auth - #680

Merged
DeFiVC merged 1 commit into
ChainLearnOfficial:mainfrom
charlesedeh021-cell:test/revoked-delegate-retake-589
Oct 21, 2026
Merged

DeFiVC merged 1 commit into
ChainLearnOfficial:mainfrom
charlesedeh021-cell:test/revoked-delegate-retake-589

Conversation

@charlesedeh021-cell

Copy link
Copy Markdown
Contributor

Closes #586
Closes #587
Closes #588
Closes #589

Summary

This PR strengthens the progress-tracker delegation regression coverage by verifying that a delegate loses all progress-authority capabilities immediately after the learner revokes delegation.

Issue #589: Delegation authorization coverage

  • Added a regression test covering retake_quiz_for after delegation is revoked.
  • The test creates a learner and delegate, records an initial quiz score, grants delegation, revokes it, and then attempts an improved score through the former delegate.
  • The call is asserted to fail, confirming that revocation is enforced by the shared learner-or-delegate authorization guard rather than only by the other progress entry points.
  • The test keeps the learner's existing progress valid before exercising the authorization failure, so the failure is specifically caused by the revoked caller.

Assigned issue references

The other assigned ChainLearn issues are included here for tracking alongside this contribution: #586, #587, and #588.

Validation

No local tests, builds, or installations were run, per request.

@drips-wave

drips-wave Bot commented Oct 21, 2026

Copy link
Copy Markdown

@charlesedeh021-cell Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@DeFiVC
DeFiVC merged commit 07ade15 into ChainLearnOfficial:main Oct 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants