Skip to content

add presuspend sanity check#275

Draft
zhongkechen wants to merge 1 commit intomainfrom
checks
Draft

add presuspend sanity check#275
zhongkechen wants to merge 1 commit intomainfrom
checks

Conversation

@zhongkechen
Copy link
Contributor

@zhongkechen zhongkechen commented Mar 26, 2026

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Issue Link, if available

Description

Just before go suspend, we do sanity check to make sure:

  • it's not suspended more than once
  • there is at least one operation pending completion, otherwise, the backend will throw an exception

This change is to help us detect possible issues with suspend in our test cases, not to prevent them.

Demo/Screenshots

Checklist

  • I have filled out every section of the PR template
  • I have thoroughly tested this change

Testing

Unit Tests

Have unit tests been written for these changes?

Integration Tests

Have integration tests been written for these changes?

Examples

Has a new example been added for the change? (if applicable)

@zhongkechen zhongkechen self-assigned this Mar 26, 2026
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.

1 participant