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.

Sign up for GitHub

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

Jump to bottom

Test: add stream-specific assertions #2836

Merged
merged 3 commits into from
Mar 7, 2022
Merged

Test: add stream-specific assertions #2836

merged 3 commits into from
Mar 7, 2022

Conversation

diesalbla
Copy link
Contributor

@diesalbla diesalbla commented Mar 4, 2022 •
edited
Loading

We add to the Fs2Suite base class some extension classes for Stream-specific assertions, such as asserting:

  • That a stream emits the same outputs as a given list or stream.
  • That a stream compilation raises an error.
  • That a stream emits, perhaps in different order, same outputs as another list, set, or stream.
  • That a stream is empty and emits no outputs.

@diesalbla diesalbla force-pushed the compiles_to_same_list branch 5 times, most recently from 6a324b5 to 7c36417 Compare March 4, 2022 23:52
@diesalbla diesalbla changed the title FS2 Suites: add an extension class for stream-specific assertion Test: add stream-specific assertions Mar 4, 2022
@diesalbla diesalbla force-pushed the compiles_to_same_list branch 4 times, most recently from 305d5c0 to 801e2b8 Compare March 5, 2022 13:22
@diesalbla diesalbla force-pushed the compiles_to_same_list branch from 801e2b8 to a98a0e5 Compare March 5, 2022 14:58
@diesalbla
FS2 Suites: add an extension class for stream-specific assertion
c7669ae 
- An assertion to check if a stream emits the same outputs as
  a given list, or as another stream (by compiling to List).
- An assertion to check if a stream compilation raises an error.
@diesalbla diesalbla force-pushed the compiles_to_same_list branch 2 times, most recently from 39735be to 4bbf220 Compare March 5, 2022 15:37
@diesalbla diesalbla force-pushed the compiles_to_same_list branch 9 times, most recently from d91b72f to 15dc560 Compare March 5, 2022 16:29
@diesalbla diesalbla force-pushed the compiles_to_same_list branch from 15dc560 to 8ad57a0 Compare March 5, 2022 19:21
@mpilquist mpilquist merged commit 48b7ba6 into typelevel:main Mar 7, 2022
@diesalbla diesalbla mentioned this pull request Mar 7, 2022
Merged
@diesalbla diesalbla deleted the compiles_to_same_list branch March 7, 2022 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mpilquist mpilquist mpilquist left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@diesalbla @mpilquist