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

test(common): migrate missing tests #1985

Merged
merged 1 commit into from
Jan 27, 2024

Conversation

erights
Copy link
Contributor

@erights erights commented Jan 19, 2024

closes: #XXXX
refs: Agoric/agoric-sdk#8771

Description

Agoric/agoric-sdk#8771 will remove some copies of code in agoric-sdk that have migrated to endo. For two of these, the corresponding tests in agoric-sdk did some minor testing beyond the corresponding tests in endo. This PR merely moves that testing logic so it doesn't get lost in the shuffle.

Security Considerations

Scaling Considerations

Documentation Considerations

Testing Considerations

Upgrade Considerations

@erights erights self-assigned this Jan 19, 2024
@erights erights force-pushed the markm-migrate-missing-common-tests branch from 481cb68 to 6b183c9 Compare January 19, 2024 01:28
@erights erights force-pushed the markm-migrate-missing-common-tests branch 3 times, most recently from 6f3e039 to f6b4b2f Compare January 21, 2024 03:29
@erights erights marked this pull request as ready for review January 21, 2024 03:33
@erights erights force-pushed the markm-migrate-missing-common-tests branch from f6b4b2f to 5295f7f Compare January 26, 2024 01:51
@erights erights merged commit 730752e into master Jan 27, 2024
14 checks passed
@erights erights deleted the markm-migrate-missing-common-tests branch January 27, 2024 04:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants