Skip to content

Wrapper for basic Stream #194

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

Merged
merged 7 commits into from
Jun 21, 2022
Merged

Wrapper for basic Stream #194

merged 7 commits into from
Jun 21, 2022

Conversation

Damtev
Copy link
Member

@Damtev Damtev commented Jun 9, 2022

Description

Added wrapper UtStream based on RangeModifiableUnlimitedArray for java.util.streams.Stream with all its methods implemented (as easy for the symbolic analysis as possible). Added UtClassMock for static methods of java.util.streams.Stream (of, empty, generate, iterate, and concat) - with using UtStream wrapper for finite streams and concrete execution for infinite streams (from generate, iterate, and concat).

Closes #145.

Type of Change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Automated Testing

See org.utbot.examples.stream.BaseStreamExampleTest

Checklist (remove irrelevant options):

  • The change followed the style guidelines of the UTBot project
  • Self-review of the code is passed
  • The change contains enough commentaries, particularly in hard-to-understand areas
  • New documentation is provided or existed one is altered
  • No new warnings
  • Tests that prove my change is effective
  • All tests pass locally with my changes

@Damtev Damtev requested a review from CaelmBleidd June 9, 2022 15:18
@Damtev Damtev force-pushed the damtev/stream_wrappers branch 3 times, most recently from ae027d0 to 42dca41 Compare June 10, 2022 10:06
Copy link
Member

@CaelmBleidd CaelmBleidd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall, LGTM, but there are several questions

@Damtev Damtev force-pushed the damtev/stream_wrappers branch 4 times, most recently from d401783 to b3c23db Compare June 20, 2022 19:26
@Damtev Damtev requested a review from CaelmBleidd June 21, 2022 11:24
@Damtev Damtev force-pushed the damtev/stream_wrappers branch from b3c23db to 8d83084 Compare June 21, 2022 14:38
@Damtev Damtev requested a review from CaelmBleidd June 21, 2022 14:41
@Damtev Damtev force-pushed the damtev/stream_wrappers branch from 8d83084 to 70a8da3 Compare June 21, 2022 16:07
@Damtev Damtev enabled auto-merge (squash) June 21, 2022 16:08
@Damtev Damtev force-pushed the damtev/stream_wrappers branch from 70a8da3 to 37305cc Compare June 21, 2022 17:16
@Damtev Damtev merged commit de99fe8 into main Jun 21, 2022
@Damtev Damtev deleted the damtev/stream_wrappers branch June 21, 2022 19:06
mmvpm pushed a commit that referenced this pull request Jun 22, 2022
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.

Wrapper for basic Stream
2 participants