Fix #174 (#178)

* Add a regression test * Fix the bug, I think * CHANGELOG * Tweak comment * Update test proofs * Add the expected values * CHANGELOG PR number * Tweak CHANGELOG a bit
smol-dot · Feb 13, 2023 · 94f4488 · 94f4488
1 parent 30744cc
commit 94f4488
Show file tree

Hide file tree

Showing 4 changed files with 14,535 additions and 7 deletions.
diff --git a/bin/wasm-node/CHANGELOG.md b/bin/wasm-node/CHANGELOG.md
@@ -8,6 +8,7 @@
 
 ### Fixed
 
+- Fix `state_getKeys` and `state_getKeysPaged` missing entries under certain conditions. ([#178](https://github.com/smol-dot/smoldot/pull/178))
 - The alternative spellings `relayChain` and `paraId` for the `relay_chain` and `para_id` fields in chain specifications are now properly accepted as intended. ([#160](https://github.com/smol-dot/smoldot/pull/160))
 
 ## 0.7.10 - 2022-02-10

diff --git a/src/trie/prefix_proof.rs b/src/trie/prefix_proof.rs
@@ -32,6 +32,8 @@ use super::{nibble, proof_decode};
 use alloc::{vec, vec::Vec};
 use core::{fmt, iter, mem};
 
+mod tests;
+
 /// Configuration to pass to [`prefix_scan`].
 pub struct Config<'a> {
     /// Prefix that all the keys must share.
@@ -188,5 +190,3 @@ pub enum Error {
     /// One or more entries in the proof are missing.
     MissingProofEntry,
 }
-
-// TODO: needs tests