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

Have store.finish() assert no received actions #3054

Merged
merged 3 commits into from
May 7, 2024

Conversation

stephencelis
Copy link
Member

store.finish() should do much of the same work as store.deinit, but asynchronously with a timeout. This PR updates things so that folks with long-living test stores (e.g. held onto by the test case) have the ability to assert that there are no unreceived actions.

`store.finish()` should do much of the same work as `store.deinit`, but
asynchronously with a timeout. This PR updates things so that folks with
long-living test stores (_e.g._ held onto by the test case) have the
ability to assert that there are no unreceived actions.
@stephencelis stephencelis requested a review from mbrandonw May 7, 2024 18:10
@stephencelis stephencelis merged commit eda999e into main May 7, 2024
7 checks passed
@stephencelis stephencelis deleted the test-store-finish-receives-actions branch May 7, 2024 19:16
cgrindel-self-hosted-renovate bot referenced this pull request in cgrindel/rules_swift_package_manager May 9, 2024
…ure to from: "1.10.3" (#1063)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
|
[pointfreeco/swift-composable-architecture](https://github.com/pointfreeco/swift-composable-architecture)
| patch | `from: "1.10.2"` -> `from: "1.10.3"` |

---

### Release Notes

<details>
<summary>pointfreeco/swift-composable-architecture
(pointfreeco/swift-composable-architecture)</summary>

###
[`v1.10.3`](https://github.com/pointfreeco/swift-composable-architecture/releases/tag/1.10.3)

[Compare
Source](https://github.com/pointfreeco/swift-composable-architecture/compare/1.10.2...1.10.3)

#### What's Changed

- Fixed: Automatically cancel in-flight `TestStore` effects when its
root feature is dismissed
([https://github.com/pointfreeco/swift-composable-architecture/pull/3044](https://github.com/pointfreeco/swift-composable-architecture/pull/3044)).
- Fixed: Propagate current dependencies to `Store.init`
([https://github.com/pointfreeco/swift-composable-architecture/pull/3049](https://github.com/pointfreeco/swift-composable-architecture/pull/3049)).
- Fixed: Runtime warn when stack integration is detected to be missing
from store's reducer
([https://github.com/pointfreeco/swift-composable-architecture/pull/3048](https://github.com/pointfreeco/swift-composable-architecture/pull/3048)).
- Fixed: `TestStore.finish()` should assert against unreceived actions
([https://github.com/pointfreeco/swift-composable-architecture/pull/3054](https://github.com/pointfreeco/swift-composable-architecture/pull/3054)).
- Fixed: Precondition when conflicting persistence keys must match value
by [@&#8203;stephencelis](https://github.com/stephencelis) in
[https://github.com/pointfreeco/swift-composable-architecture/pull/3051](https://github.com/pointfreeco/swift-composable-architecture/pull/3051).

    > **Important:**
> If you have defined a custom persistence key, you may need to
explicitly conform it to `Hashable` after upgrading to this version.
- Fixed: `@Shared` no longer eagerly evaluates its initial value (thanks
[@&#8203;seanmrich](https://github.com/seanmrich),
[https://github.com/pointfreeco/swift-composable-architecture/pull/3057](https://github.com/pointfreeco/swift-composable-architecture/pull/3057)).
- Infrastructure: Added a brand new tutorial: Building SyncUps
([https://github.com/pointfreeco/swift-composable-architecture/pull/3039](https://github.com/pointfreeco/swift-composable-architecture/pull/3039)).
- Infrastructure: Add store invalidation test
([https://github.com/pointfreeco/swift-composable-architecture/pull/3038](https://github.com/pointfreeco/swift-composable-architecture/pull/3038)).
- Infrastructure: Document `@CasePathable @&#8203;dynamicMemberLookup`
in 1.4 migration guide
([https://github.com/pointfreeco/swift-composable-architecture/pull/3043](https://github.com/pointfreeco/swift-composable-architecture/pull/3043)).
- Infrastructure: Document nested enum reducers
([https://github.com/pointfreeco/swift-composable-architecture/pull/3050](https://github.com/pointfreeco/swift-composable-architecture/pull/3050)).

#### New Contributors

- [@&#8203;seanmrich](https://github.com/seanmrich) made their first
contribution in
[https://github.com/pointfreeco/swift-composable-architecture/pull/3057](https://github.com/pointfreeco/swift-composable-architecture/pull/3057)

**Full Changelog**:
pointfreeco/swift-composable-architecture@1.10.2...1.10.3

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

👻 **Immortal**: This PR will be recreated if closed unmerged. Get
[config help](https://github.com/renovatebot/renovate/discussions) if
that's undesired.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://github.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi4xMDkuNCIsInVwZGF0ZWRJblZlciI6IjM2LjEwOS40IiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: Self-hosted Renovate Bot <361546+cgrindel-self-hosted-renovate[bot]@users.noreply.github.enterprise.com>
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