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

feat: MAD Precompile fuzzing #858

Merged
merged 3 commits into from
Dec 27, 2024
Merged

feat: MAD Precompile fuzzing #858

merged 3 commits into from
Dec 27, 2024

Conversation

shekohex
Copy link
Contributor

@shekohex shekohex commented Dec 26, 2024

Summary of changes
Changes introduced in this pull request:
This pull request includes several updates and additions to the precompiles/multi-asset-delegation module, focusing on dependency management, feature enhancements, and code refactoring. The key changes are grouped by theme as follows:

Dependency Management and Configuration:

  • Added multiple optional dependencies to precompiles/multi-asset-delegation/Cargo.toml to support various features and improve modularity.
  • Introduced a new fuzzing feature in precompiles/multi-asset-delegation/Cargo.toml to include dependencies required for fuzz testing.
  • Created a new Cargo.toml file for the pallet-evm-precompile-multi-asset-delegation-fuzzer package to set up a fuzzer for the precompile.

Code Refactoring:

  • Changed the type of AssetId from u32 to u128 across multiple files to support larger asset identifiers. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]
  • Updated the mock and mock_evm modules to be included when either testing or fuzzing is enabled, instead of only during tests.

Feature Enhancements:

  • Added a new member precompiles/multi-asset-delegation/fuzzer to the members list in the main Cargo.toml file to include the new fuzzer package.

These changes collectively enhance the functionality, maintainability, and testability of the precompiles/multi-asset-delegation module.

Reference issue to close (if applicable)

Closes

@shekohex shekohex self-assigned this Dec 26, 2024
@shekohex shekohex marked this pull request as ready for review December 26, 2024 20:13
Cargo.toml Outdated Show resolved Hide resolved
precompiles/multi-asset-delegation/Cargo.toml Outdated Show resolved Hide resolved
@shekohex shekohex merged commit 5dc9766 into main Dec 27, 2024
3 checks passed
@shekohex shekohex deleted the shady/mad-precompile-fuzz branch December 27, 2024 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Completed ✅
Development

Successfully merging this pull request may close these issues.

2 participants