Skip to content

Commit

Permalink
fix: Circuit Breaker flaky test (#2770)
Browse files Browse the repository at this point in the history
allow CB test responses to converge

Signed-off-by: Guy Daich <[email protected]>
  • Loading branch information
guydc authored Mar 4, 2024
1 parent ca572b8 commit e8a2b5e
Showing 1 changed file with 1 addition and 9 deletions.
10 changes: 1 addition & 9 deletions test/e2e/tests/circuitbreaker.go
Original file line number Diff line number Diff line change
Expand Up @@ -45,15 +45,7 @@ var CircuitBreakerTest = suite.ConformanceTest{
Namespace: ns,
}

req := http.MakeRequest(t, &expectedResponse, gwAddr, "HTTP", "http")
cReq, cResp, err := suite.RoundTripper.CaptureRoundTrip(req)
if err != nil {
t.Errorf("failed to get expected response: %v", err)
}

if err := http.CompareRequest(t, &req, cReq, cResp, expectedResponse); err != nil {
t.Errorf("failed to compare request and response: %v", err)
}
http.MakeRequestAndExpectEventuallyConsistentResponse(t, suite.RoundTripper, suite.TimeoutConfig, gwAddr, expectedResponse)
})
},
}

0 comments on commit e8a2b5e

Please sign in to comment.