astronomer / astro-cli / 9ad99b8e-3845-40f1-8a24-6318621f6405

39%

main: 39%

Pull #2071

circleci

Move shared request editor and API error logic into pkg/httputil

Consolidates the duplicated TrueString constant, ErrorRequest/ErrorBaseURL/
HTTPStatus200/HTTPStatus204 vars, NormalizeAPIError function, and request
editor function from all three astro-client-*-core packages into a new
pkg/httputil/request_editor.go. Each client package now re-exports the
vars/const and delegates NormalizeAPIError to httputil, and constructs its
request editor via httputil.NewRequestEditorFn with its API path argument.

Also fixes the double x-astro-client-version header bug in
astro-client-core/client.go. The original code called Header.Add (which
appends, not overwrites) unconditionally after the switch that already set
it -- so every request carried two x-astro-client-version values. In the
deploy-action case the server would see both the deploy action version and
CurrVersion.

3 of 49 new or added lines in 6 files covered. (6.12%)

24842 of 63141 relevant lines covered (39.34%)

9.51 hits per line