AwaitGatewayWithStatus exits prematurely

If the Gateway is found but doesn't have the expected status, it returns
an error which stops the retry loop and fails. It should return a nil
error with a message string so it keeps retrying.

Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

test/e2e/framework/gateways.go

@@ -57,8 +57,7 @@ func (f *Framework) AwaitGatewayWithStatus(cluster ClusterIndex, name, status st
 			gw := result.(*unstructured.Unstructured)
 			haStatus := NestedString(gw.Object, "status", "haStatus")
 			if haStatus != status {
-				return false, "", fmt.Errorf("gateway %q exists but has wrong status %q, expected %q",
-					gw.GetName(), haStatus, status)
+				return false, fmt.Sprintf("gateway %q exists but has wrong status %q, expected %q", gw.GetName(), haStatus, status), nil
 			}
 			return true, "", nil
 		})