Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add summary tests for ListWrapperReturnsVoidExample#444

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
amandelpie merged 2 commits intomainfrom437-summary-tests
Jul 8, 2022

Conversation

@onewhl
Copy link
Member

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

@onewhlonewhl added the comp-summariesSomething related to the method names, code comments and display names generation labelJul 6, 2022
@onewhlonewhl requested a review fromamandelpieJuly 6, 2022 15:07
Copy link
Collaborator

@amandelpieamandelpie 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

@amandelpieamandelpie merged commit7010e64 intomainJul 8, 2022
@amandelpieamandelpie deleted the 437-summary-tests branchJuly 8, 2022 11:30
Vassiliy-Kudryashov pushed a commit that referenced this pull requestJul 11, 2022
* Add summary tests for ListWrapperReturnsVoidExample* Review fixes: add KDoc with the link to the corresponding issue on GitHub
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@amandelpieamandelpieamandelpie approved these changes

Assignees

No one assigned

Labels

comp-summariesSomething related to the method names, code comments and display names generation

Projects

Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@onewhl@amandelpie

[8]ページ先頭

©2009-2025 Movatter.jp