Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✨ Allow mix instructions for multiple programs in workspace #200

Merged
merged 1 commit into from
Sep 14, 2024

Conversation

lukacan
Copy link
Collaborator

@lukacan lukacan commented Sep 14, 2024

Description

This PR improves the logic of generating fuzz_instructions.rs. Currently, in case of multiple programs in the workspace, each program has its module and separate FuzzInstructions. This PR merges the FuzzInstructions generated in fuzz_instructions.rs in that case, if multiple programs are present in a workspace it is possible to create instructions sequences of calling both programs not only one. Moreover, as introduced in the previous PR, the get_program_id function was implemented for IxOps trait , this ensures that the Instructions contain the program ID of their corresponding Program.

Notes:

  • In case of the same account names across multiple programs, account storage in FuzzAccounts will be generated as _ (appended snake case program name)
  • In case of the same instruction names across programs, the Instruction Variants in FuzzInstructions are generated as InstructionNameProgramName (appended as Upper Camel)

Related Tickets & Documents

  • Related Issue #

  • Closes #

  • I clicked on "Allow edits from maintainers"

@lukacan lukacan force-pushed the impr/fuzz-ixs-multiple-programs branch from 1eed6ed to 9b557f6 Compare September 14, 2024 11:44
@lukacan lukacan changed the title ✨ Improve IDL parsing logic with possibility to mix instructions from… ✨ Allow mix instructions for multiple programs in workspace Sep 14, 2024
@lukacan lukacan merged commit f62e372 into develop Sep 14, 2024
6 checks passed
@lukacan lukacan deleted the impr/fuzz-ixs-multiple-programs branch September 14, 2024 17:39
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.

1 participant