Skip to content

feat(load): token transfer load tests#617

Draft
rodrigombsoares wants to merge 1 commit into
mainfrom
load-tokentransfer
Draft

feat(load): token transfer load tests#617
rodrigombsoares wants to merge 1 commit into
mainfrom
load-tokentransfer

Conversation

@rodrigombsoares
Copy link
Copy Markdown
Contributor

No description provided.

t.Fatal("could not find BurnMint 2.0.0 to LockRelease 2.0.0 token qualifier")

return ""
return devenvtests.DefaultDevenvTokenLane(t, in, lib, srcSelector, destSelector)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO: this can be called directly, this wrapper is not needed

Comment on lines +62 to +65
firstDest := destinations[0]
receiverBalanceBefore, err := firstDest.Chain.GetTokenBalance(ctx, firstDest.Receiver, lane.DestToken)
require.NoError(t, err)
require.NotNil(t, receiverBalanceBefore)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO: should check for all destinations

}
}

func cantonTokenLoadDestination(chain cciptestinterfaces.CCIP17, receiver protocol.UnknownAddress, lane devenvtests.TokenLane) Destination {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO: We can collapse this and cantonLoadDestination in a single function with optional lane.

Comment on lines +24 to +28
// Default transfer amounts (lane identity is discovery-driven; amounts are overridable later via env).
CantonToEVMFeeAmount int64 = 2_000
CantonToEVMTokenTransferAmount int64 = 1_000
EVMToCantonTransferAmount int64 = 100_000_000_000
EVMDecimalsScale int64 = 1_000_000_000_000_000_000
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO: E2E should use the same defaults. This looks duplicated and maybe should be defined elsewhere.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant