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(async-flow): demonstrate #9465 #9474

Merged
merged 1 commit into from
Jun 9, 2024

Conversation

erights
Copy link
Member

@erights erights commented Jun 9, 2024

closes: #XXXX
refs: #9465

Description

This PR just captures the test from #9465 , turning it into an expected failure. A later PR that fixes the bug would then remove the .failing.

Security Considerations

The bug itself endangers integrity, so fixing it will help security. This PR though just adds the test that demonstrates the bug.

Scaling Considerations

none

Documentation Considerations

none

Testing Considerations

The point.

Upgrade Considerations

The bug means that an asyncFlow activation that does return early would be committed to the done state, preventing a later upgrade from repairing it to get beyond the problem. But this PR just adds the test for that bug.

@erights erights self-assigned this Jun 9, 2024
Copy link

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: 76e7433
Status: ✅  Deploy successful!
Preview URL: https://6df4e867.agoric-sdk.pages.dev
Branch Preview URL: https://markm-asyncflow-test-early-c.agoric-sdk.pages.dev

View logs

@erights erights marked this pull request as ready for review June 9, 2024 04:34
@erights erights requested a review from mhofman June 9, 2024 04:34
Copy link
Member

@mhofman mhofman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. I should have made a PR when I filed the issue

@erights erights added the automerge:squash Automatically squash merge label Jun 9, 2024
@mergify mergify bot merged commit 27cc9b8 into master Jun 9, 2024
79 checks passed
@mergify mergify bot deleted the markm-asyncFlow-test-early-completion branch June 9, 2024 05:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge:squash Automatically squash merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants