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

Add a BigMap contract for testing the storage trie #295

Open
wants to merge 1 commit into
base: v2-main
Choose a base branch
from

Conversation

eljobe
Copy link
Member

@eljobe eljobe commented Jan 31, 2025

This contracts makes it easy to add a bunch of storage slots in a single transation, and then in a separate transaction to free a bunch of those slots and add some others.

The idea is that this sort of operation will pass with the deterministic flag on the storage db implementation set and fail when it is not set.

Part of NIT-3068

@cla-bot cla-bot bot added the s label Jan 31, 2025
@eljobe eljobe changed the base branch from main to pre-bold January 31, 2025 17:14
@eljobe
Copy link
Member Author

eljobe commented Jan 31, 2025

@PlasmaPower, I'm not sure what all the CI failures mean. Did I do something wrong?

@PlasmaPower
Copy link
Contributor

Some of the failures are because the contract isn't formatted correctly. I'd fix that and see what's still failing. The audit is probably just not passing on the base branch, so I think we can merge it without that one passing.

src/mocks/BigMap.sol Outdated Show resolved Hide resolved
@eljobe eljobe requested a review from PlasmaPower February 1, 2025 14:52
PlasmaPower
PlasmaPower previously approved these changes Feb 3, 2025
Copy link
Contributor

@PlasmaPower PlasmaPower left a comment

Choose a reason for hiding this comment

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

LGTM

@gzeoneth gzeoneth changed the base branch from pre-bold to v2-main February 5, 2025 15:08
@gzeoneth
Copy link
Member

gzeoneth commented Feb 5, 2025

sorry, I missed this PR when releasing 2.1.3; do we need a release tag for this mock or we are ok with just merging this into the permanent v2-main branch? I can also tag 2.1.4 if that is preferred.

@gzeoneth gzeoneth changed the base branch from v2-main to main February 5, 2025 15:19
@gzeoneth gzeoneth dismissed PlasmaPower’s stale review February 5, 2025 15:19

The base branch was changed.

@gzeoneth gzeoneth changed the base branch from main to v2-main February 5, 2025 15:19
@PlasmaPower
Copy link
Contributor

We don't need a release for this, this is just for nitro unit tests so it just needs to be in the nitro pin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants