Skip to content

test: add role/function authorization matrix tests#287

Open
ogbesylvester0-gif wants to merge 1 commit into
HubDApp:mainfrom
ogbesylvester0-gif:feat/auth-matrix-tests-215
Open

test: add role/function authorization matrix tests#287
ogbesylvester0-gif wants to merge 1 commit into
HubDApp:mainfrom
ogbesylvester0-gif:feat/auth-matrix-tests-215

Conversation

@ogbesylvester0-gif

Copy link
Copy Markdown
  • Add auth_matrix.rs with 35 tests covering all mutating functions
  • Matrix covers admin, owner, maintainer, reviewer, and stranger roles
  • Assert unauthorized calls return correct errors and do not mutate state
  • Fix pre-existing bugs: |} syntax in types.rs, missing ProjectBountyUrl storage key, duplicate/misplaced bounty_url blocks in project_registry
  • Add bounty_url field to existing test struct initializers

Closes #215

- Add auth_matrix.rs with 35 tests covering all mutating functions
- Matrix covers admin, owner, maintainer, reviewer, and stranger roles
- Assert unauthorized calls return correct errors and do not mutate state
- Fix pre-existing bugs: |} syntax in types.rs, missing ProjectBountyUrl
  storage key, duplicate/misplaced bounty_url blocks in project_registry
- Add bounty_url field to existing test struct initializers

Closes HubDApp#215
@drips-wave

drips-wave Bot commented Jun 26, 2026

Copy link
Copy Markdown

@ogbesylvester0-gif 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

@Naomi-Gift

Copy link
Copy Markdown
Contributor

@ogbesylvester0-gif Please resolve conflicts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Authorization Matrix Tests

2 participants