Skip to content

Commit b5e980f

Browse files
committed
fixup! Add comprehensive testing for TierStore
delegate to dynstoretrait's sync methods
1 parent 72d38ea commit b5e980f

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

tests/common/mod.rs

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -371,25 +371,28 @@ impl ForeignDynStoreTrait for TestForeignDynStoreAdapter {
371371
fn read(
372372
&self, primary_namespace: String, secondary_namespace: String, key: String,
373373
) -> Result<Vec<u8>, IOError> {
374-
self.0.read(&primary_namespace, &secondary_namespace, &key).map_err(Into::into)
374+
DynStoreTrait::read(&*self.0, &primary_namespace, &secondary_namespace, &key)
375+
.map_err(Into::into)
375376
}
376377

377378
fn write(
378379
&self, primary_namespace: String, secondary_namespace: String, key: String, buf: Vec<u8>,
379380
) -> Result<(), IOError> {
380-
self.0.write(&primary_namespace, &secondary_namespace, &key, buf).map_err(Into::into)
381+
DynStoreTrait::write(&*self.0, &primary_namespace, &secondary_namespace, &key, buf)
382+
.map_err(Into::into)
381383
}
382384

383385
fn remove(
384386
&self, primary_namespace: String, secondary_namespace: String, key: String, lazy: bool,
385387
) -> Result<(), IOError> {
386-
self.0.remove(&primary_namespace, &secondary_namespace, &key, lazy).map_err(Into::into)
388+
DynStoreTrait::remove(&*self.0, &primary_namespace, &secondary_namespace, &key, lazy)
389+
.map_err(Into::into)
387390
}
388391

389392
fn list(
390393
&self, primary_namespace: String, secondary_namespace: String,
391394
) -> Result<Vec<String>, IOError> {
392-
self.0.list(&primary_namespace, &secondary_namespace).map_err(Into::into)
395+
DynStoreTrait::list(&*self.0, &primary_namespace, &secondary_namespace).map_err(Into::into)
393396
}
394397
}
395398

0 commit comments

Comments
 (0)