Skip to content

Conversation

lukacan
Copy link
Collaborator

@lukacan lukacan commented Jun 6, 2025

Description

This PR aims to add support for lamports, space, and owner attributes to the account attribute macro used within fuzz test account contexts.

It will simplify the process of creating empty data accounts expected to be created before the initialization.

Related Tickets & Documents

  • Related Issue #

  • Closes #

  • I clicked on "Allow edits from maintainers"

@lukacan lukacan requested review from 0xm4ud and hyckomatej June 6, 2025 12:20
Copy link
Contributor

@hyckomatej hyckomatej left a comment

Choose a reason for hiding this comment

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

if possible it would be nice if account types had a derive macro that would calculate space automatically since we r adding the space attribute

if constraints.space.is_some() && constraints.owner.is_none() {
return Err(ParseError::new(
proc_macro2::Span::call_site(),
"space requires non-optional storage attribute",
Copy link
Contributor

Choose a reason for hiding this comment

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

storage or owner?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

good point , both , willl fix

@lukacan lukacan requested a review from hyckomatej June 6, 2025 12:42
Copy link

@0xm4ud 0xm4ud left a comment

Choose a reason for hiding this comment

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

reviewed, lgtm!

@lukacan lukacan merged commit e182e95 into develop Jun 6, 2025
9 checks passed
@lukacan lukacan deleted the feat/account-metadata-options branch June 6, 2025 16:12
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.

3 participants