Skip to content
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

tests: inject delegation using admin #13

Merged
merged 12 commits into from
Jun 13, 2024
Merged

tests: inject delegation using admin #13

merged 12 commits into from
Jun 13, 2024

Conversation

gusin13
Copy link

@gusin13 gusin13 commented Jun 12, 2024

Adds test to inject fp and delegation in smart contract using admin

tests/e2e/types/datagen.go Dismissed Show dismissed Hide dismissed
tests/e2e/types/datagen.go Dismissed Show dismissed Hide dismissed
Copy link
Member

@SebastianElvis SebastianElvis left a comment

Choose a reason for hiding this comment

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

Tbh I have no idea why this works and cannot tell the difference between the protobuf objects in babylon and these ones, but great work 👍 These are only for testing purposes so hacking around should be fine

@gusin13
Copy link
Author

gusin13 commented Jun 13, 2024

@SebastianElvis so for the fields which are Binary in rust we need to send b64 string from Go, in the protobuf struct for example in case of delegation the staking tx is []bytes ref but we need it to be a string so had to create a custom struct and then b64 encode the staking tx.

@SebastianElvis
Copy link
Member

OK, that means we need to avoid []byte completely when submitting ExecMessage to contracts 😢 thanks for figuring this out

@gusin13 gusin13 merged commit 4f27c48 into main Jun 13, 2024
2 of 5 checks passed
@gusin13 gusin13 deleted the gusin13/debug-exec branch June 13, 2024 16:10
@gusin13 gusin13 requested a review from maurolacy June 13, 2024 16:19
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.

2 participants