Add more context in error messages

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
This commit is contained in:
Matthias Beyer 2019-05-18 00:14:34 +02:00
parent 112a2d6af8
commit 5f07d7d21a

View file

@ -27,6 +27,7 @@
use regex::Regex; use regex::Regex;
use failure::Fallible as Result; use failure::Fallible as Result;
use failure::ResultExt;
use failure::Error; use failure::Error;
use libimagstore::store::Store; use libimagstore::store::Store;
@ -52,19 +53,28 @@ impl<'a> BookmarkCollectionStore<'a> for Store {
fn new(&'a self, name: &str) -> Result<FileLockEntry<'a>> { fn new(&'a self, name: &str) -> Result<FileLockEntry<'a>> {
crate::module_path::new_id(name) crate::module_path::new_id(name)
.and_then(|id| self.create(id).map_err(Error::from)) .and_then(|id| self.create(id)
.context("Failed to create FileLockEntry")
.map_err(Error::from))
.context("Failed to create Id for new Bookmark Collection")
.map_err(Error::from) .map_err(Error::from)
} }
fn get(&'a self, name: &str) -> Result<Option<FileLockEntry<'a>>> { fn get(&'a self, name: &str) -> Result<Option<FileLockEntry<'a>>> {
crate::module_path::new_id(name) crate::module_path::new_id(name)
.and_then(|id| self.get(id).map_err(Error::from)) .and_then(|id| self.get(id)
.context("Failed to get FileLockEntry")
.map_err(Error::from))
.context("Failed to get Bookmark Collection")
.map_err(Error::from) .map_err(Error::from)
} }
fn delete(&'a self, name: &str) -> Result<()> { fn delete(&'a self, name: &str) -> Result<()> {
crate::module_path::new_id(name) crate::module_path::new_id(name)
.and_then(|id| self.delete(id).map_err(Error::from)) .and_then(|id| self.delete(id)
.context("Failed to delete FileLockEntry")
.map_err(Error::from))
.context("Failed to delete Bookmark Collection")
.map_err(Error::from) .map_err(Error::from)
} }