Include latest error in AllAttemptsErrored

  $ git grep AllAttemptsErrored
  src/client.rs:                        return Err(Error::AllAttemptsErrored(errors));
  src/client.rs:                                        return Err(Error::AllAttemptsErrored(errors));
fixed here
  src/raw_client.rs:    Err(Error::AllAttemptsErrored(errors))
already correctly collected
  src/types.rs:    AllAttemptsErrored(Vec<Error>),
  src/types.rs:            Error::AllAttemptsErrored(errors) => {
consumers

Closes #186

Author: nabijaczleweli

src/client.rs

@@ -53,7 +53,9 @@ macro_rules! impl_inner_call {
                     return res;
                 },
                 Err(e) => {
-                    let failed_attempts = errors.len() + 1;
+                    errors.push(e);
+                    let e = errors[errors.len() - 1];
+                    let failed_attempts = errors.len();
 
                     if retries_exhausted(failed_attempts, $self.config.retry()) {
                         warn!("call '{}' failed after {} attempts", stringify!($name), failed_attempts);
@@ -62,8 +64,6 @@ macro_rules! impl_inner_call {
 
                     warn!("call '{}' failed with {}, retry: {}/{}", stringify!($name), e, failed_attempts, $self.config.retry());
 
-                    errors.push(e);
-
                     // Only one thread will try to recreate the client getting the write lock,
                     // other eventual threads will get Err and will block at the beginning of
                     // previous loop when trying to read()
@@ -77,16 +77,16 @@ macro_rules! impl_inner_call {
                                     break;
                                 },
                                 Err(e) => {
-                                    let failed_attempts = errors.len() + 1;
+                                    errors.push(e);
+                                    let e = errors[errors.len() - 1];
+                                    let failed_attempts = errors.len();
 
                                     if retries_exhausted(failed_attempts, $self.config.retry()) {
                                         warn!("re-creating client failed after {} attempts", failed_attempts);
                                         return Err(Error::AllAttemptsErrored(errors));
                                     }
 
                                     warn!("re-creating client failed with {}, retry: {}/{}", e, failed_attempts, $self.config.retry());
-
-                                    errors.push(e);
                                 }
                             }
                         }