Skip to content

feat(contract): add get_admin() and is_initialized() query functions#172

Merged
Fatimasanusi merged 3 commits into
Trust-Analysis:mainfrom
KarenZita01:feat/get-admin-is-initialized
Jun 28, 2026
Merged

feat(contract): add get_admin() and is_initialized() query functions#172
Fatimasanusi merged 3 commits into
Trust-Analysis:mainfrom
KarenZita01:feat/get-admin-is-initialized

Conversation

@KarenZita01

Copy link
Copy Markdown
Contributor

Summary

Closes #48

Adds two public query functions to the RwaMarketplace smart contract.

Changes

get_admin() -> Address

  • Returns the stored admin Address
  • Panics with "Contract not initialized: admin" if called before init()

is_initialized() -> bool

  • Returns true when DataKey::Admin exists in instance storage, false otherwise
  • Never panics

Tests (added to existing test module)

Test Covers
test_is_initialized_false_before_init returns false before init()
test_is_initialized_true_after_init returns true after init()
test_get_admin_returns_correct_address returns the address passed to init()
test_get_admin_panics_before_init panics with correct message before init()

Checklist

  • Implement get_admin() — return admin address or panic
  • Implement is_initialized() — return whether Admin key exists
  • Write unit tests

Closes Trust-Analysis#48

- get_admin(): returns admin Address, panics if not initialized
- is_initialized(): returns bool based on presence of DataKey::Admin
- 4 unit tests covering both pre- and post-init behaviour
@drips-wave

drips-wave Bot commented Jun 28, 2026

Copy link
Copy Markdown

@KarenZita01 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

@Fatimasanusi
Fatimasanusi merged commit aa30006 into Trust-Analysis:main Jun 28, 2026
0 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.

Add get_admin and is_initialized public query functions

2 participants