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

Implement new approach to create variables in parametrized tests #547

Merged
merged 4 commits into from
Jul 20, 2022

Conversation

EgorkaKulikov
Copy link
Collaborator

@EgorkaKulikov EgorkaKulikov commented Jul 18, 2022

Description

Currently parametrized test variables contains a lot of hacks in variable creation process.
For example, expectedResult in method parameters and asserts is represented by to different CgVariable instances
Also there is a strange hack with wrappers for actual variable.

We need to remove all this ugly hacks.

Also some fixes in deepEquals method related to primitive wrappers are suggested.

Fixes # (562)

Type of Change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

Automated Testing

utbot-samples and ContestEstimator

Manual Scenario

Smoke regression testing of Codegen

@EgorkaKulikov EgorkaKulikov force-pushed the egor/parametrized_tests_variables branch 3 times, most recently from 82decc4 to 275697c Compare July 20, 2022 08:30
@EgorkaKulikov EgorkaKulikov force-pushed the egor/parametrized_tests_variables branch from 275697c to 08928ee Compare July 20, 2022 09:42
@EgorkaKulikov EgorkaKulikov marked this pull request as ready for review July 20, 2022 09:43
@EgorkaKulikov EgorkaKulikov requested a review from ArsenHD July 20, 2022 09:44
@EgorkaKulikov EgorkaKulikov force-pushed the egor/parametrized_tests_variables branch from 17cce95 to dbbff7c Compare July 20, 2022 11:37
@EgorkaKulikov EgorkaKulikov enabled auto-merge (squash) July 20, 2022 12:08
@EgorkaKulikov EgorkaKulikov merged commit 74e7a34 into main Jul 20, 2022
@EgorkaKulikov EgorkaKulikov deleted the egor/parametrized_tests_variables branch July 20, 2022 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Create variables in parametrized tests similarly as in not parametrized
2 participants