//
// imag - the personal information management suite for the commandline
// Copyright (C) 2015, 2016 Matthias Beyer <mail@beyermatthias.de> and contributors
//
// This library is free software; you can redistribute it and/or
// modify it under the terms of the GNU Lesser General Public
// License as published by the Free Software Foundation; version
// 2.1 of the License.
//
// This library is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
// Lesser General Public License for more details.
//
// You should have received a copy of the GNU Lesser General Public
// License along with this library; if not, write to the Free Software
// Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
//

use std::collections::HashMap;
use std::ops::Drop;
use std::path::PathBuf;
use std::result::Result as RResult;
use std::sync::Arc;
use std::sync::RwLock;
use std::collections::BTreeMap;
use std::io::Read;
use std::convert::From;
use std::convert::Into;
use std::sync::Mutex;
use std::ops::Deref;
use std::ops::DerefMut;
use std::fmt::Formatter;
use std::fmt::Debug;
use std::fmt::Error as FMTError;

use toml::{Table, Value};
use regex::Regex;
use glob::glob;
use walkdir::WalkDir;
use walkdir::Iter as WalkDirIter;

use error::{ParserErrorKind, ParserError};
use error::{StoreError as SE, StoreErrorKind as SEK};
use error::MapErrInto;
use storeid::{IntoStoreId, StoreId, StoreIdIterator};
use file_abstraction::FileAbstraction;
use toml_ext::*;

use hook::aspect::Aspect;
use hook::error::HookErrorKind;
use hook::result::HookResult;
use hook::accessor::{ MutableHookDataAccessor,
            StoreIdAccessor};
use hook::position::HookPosition;
use hook::Hook;

use libimagerror::into::IntoError;
use libimagerror::trace::trace_error;
use libimagutil::iter::FoldResult;
use libimagutil::debug_result::*;

use self::glob_store_iter::*;

/// The Result Type returned by any interaction with the store that could fail
pub type Result<T> = RResult<T, SE>;


#[derive(Debug, PartialEq)]
enum StoreEntryStatus {
    Present,
    Borrowed
}

/// A store entry, depending on the option type it is either borrowed currently
/// or not.
#[derive(Debug)]
struct StoreEntry {
    id: StoreId,
    file: FileAbstraction,
    status: StoreEntryStatus,
}

pub enum StoreObject {
    Id(StoreId),
    Collection(PathBuf),
}

pub struct Walk {
    store_path: PathBuf,
    dirwalker: WalkDirIter,
}

impl Walk {

    fn new(mut store_path: PathBuf, mod_name: &str) -> Walk {
        let pb = store_path.clone();
        store_path.push(mod_name);
        Walk {
            store_path: pb,
            dirwalker: WalkDir::new(store_path).into_iter(),
        }
    }
}

impl ::std::ops::Deref for Walk {
    type Target = WalkDirIter;

    fn deref(&self) -> &Self::Target {
        &self.dirwalker
    }
}

impl Iterator for Walk {
    type Item = StoreObject;

    fn next(&mut self) -> Option<Self::Item> {
        while let Some(something) = self.dirwalker.next() {
            match something {
                Ok(next) => if next.file_type().is_dir() {
                                return Some(StoreObject::Collection(next.path().to_path_buf()))
                            } else if next.file_type().is_file() {
                                let n   = next.path().to_path_buf();
                                let sid = match StoreId::new(Some(self.store_path.clone()), n) {
                                    Err(e) => {
                                        trace_error(&e);
                                        continue;
                                    },
                                    Ok(o) => o,
                                };
                                return Some(StoreObject::Id(sid))
                            },
                Err(e) => {
                    warn!("Error in Walker");
                    debug!("{:?}", e);
                    return None;
                }
            }
        }

        return None;
    }
}


impl StoreEntry {

    fn new(id: StoreId) -> Result<StoreEntry> {
        let pb = try!(id.clone().into_pathbuf());
        Ok(StoreEntry {
            id: id,
            file: FileAbstraction::Absent(pb),
            status: StoreEntryStatus::Present,
        })
    }

    /// The entry is currently borrowed, meaning that some thread is currently
    /// mutating it
    fn is_borrowed(&self) -> bool {
        self.status == StoreEntryStatus::Borrowed
    }

    fn get_entry(&mut self) -> Result<Entry> {
        let id = &self.id.clone();
        if !self.is_borrowed() {
            self.file
                .get_file_content()
                .and_then(|mut file| Entry::from_reader(id.clone(), &mut file))
                .or_else(|err| if err.err_type() == SEK::FileNotFound {
                    Ok(Entry::new(id.clone()))
                } else {
                    Err(err)
                })
        } else {
            Err(SE::new(SEK::EntryAlreadyBorrowed, None))
        }
    }

    fn write_entry(&mut self, entry: &Entry) -> Result<()> {
        if self.is_borrowed() {
            assert_eq!(self.id, entry.location);
            self.file.write_file_content(entry.to_str().as_bytes())
                .map_err_into(SEK::FileError)
                .map(|_| ())
        } else {
            Ok(())
        }
    }
}

/// The Store itself, through this object one can interact with IMAG's entries
pub struct Store {
    location: PathBuf,

    /**
     * Configuration object of the store
     */
    configuration: Option<Value>,

    /*
     * Registered hooks
     */

    store_unload_aspects  : Arc<Mutex<Vec<Aspect>>>,

    pre_create_aspects    : Arc<Mutex<Vec<Aspect>>>,
    post_create_aspects   : Arc<Mutex<Vec<Aspect>>>,
    pre_retrieve_aspects  : Arc<Mutex<Vec<Aspect>>>,
    post_retrieve_aspects : Arc<Mutex<Vec<Aspect>>>,
    pre_update_aspects    : Arc<Mutex<Vec<Aspect>>>,
    post_update_aspects   : Arc<Mutex<Vec<Aspect>>>,
    pre_delete_aspects    : Arc<Mutex<Vec<Aspect>>>,
    post_delete_aspects   : Arc<Mutex<Vec<Aspect>>>,
    pre_move_aspects      : Arc<Mutex<Vec<Aspect>>>,
    post_move_aspects     : Arc<Mutex<Vec<Aspect>>>,

    /**
     * Internal Path->File cache map
     *
     * Caches the files, so they remain flock()ed
     *
     * Could be optimized for a threadsafe HashMap
     */
    entries: Arc<RwLock<HashMap<StoreId, StoreEntry>>>,
}

impl Store {

    /// Create a new Store object
    pub fn new(location: PathBuf, store_config: Option<Value>) -> Result<Store> {
        use configuration::*;

        debug!("Validating Store configuration");
        let _ = try!(config_is_valid(&store_config).map_err_into(SEK::ConfigurationError));

        debug!("Building new Store object");
        if !location.exists() {
            if !config_implicit_store_create_allowed(store_config.as_ref()) {
                warn!("Implicitely creating store directory is denied");
                warn!(" -> Either because configuration does not allow it");
                warn!(" -> or because there is no configuration");
                return Err(SEK::CreateStoreDirDenied.into_error())
                    .map_err_into(SEK::FileError)
                    .map_err_into(SEK::IoError);
            }

            try!(FileAbstraction::create_dir_all(&location)
                 .map_err_into(SEK::StorePathCreate)
                 .map_dbg_err_str("Failed"));
        } else if location.is_file() {
            debug!("Store path exists as file");
            return Err(SEK::StorePathExists.into_error());
        }

        let store_unload_aspects = get_store_unload_aspect_names(&store_config)
            .into_iter().map(|n| {
                let cfg = AspectConfig::get_for(&store_config, n.clone());
                Aspect::new(n, cfg)
            }).collect();

        let pre_create_aspects = get_pre_create_aspect_names(&store_config)
            .into_iter().map(|n| {
                let cfg = AspectConfig::get_for(&store_config, n.clone());
                Aspect::new(n, cfg)
            }).collect();

        let post_create_aspects = get_post_create_aspect_names(&store_config)
            .into_iter().map(|n| {
                let cfg = AspectConfig::get_for(&store_config, n.clone());
                Aspect::new(n, cfg)
            }).collect();

        let pre_retrieve_aspects = get_pre_retrieve_aspect_names(&store_config)
            .into_iter().map(|n| {
                let cfg = AspectConfig::get_for(&store_config, n.clone());
                Aspect::new(n, cfg)
            }).collect();

        let post_retrieve_aspects = get_post_retrieve_aspect_names(&store_config)
            .into_iter().map(|n| {
                let cfg = AspectConfig::get_for(&store_config, n.clone());
                Aspect::new(n, cfg)
            }).collect();

        let pre_update_aspects = get_pre_update_aspect_names(&store_config)
            .into_iter().map(|n| {
                let cfg = AspectConfig::get_for(&store_config, n.clone());
                Aspect::new(n, cfg)
            }).collect();

        let post_update_aspects = get_post_update_aspect_names(&store_config)
            .into_iter().map(|n| {
                let cfg = AspectConfig::get_for(&store_config, n.clone());
                Aspect::new(n, cfg)
            }).collect();

        let pre_delete_aspects = get_pre_delete_aspect_names(&store_config)
            .into_iter().map(|n| {
                let cfg = AspectConfig::get_for(&store_config, n.clone());
                Aspect::new(n, cfg)
            }).collect();

        let post_delete_aspects = get_post_delete_aspect_names(&store_config)
            .into_iter().map(|n| {
                let cfg = AspectConfig::get_for(&store_config, n.clone());
                Aspect::new(n, cfg)
            }).collect();

        let pre_move_aspects = get_pre_move_aspect_names(&store_config)
            .into_iter().map(|n| {
                let cfg = AspectConfig::get_for(&store_config, n.clone());
                Aspect::new(n, cfg)
            }).collect();

        let post_move_aspects = get_post_move_aspect_names(&store_config)
            .into_iter().map(|n| {
                let cfg = AspectConfig::get_for(&store_config, n.clone());
                Aspect::new(n, cfg)
            }).collect();

        let store = Store {
            location: location.clone(),
            configuration: store_config,

            store_unload_aspects  : Arc::new(Mutex::new(store_unload_aspects)),

            pre_create_aspects    : Arc::new(Mutex::new(pre_create_aspects)),
            post_create_aspects   : Arc::new(Mutex::new(post_create_aspects)),
            pre_retrieve_aspects  : Arc::new(Mutex::new(pre_retrieve_aspects)),
            post_retrieve_aspects : Arc::new(Mutex::new(post_retrieve_aspects)),
            pre_update_aspects    : Arc::new(Mutex::new(pre_update_aspects)),
            post_update_aspects   : Arc::new(Mutex::new(post_update_aspects)),
            pre_delete_aspects    : Arc::new(Mutex::new(pre_delete_aspects)),
            post_delete_aspects   : Arc::new(Mutex::new(post_delete_aspects)),
            pre_move_aspects    : Arc::new(Mutex::new(pre_move_aspects)),
            post_move_aspects   : Arc::new(Mutex::new(post_move_aspects)),
            entries: Arc::new(RwLock::new(HashMap::new())),
        };

        debug!("Store building succeeded");
        debug!("------------------------");
        debug!("{:?}", store);
        debug!("------------------------");

        Ok(store)
    }

    /// Get the store configuration
    pub fn config(&self) -> Option<&Value> {
        self.configuration.as_ref()
    }

    /// Verify the store.
    ///
    /// This function is not intended to be called by normal programs but only by `imag-store`.
    #[cfg(feature = "verify")]
    pub fn verify(&self) -> bool {
        info!("Header | Content length | Path");
        info!("-------+----------------+-----");

        WalkDir::new(self.location.clone())
            .into_iter()
            .map(|res| {
                match res {
                    Ok(dent) => {
                        if dent.file_type().is_file() {
                            match self.get(PathBuf::from(dent.path())) {
                                Ok(Some(fle)) => {
                                    let p           = fle.get_location();
                                    let content_len = fle.get_content().len();
                                    let header      = if fle.get_header().verify().is_ok() {
                                        "ok"
                                    } else {
                                        "broken"
                                    };

                                    info!("{: >6} | {: >14} | {:?}", header, content_len, p.deref());
                                },

                                Ok(None) => {
                                    info!("{: >6} | {: >14} | {:?}", "?", "couldn't load", dent.path());
                                },

                                Err(e) => {
                                    debug!("{:?}", e);
                                },
                            }
                        } else {
                            info!("{: >6} | {: >14} | {:?}", "?", "<no file>", dent.path());
                        }
                    },

                    Err(e) => {
                        debug!("{:?}", e);
                    },
                }

                true
            })
            .all(|b| b)
    }

    /// Creates the Entry at the given location (inside the entry)
    pub fn create<'a, S: IntoStoreId>(&'a self, id: S) -> Result<FileLockEntry<'a>> {
        let id = try!(id.into_storeid()).with_base(self.path().clone());
        if let Err(e) = self.execute_hooks_for_id(self.pre_create_aspects.clone(), &id) {
            return Err(e)
                .map_err_into(SEK::PreHookExecuteError)
                .map_err_into(SEK::HookExecutionError)
                .map_err_into(SEK::CreateCallError)
        }

        {
            let mut hsmap = match self.entries.write() {
                Err(_) => return Err(SEK::LockPoisoned.into_error()).map_err_into(SEK::CreateCallError),
                Ok(s) => s,
            };

            if hsmap.contains_key(&id) {
                return Err(SEK::EntryAlreadyExists.into_error()).map_err_into(SEK::CreateCallError);
            }
            hsmap.insert(id.clone(), {
                let mut se = try!(StoreEntry::new(id.clone()));
                se.status = StoreEntryStatus::Borrowed;
                se
            });
        }

        let mut fle = FileLockEntry::new(self, Entry::new(id));
        self.execute_hooks_for_mut_file(self.post_create_aspects.clone(), &mut fle)
            .map_err_into(SEK::PostHookExecuteError)
            .map_err_into(SEK::HookExecutionError)
            .map_err_into(SEK::CreateCallError)
            .map(|_| fle)
    }

    /// Borrow a given Entry. When the `FileLockEntry` is either `update`d or
    /// dropped, the new Entry is written to disk
    ///
    /// Implicitely creates a entry in the store if there is no entry with the id `id`. For a
    /// non-implicitely-create look at `Store::get`.
    pub fn retrieve<'a, S: IntoStoreId>(&'a self, id: S) -> Result<FileLockEntry<'a>> {
        let id = try!(id.into_storeid()).with_base(self.path().clone());
        if let Err(e) = self.execute_hooks_for_id(self.pre_retrieve_aspects.clone(), &id) {
            return Err(e)
                .map_err_into(SEK::PreHookExecuteError)
                .map_err_into(SEK::HookExecutionError)
                .map_err_into(SEK::RetrieveCallError)
        }

        let entry = try!({
            self.entries
                .write()
                .map_err(|_| SE::new(SEK::LockPoisoned, None))
                .and_then(|mut es| {
                    let new_se = try!(StoreEntry::new(id.clone()));
                    let mut se = es.entry(id.clone()).or_insert(new_se);
                    let entry = se.get_entry();
                    se.status = StoreEntryStatus::Borrowed;
                    entry
                })
                .map_err_into(SEK::RetrieveCallError)
        });

        let mut fle = FileLockEntry::new(self, entry);
        self.execute_hooks_for_mut_file(self.post_retrieve_aspects.clone(), &mut fle)
            .map_err_into(SEK::PostHookExecuteError)
            .map_err_into(SEK::HookExecutionError)
            .map_err_into(SEK::RetrieveCallError)
            .and(Ok(fle))
    }

    /// Get an entry from the store if it exists.
    ///
    /// This executes the {pre,post}_retrieve_aspects hooks.
    pub fn get<'a, S: IntoStoreId + Clone>(&'a self, id: S) -> Result<Option<FileLockEntry<'a>>> {
        let id = try!(id.into_storeid()).with_base(self.path().clone());

        let exists = try!(self.entries
            .read()
            .map(|map| map.contains_key(&id))
            .map_err(|_| SE::new(SEK::LockPoisoned, None))
            .map_err_into(SEK::GetCallError)
        );

        if !exists && !id.exists() {
            debug!("Does not exist in internal cache or filesystem: {:?}", id);
            return Ok(None);
        }

        self.retrieve(id).map(Some).map_err_into(SEK::GetCallError)
    }

    /// Iterate over all StoreIds for one module name
    pub fn retrieve_for_module(&self, mod_name: &str) -> Result<StoreIdIterator> {
        let mut path = self.path().clone();
        path.push(mod_name);

        path.to_str()
            .ok_or(SE::new(SEK::EncodingError, None))
            .and_then(|path| {
                let path = [ path, "/**/*" ].join("");
                debug!("glob()ing with '{}'", path);
                glob(&path[..]).map_err_into(SEK::GlobError)
            })
            .map(|paths| GlobStoreIdIterator::new(paths, self.path().clone()).into())
            .map_err_into(SEK::GlobError)
            .map_err_into(SEK::RetrieveForModuleCallError)
    }

    // Walk the store tree for the module
    pub fn walk<'a>(&'a self, mod_name: &str) -> Walk {
        Walk::new(self.path().clone(), mod_name)
    }

    /// Return the `FileLockEntry` and write to disk
    pub fn update<'a>(&'a self, mut entry: FileLockEntry<'a>) -> Result<()> {
        self._update(&mut entry, false).map_err_into(SEK::UpdateCallError)
    }

    /// Internal method to write to the filesystem store.
    ///
    /// # Assumptions
    /// This method assumes that entry is dropped _right after_ the call, hence
    /// it is not public.
    fn _update<'a>(&'a self, mut entry: &mut FileLockEntry<'a>, modify_presence: bool) -> Result<()> {
        let _ = try!(self.execute_hooks_for_mut_file(self.pre_update_aspects.clone(), &mut entry)
            .map_err_into(SEK::PreHookExecuteError)
            .map_err_into(SEK::HookExecutionError)
            .map_err_into(SEK::UpdateCallError)
        );

        let mut hsmap = match self.entries.write() {
            Err(_) => return Err(SE::new(SEK::LockPoisoned, None)),
            Ok(e) => e,
        };

        let mut se = try!(hsmap.get_mut(&entry.location).ok_or(SE::new(SEK::IdNotFound, None)));

        assert!(se.is_borrowed(), "Tried to update a non borrowed entry.");

        debug!("Verifying Entry");
        try!(entry.entry.verify());

        debug!("Writing Entry");
        try!(se.write_entry(&entry.entry));
        if modify_presence {
            se.status = StoreEntryStatus::Present;
        }


        self.execute_hooks_for_mut_file(self.post_update_aspects.clone(), &mut entry)
            .map_err_into(SEK::PostHookExecuteError)
            .map_err_into(SEK::HookExecutionError)
            .map_err_into(SEK::UpdateCallError)
    }

    /// Retrieve a copy of a given entry, this cannot be used to mutate
    /// the one on disk
    pub fn retrieve_copy<S: IntoStoreId>(&self, id: S) -> Result<Entry> {
        let id = try!(id.into_storeid()).with_base(self.path().clone());
        let entries = match self.entries.write() {
            Err(_) => {
                return Err(SE::new(SEK::LockPoisoned, None))
                    .map_err_into(SEK::RetrieveCopyCallError);
            },
            Ok(e) => e,
        };

        // if the entry is currently modified by the user, we cannot drop it
        if entries.get(&id).map(|e| e.is_borrowed()).unwrap_or(false) {
            return Err(SE::new(SEK::IdLocked, None)).map_err_into(SEK::RetrieveCopyCallError);
        }

        try!(StoreEntry::new(id)).get_entry()
    }

    /// Delete an entry
    pub fn delete<S: IntoStoreId>(&self, id: S) -> Result<()> {
        let id = try!(id.into_storeid()).with_base(self.path().clone());
        if let Err(e) = self.execute_hooks_for_id(self.pre_delete_aspects.clone(), &id) {
            return Err(e)
                .map_err_into(SEK::PreHookExecuteError)
                .map_err_into(SEK::HookExecutionError)
                .map_err_into(SEK::DeleteCallError)
        }

        {
            let mut entries = match self.entries.write() {
                Err(_) => return Err(SE::new(SEK::LockPoisoned, None))
                    .map_err_into(SEK::DeleteCallError),
                Ok(e) => e,
            };

            // if the entry is currently modified by the user, we cannot drop it
            match entries.get(&id) {
                None => {
                    return Err(SEK::FileNotFound.into_error()).map_err_into(SEK::DeleteCallError)
                },
                Some(e) => if e.is_borrowed() {
                    return Err(SE::new(SEK::IdLocked, None)).map_err_into(SEK::DeleteCallError)
                }
            }

            // remove the entry first, then the file
            entries.remove(&id);
            let pb = try!(id.clone().with_base(self.path().clone()).into_pathbuf());
            if let Err(e) = FileAbstraction::remove_file(&pb) {
                return Err(SEK::FileError.into_error_with_cause(Box::new(e)))
                    .map_err_into(SEK::DeleteCallError);
            }
        }

        self.execute_hooks_for_id(self.post_delete_aspects.clone(), &id)
            .map_err_into(SEK::PostHookExecuteError)
            .map_err_into(SEK::HookExecutionError)
            .map_err_into(SEK::DeleteCallError)
    }

    /// Save a copy of the Entry in another place
    /// Executes the post_move_aspects for the new id
    pub fn save_to(&self, entry: &FileLockEntry, new_id: StoreId) -> Result<()> {
        self.save_to_other_location(entry, new_id, false)
    }

    /// Save an Entry in another place
    /// Removes the original entry
    /// Executes the post_move_aspects for the new id
    pub fn save_as(&self, entry: FileLockEntry, new_id: StoreId) -> Result<()> {
        self.save_to_other_location(&entry, new_id, true)
    }

    fn save_to_other_location(&self, entry: &FileLockEntry, new_id: StoreId, remove_old: bool)
        -> Result<()>
    {
        let new_id = new_id.with_base(self.path().clone());
        let hsmap = try!(
            self.entries
                .write()
                .map_err(|_| SEK::LockPoisoned.into_error())
                .map_err_into(SEK::MoveCallError)
        );

        if hsmap.contains_key(&new_id) {
            return Err(SEK::EntryAlreadyExists.into_error()).map_err_into(SEK::MoveCallError)
        }

        let old_id = entry.get_location().clone();

        let old_id_as_path = try!(old_id.clone().with_base(self.path().clone()).into_pathbuf());
        let new_id_as_path = try!(new_id.clone().with_base(self.path().clone()).into_pathbuf());
        FileAbstraction::copy(&old_id_as_path, &new_id_as_path)
            .and_then(|_| {
                if remove_old {
                    FileAbstraction::remove_file(&old_id_as_path)
                } else {
                    Ok(())
                }
            })
            .map_err_into(SEK::FileError)
            .and_then(|_| self.execute_hooks_for_id(self.post_move_aspects.clone(), &new_id)
                    .map_err_into(SEK::PostHookExecuteError)
                    .map_err_into(SEK::HookExecutionError))
            .map_err_into(SEK::MoveCallError)
    }

    /// Move an entry without loading
    ///
    /// This function moves an entry from one path to another.
    ///
    /// Generally, this function shouldn't be used by library authors, if they "just" want to move
    /// something around. A library for moving entries while caring about meta-data and links.
    ///
    /// # Errors
    ///
    /// This function returns an error in certain cases:
    ///
    /// * If pre-move-hooks error (if they return an error which indicates that the action should be
    ///   aborted)
    /// * If the about-to-be-moved entry is borrowed
    /// * If the lock on the internal data structure cannot be aquired
    /// * If the new path already exists
    /// * If the about-to-be-moved entry does not exist
    /// * If the FS-operation failed
    /// * If the post-move-hooks error (though the operation has succeeded then).
    ///
    /// # Warnings
    ///
    /// This should be used with _great_ care, as moving an entry from `a` to `b` might result in
    /// dangling links (see below).
    ///
    /// ## Moving linked entries
    ///
    /// If the entry which is moved is linked to another entry, these links get invalid (but we do
    /// not detect this here). As links are always two-way-links, so `a` is not only linked to `b`,
    /// but also the other way round, moving `b` to `c` results in the following scenario:
    ///
    /// * `a` links to `b`, which does not exist anymore.
    /// * `c` links to `a`, which does exist.
    ///
    /// So the link is _partly dangling_, so to say.
    ///
    pub fn move_by_id(&self, old_id: StoreId, new_id: StoreId) -> Result<()> {
        let new_id = new_id.with_base(self.path().clone());
        let old_id = old_id.with_base(self.path().clone());

        if let Err(e) = self.execute_hooks_for_id(self.pre_move_aspects.clone(), &old_id) {
            return Err(e)
                .map_err_into(SEK::PreHookExecuteError)
                .map_err_into(SEK::HookExecutionError)
                .map_err_into(SEK::MoveByIdCallError)
        }

        {
            let mut hsmap = match self.entries.write() {
                Err(_) => return Err(SE::new(SEK::LockPoisoned, None)),
                Ok(m)  => m,
            };

            if hsmap.contains_key(&new_id) {
                return Err(SEK::EntryAlreadyExists.into_error());
            }

            // if we do not have an entry here, we fail in `FileAbstraction::rename()` below.
            // if we have one, but it is borrowed, we really should not rename it, as this might
            // lead to strange errors
            if hsmap.get(&old_id).map(|e| e.is_borrowed()).unwrap_or(false) {
                return Err(SEK::EntryAlreadyBorrowed.into_error());
            }

            let old_id_pb = try!(old_id.clone().with_base(self.path().clone()).into_pathbuf());
            let new_id_pb = try!(new_id.clone().with_base(self.path().clone()).into_pathbuf());

            match FileAbstraction::rename(&old_id_pb, &new_id_pb) {
                Err(e) => return Err(SEK::EntryRenameError.into_error_with_cause(Box::new(e))),
                Ok(_) => {
                    debug!("Rename worked on filesystem");

                    // assert enforced through check hsmap.contains_key(&new_id) above.
                    // Should therefor never fail
                    assert!(hsmap
                            .remove(&old_id)
                            .and_then(|mut entry| {
                                entry.id = new_id.clone();
                                hsmap.insert(new_id.clone(), entry)
                            }).is_none())
                }
            }

        }

        self.execute_hooks_for_id(self.pre_move_aspects.clone(), &new_id)
            .map_err_into(SEK::PostHookExecuteError)
            .map_err_into(SEK::HookExecutionError)
            .map_err_into(SEK::MoveByIdCallError)
    }

    /// Gets the path where this store is on the disk
    pub fn path(&self) -> &PathBuf {
        &self.location
    }

    pub fn register_hook(&mut self,
                         position: HookPosition,
                         aspect_name: &str,
                         mut h: Box<Hook>)
        -> Result<()>
    {
        debug!("Registering hook: {:?}", h);
        debug!("     in position: {:?}", position);
        debug!("     with aspect: {:?}", aspect_name);

        let guard = match position {
                HookPosition::StoreUnload  => self.store_unload_aspects.clone(),

                HookPosition::PreCreate    => self.pre_create_aspects.clone(),
                HookPosition::PostCreate   => self.post_create_aspects.clone(),
                HookPosition::PreRetrieve  => self.pre_retrieve_aspects.clone(),
                HookPosition::PostRetrieve => self.post_retrieve_aspects.clone(),
                HookPosition::PreUpdate    => self.pre_update_aspects.clone(),
                HookPosition::PostUpdate   => self.post_update_aspects.clone(),
                HookPosition::PreDelete    => self.pre_delete_aspects.clone(),
                HookPosition::PostDelete   => self.post_delete_aspects.clone(),
            };

        let mut guard = match guard.deref().lock().map_err(|_| SE::new(SEK::LockError, None)) {
            Err(e) => return Err(SEK::HookRegisterError.into_error_with_cause(Box::new(e))),
            Ok(g) => g,
        };

        for mut aspect in guard.deref_mut() {
            if aspect.name().clone() == aspect_name.clone() {
                debug!("Trying to find configuration for hook: {:?}", h);
                self.get_config_for_hook(h.name()).map(|config| h.set_config(config));
                debug!("Trying to register hook in aspect: {:?} <- {:?}", aspect, h);
                aspect.register_hook(h);
                return Ok(());
            }
        }

        let annfe = SEK::AspectNameNotFoundError.into_error();
        Err(SEK::HookRegisterError.into_error_with_cause(Box::new(annfe)))
    }

    fn get_config_for_hook(&self, name: &str) -> Option<&Value> {
        match self.configuration {
            Some(Value::Table(ref tabl)) => {
                debug!("Trying to head 'hooks' section from {:?}", tabl);
                tabl.get("hooks")
                    .map(|hook_section| {
                        debug!("Found hook section:  {:?}", hook_section);
                        debug!("Reading section key: {:?}", name);
                        match *hook_section {
                            Value::Table(ref tabl) => tabl.get(name),
                            _ => None
                        }
                    })
                    .unwrap_or(None)
            },
            _ => None,
        }
    }

    fn execute_hooks_for_id(&self,
                            aspects: Arc<Mutex<Vec<Aspect>>>,
                            id: &StoreId)
        -> HookResult<()>
    {
        match aspects.lock() {
            Err(_) => return Err(HookErrorKind::HookExecutionError.into()),
            Ok(g) => g
        }.iter().fold_defresult(|aspect| {
            debug!("[Aspect][exec]: {:?}", aspect);
            (aspect as &StoreIdAccessor).access(id)
        }).map_err(Box::new)
            .map_err(|e| HookErrorKind::HookExecutionError.into_error_with_cause(e))
    }

    fn execute_hooks_for_mut_file(&self,
                                  aspects: Arc<Mutex<Vec<Aspect>>>,
                                  fle: &mut FileLockEntry)
        -> HookResult<()>
    {
        match aspects.lock() {
            Err(_) => return Err(HookErrorKind::HookExecutionError.into()),
            Ok(g) => g
        }.iter().fold_defresult(|aspect| {
            debug!("[Aspect][exec]: {:?}", aspect);
            aspect.access_mut(fle)
        }).map_err(Box::new)
            .map_err(|e| HookErrorKind::HookExecutionError.into_error_with_cause(e))
    }

}

impl Debug for Store {

    fn fmt(&self, fmt: &mut Formatter) -> RResult<(), FMTError> {
        try!(write!(fmt, " --- Store ---\n"));
        try!(write!(fmt, "\n"));
        try!(write!(fmt, " - location               : {:?}\n", self.location));
        try!(write!(fmt, " - configuration          : {:?}\n", self.configuration));
        try!(write!(fmt, " - pre_create_aspects     : {:?}\n", self.pre_create_aspects    ));
        try!(write!(fmt, " - post_create_aspects    : {:?}\n", self.post_create_aspects   ));
        try!(write!(fmt, " - pre_retrieve_aspects   : {:?}\n", self.pre_retrieve_aspects  ));
        try!(write!(fmt, " - post_retrieve_aspects  : {:?}\n", self.post_retrieve_aspects ));
        try!(write!(fmt, " - pre_update_aspects     : {:?}\n", self.pre_update_aspects    ));
        try!(write!(fmt, " - post_update_aspects    : {:?}\n", self.post_update_aspects   ));
        try!(write!(fmt, " - pre_delete_aspects     : {:?}\n", self.pre_delete_aspects    ));
        try!(write!(fmt, " - post_delete_aspects    : {:?}\n", self.post_delete_aspects   ));
        try!(write!(fmt, "\n"));
        try!(write!(fmt, "Entries:\n"));
        try!(write!(fmt, "{:?}", self.entries));
        try!(write!(fmt, "\n"));
        Ok(())
    }

}

impl Drop for Store {

    /**
     * Unlock all files on drop
     *
     * TODO: Unlock them
     */
    fn drop(&mut self) {
        match StoreId::new(Some(self.location.clone()), PathBuf::from(".")) {
            Err(e) => {
                trace_error(&e);
                warn!("Cannot construct StoreId for Store to execute hooks!");
                warn!("Will close Store without executing hooks!");
            },
            Ok(store_id) => {
                if let Err(e) = self.execute_hooks_for_id(self.store_unload_aspects.clone(), &store_id) {
                    debug!("Store-load hooks execution failed. Cannot create store object.");
                    warn!("Store Unload Hook error: {:?}", e);
                }
            },
        };

        debug!("Dropping store");
    }

}

/// A struct that allows you to borrow an Entry
pub struct FileLockEntry<'a> {
    store: &'a Store,
    entry: Entry,
}

impl<'a> FileLockEntry<'a, > {
    fn new(store: &'a Store, entry: Entry) -> FileLockEntry<'a> {
        FileLockEntry {
            store: store,
            entry: entry,
        }
    }
}

impl<'a> Debug for FileLockEntry<'a> {
    fn fmt(&self, fmt: &mut Formatter) -> RResult<(), FMTError> {
        write!(fmt, "FileLockEntry(Store = {})", self.store.location.to_str()
               .unwrap_or("Unknown Path"))
    }
}

impl<'a> Deref for FileLockEntry<'a> {
    type Target = Entry;

    fn deref(&self) -> &Self::Target {
        &self.entry
    }
}

impl<'a> DerefMut for FileLockEntry<'a> {
    fn deref_mut(&mut self) -> &mut Self::Target {
        &mut self.entry
    }
}

#[cfg(not(test))]
impl<'a> Drop for FileLockEntry<'a> {
    /// This will silently ignore errors, use `Store::update` if you want to catch the errors
    fn drop(&mut self) {
        let _ = self.store._update(self, true);
    }
}

#[cfg(test)]
impl<'a> Drop for FileLockEntry<'a> {
    /// This will not silently ignore errors but prints the result of the _update() call for testing
    fn drop(&mut self) {
        let _ = self.store._update(self, true).map_err(|e| trace_error(&e));
    }
}


/// `EntryContent` type
pub type EntryContent = String;

/// `EntryHeader`
///
/// This is basically a wrapper around `toml::Table` which provides convenience to the user of the
/// library.
#[derive(Debug, Clone, PartialEq)]
pub struct EntryHeader(Value);

pub type EntryResult<V> = RResult<V, ParserError>;

/**
 * Wrapper type around file header (TOML) object
 */
impl EntryHeader {

    pub fn new() -> EntryHeader {
        EntryHeader(build_default_header())
    }

    pub fn header(&self) -> &Value {
        &self.0
    }

    fn from_table(t: Table) -> EntryHeader {
        EntryHeader(Value::Table(t))
    }

    pub fn parse(s: &str) -> EntryResult<EntryHeader> {
        use toml::Parser;

        let mut parser = Parser::new(s);
        parser.parse()
            .ok_or(ParserErrorKind::TOMLParserErrors.into())
            .and_then(verify_header_consistency)
            .map(EntryHeader::from_table)
    }

    pub fn verify(&self) -> Result<()> {
        match self.0 {
            Value::Table(ref t) => verify_header(&t),
            _ => Err(SE::new(SEK::HeaderTypeFailure, None)),
        }
    }

    #[inline]
    pub fn insert_with_sep(&mut self, spec: &str, sep: char, v: Value) -> Result<bool> {
        self.0.insert_with_sep(spec, sep, v)
    }

    #[inline]
    pub fn set_with_sep(&mut self, spec: &str, sep: char, v: Value) -> Result<Option<Value>> {
        self.0.set_with_sep(spec, sep, v)
    }

    #[inline]
    pub fn read_with_sep(&self, spec: &str, splitchr: char) -> Result<Option<Value>> {
        self.0.read_with_sep(spec, splitchr)
    }

    #[inline]
    pub fn delete_with_sep(&mut self, spec: &str, splitchr: char) -> Result<Option<Value>> {
        self.0.delete_with_sep(spec, splitchr)
    }

    #[inline]
    pub fn insert(&mut self, spec: &str, v: Value) -> Result<bool> {
        self.0.insert(spec, v)
    }

    #[inline]
    pub fn set(&mut self, spec: &str, v: Value) -> Result<Option<Value>> {
        self.0.set(spec, v)
    }

    #[inline]
    pub fn read(&self, spec: &str) -> Result<Option<Value>> {
        self.0.read(spec)
    }

    #[inline]
    pub fn delete(&mut self, spec: &str) -> Result<Option<Value>> {
        self.0.delete(spec)
    }

}

impl Into<Table> for EntryHeader {

    fn into(self) -> Table {
        match self.0 {
            Value::Table(t) => t,
            _ => panic!("EntryHeader is not a table!"),
        }
    }

}

impl From<Table> for EntryHeader {

    fn from(t: Table) -> EntryHeader {
        EntryHeader(Value::Table(t))
    }

}

fn build_default_header() -> Value { // BTreeMap<String, Value>
    let mut m = BTreeMap::new();

    m.insert(String::from("imag"), {
        let mut imag_map = BTreeMap::<String, Value>::new();

        imag_map.insert(String::from("version"), Value::String(String::from(version!())));
        imag_map.insert(String::from("links"), Value::Array(vec![]));

        Value::Table(imag_map)
    });

    Value::Table(m)
}
fn verify_header(t: &Table) -> Result<()> {
    if !has_main_section(t) {
        Err(SE::from(ParserErrorKind::MissingMainSection.into_error()))
    } else if !has_imag_version_in_main_section(t) {
        Err(SE::from(ParserErrorKind::MissingVersionInfo.into_error()))
    } else if !has_only_tables(t) {
        debug!("Could not verify that it only has tables in its base table");
        Err(SE::from(ParserErrorKind::NonTableInBaseTable.into_error()))
    } else {
        Ok(())
    }
}

fn verify_header_consistency(t: Table) -> EntryResult<Table> {
    verify_header(&t)
        .map_err(Box::new)
        .map_err(|e| ParserErrorKind::HeaderInconsistency.into_error_with_cause(e))
        .map(|_| t)
}

fn has_only_tables(t: &Table) -> bool {
    debug!("Verifying that table has only tables");
    t.iter().all(|(_, x)| if let Value::Table(_) = *x { true } else { false })
}

fn has_main_section(t: &Table) -> bool {
    t.contains_key("imag") &&
        match t.get("imag") {
            Some(&Value::Table(_)) => true,
            Some(_)                => false,
            None                   => false,
        }
}

fn has_imag_version_in_main_section(t: &Table) -> bool {
    use semver::Version;

    match *t.get("imag").unwrap() {
        Value::Table(ref sec) => {
            sec.get("version")
                .and_then(|v| {
                    match *v {
                        Value::String(ref s) => Some(Version::parse(&s[..]).is_ok()),
                        _                    => Some(false),
                    }
                })
            .unwrap_or(false)
        }
        _ => false,
    }
}

/**
 * An Entry of the store
 *
 * Contains location, header and content part.
 */
#[derive(Debug, Clone)]
pub struct Entry {
    location: StoreId,
    header: EntryHeader,
    content: EntryContent,
}

impl Entry {

    pub fn new(loc: StoreId) -> Entry {
        Entry {
            location: loc,
            header: EntryHeader::new(),
            content: EntryContent::new()
        }
    }

    pub fn from_reader<S: IntoStoreId>(loc: S, file: &mut Read) -> Result<Entry> {
        let text = {
            let mut s = String::new();
            try!(file.read_to_string(&mut s));
            s
        };
        Self::from_str(loc, &text[..])
    }

    pub fn from_str<S: IntoStoreId>(loc: S, s: &str) -> Result<Entry> {
        debug!("Building entry from string");
        lazy_static! {
            static ref RE: Regex = Regex::new(r"(?smx)
                ^---$
                (?P<header>.*) # Header
                ^---$\n
                (?P<content>.*) # Content
            ").unwrap();
        }

        let matches = match RE.captures(s) {
            None    => return Err(SE::new(SEK::MalformedEntry, None)),
            Some(s) => s,
        };

        let header = match matches.name("header") {
            None    => return Err(SE::new(SEK::MalformedEntry, None)),
            Some(s) => s
        };

        let content = matches.name("content").unwrap_or("");

        debug!("Header and content found. Yay! Building Entry object now");
        Ok(Entry {
            location: try!(loc.into_storeid()),
            header: try!(EntryHeader::parse(header)),
            content: content.into(),
        })
    }

    pub fn to_str(&self) -> String {
        format!("---\n{header}---\n{content}",
                header  = ::toml::encode_str(&self.header.0),
                content = self.content)
    }

    pub fn get_location(&self) -> &StoreId {
        &self.location
    }

    pub fn get_header(&self) -> &EntryHeader {
        &self.header
    }

    pub fn get_header_mut(&mut self) -> &mut EntryHeader {
        &mut self.header
    }

    pub fn get_content(&self) -> &EntryContent {
        &self.content
    }

    pub fn get_content_mut(&mut self) -> &mut EntryContent {
        &mut self.content
    }

    pub fn verify(&self) -> Result<()> {
        self.header.verify()
    }

}

impl PartialEq for Entry {

    fn eq(&self, other: &Entry) -> bool {
        self.location == other.location && // As the location only compares from the store root
            self.header == other.header && // and the other Entry could be from another store (not
            self.content == other.content  // implemented by now, but we think ahead here)
    }

}

mod glob_store_iter {
    use std::fmt::{Debug, Formatter};
    use std::fmt::Error as FmtError;
    use std::path::PathBuf;
    use glob::Paths;
    use storeid::StoreId;
    use storeid::StoreIdIterator;

    use error::StoreErrorKind as SEK;
    use error::MapErrInto;

    use libimagerror::trace::trace_error;

    pub struct GlobStoreIdIterator {
        store_path: PathBuf,
        paths: Paths,
    }

    impl Debug for GlobStoreIdIterator {

        fn fmt(&self, fmt: &mut Formatter) -> Result<(), FmtError> {
            write!(fmt, "GlobStoreIdIterator")
        }

    }

    impl Into<StoreIdIterator> for GlobStoreIdIterator {

        fn into(self) -> StoreIdIterator {
            StoreIdIterator::new(Box::new(self))
        }

    }

    impl GlobStoreIdIterator {

        pub fn new(paths: Paths, store_path: PathBuf) -> GlobStoreIdIterator {
            debug!("Create a GlobStoreIdIterator(store_path = {:?}, /* ... */)", store_path);

            GlobStoreIdIterator {
                store_path: store_path,
                paths: paths,
            }
        }

    }

    impl Iterator for GlobStoreIdIterator {
        type Item = StoreId;

        fn next(&mut self) -> Option<StoreId> {
            self.paths
                .next()
                .and_then(|o| {
                    debug!("GlobStoreIdIterator::next() => {:?}", o);
                    o.map_err_into(SEK::StoreIdHandlingError)
                        .and_then(|p| StoreId::from_full_path(&self.store_path, p))
                        .map_err(|e| {
                            debug!("GlobStoreIdIterator error: {:?}", e);
                            trace_error(&e);
                        }).ok()
                })
        }

    }

}


#[cfg(test)]
mod test {
    extern crate env_logger;

    use std::collections::BTreeMap;
    use storeid::StoreId;

    use toml::Value;

    #[test]
    fn test_imag_section() {
        use super::has_main_section;

        let mut map = BTreeMap::new();
        map.insert("imag".into(), Value::Table(BTreeMap::new()));

        assert!(has_main_section(&map));
    }

    #[test]
    fn test_imag_invalid_section_type() {
        use super::has_main_section;

        let mut map = BTreeMap::new();
        map.insert("imag".into(), Value::Boolean(false));

        assert!(!has_main_section(&map));
    }

    #[test]
    fn test_imag_abscent_main_section() {
        use super::has_main_section;

        let mut map = BTreeMap::new();
        map.insert("not_imag".into(), Value::Boolean(false));

        assert!(!has_main_section(&map));
    }

    #[test]
    fn test_main_section_without_version() {
        use super::has_imag_version_in_main_section;

        let mut map = BTreeMap::new();
        map.insert("imag".into(), Value::Table(BTreeMap::new()));

        assert!(!has_imag_version_in_main_section(&map));
    }

    #[test]
    fn test_main_section_with_version() {
        use super::has_imag_version_in_main_section;

        let mut map = BTreeMap::new();
        let mut sub = BTreeMap::new();
        sub.insert("version".into(), Value::String("0.0.0".into()));
        map.insert("imag".into(), Value::Table(sub));

        assert!(has_imag_version_in_main_section(&map));
    }

    #[test]
    fn test_main_section_with_version_in_wrong_type() {
        use super::has_imag_version_in_main_section;

        let mut map = BTreeMap::new();
        let mut sub = BTreeMap::new();
        sub.insert("version".into(), Value::Boolean(false));
        map.insert("imag".into(), Value::Table(sub));

        assert!(!has_imag_version_in_main_section(&map));
    }

    #[test]
    fn test_verification_good() {
        use super::verify_header_consistency;

        let mut header = BTreeMap::new();
        let sub = {
            let mut sub = BTreeMap::new();
            sub.insert("version".into(), Value::String(String::from("0.0.0")));

            Value::Table(sub)
        };

        header.insert("imag".into(), sub);

        assert!(verify_header_consistency(header).is_ok());
    }

    #[test]
    fn test_verification_invalid_versionstring() {
        use super::verify_header_consistency;

        let mut header = BTreeMap::new();
        let sub = {
            let mut sub = BTreeMap::new();
            sub.insert("version".into(), Value::String(String::from("000")));

            Value::Table(sub)
        };

        header.insert("imag".into(), sub);

        assert!(!verify_header_consistency(header).is_ok());
    }


    #[test]
    fn test_verification_current_version() {
        use super::verify_header_consistency;

        let mut header = BTreeMap::new();
        let sub = {
            let mut sub = BTreeMap::new();
            sub.insert("version".into(), Value::String(String::from(version!())));

            Value::Table(sub)
        };

        header.insert("imag".into(), sub);

        assert!(verify_header_consistency(header).is_ok());
    }

    static TEST_ENTRY : &'static str = "---
[imag]
version = \"0.0.3\"
---
Hai";

    #[test]
    fn test_entry_from_str() {
        use super::Entry;
        use std::path::PathBuf;
        println!("{}", TEST_ENTRY);
        let entry = Entry::from_str(StoreId::new_baseless(PathBuf::from("test/foo~1.3")).unwrap(),
                                    TEST_ENTRY).unwrap();

        assert_eq!(entry.content, "Hai");
    }

    #[test]
    fn test_entry_to_str() {
        use super::Entry;
        use std::path::PathBuf;
        println!("{}", TEST_ENTRY);
        let entry = Entry::from_str(StoreId::new_baseless(PathBuf::from("test/foo~1.3")).unwrap(),
                                    TEST_ENTRY).unwrap();
        let string = entry.to_str();

        assert_eq!(TEST_ENTRY, string);
    }

}

#[cfg(test)]
mod store_tests {
    use std::path::PathBuf;

    use super::Store;

    pub fn get_store() -> Store {
        Store::new(PathBuf::from("/"), None).unwrap()
    }

    #[test]
    fn test_store_instantiation() {
        let store = get_store();

        assert_eq!(store.location, PathBuf::from("/"));
        assert!(store.entries.read().unwrap().is_empty());

        assert!(store.store_unload_aspects.lock().unwrap().is_empty());

        assert!(store.pre_create_aspects.lock().unwrap().is_empty());
        assert!(store.post_create_aspects.lock().unwrap().is_empty());
        assert!(store.pre_retrieve_aspects.lock().unwrap().is_empty());
        assert!(store.post_retrieve_aspects.lock().unwrap().is_empty());
        assert!(store.pre_update_aspects.lock().unwrap().is_empty());
        assert!(store.post_update_aspects.lock().unwrap().is_empty());
        assert!(store.pre_delete_aspects.lock().unwrap().is_empty());
        assert!(store.post_delete_aspects.lock().unwrap().is_empty());
        assert!(store.pre_move_aspects.lock().unwrap().is_empty());
        assert!(store.post_move_aspects.lock().unwrap().is_empty());
    }

    #[test]
    fn test_store_create() {
        let store = get_store();

        for n in 1..100 {
            let s = format!("test-{}", n);
            let entry = store.create(PathBuf::from(s.clone())).unwrap();
            assert!(entry.verify().is_ok());
            let loc = entry.get_location().clone().into_pathbuf().unwrap();
            assert!(loc.starts_with("/"));
            assert!(loc.ends_with(s));
        }
    }

    #[test]
    fn test_store_get_create_get_delete_get() {
        let store = get_store();

        for n in 1..100 {
            let res = store.get(PathBuf::from(format!("test-{}", n)));
            assert!(match res { Ok(None) => true, _ => false, })
        }

        for n in 1..100 {
            let s = format!("test-{}", n);
            let entry = store.create(PathBuf::from(s.clone())).unwrap();

            assert!(entry.verify().is_ok());

            let loc = entry.get_location().clone().into_pathbuf().unwrap();

            assert!(loc.starts_with("/"));
            assert!(loc.ends_with(s));
        }

        for n in 1..100 {
            let res = store.get(PathBuf::from(format!("test-{}", n)));
            assert!(match res { Ok(Some(_)) => true, _ => false, })
        }

        for n in 1..100 {
            assert!(store.delete(PathBuf::from(format!("test-{}", n))).is_ok())
        }

        for n in 1..100 {
            let res = store.get(PathBuf::from(format!("test-{}", n)));
            assert!(match res { Ok(None) => true, _ => false, })
        }
    }

    #[test]
    fn test_store_create_twice() {
        use error::StoreErrorKind as SEK;

        let store = get_store();

        for n in 1..100 {
            let s = format!("test-{}", n % 50);
            store.create(PathBuf::from(s.clone()))
                .map_err(|e| assert!(is_match!(e.err_type(), SEK::CreateCallError) && n >= 50))
                .ok()
                .map(|entry| {
                    assert!(entry.verify().is_ok());
                    let loc = entry.get_location().clone().into_pathbuf().unwrap();
                    assert!(loc.starts_with("/"));
                    assert!(loc.ends_with(s));
                });
        }
    }

    #[test]
    fn test_store_create_in_hm() {
        use storeid::StoreId;

        let store = get_store();

        for n in 1..100 {
            let pb = StoreId::new_baseless(PathBuf::from(format!("test-{}", n))).unwrap();

            assert!(store.entries.read().unwrap().get(&pb).is_none());
            assert!(store.create(pb.clone()).is_ok());

            let pb = pb.with_base(store.path().clone());
            assert!(store.entries.read().unwrap().get(&pb).is_some());
        }
    }

    #[test]
    fn test_store_retrieve_in_hm() {
        use storeid::StoreId;

        let store = get_store();

        for n in 1..100 {
            let pb = StoreId::new_baseless(PathBuf::from(format!("test-{}", n))).unwrap();

            assert!(store.entries.read().unwrap().get(&pb).is_none());
            assert!(store.retrieve(pb.clone()).is_ok());

            let pb = pb.with_base(store.path().clone());
            assert!(store.entries.read().unwrap().get(&pb).is_some());
        }
    }

    #[test]
    fn test_get_none() {
        let store = get_store();

        for n in 1..100 {
            match store.get(PathBuf::from(format!("test-{}", n))) {
                Ok(None) => assert!(true),
                _        => assert!(false),
            }
        }
    }

    #[test]
    fn test_delete_none() {
        let store = get_store();

        for n in 1..100 {
            match store.delete(PathBuf::from(format!("test-{}", n))) {
                Err(_) => assert!(true),
                _      => assert!(false),
            }
        }
    }

    // Disabled because we cannot test this by now, as we rely on glob() in
    // Store::retieve_for_module(), which accesses the filesystem and tests run in-memory, so there
    // are no files on the filesystem in this test after Store::create().
    //
    // #[test]
    // fn test_retrieve_for_module() {
    //     let pathes = vec![
    //         "foo/1", "foo/2", "foo/3", "foo/4", "foo/5",
    //         "bar/1", "bar/2", "bar/3", "bar/4", "bar/5",
    //         "bla/1", "bla/2", "bla/3", "bla/4", "bla/5",
    //         "boo/1", "boo/2", "boo/3", "boo/4", "boo/5",
    //         "glu/1", "glu/2", "glu/3", "glu/4", "glu/5",
    //     ];

    //     fn test(store: &Store, modulename: &str) {
    //         use std::path::Component;
    //         use storeid::StoreId;

    //         let retrieved = store.retrieve_for_module(modulename);
    //         assert!(retrieved.is_ok());
    //         let v : Vec<StoreId> = retrieved.unwrap().collect();
    //         println!("v = {:?}", v);
    //         assert!(v.len() == 5);

    //         let retrieved = store.retrieve_for_module(modulename);
    //         assert!(retrieved.is_ok());

    //         assert!(retrieved.unwrap().all(|e| {
    //             let first = e.components().next();
    //             assert!(first.is_some());
    //             match first.unwrap() {
    //                 Component::Normal(s) => s == modulename,
    //                 _                    => false,
    //             }
    //         }))
    //     }

    //     let store = get_store();
    //     for path in pathes {
    //         assert!(store.create(PathBuf::from(path)).is_ok());
    //     }

    //     test(&store, "foo");
    //     test(&store, "bar");
    //     test(&store, "bla");
    //     test(&store, "boo");
    //     test(&store, "glu");
    // }

    #[test]
    fn test_store_move_moves_in_hm() {
        use storeid::StoreId;

        let store = get_store();

        for n in 1..100 {
            if n % 2 == 0 { // every second
                let id    = StoreId::new_baseless(PathBuf::from(format!("t-{}", n))).unwrap();
                let id_mv = StoreId::new_baseless(PathBuf::from(format!("t-{}", n - 1))).unwrap();

                {
                    assert!(store.entries.read().unwrap().get(&id).is_none());
                }

                {
                    assert!(store.create(id.clone()).is_ok());
                }

                {
                    let id_with_base = id.clone().with_base(store.path().clone());
                    assert!(store.entries.read().unwrap().get(&id_with_base).is_some());
                }

                let r = store.move_by_id(id.clone(), id_mv.clone());
                assert!(r.map_err(|e| println!("ERROR: {:?}", e)).is_ok());

                {
                    let id_mv_with_base = id_mv.clone().with_base(store.path().clone());
                    assert!(store.entries.read().unwrap().get(&id_mv_with_base).is_some());
                }

                assert!(match store.get(id.clone()) { Ok(None) => true, _ => false },
                        "Moved id ({:?}) is still there", id);
                assert!(match store.get(id_mv.clone()) { Ok(Some(_)) => true, _ => false },
                        "New id ({:?}) is not in store...", id_mv);
            }
        }
    }

}

#[cfg(test)]
mod store_hook_tests {

    mod test_hook {
        use hook::Hook;
        use hook::accessor::HookDataAccessor;
        use hook::accessor::HookDataAccessorProvider;
        use hook::position::HookPosition;

        use self::accessor::TestHookAccessor as DHA;

        use toml::Value;

        #[derive(Debug)]
        pub struct TestHook {
            position: HookPosition,
            accessor: DHA,
        }

        impl TestHook {

            pub fn new(pos: HookPosition, succeed: bool, error_aborting: bool) -> TestHook {
                TestHook { position: pos.clone(), accessor: DHA::new(pos, succeed, error_aborting) }
            }

        }

        impl Hook for TestHook {
            fn name(&self) -> &'static str { "testhook_succeeding" }
            fn set_config(&mut self, _: &Value) { }
        }

        impl HookDataAccessorProvider for TestHook {

            fn accessor(&self) -> HookDataAccessor {
                use hook::position::HookPosition as HP;
                use hook::accessor::HookDataAccessor as HDA;

                match self.position {
                    HP::StoreUnload  |
                    HP::PreCreate    |
                    HP::PreRetrieve  |
                    HP::PreDelete    |
                    HP::PostDelete   => HDA::StoreIdAccess(&self.accessor),
                    HP::PostCreate   |
                    HP::PostRetrieve |
                    HP::PreUpdate    |
                    HP::PostUpdate   => HDA::MutableAccess(&self.accessor),
                }
            }

        }

        pub mod accessor {
            use hook::result::HookResult;
            use hook::accessor::MutableHookDataAccessor;
            use hook::accessor::NonMutableHookDataAccessor;
            use hook::accessor::StoreIdAccessor;
            use hook::position::HookPosition;
            use store::FileLockEntry;
            use storeid::StoreId;
            use hook::error::HookErrorKind as HEK;
            use hook::error::CustomData;
            use libimagerror::into::IntoError;

            #[derive(Debug)]
            pub struct TestHookAccessor {
                pos: HookPosition,
                succeed: bool,
                error_aborting: bool
            }

            impl TestHookAccessor {

                pub fn new(position: HookPosition, succeed: bool, error_aborting: bool)
                    -> TestHookAccessor
                {
                    TestHookAccessor {
                        pos: position,
                        succeed: succeed,
                        error_aborting: error_aborting,
                    }
                }

            }

            fn get_result(succeed: bool, abort: bool) -> HookResult<()> {
                println!("Generting result: succeed = {}, abort = {}", succeed, abort);
                if succeed {
                    println!("Generating result: Ok(())");
                    Ok(())
                } else {
                    if abort {
                        println!("Generating result: Err(_), aborting");
                        Err(HEK::HookExecutionError.into_error())
                    } else {
                        println!("Generating result: Err(_), not aborting");
                        let custom = CustomData::default().aborting(false);
                        Err(HEK::HookExecutionError.into_error().with_custom_data(custom))
                    }
                }
            }

            impl StoreIdAccessor for TestHookAccessor {

                fn access(&self, _: &StoreId) -> HookResult<()> {
                    get_result(self.succeed, self.error_aborting)
                }

            }

            impl MutableHookDataAccessor for TestHookAccessor {

                fn access_mut(&self, _: &mut FileLockEntry) -> HookResult<()> {
                    get_result(self.succeed, self.error_aborting)
                }

            }

            impl NonMutableHookDataAccessor for TestHookAccessor {

                fn access(&self, _: &FileLockEntry) -> HookResult<()> {
                    get_result(self.succeed, self.error_aborting)
                }

            }

        }

    }

    use std::path::PathBuf;

    use hook::position::HookPosition as HP;
    use storeid::StoreId;
    use store::Store;

    use self::test_hook::TestHook;

    fn get_store_with_config() -> Store {
        use toml::Parser;

        let cfg = Parser::new(mini_config()).parse().unwrap();
        println!("Config parsed: {:?}", cfg);
        Store::new(PathBuf::from("/"), Some(cfg.get("store").cloned().unwrap())).unwrap()
    }

    fn mini_config() -> &'static str {
        r#"
[store]
store-unload-hook-aspects  = [ "test" ]
pre-create-hook-aspects    = [ "test" ]
post-create-hook-aspects   = [ "test" ]
pre-move-hook-aspects      = [ "test" ]
post-move-hook-aspects     = [ "test" ]
pre-retrieve-hook-aspects  = [ "test" ]
post-retrieve-hook-aspects = [ "test" ]
pre-update-hook-aspects    = [ "test" ]
post-update-hook-aspects   = [ "test" ]
pre-delete-hook-aspects    = [ "test" ]
post-delete-hook-aspects   = [ "test" ]

[store.aspects.test]
parallel = false
mutable_hooks = true

[store.hooks.testhook_succeeding]
aspect = "test"
        "#
    }

    fn test_hook_execution(hook_positions: &[HP], storeid_name: &str) {
        let mut store = get_store_with_config();
        let pos       = HP::PreCreate;

        println!("Registering hooks...");
        for pos in hook_positions {
            let hook = TestHook::new(pos.clone(), true, false);
            println!("\tRegistering: {:?}", pos);
            assert!(store.register_hook(pos.clone(), "test", Box::new(hook))
                    .map_err(|e| println!("{:?}", e))
                    .is_ok()
            );
        }
        println!("... done.");

        let pb       = StoreId::new_baseless(PathBuf::from(storeid_name)).unwrap();
        let pb_moved = StoreId::new_baseless(PathBuf::from(format!("{}-moved", storeid_name))).unwrap();

        println!("Creating {:?}", pb);
        assert!(store.create(pb.clone()).is_ok());

        {
            println!("Getting {:?} -> Some?", pb);
            assert!(match store.get(pb.clone()) {
                Ok(Some(_)) => true,
                _           => false,
            });
        }

        {
            println!("Getting {:?} -> None?", pb_moved);
            assert!(match store.get(pb_moved.clone()) {
                Ok(None) => true,
                _        => false,
            });
        }

        {
            println!("Moving {:?} -> {:?}", pb, pb_moved);
            assert!(store.move_by_id(pb.clone(), pb_moved.clone()).map_err(|e| println!("ERROR MOVING: {:?}", e)).is_ok());
        }

        {
            println!("Getting {:?} -> None", pb);
            assert!(match store.get(pb.clone()) {
                Ok(None) => true,
                _        => false,
            });
        }

        {
            println!("Getting {:?} -> Some", pb_moved);
            assert!(match store.get(pb_moved.clone()) {
                Ok(Some(_)) => true,
                _           => false,
            });
        }

        {
            println!("Getting {:?} -> Some -> updating", pb_moved);
            assert!(match store.get(pb_moved.clone()).map_err(|e| println!("ERROR GETTING: {:?}", e)) {
                Ok(Some(fle)) => store.update(fle).map_err(|e| println!("ERROR UPDATING: {:?}", e)).is_ok(),
                _             => false,
            });
        }

        println!("Deleting {:?}", pb_moved);
        assert!(store.delete(pb_moved).is_ok());
    }

    #[test]
    fn test_storeunload() {
        test_hook_execution(&[HP::StoreUnload], "test_storeunload");
    }

    #[test]
    fn test_precreate() {
        test_hook_execution(&[HP::PreCreate], "test_precreate");
    }

    #[test]
    fn test_postcreate() {
        test_hook_execution(&[HP::PostCreate], "test_postcreate");
    }

    #[test]
    fn test_preretrieve() {
        test_hook_execution(&[HP::PreRetrieve], "test_preretrieve");
    }

    #[test]
    fn test_postretrieve() {
        test_hook_execution(&[HP::PostRetrieve], "test_postretrieve");
    }

    #[test]
    fn test_preupdate() {
        test_hook_execution(&[HP::PreUpdate], "test_preupdate");
    }

    #[test]
    fn test_postupdate() {
        test_hook_execution(&[HP::PostUpdate], "test_postupdate");
    }

    #[test]
    fn test_predelete() {
        test_hook_execution(&[HP::PreDelete], "test_predelete");
    }

    #[test]
    fn test_postdelete() {
        test_hook_execution(&[HP::PostDelete], "test_postdelete");
    }

    #[test]
    fn test_multiple_same_position() {
        let positions = [ HP::StoreUnload, HP::PreCreate, HP::PostCreate, HP::PreRetrieve,
            HP::PostRetrieve, HP::PreUpdate, HP::PostUpdate, HP::PreDelete, HP::PostDelete ];

        for position in positions.iter() {
            for n in 2..10 {
                let mut v = Vec::with_capacity(n);
                for _ in 0..n { v.push(position.clone()); }

                test_hook_execution(&v, "test_multiple_same_position");
            }
        }
    }


    fn get_store_with_aborting_hook_at_pos(pos: HP) -> Store {
        let mut store = get_store_with_config();
        let hook      = TestHook::new(pos.clone(), false, true);

        assert!(store.register_hook(pos, "test", Box::new(hook)).map_err(|e| println!("{:?}", e)).is_ok());
        store
    }

    #[test]
    fn test_pre_create_error() {
        let storeid = StoreId::new_baseless(PathBuf::from("test_pre_create_error")).unwrap();
        let store   = get_store_with_aborting_hook_at_pos(HP::PreCreate);
        assert!(store.create(storeid).is_err());
    }

    #[test]
    fn test_pre_retrieve_error() {
        let storeid = StoreId::new_baseless(PathBuf::from("test_pre_retrieve_error")).unwrap();
        let store   = get_store_with_aborting_hook_at_pos(HP::PreRetrieve);
        assert!(store.retrieve(storeid).is_err());
    }

    #[test]
    fn test_pre_delete_error() {
        let storeid = StoreId::new_baseless(PathBuf::from("test_pre_delete_error")).unwrap();
        let store   = get_store_with_aborting_hook_at_pos(HP::PreDelete);
        assert!(store.delete(storeid).is_err());
    }

    #[test]
    fn test_pre_update_error() {
        let storeid = StoreId::new_baseless(PathBuf::from("test_pre_update_error")).unwrap();
        let store   = get_store_with_aborting_hook_at_pos(HP::PreUpdate);
        let fle     = store.create(storeid).unwrap();

        assert!(store.update(fle).is_err());
    }

    #[test]
    fn test_post_create_error() {
        let store   = get_store_with_aborting_hook_at_pos(HP::PostCreate);
        let pb      = StoreId::new_baseless(PathBuf::from("test_post_create_error")).unwrap();

        assert!(store.create(pb.clone()).is_err());

        // But the entry exists, as the hook fails post-create
        assert!(store.entries.read().unwrap().get(&pb.with_base(store.path().clone())).is_some());
    }

    #[test]
    fn test_post_retrieve_error() {
        let store   = get_store_with_aborting_hook_at_pos(HP::PostRetrieve);
        let pb      = StoreId::new_baseless(PathBuf::from("test_post_retrieve_error")).unwrap();

        assert!(store.retrieve(pb.clone()).is_err());

        // But the entry exists, as the hook fails post-retrieve
        assert!(store.entries.read().unwrap().get(&pb.with_base(store.path().clone())).is_some());
    }

    #[test]
    fn test_post_delete_error() {
        let store   = get_store_with_aborting_hook_at_pos(HP::PostDelete);
        let pb      = StoreId::new_baseless(PathBuf::from("test_post_delete_error")).unwrap();

        assert!(store.create(pb.clone()).is_ok());
        let pb = pb.with_base(store.path().clone());
        assert!(store.entries.read().unwrap().get(&pb).is_some());

        assert!(store.delete(pb.clone()).is_err());
        // But the entry is removed, as we fail post-delete
        assert!(store.entries.read().unwrap().get(&pb).is_none());
    }

    #[test]
    fn test_post_update_error() {
        let store   = get_store_with_aborting_hook_at_pos(HP::PostUpdate);
        let pb      = StoreId::new_baseless(PathBuf::from("test_post_update_error")).unwrap();
        let fle     = store.create(pb.clone()).unwrap();
        let pb      = pb.with_base(store.path().clone());

        assert!(store.entries.read().unwrap().get(&pb).is_some());
        assert!(store.update(fle).is_err());
    }

    fn get_store_with_allowed_error_hook_at_pos(pos: HP) -> Store {
        let mut store = get_store_with_config();
        let hook      = TestHook::new(pos.clone(), false, false);

        assert!(store.register_hook(pos, "test", Box::new(hook)).map_err(|e| println!("{:?}", e)).is_ok());
        store
    }

    #[test]
    fn test_pre_create_allowed_error() {
        let storeid = StoreId::new_baseless(PathBuf::from("test_pre_create_allowed_error")).unwrap();
        let store   = get_store_with_allowed_error_hook_at_pos(HP::PreCreate);
        assert!(store.create(storeid).is_ok());
    }

    #[test]
    fn test_pre_retrieve_allowed_error() {
        let storeid = StoreId::new_baseless(PathBuf::from("test_pre_retrieve_allowed_error")).unwrap();
        let store   = get_store_with_allowed_error_hook_at_pos(HP::PreRetrieve);
        assert!(store.retrieve(storeid).is_ok());
    }

    #[test]
    fn test_pre_delete_allowed_error() {
        let storeid = StoreId::new_baseless(PathBuf::from("test_pre_delete_allowed_error")).unwrap();
        let store   = get_store_with_allowed_error_hook_at_pos(HP::PreDelete);
        assert!(store.retrieve(storeid.clone()).is_ok());
        assert!(store.delete(storeid).map_err(|e| println!("{:?}", e)).is_ok());
    }

    #[test]
    fn test_pre_update_allowed_error() {
        let storeid = StoreId::new_baseless(PathBuf::from("test_pre_update_allowed_error")).unwrap();
        let store   = get_store_with_allowed_error_hook_at_pos(HP::PreUpdate);
        let fle     = store.create(storeid).unwrap();

        assert!(store.update(fle).is_ok());
    }

    #[test]
    fn test_post_create_allowed_error() {
        let store   = get_store_with_allowed_error_hook_at_pos(HP::PostCreate);
        let pb      = StoreId::new_baseless(PathBuf::from("test_pre_create_allowed_error")).unwrap();

        assert!(store.create(pb.clone()).is_ok());

        // But the entry exists, as the hook fails post-create
        assert!(store.entries.read().unwrap().get(&pb.with_base(store.path().clone())).is_some());
    }

    #[test]
    fn test_post_retrieve_allowed_error() {
        let store   = get_store_with_allowed_error_hook_at_pos(HP::PostRetrieve);
        let pb      = StoreId::new_baseless(PathBuf::from("test_pre_retrieve_allowed_error")).unwrap();

        assert!(store.retrieve(pb.clone()).is_ok());

        // But the entry exists, as the hook fails post-retrieve
        assert!(store.entries.read().unwrap().get(&pb.with_base(store.path().clone())).is_some());
    }

    #[test]
    fn test_post_delete_allowed_error() {
        let store   = get_store_with_allowed_error_hook_at_pos(HP::PostDelete);
        let pb      = StoreId::new_baseless(PathBuf::from("test_post_delete_allowed_error")).unwrap();

        assert!(store.create(pb.clone()).is_ok());
        let pb = pb.with_base(store.path().clone());
        assert!(store.entries.read().unwrap().get(&pb).is_some());

        assert!(store.delete(pb.clone()).is_ok());
        // But the entry is removed, as we fail post-delete
        assert!(store.entries.read().unwrap().get(&pb).is_none());
    }

    #[test]
    fn test_post_update_allowed_error() {
        let store   = get_store_with_allowed_error_hook_at_pos(HP::PostUpdate);
        let pb      = StoreId::new_baseless(PathBuf::from("test_pre_update_allowed_error")).unwrap();
        let fle     = store.create(pb.clone()).unwrap();
        let pb      = pb.with_base(store.path().clone());

        assert!(store.entries.read().unwrap().get(&pb).is_some());
        assert!(store.update(fle).is_ok());
    }
}