Currently returns nothing - should instead return Result<(), & 'static str> to ease testing and code quality (means we don't need to use ensure macros - we can just use ?).