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

replace swingsetTests #10906

Open
turadg opened this issue Jan 28, 2025 · 3 comments
Open

replace swingsetTests #10906

turadg opened this issue Jan 28, 2025 · 3 comments

Comments

@turadg
Copy link
Member

turadg commented Jan 28, 2025

What is the Problem Being Solved?

The tests in test/swingsetTests are brittle and made before we had SwingSet RunUtils. Much of what they cover is now covered elsewhere now.

Exos have good coverage of behavior without SwingSet integration.
For SwingSet integration we have @agoric/boot with RunUtils.
We may soon have a way to test RunUtils picking up from a state: #10807

Related:

Description of the Design

Remove swingsetTests that are covered elsewhere.

For what's left, convert to modern tests. They can have their own very light bootstrap config.
TBD whether they live in swingsetTests as a new meaning of the term.

Security Considerations

Scaling Considerations

Test Plan

No tests that rely on a log dump from buildVatController except in SwingSet packages.

Upgrade Considerations

@mhofman
Copy link
Member

mhofman commented Jan 29, 2025

I'm actually not sure we should do this. I actually have a plan to update the fake liveslots environment to make it easier to built tests with it, in particular upgrade tests of a single vat. Afaik, this is still pretty difficult and costly to do with RunUtils based tests.

Edit: maybe I'm confused as to what "swingsetTests" actually are? Are they different than fake liveslots tests?

@turadg
Copy link
Member Author

turadg commented Jan 29, 2025

Yeah, different than fake liveslots tests. They actually use a vat controller with XS workers, but since we didn't have EV yet they have the behavior within the core-eval and the test is just a golden log.

Happy to have better fake liveslots to make unit testing exos and vats higher fidelity, but this is different.

@mhofman
Copy link
Member

mhofman commented Jan 29, 2025

Perfect. No concerns then. It sounds like RunUtils test right now are strictly superior and cover the same cases as these legacy swingset tests.

@turadg turadg changed the title remove swingsetTests replace swingsetTests Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants