Skip to content

Removed unformatted symbols from code descriptions to avoid fail with code formatter from OpenAPI #318

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
Merged
merged 1 commit into from
Jun 28, 2022
Merged

Removed unformatted symbols from code descriptions to avoid fail with code formatter from OpenAPI #318

merged 1 commit into from
Jun 28, 2022

Conversation

amandelpie
Copy link
Collaborator

@amandelpie amandelpie commented Jun 27, 2022 •
edited
Loading

Description

Fixes #271

Fixes # (issue)

Type of Change

Minor bug fix (non-breaking small changes)

How Has This Been Tested?

Run test generation for the public static Stats of(double... values) method in guava-26, before bug fix it fails with the error described in the issue, after bug fix it produces tests well

Checklist (remove irrelevant options):

  • The change followed the style guidelines of the UTBot project
  • Self-review of the code is passed
  • No new warnings
  • Tests that prove my change is effective
  • All tests pass locally with my changes

@amandelpie amandelpie added ctg-bug Issue is a bug comp-summaries Something related to the method names, code comments and display names generation labels Jun 27, 2022
@amandelpie amandelpie requested a review from Damtev June 28, 2022 08:58
Damtev
Damtev approved these changes Jun 28, 2022
View reviewed changes
Copy link
Member

@Damtev Damtev left a comment

Choose a reason for hiding this comment

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

LGTM, but is a bit strange coverage matcher is never actually used. Perhaps we should refactor the check method someday.

@amandelpie
Copy link
Collaborator Author

LGTM, but is a bit strange coverage matcher is never actually used. Perhaps we should refactor the check method someday.

This module is full of problems. But the refactoring is not goal of this PR.

Thanks

@amandelpie amandelpie merged commit 1728ec0 into main Jun 28, 2022
@amandelpie amandelpie deleted the feature-271 branch June 28, 2022 13:35
dtim pushed a commit that referenced this pull request Jun 28, 2022
@amandelpie @dtim
Removed unformatted symbols from code descriptions (#318)
c7a0d9a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Damtev Damtev Damtev approved these changes

Assignees
No one assigned
Labels
comp-summaries Something related to the method names, code comments and display names generation ctg-bug Issue is a bug
Projects
UTBot Java
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Empty test for Stats of(double... values) due to wrong documentation with /r/n
2 participants
@amandelpie @Damtev