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

Split current tests file #48

Merged
merged 3 commits into from
Oct 4, 2023
Merged

Split current tests file #48

merged 3 commits into from
Oct 4, 2023

Conversation

shtaif
Copy link
Owner

@shtaif shtaif commented Oct 4, 2023

Split the single current test file into two - one having all the tests relevant only for the iterified function and the other all the tests relevant only for iterifiedUnwrapped.

In the former unified test suite, some tests where written based on iterified and others based on iterifiedUnwrapped based on concerns of convenience. Now, each of these has direct coverage for all of its behaviors individually, even for the features overlapping with the other's.

…hannel's iterator as closed for further pull attempts as soon as it realizes its parent channel itself had already ended/errored out
…s relevant only for the `iterified` function and the other all the tests relevant only for `iterifiedUnwrapped`
@shtaif shtaif self-assigned this Oct 4, 2023
@shtaif shtaif marked this pull request as ready for review October 4, 2023 14:07
@shtaif shtaif merged commit 815d357 into master Oct 4, 2023
6 checks passed
@shtaif shtaif deleted the split-current-tests-file branch October 4, 2023 14:16
shtaif pushed a commit that referenced this pull request Jun 22, 2024
## [1.2.5](v1.2.4...v1.2.5) (2024-06-22)

### Bug Fixes

* fix ESM build to contain `import` statements with full file extensions ([#49](#49)) ([2aad91d](2aad91d))

### Refactor

* internal logic fix in the multicast channel code (no public behavior impact) - ensure to mark a channel's iterator as closed for further pull attempts as soon as it realizes its parent channel itself had already ended/errored out ([#47](#47)) ([d11a74b](d11a74b))

### Tests

* split the single current test file into two - one having all the tests relevant only for the `iterified` function and the other all the tests relevant only for `iterifiedUnwrapped` ([#48](#48)) ([815d357](815d357))
Copy link

🎉 This PR is included in version 1.2.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant