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

Lessen thisInstance creation in parameterized test generation #1065

Merged
merged 5 commits into from
Oct 7, 2022

Conversation

sofurihafe
Copy link
Member

@sofurihafe sofurihafe commented Oct 4, 2022

Description

This PR lessens thisInstance creations in parameterized test generation moving them from data providers to generic test frame.

Type of Change

  • New feature/optimization (non-breaking change which adds functionality)

How Has This Been Tested?

Automated Testing

utbot-samples

Manual Scenario

Enable parameterized test generation. Run, for example, StaticSubstitutionExamplesTest and verify that generic test frame has thisInstance initialization, while data providers don't have it.

@sofurihafe sofurihafe marked this pull request as ready for review October 4, 2022 07:44
@EgorkaKulikov EgorkaKulikov merged commit 56bf6ab into main Oct 7, 2022
@EgorkaKulikov EgorkaKulikov deleted the andrey-t/parameterized-tests-this-instance branch October 7, 2022 13:11
@alisevych alisevych added this to the 2022.10.1 Release milestone Oct 10, 2022
AbdullinAM pushed a commit to AbdullinAM/UTBotJava that referenced this pull request Oct 17, 2022
…stBot#1065)

* Lessen thisInstance creation in parameterized test generation

* Avoid incorrect configuration

Co-authored-by: Egor Kulikov <egor.k.kulikov@gmail.com>
AbdullinAM pushed a commit to AbdullinAM/UTBotJava that referenced this pull request Oct 17, 2022
…stBot#1065)

* Lessen thisInstance creation in parameterized test generation

* Avoid incorrect configuration

Co-authored-by: Egor Kulikov <egor.k.kulikov@gmail.com>
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.

3 participants