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

Add an integration test for forger when migrating protocol with new pseudo entrypoints #2850

Closed
hui-an-yang opened this issue Feb 29, 2024 · 1 comment · Fixed by #2938
Closed
Assignees
Labels
Paris testing🧪 Work related to testing, manual or automated
Milestone

Comments

@hui-an-yang
Copy link
Collaborator

hui-an-yang commented Feb 29, 2024

Write a smart contract with each pseudo entrypoints listed here
Originate the smart contract on chain and test for localForger result of making contract call to each entrypoints are the same as rpcForger
Especially between protocol migration, so we need to target protocol specific testnets separately
Also, in the test need to add comment for every new protocol need to check in openAPI for example oxford the value under 018-Proxford.entrypoint for new entrypoint added to write a new contract covering all listed pseudo entrypoint names

@hui-an-yang hui-an-yang changed the title Add an integration test for forger consistency migrating protocol with new entrypoints Add an integration test for forger when migrating protocol with new pseudo entrypoints Feb 29, 2024
@hui-an-yang hui-an-yang added the testing🧪 Work related to testing, manual or automated label Mar 1, 2024
@dsawali dsawali added this to the v20 milestone Mar 7, 2024
@hui-an-yang
Copy link
Collaborator Author

michelson json contract code with each pseudo entrypiont name as parameter
also deployed on ghostnet for reference https://better-call.dev/ghostnet/KT1QYn1mFHynbangmvaGL4BDA66u766rLR1X/operations

[{"prim":"parameter","args":[{"prim":"or","args":[{"prim":"unit","annots":["%default"]},{"prim":"or","args":[{"prim":"unit","annots":["%root"]},{"prim":"or","args":[{"prim":"unit","annots":["%do"]},{"prim":"or","args":[{"prim":"unit","annots":["%set_delegate"]},{"prim":"or","args":[{"prim":"unit","annots":["%remove_delegate"]},{"prim":"or","args":[{"prim":"unit","annots":["%deposit"]},{"prim":"or","args":[{"prim":"unit","annots":["%stake"]},{"prim":"or","args":[{"prim":"unit","annots":["%unstake"]},{"prim":"or","args":[{"prim":"unit","annots":["%finalize_unstake"]},{"prim":"unit","annots":["%set_delegate_parameters"]}]}]}]}]}]}]}]}]}]}]},{"prim":"storage","args":[{"prim":"string"}]},{"prim":"code","args":[[{"prim":"CAR"},{"prim":"IF_LEFT","args":[[{"prim":"DROP"},{"prim":"PUSH","args":[{"prim":"string"},{"string":"default"}]}],[{"prim":"IF_LEFT","args":[[{"prim":"DROP"},{"prim":"PUSH","args":[{"prim":"string"},{"string":"root"}]}],[{"prim":"IF_LEFT","args":[[{"prim":"DROP"},{"prim":"PUSH","args":[{"prim":"string"},{"string":"do"}]}],[{"prim":"IF_LEFT","args":[[{"prim":"DROP"},{"prim":"PUSH","args":[{"prim":"string"},{"string":"set_delegate"}]}],[{"prim":"IF_LEFT","args":[[{"prim":"DROP"},{"prim":"PUSH","args":[{"prim":"string"},{"string":"remove_delegate"}]}],[{"prim":"IF_LEFT","args":[[{"prim":"DROP"},{"prim":"PUSH","args":[{"prim":"string"},{"string":"deposit"}]}],[{"prim":"IF_LEFT","args":[[{"prim":"DROP"},{"prim":"PUSH","args":[{"prim":"string"},{"string":"stake"}]}],[{"prim":"IF_LEFT","args":[[{"prim":"DROP"},{"prim":"PUSH","args":[{"prim":"string"},{"string":"unstake"}]}],[{"prim":"IF_LEFT","args":[[{"prim":"DROP"},{"prim":"PUSH","args":[{"prim":"string"},{"string":"finalize_unstake"}]}],[{"prim":"DROP"},{"prim":"PUSH","args":[{"prim":"string"},{"string":"set_delegate_parameters"}]}]]}]]}]]}]]}]]}]]}]]}]]}]]},{"prim":"NIL","args":[{"prim":"operation"}]},{"prim":"PAIR"}]]}]

@hui-an-yang hui-an-yang self-assigned this Apr 25, 2024
@hui-an-yang hui-an-yang mentioned this issue Apr 25, 2024
9 tasks
@hui-an-yang hui-an-yang linked a pull request Apr 25, 2024 that will close this issue
9 tasks
@ac10n ac10n moved this to Done in Taquito Dev Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Paris testing🧪 Work related to testing, manual or automated
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants