Skip to content
build-refactor

tests: update unit tests #91

Sign in to view logs

tests: update unit tests

tests: update unit tests #91

Re-run triggered March 1, 2024 03:44
@ainghazalainghazal
#68
ainghazal:test/complete-coverage
Status Success
Total duration 35s
Artifacts –

build-refactor.yml

on: pull_request
short-tests
25s
short-tests
lint
7s
lint
gosec
40s
gosec
coverage-threshold
18s
coverage-threshold
integration
18s
integration
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
lint: internal/runtimex/runtimex_test.go#L13
parameter 't' seems to be unused, consider removing or renaming it as _
lint: internal/runtimex/runtimex_test.go#L31
parameter 't' seems to be unused, consider removing or renaming it as _
lint: internal/runtimex/runtimex_test.go#L22
parameter 't' seems to be unused, consider removing or renaming it as _
lint: internal/session/keysource_test.go#L34
should replace i += 1 with i++
lint: internal/runtimex/runtimex_test.go#L40
parameter 't' seems to be unused, consider removing or renaming it as _
lint: internal/reliabletransport/constants.go#L4
comment on exported const RELIABLE_SEND_BUFFER_SIZE should be of the form "RELIABLE_SEND_BUFFER_SIZE ..."
lint: internal/bytesx/bytesx.go#L38
comment on exported function GenRandomBytes should be of the form "GenRandomBytes ..."
lint: internal/reliabletransport/constants.go#L12
comment on exported const MAX_ACKS_PER_OUTGOING_PACKET should be of the form "MAX_ACKS_PER_OUTGOING_PACKET ..."
lint: internal/tlssession/tlshandshake.go#L160
parameter 'verifiedChains' seems to be unused, consider removing or renaming it as _
lint: internal/networkio/common_test.go#L27
parameter 'ctx' seems to be unused, consider removing or renaming it as _