Skip to content

test(contracts): add reputation profile tests#208

Merged
Kaylahray merged 6 commits into
Quid-proquo:mainfrom
nonchalanttee22-lgtm:contracts/reputation-profile-tests
Jun 4, 2026
Merged

test(contracts): add reputation profile tests#208
Kaylahray merged 6 commits into
Quid-proquo:mainfrom
nonchalanttee22-lgtm:contracts/reputation-profile-tests

Conversation

@nonchalanttee22-lgtm

Copy link
Copy Markdown
Contributor
  • Add quid-reputation contract package (Cargo.toml, error.rs, types.rs, lib.rs)
  • Add src/test.rs with 18 tests covering:
    • Admin bootstrap: set_admin and get_admin flows
    • Profile upsert: create, overwrite, owner assertion, auth guard
    • increment_success: single, multiple, missing profile, auth guard
    • record_rejection: single, multiple, missing profile, auth guard
    • Combined: full lifecycle and independent profile isolation

Closes #199

- Add quid-reputation contract package (Cargo.toml, error.rs, types.rs, lib.rs)
- Add src/test.rs with 18 tests covering:
  - Admin bootstrap: set_admin and get_admin flows
  - Profile upsert: create, overwrite, owner assertion, auth guard
  - increment_success: single, multiple, missing profile, auth guard
  - record_rejection: single, multiple, missing profile, auth guard
  - Combined: full lifecycle and independent profile isolation

Closes Quid-proquo#199
@drips-wave

drips-wave Bot commented Jun 1, 2026

Copy link
Copy Markdown

@nonchalanttee22-lgtm 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

@Kaylahray Kaylahray left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @nonchalanttee22-lgtm Please resolve conflicts n workflow

@Kaylahray

Copy link
Copy Markdown
Contributor

@nonchalanttee22-lgtm

@Kaylahray

Copy link
Copy Markdown
Contributor

Knock Knock @nonchalanttee22-lgtm

@nonchalanttee22-lgtm

Copy link
Copy Markdown
Contributor Author

yes yes sorry solving em

@Kaylahray Kaylahray merged commit 068050e into Quid-proquo:main Jun 4, 2026
2 of 3 checks passed
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.

Reputation profile mutation tests

3 participants