Skip to content

Commit

Permalink
[TT-628] Close the EVMClient connections on test cleanup (#10911)
Browse files Browse the repository at this point in the history
* [TT-628] Close the evmclient connections on cleanup
They can sometimes log transactions aftet the test has completed due to them happening in a go routine if you don't close them beforehand.

* merge conflict fixes
  • Loading branch information
tateexon authored Oct 13, 2023
1 parent 2e825c4 commit d8aa339
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 3 deletions.
6 changes: 6 additions & 0 deletions integration-tests/docker/test_env/test_env.go
Original file line number Diff line number Diff line change
Expand Up @@ -261,6 +261,12 @@ func (te *CLClusterTestEnv) Cleanup() error {
}
}

// close EVMClient connections
if te.EVMClient != nil {
err := te.EVMClient.Close()
return err
}

return nil
}

Expand Down
2 changes: 1 addition & 1 deletion integration-tests/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ require (
github.com/rs/zerolog v1.30.0
github.com/slack-go/slack v0.12.2
github.com/smartcontractkit/chainlink-env v0.38.3
github.com/smartcontractkit/chainlink-testing-framework v1.17.9
github.com/smartcontractkit/chainlink-testing-framework v1.17.10
github.com/smartcontractkit/chainlink/v2 v2.0.0-00010101000000-000000000000
github.com/smartcontractkit/libocr v0.0.0-20230922131214-122accb19ea6
github.com/smartcontractkit/ocr2keepers v0.7.27
Expand Down
4 changes: 2 additions & 2 deletions integration-tests/go.sum
Original file line number Diff line number Diff line change
Expand Up @@ -2368,8 +2368,8 @@ github.com/smartcontractkit/chainlink-solana v1.0.3-0.20231010203454-896f5c3c04d
github.com/smartcontractkit/chainlink-solana v1.0.3-0.20231010203454-896f5c3c04d1/go.mod h1:RIUJXn7EVp24TL2p4FW79dYjyno23x5mjt1nKN+5WEk=
github.com/smartcontractkit/chainlink-starknet/relayer v0.0.1-beta-test.0.20230901115736-bbabe542a918 h1:ByVauKFXphRlSNG47lNuxZ9aicu+r8AoNp933VRPpCw=
github.com/smartcontractkit/chainlink-starknet/relayer v0.0.1-beta-test.0.20230901115736-bbabe542a918/go.mod h1:/yp/sqD8Iz5GU5fcercjrw0ivJF7HDcupYg+Gjr7EPg=
github.com/smartcontractkit/chainlink-testing-framework v1.17.9 h1:dU5F457J4JtLziQszkyFdgrpX7BE0SweEQ+QFlRHiR4=
github.com/smartcontractkit/chainlink-testing-framework v1.17.9/go.mod h1:Q6vSDsN8qWs8K5+XYvbUAkvl1PyW6tqcYwLpCX8PV44=
github.com/smartcontractkit/chainlink-testing-framework v1.17.10 h1:FZJMIOTcmdCtaqTXisz430uq3IrC4VZQDE9krmhpm+c=
github.com/smartcontractkit/chainlink-testing-framework v1.17.10/go.mod h1:Q6vSDsN8qWs8K5+XYvbUAkvl1PyW6tqcYwLpCX8PV44=
github.com/smartcontractkit/go-plugin v0.0.0-20231003134350-e49dad63b306 h1:ko88+ZznniNJZbZPWAvHQU8SwKAdHngdDZ+pvVgB5ss=
github.com/smartcontractkit/go-plugin v0.0.0-20231003134350-e49dad63b306/go.mod h1:w1sAEES3g3PuV/RzUrgow20W2uErMly84hhD3um1WL4=
github.com/smartcontractkit/grpc-proxy v0.0.0-20230731113816-f1be6620749f h1:hgJif132UCdjo8u43i7iPN1/MFnu49hv7lFGFftCHKU=
Expand Down

0 comments on commit d8aa339

Please sign in to comment.