cmd/gitops-pusher: fix minor bug with ACL tests (#5123)

Signed-off-by: Xe <xe@tailscale.com>
pull/5126/head
Xe Iaso 2 years ago committed by GitHub
parent ba91f57ddd
commit 92357a54ec
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -237,12 +237,6 @@ func testNewACLs(ctx context.Context, tailnet, apiKey, policyFname string) error
} }
defer resp.Body.Close() defer resp.Body.Close()
got := resp.StatusCode
want := http.StatusOK
if got != want {
return fmt.Errorf("wanted HTTP status code %d but got %d", want, got)
}
var ate ACLTestError var ate ACLTestError
err = json.NewDecoder(resp.Body).Decode(&ate) err = json.NewDecoder(resp.Body).Decode(&ate)
if err != nil { if err != nil {
@ -253,6 +247,12 @@ func testNewACLs(ctx context.Context, tailnet, apiKey, policyFname string) error
return ate return ate
} }
got := resp.StatusCode
want := http.StatusOK
if got != want {
return fmt.Errorf("wanted HTTP status code %d but got %d", want, got)
}
return nil return nil
} }

Loading…
Cancel
Save