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

Add summary tests for ListWrapperReturnsVoidExample #444

Merged
merged 2 commits into from
Jul 8, 2022

Conversation

onewhl
Copy link
Member

@onewhl onewhl commented Jul 6, 2022

Description

I investigated issue 437 and didn't reproduced the error.
To capture the current behavior , I wrote summary tests for ListWrapperReturnsVoidExample class.

Type of Change

  • New unit tests

How Has This Been Tested?

I added summary tests for each method in the ListWrapperReturnsVoidExample class.

Automated Testing

Tests are located in examples.collections.SummaryListWrapperReturnsVoidTest

Manual Scenario

You can run UTBot on the ListWrapperReturnsVoidExample class and check the summaries you get with the ones I provide in summary tests.

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

@onewhl onewhl added the comp-summaries Something related to the method names, code comments and display names generation label Jul 6, 2022
@onewhl onewhl requested a review from amandelpie July 6, 2022 15:07
Copy link
Collaborator

@amandelpie amandelpie left a comment

Choose a reason for hiding this comment

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

Please add a KDoc here with the link to the issue related to the newly added test and postpone the merging for a few days

@amandelpie amandelpie merged commit 7010e64 into main Jul 8, 2022
@amandelpie amandelpie deleted the 437-summary-tests branch July 8, 2022 11:30
Vassiliy-Kudryashov pushed a commit that referenced this pull request Jul 11, 2022
* Add summary tests for ListWrapperReturnsVoidExample

* Review fixes: add KDoc with the link to the corresponding issue on GitHub
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp-summaries Something related to the method names, code comments and display names generation
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants