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

Unittests for await logic (Creation) #2791

Closed
11 tasks done
Tracked by #2774
EronWright opened this issue Jan 30, 2024 · 0 comments · Fixed by #2797
Closed
11 tasks done
Tracked by #2774

Unittests for await logic (Creation) #2791

EronWright opened this issue Jan 30, 2024 · 0 comments · Fixed by #2797
Assignees
Labels
area/tests kind/task Work that's part of an ongoing epic resolution/fixed This issue was fixed

Comments

@EronWright
Copy link
Contributor

EronWright commented Jan 30, 2024

New test cases for the await package.

  • Test_Creation
    • Test_Creation (1.13s)
    • Test_Creation/NoMatchError (0.13s)
    • Test_Creation/ServiceUnavailable (0.12s)
    • Test_Creation/RemovedAPI (0.12s)
    • Test_Creation/Namespaced (0.12s)
    • Test_Creation/NonNamespaced (0.16s)
    • Test_Creation/SkipAwait (0.12s)
    • Test_Creation/NoAwaiter (0.12s)
    • Test_Creation/AwaitError (0.12s)
    • Test_Creation/Preview (0.12s)
@EronWright EronWright self-assigned this Jan 30, 2024
@EronWright EronWright added kind/task Work that's part of an ongoing epic area/tests labels Jan 30, 2024
@EronWright EronWright changed the title Unittests for await logic Unittests for await logic (Creation) Jan 31, 2024
EronWright added a commit that referenced this issue Feb 1, 2024
<!--Thanks for your contribution. See [CONTRIBUTING](CONTRIBUTING.md)
    for Pulumi's contribution guidelines.

    Help us merge your changes more quickly by adding more details such
    as labels, milestones, and reviewers.-->

### Proposed changes

<!--Give us a brief description of what you've done and what it solves.
-->

This PR implements unit tests for the await logic, i.e. the flow of
hitting the API server (with retry), then invoking the registered
awaiter to poll for success, then refreshing the live object.

### Related issues (optional)

<!--Refer to related PRs or issues: #1234, or 'Fixes #1234' or 'Closes
#1234'.
Or link to full URLs to issues or pull requests in other GitHub
repositories. -->

Closes #2791
@pulumi-bot pulumi-bot added the resolution/fixed This issue was fixed label Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tests kind/task Work that's part of an ongoing epic resolution/fixed This issue was fixed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants