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

Implement feedback from community #43

Merged
merged 5 commits into from
Sep 1, 2024
Merged

Implement feedback from community #43

merged 5 commits into from
Sep 1, 2024

Conversation

Jac0xb
Copy link
Owner

@Jac0xb Jac0xb commented Jun 5, 2024

PR

  • New program version of program being deployed L2TExMFKdjpN9kozasaurPirfHy9P8sbXoAN1qA3S95.

  • Implement vscode scripts to step-through debug rust tests.

  • AssertAccountDataMulti instruction.

  • Add new log levels, which only log on failure:

    • FailedPlaintextMessage
    • FailedEncodedMessage
    • FailedEncodedNoop
  • Implement registry which codegens lighthouse assertions for rust/web3.js@tp4/web3.js@legacy.

    • Added vote account assertion to registry for AuthorizedWithdrawer
    • Example:
export const voteAccountTemplateArgs: AssertionTemplateArgs = {
  assertionName: 'VoteAccount',
  variants: [
    {
      name: 'AuthorizedWithdrawer',
      kind: 'Pubkey',
      assertions: [
        {
          kind: 'U8',
          value: 2,
          valueOperator: { type: 'Integer', enum: IntegerOperator.Equal },
          offset: 0,
        },
        {
          kind: 'Pubkey',
          value: DefaultValueFn,
          valueOperator: DefaultValueOperatorFn,
          offset: 36,
        },
      ],
    },
  ],
};

@Jac0xb
Copy link
Owner Author

Jac0xb commented Jul 7, 2024

Need to test:

  • [-] Vote account for typescript legacy / typescript preview
  • [-] Write tests for new assertion failures (create tests for old log levels and new ones)
  • [-] Vote account assertion
  • [-] Write test for multi account data assertion

@Jac0xb Jac0xb marked this pull request as ready for review July 8, 2024 03:03
@Jac0xb Jac0xb merged commit 24bc484 into main Sep 1, 2024
1 check 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.

1 participant