Skip to content

Conversation

@ordian
Copy link
Contributor

@ordian ordian commented Jan 2, 2020

This PR:

  • extract common tests into a separate crate kvdb-test-utils
  • kvdb_rocksdb::get now returns an error if the column is out of bounds to match the kvdb-memorydb implementation
  • some unit tests are updated to return an Error instead of unwrapping

Copy link
Contributor

@dvdplm dvdplm left a comment

Choose a reason for hiding this comment

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

lgtm

Maybe you could add a simple macro to reduce the boilerplate a bit? utils:impl_test!("this", "and_this_one_too");?

@ordian ordian merged commit 072d8e8 into master Jan 3, 2020
@ordian ordian deleted the ao-extract-kvdb-tests branch January 3, 2020 07:35
dvdplm added a commit that referenced this pull request Jan 14, 2020
* master: (27 commits)
  update parity-util-mem (#309)
  Update features and feature dependencies (#307)
  Use proper memory queries to rocksdb (#308)
  Draft version updates and changelog (#299)
  Use custom error type for `from_hex` (#305)
  Fix typo. (#303)
  kvdb: remove KeyValueDBHandler (#304)
  extract common kvdb tests into a crate (#301)
  Expose to_hex and from_hex from impl-serde (#302)
  Add a top level function to parity-util-mem (#298)
  I/O statistic for key-value databases (#294)
  use for_tuples (#300)
  Add memory extensions for LRUCache, hashbrown (#293)
  Add memory stats for kvdb-s (#292)
  Provide #[derive(MallocSizeOf)] that is actually working (#291)
  Move and extend impls for locking primitives (#290)
  Update uint README (#288)
  Update README.md (#287)
  [ci]: remove feature flags in virtual workspace (#289)
  Prepare releases (#286)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants