code review

Signed-off-by: Cyrill Leutwiler <[email protected]>

crates/storage/src/lazy/vec.rs

@@ -206,6 +206,8 @@ where
     /// trigger additional storage reads.
     #[inline]
     pub fn len(&self) -> u32 {
+        debug_assert!(self.len_cached.is_none() || self.len.get() == self.len_cached);
+
         self.len_cached
             .unwrap_or_else(|| self.len.get().unwrap_or(u32::MIN))
     }
@@ -444,6 +446,7 @@ where
         f.debug_struct("StorageVec")
             .field("key", &KeyType::KEY)
             .field("len", &self.len)
+            .field("len_cached", &self.len_cached)
             .finish()
     }
 }

integration-tests/lazyvec-integration-test/lib.rs

@@ -40,7 +40,8 @@ mod lazyvec_integration_tests {
         /// Checks whether given account is allowed to vote and didn't already
         /// participate.
         fn is_eligible(&self, _voter: AccountId) -> bool {
-            // todo
+            // ToDo: In production, the contract would actually verify eligible voters.
+            // For example, a merkle proof could be an efficient way to do this.
             true
         }