refactor

Author: marvin-j97

src/merge.rs

@@ -9,9 +9,10 @@ type IterItem = crate::Result<InternalValue>;
 
 pub type BoxedIterator<'a> = Box<dyn DoubleEndedIterator<Item = IterItem> + Send + 'a>;
 
-#[derive(Eq)]
 struct HeapItem(usize, InternalValue);
 
+impl Eq for HeapItem {}
+
 impl PartialEq for HeapItem {
     fn eq(&self, other: &Self) -> bool {
         self.1.key == other.1.key

src/table/id.rs

@@ -26,3 +26,19 @@ impl From<(TreeId, TableId)> for GlobalTableId {
         Self(tid, sid)
     }
 }
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+    use test_log::test;
+
+    #[test]
+    fn global_table_id_accessors() {
+        let tree_id = 42;
+        let table_id: TableId = 7;
+        let global_table_id = GlobalTableId::from((tree_id, table_id));
+
+        assert_eq!(global_table_id.tree_id(), 42);
+        assert_eq!(global_table_id.table_id(), 7);
+    }
+}

src/value.rs

@@ -22,7 +22,7 @@ pub type UserValue = Slice;
 pub type SeqNo = u64;
 
 /// Internal representation of KV pairs
-#[derive(Clone, Eq)]
+#[derive(Clone)]
 pub struct InternalValue {
     /// Internal key
     pub key: InternalKey,
@@ -93,6 +93,8 @@ impl InternalValue {
     }
 }
 
+#[cfg(test)]
+#[cfg_attr(coverage_nightly, coverage(off))]
 impl PartialEq for InternalValue {
     fn eq(&self, other: &Self) -> bool {
         self.key == other.key