Refactor test infrastructure and consolidate test utilities · sean-/opensearch-go@17ab399

Triggered via push January 29, 2026 17:47

sean-

gh-786

Status Failure

Total duration 31s

Artifacts –

lint.yml

on: push

10 errors

Lint: opensearchutil/testutil/mockhttp/server.go#L265

net/http.Get must not be called. use net/http.NewRequestWithContext and (*net/http.Client).Do(*http.Request) (noctx)

Lint: opensearchutil/testutil/helpers.go#L171

net/http.Get must not be called. use net/http.NewRequestWithContext and (*net/http.Client).Do(*http.Request) (noctx)

Lint: opensearchutil/testutil/mockhttp/port.go#L87

G115: integer overflow conversion int -> uint32 (gosec)

Lint: opensearchutil/testutil/helpers.go#L93

G115: integer overflow conversion int -> uint32 (gosec)

Lint: opensearchutil/testutil/mockhttp/port.go#L44

don't use `init` function (gochecknoinits)

Lint: opensearchutil/testutil/helpers.go#L77

don't use `init` function (gochecknoinits)

Lint: opensearchutil/testutil/mockhttp/port.go#L40

allocatedPorts is a global variable (gochecknoglobals)

Lint: opensearchutil/testutil/mockhttp/port.go#L39

mockPortPool is a global variable (gochecknoglobals)

Lint: opensearchutil/testutil/helpers.go#L74

allocatedPorts is a global variable (gochecknoglobals)

Lint: opensearchutil/testutil/helpers.go#L71

portPools is a global variable (gochecknoglobals)