-
Notifications
You must be signed in to change notification settings - Fork 53
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge integration-tests Folder #1159
Conversation
@@ -1,15 +1,12 @@ | |||
module github.com/smartcontractkit/ccip/integration-tests | |||
module github.com/smartcontractkit/chainlink/integration-tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should not change
@@ -505,6 +501,4 @@ replace ( | |||
|
|||
// type func(a Label, b Label) bool of func(a, b Label) bool {…} does not match inferred type func(a Label, b Label) int for func(a E, b E) int | |||
github.com/prometheus/prometheus => github.com/prometheus/prometheus v0.47.2-0.20231010075449-4b9c19fe5510 | |||
|
|||
github.com/smartcontractkit/chainlink/integration-tests => ../integration-tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should not change
@@ -728,7 +685,7 @@ func TeardownSuite( | |||
l.Warn().Msgf("Error deleting jobs %+v", err) | |||
} | |||
|
|||
if chainlinkNodes != nil && chainClient != nil { | |||
if chainlinkNodes != nil { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should not change
@@ -740,29 +697,6 @@ func TeardownSuite( | |||
} else { | |||
l.Info().Msg("Successfully returned funds from chainlink nodes to default network wallets") | |||
} | |||
// The following is needed for tests using EVMClient, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to bring this back
@@ -667,45 +664,6 @@ func ConfigureOCRv2AggregatorContracts( | |||
return nil | |||
} | |||
|
|||
// ReturnFunds attempts to return all the funds from the chainlink nodes to the network's default address |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to bring this back
@@ -10,7 +10,7 @@ import ( | |||
"github.com/spf13/cobra" | |||
"github.com/testcontainers/testcontainers-go" | |||
|
|||
"github.com/smartcontractkit/ccip/integration-tests/docker/cmd/internal" | |||
"github.com/smartcontractkit/chainlink/integration-tests/docker/cmd/internal" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should not change
Motivation
Solution