test: add unit tests for root owners finding (#80) #9
Jiahui-Zhang-20Annotations
10 errors and 10 warnings
|
lint:
crons.go#L58
Error return value is not checked (errcheck)
|
|
lint:
crons.go#L62
Error return value is not checked (errcheck)
|
|
lint:
crons.go#L94
Error return value of `cronsMonitorData.addJob` is not checked (errcheck)
|
|
lint:
crons_monitor_data_test.go#L71
Error return value of `cronsMonitorData.addJob` is not checked (errcheck)
|
|
lint:
enhancers_common.go#L125
Error return value is not checked (errcheck)
|
|
lint:
sentry_dsn_data_test.go#L42
Error return value of `clientMapping.AddClientToMap` is not checked (errcheck)
|
|
lint:
sentry_dsn_data_test.go#L52
Error return value of `clientMapping.AddClientToMap` is not checked (errcheck)
|
|
lint:
sentry_dsn_data_test.go#L70
Error return value of `clientMapping.AddClientToMap` is not checked (errcheck)
|
|
lint:
crons_monitor_data_test.go#L39
string `* * * * *` has 5 occurrences, make it a constant (goconst)
|
|
lint:
crons_monitor_data_test.go#L12
string `080181f33ca343f89b0bf55d50abfeee` has 3 occurrences, make it a constant (goconst)
|
|
lint:
utils.go#L12
var-declaration: should omit type map[string]struct{} from declaration of var truthyStrings; it will be inferred from the right-hand side (revive)
|
|
lint:
utils.go#L24
var-naming: func prettyJson should be prettyJSON (revive)
|
|
lint:
informer_deployments.go#L11
unused-parameter: parameter 'namespace' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
enhancers_common_test.go#L7
unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
mocks_test.go#L15
unused-parameter: parameter 'options' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
mocks_test.go#L21
unused-parameter: parameter 'timeout' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
cluster_config.go#L53
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
|
|
lint:
context.go#L28
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
|
|
lint:
cluster_config.go#L83
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
|
|
lint:
informer_replicasets.go#L12
unused-parameter: parameter 'namespace' seems to be unused, consider removing or renaming it as _ (revive)
|