You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
#8453 is ABD. The main thing that's missing at this point is a test that demonstrates that it works in the upgrade environment.
Description of the Design
A test that starts from (a copy of) mainNet. It should ensure there's a contract with outstanding offers or seats. I think vaults would work, or it could use coveredCall, atomicSwap, or a purpose-built contract in order to have additional instrumentation. After upgrading, the outstanding offers & seats should still be alive, the contract should still be responsive, and the offers should be exercisable.
Security Considerations
We're concerned about offer-safety.
Scaling Considerations
Not a big issue here.
Test Plan
See above.
Upgrade Considerations
Ditto.
The text was updated successfully, but these errors were encountered:
The SDK PR tests head against head. It shows that the bug with promise watchers is gone but can't test the "repair" because it doesn't have the earlier state to repair.
So this test needs to:
Make an open offer (e.g. bid)
Core-eval to restart vat-admin vat
Make sure the offer stays open (demonstrating repair)
What is the Problem Being Solved?
#8453 is ABD. The main thing that's missing at this point is a test that demonstrates that it works in the upgrade environment.
Description of the Design
A test that starts from (a copy of) mainNet. It should ensure there's a contract with outstanding offers or seats. I think vaults would work, or it could use coveredCall, atomicSwap, or a purpose-built contract in order to have additional instrumentation. After upgrading, the outstanding offers & seats should still be alive, the contract should still be responsive, and the offers should be exercisable.
Security Considerations
We're concerned about offer-safety.
Scaling Considerations
Not a big issue here.
Test Plan
See above.
Upgrade Considerations
Ditto.
The text was updated successfully, but these errors were encountered: