Open turadg opened 8 months ago
Let's be sure to move tests like core-eval.test.js
from upgrade-next. E.g. to z:acceptance
.
https://github.com/Agoric/agoric-sdk/pull/9204/#discussion_r1566303891
perhaps a relevant subtask:
another relevant subtask:
I'm pretty sure such a test was built for upgrade-14, but it no longer runs for new releases.
Yeah I think that test got removed in #9204. Might be worth to re-add it in a better place now that we have #9257
One thing to consider is whether some tests should be able to setup some state before the upgrade so that it can be used by the acceptance layer after the upgrade. That would likely require setting up a "empty core eval" layer before the upgrade-next
one.
This is will be worked on by BytePitch folks(@anilhelvaci and @Jorge-Lopes)
What is the Problem Being Solved?
Our functional tests are all of HEAD. In
/a3p-integration
we now have robust testing of upgrades but they only cover the expected changes. We need coverage that the whole product still works as expected after the pending upgrade goes on chain.Description of the Design
Have a virtual (empty) proposal that runs after all the others to test functionality that should always work regardless of upgrades performed.
Call it
z:acceptance
so it will always run last.Its tests should verify that these important functions still work:
Security Considerations
Scaling Considerations
Test Plan
To be determined with Product
Upgrade Considerations