Skip to content

tests: adapt integration tests to run against the refactored code #73

tests: adapt integration tests to run against the refactored code

tests: adapt integration tests to run against the refactored code #73

Triggered via pull request February 13, 2024 17:03
Status Failure
Total duration 54s
Artifacts

build-refactor.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

2 errors and 10 warnings
integration: tests/integration/main.go#L130
vpnConfig.Remote().AddrPort undefined (type *model.Remote has no field or method AddrPort)
integration
Process completed with exit code 1.
lint: internal/bytesx/bytesx.go#L115
comment on exported function BytesPadPKCS7 should be of the form "BytesPadPKCS7 ..."
lint: internal/bytesx/bytesx.go#L38
comment on exported function GenRandomBytes should be of the form "GenRandomBytes ..."
lint: internal/model/packet.go#L24
don't use ALL_CAPS in Go names; use CamelCase
lint: internal/session/manager.go#L142
if block ends with a return statement, so drop this else and outdent its block
lint: internal/model/session.go#L10
comment on exported const S_UNDEF should be of the form "S_UNDEF ..."
lint: internal/model/session.go#L8
don't use ALL_CAPS in Go names; use CamelCase
lint: internal/model/packet.go#L25
don't use ALL_CAPS in Go names; use CamelCase
lint: internal/tlssession/tlshandshake.go#L160
parameter 'verifiedChains' seems to be unused, consider removing or renaming it as _
lint: internal/model/packet.go#L26
don't use ALL_CAPS in Go names; use CamelCase
lint: internal/model/session.go#L11
don't use ALL_CAPS in Go names; use CamelCase