WIP > Fix test naming mismatch

elastic · Oct 4, 2019 · 2503593 · 2503593
1 parent 803eeca
commit 2503593
Showing 1 changed file with 17 additions and 17 deletions.
diff --git a/estransport/estransport_internal_test.go b/estransport/estransport_internal_test.go
@@ -350,14 +350,21 @@ func TestTransportPerformRetries(t *testing.T) {
 					i++
 					fmt.Printf("Request #%d", i)
 					fmt.Print(": ERR\n")
-					return nil, &mockNetError{error: fmt.Errorf("Mock network error (%d)", i)}
+					return nil, fmt.Errorf("Mock regular error (%d)", i)
 				},
-			},
-			DisableRetry: true,
-		})
+			}})
 
 		req, _ := http.NewRequest("GET", "/abc", nil)
-		tp.Perform(req)
+
+		res, err := tp.Perform(req)
+
+		if err == nil {
+			t.Fatalf("Expected error, got: %v", err)
+		}
+
+		if res != nil {
+			t.Errorf("Unexpected response: %+v", res)
+		}
 
 		if i != 1 {
 			t.Errorf("Unexpected number of requests, want=%d, got=%d", 1, i)
@@ -375,21 +382,14 @@ func TestTransportPerformRetries(t *testing.T) {
 					i++
 					fmt.Printf("Request #%d", i)
 					fmt.Print(": ERR\n")
-					return nil, fmt.Errorf("Mock regular error (%d)", i)
+					return nil, &mockNetError{error: fmt.Errorf("Mock network error (%d)", i)}
 				},
-			}})
+			},
+			DisableRetry: true,
+		})
 
 		req, _ := http.NewRequest("GET", "/abc", nil)
-
-		res, err := tp.Perform(req)
-
-		if err == nil {
-			t.Fatalf("Expected error, got: %v", err)
-		}
-
-		if res != nil {
-			t.Errorf("Unexpected response: %+v", res)
-		}
+		tp.Perform(req)
 
 		if i != 1 {
 			t.Errorf("Unexpected number of requests, want=%d, got=%d", 1, i)