Skip to content

Conversation

@rustopian
Copy link

@rustopian rustopian commented Oct 31, 2025

REPLACED BY #161

@rustopian rustopian force-pushed the mollusk-tests-1-initialize branch from 9034445 to 2d518cd Compare October 31, 2025 13:49
_ => &default_checks,
};

let test_missing_signers = self.test_missing_signers.unwrap_or(true);
Copy link
Author

@rustopian rustopian Nov 1, 2025

Choose a reason for hiding this comment

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

The idea here is to make test-writing safer. test_missing_signers, where the instruction is run with every signer missing to ensure MissingRequiredSignature, is now the default, and must be explicitly disabled with .test_missing_signers(false)

This prevents a test writer from simply forgetting to test all missing signers. .test_missing_signers(true) is still used to keep test intentions transparent.

StakeTestContext to reduce boilerplate

InstructionConfig builder pattern

improve InitializeChecked coverage
@rustopian rustopian force-pushed the mollusk-tests-1-initialize branch from 7329649 to 58d7c99 Compare November 1, 2025 13:43
@rustopian rustopian changed the title Mollusk tests, part 1: Initialize Mollusk tests, part 1: Initialize, harness Nov 1, 2025
@rustopian rustopian changed the title Mollusk tests, part 1: Initialize, harness Mollusk tests, part 1: Initialize, InitializeChecked, harness Nov 3, 2025
@rustopian rustopian force-pushed the mollusk-tests-1-initialize branch from 326ffd0 to 6d33056 Compare November 4, 2025 00:24
@rustopian rustopian closed this Nov 4, 2025
@rustopian rustopian deleted the mollusk-tests-1-initialize branch November 4, 2025 09:33
@rustopian rustopian changed the title Mollusk tests, part 1: Initialize, InitializeChecked, harness [REPLACED] Mollusk tests, part 1: Initialize, InitializeChecked, harness Nov 4, 2025
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.

2 participants