do not return short redirect chains as test all rules errors

Author: xavierlacot

src/api/redirection_loop.rs

@@ -34,6 +34,12 @@ impl RedirectionLoop {
     pub fn has_error(&self) -> bool {
         self.error.is_some()
     }
+    pub fn has_error_too_many_hops(&self) -> bool {
+        self.error.is_some() && matches!(self.error, Some(RedirectionError::TooManyHops))
+    }
+    pub fn has_error_loop(&self) -> bool {
+        self.error.is_some() && matches!(self.error, Some(RedirectionError::Loop))
+    }
     fn compute(router: &Router<Rule>, max_hops: u8, example: &Example) -> RedirectionLoop {
         let mut current_url = example.url.clone();
         let mut current_method = example.method.clone().unwrap_or(String::from("GET"));

src/api/test_examples.rs

@@ -174,7 +174,7 @@ impl TestExamplesOutput {
         } else {
             let redirection_loop = RedirectionLoop::from_example(router, max_hops, example);
 
-            if redirection_loop.has_error() {
+            if redirection_loop.has_error_too_many_hops() || redirection_loop.has_error_loop(){
                 results.add_failed_example(
                     route.handler(),
                     example.clone(),