fix(test): Add unit test for kvevents pool.go #29
gyliu513Annotations
9 errors and 1 warning
|
lint
Process completed with exit code 2.
|
|
lint:
pkg/kvevents/pool_test.go#L633
variable name 'p' is too short for the scope of its usage (varnamelen)
|
|
lint:
pkg/kvevents/pool_test.go#L560
variable name 'p' is too short for the scope of its usage (varnamelen)
|
|
lint:
pkg/kvevents/pool_test.go#L515
variable name 'p' is too short for the scope of its usage (varnamelen)
|
|
lint:
pkg/kvevents/pool_test.go#L17
package should be `kvevents_test` instead of `kvevents` (testpackage)
|
|
lint:
pkg/kvevents/pool_test.go#L85
return both a `nil` error and an invalid value: use a sentinel error instead (nilnil)
|
|
lint:
pkg/kvevents/pool_test.go#L669
The line is 137 characters long, which exceeds the maximum of 130 characters. (lll)
|
|
lint:
pkg/kvevents/pool_test.go#L84
The line is 140 characters long, which exceeds the maximum of 130 characters. (lll)
|
|
lint:
pkg/kvevents/pool_test.go#L125
unnamedResult: consider giving a name to these results (gocritic)
|
|
lint
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/cache@v4, actions/setup-go@v5, golangci/golangci-lint-action@v8. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|