-
Notifications
You must be signed in to change notification settings - Fork 119
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
farming rpc test #1217
farming rpc test #1217
Conversation
Signed-off-by: zqhxuyuan <[email protected]>
✅ Congestion test: 1-day congestion cost (manta-runtime) is above the target daily congestion cost |
|
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.
Can't we make the test run for custom/custom asset pair instead of the native/custom pair.
Then we can have it run for Calamari as well ?
Btw ill open a PR to uncomment the rpc correctness and performance test you mentioned in the description - #1221
Signed-off-by: zqhxuyuan <[email protected]>
…o farming_rpc_test
Signed-off-by: zqhxuyuan <[email protected]>
Signed-off-by: zqhxuyuan <[email protected]>
Signed-off-by: zqhxuyuan <[email protected]>
Signed-off-by: zqhxuyuan <[email protected]>
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.
looks good but seems that ci is failing
Signed-off-by: zqhxuyuan <[email protected]>
Signed-off-by: zqhxuyuan <[email protected]>
Signed-off-by: zqhxuyuan <[email protected]>
Signed-off-by: zqhxuyuan <[email protected]>
Signed-off-by: zqhxuyuan <[email protected]>
Signed-off-by: zqhxuyuan <[email protected]>
Seems mantapay rpc tests didn't get passed. |
They rely on starting from a clean state, that's why they should be first. |
Ah, But I have this timeout issue when put farming test after manta pay test. https://github.com/Manta-Network/Manta/actions/runs/5595302834/jobs/10231502632 |
Signed-off-by: zqhxuyuan <[email protected]>
It seems it hangs up on this loop after the first attempt to execute a runtime upgrade https://github.com/Manta-Network/Manta/blob/farming_rpc_test/tests/test-farming.ts#L136-L139 |
Signed-off-by: zqhxuyuan <[email protected]>
…o farming_rpc_test
Signed-off-by: zqhxuyuan <[email protected]>
Signed-off-by: zqhxuyuan <[email protected]>
Signed-off-by: zqhxuyuan <[email protected]>
…o farming_rpc_test
Description
not add to manta CI yet, asrun test suites
not opened for now:This rpc testcase only for manta runtime.
Before we can approve this PR for merge, please make sure that all the following items have been checked off:
L-
group to this PRA-
andC-
groups to this PRA-calamari
and/orA-manta
if your changes are meant for/impact either of these (CI depends on it)Files changed
in the Github PR explorer.Situational Notes:
BaseFilter
. Ensure every extrinsic works from front-end. If there's corresponding tool, ensure both work for each other.try-runtime
. This includes migrations inherited from upstream changes, and you can search the diffs for modifications of#[pallet::storage]
items to check for any.