Skip to content

Test prefix removal in the trie cache#2666

Open
tomaka wants to merge 1 commit into
paritytech:mainfrom
tomaka:test-remove-prefix
Open

Test prefix removal in the trie cache#2666
tomaka wants to merge 1 commit into
paritytech:mainfrom
tomaka:test-remove-prefix

Conversation

@tomaka

@tomaka tomaka commented Aug 19, 2022

Copy link
Copy Markdown
Contributor

Tweaks the trie cache test in order to test prefix removal operations.

As I'm opening this PR, the test is failing, and it clearly indicates a bug in the trie cache.
Needs fixing.

@mergify mergify Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Automatically approving tomaka's pull requests. This auto-approval will be removed once more maintainers are active.

@github-actions

Copy link
Copy Markdown
Contributor

twiggy diff report

Difference in .wasm size before and after this pull request.


 Delta Bytes │ Item
─────────────┼──────────────────
          +0 ┊ Σ [0 Total Rows]

@melekes melekes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

mergify Bot added a commit that referenced this pull request Sep 5, 2022
Adds some tests regarding the code added in #2663

I wanted to maybe figure out why #2666 was failing, but didn't manage to find a failing case.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
@dmitry-markin
dmitry-markin force-pushed the main branch 2 times, most recently from dffa75f to f471baa Compare March 16, 2026 14:19
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