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 orch flow upgrades by bridge message deferral #9755

Merged
merged 6 commits into from
Jul 26, 2024

Conversation

turadg
Copy link
Member

@turadg turadg commented Jul 22, 2024

refs: #9303

Description

Tests restarting an orchestration flow while an IBC message is pending response.

#9303 will be closed when this and #9719 have landed.

Security Considerations

none

Scaling Considerations

none

Documentation Considerations

none

Testing Considerations

per se

Upgrade Considerations

helps, no change

Copy link

cloudflare-workers-and-pages bot commented Jul 22, 2024

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: 4545c95
Status: ✅  Deploy successful!
Preview URL: https://072d64e9.agoric-sdk.pages.dev
Branch Preview URL: https://9303-orch-bridge-upgrade.agoric-sdk.pages.dev

View logs

Copy link

codecov bot commented Jul 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.39%. Comparing base (77fcd1a) to head (4545c95).

Additional details and impacted files
Components Coverage Δ
SwingSet/kernel 73.36% <ø> (ø)
ERTP 92.45% <ø> (ø)
Orchestration 94.62% <100.00%> (+<0.01%) ⬆️
swing-store 95.95% <ø> (ø)
Files Coverage Δ
packages/boot/tools/supports.ts 100.00% <100.00%> (+0.15%) ⬆️
...hestration/src/exos/local-orchestration-account.js 87.85% <100.00%> (+0.07%) ⬆️

@turadg turadg force-pushed the 9303-orch-bridge-upgrade branch 4 times, most recently from 0df96b0 to ef9165a Compare July 24, 2024 23:31
@turadg turadg mentioned this pull request Jul 24, 2024
mergify bot added a commit that referenced this pull request Jul 25, 2024
refs: #9303

## Description

A bunch of refactors and testing utilities to support #9303.

It also adds a couple upgrade tests of orchestration contracts, but neither of them cover restarting orchestration flows so I'll leave #9755 for that. 

### Security Considerations
none

### Scaling Considerations
none

### Documentation Considerations
none

### Testing Considerations
per se

### Upgrade Considerations
Helps to test upgrades, should not affect any.
@turadg turadg force-pushed the 9303-orch-bridge-upgrade branch from ef9165a to 3845eac Compare July 25, 2024 21:41
@turadg turadg marked this pull request as ready for review July 25, 2024 22:25
@turadg turadg requested review from dckc and 0xpatrickdev July 25, 2024 22:25
@turadg turadg force-pushed the 9303-orch-bridge-upgrade branch from 3845eac to 99738b3 Compare July 26, 2024 14:29
@turadg turadg force-pushed the 9303-orch-bridge-upgrade branch from 99738b3 to 3043994 Compare July 26, 2024 14:37
packages/boot/tools/supports.ts Outdated Show resolved Hide resolved
@turadg turadg force-pushed the 9303-orch-bridge-upgrade branch from 3043994 to 4545c95 Compare July 26, 2024 22:12
@turadg turadg added the automerge:rebase Automatically rebase updates, then merge label Jul 26, 2024
@mergify mergify bot merged commit 1875030 into master Jul 26, 2024
92 checks passed
@mergify mergify bot deleted the 9303-orch-bridge-upgrade branch July 26, 2024 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge:rebase Automatically rebase updates, then merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants