Skip to content

Rename UtTestCase #519 - #517

Merged
sergeypospelov merged 3 commits into
mainfrom
sergey/ut-test-case-refactoring
Jul 14, 2022
Merged

Rename UtTestCase #519#517
sergeypospelov merged 3 commits into
mainfrom
sergey/ut-test-case-refactoring

Conversation

@sergeypospelov

@sergeypospelovsergeypospelov commented Jul 13, 2022

Copy link
Copy Markdown
Member

Description

  • Renamed UtTestCase class to UtMethodTestSet.
  • Also renamed all related local variables and functions to ...testSet....
  • Along with this, renamed UtValueTestCase to UtMethodValueTestSet. FYI: it's only used in our tests.

Fixes#519

Type of Change

  • Refactoring (typos and non-functional changes)

Checklist:

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

@sergeypospelov
sergeypospelovforce-pushed the sergey/ut-test-case-refactoring branch from 9fe794a to ba1ac51CompareJuly 13, 2022 12:17
@sergeypospelovsergeypospelov changed the title UtTestCase refactorRename UtTestCaseJul 13, 2022
@sergeypospelovsergeypospelov changed the title Rename UtTestCaseRename UtTestCase #519Jul 13, 2022

@mmvpmmmvpm left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please rename the following as well:

  • org/utbot/sarif/SarifReport.kt:73
  • org/utbot/framework/plugin/sarif/GenerateTestsAndSarifReportFacade.kt:35
  • org/utbot/framework/plugin/sarif/GenerateTestsAndSarifReportFacade.kt:72
  • org/utbot/contest/Contest.kt:336
  • org/utbot/external/api/UtBotJavaApi.kt:111

@mmvpm

Copy link
Copy Markdown
Collaborator

Also:

  • examples/SummaryTestCaseGeneratorTest.kt:100
  • org/utbot/framework/codegen/model/constructor/tree/CgTestClassConstructor.kt:53
  • org/utbot/examples/UtModelTestCaseChecker.kt:96
  • org/utbot/examples/UtValueTestCaseChecker.kt:2295

@mmvpm

Copy link
Copy Markdown
Collaborator

Also:

  • org/utbot/examples/UtValueTestCaseChecker.kt:2339
  • org/utbot/examples/UtValueTestCaseChecker.kt:2479
  • org/utbot/framework/plugin/api/FuzzerTestCaseGeneratorTest.kt:18

@sergeypospelov

Copy link
Copy Markdown
MemberAuthor

Looks like

  • org/utbot/contest/Contest.kt:336

shouldn't be renamed

@mmvpm

Copy link
Copy Markdown
Collaborator

Also:

  • org/utbot/cli/BunchTestGeneratorCommand.kt:107
  • org/utbot/cli/GenerateTestsCommand.kt:105
  • org.utbot.cli.GenerateTestsAbstractCommand#generateTestSets
  • org/utbot/framework/coverage/CoverageCalculator.kt:143
  • org/utbot/intellij/plugin/generator/CodeGenerationController.kt:98

@mmvpmmmvpm left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Also:

  • org/utbot/external/api/UtBotJavaApi.kt:153 (with comment)
  • org/utbot/external/api/UtBotJavaApi.kt:105
  • (?) org/utbot/framework/codegen/model/constructor/tree/CgMethodConstructor.kt:1365
  • (?) org/utbot/framework/codegen/model/constructor/tree/CgMethodConstructor.kt:1312
  • (?) org/utbot/framework/codegen/model/constructor/tree/CgTestClassConstructor.kt:44

@sergeypospelov

Copy link
Copy Markdown
MemberAuthor
  • (?) org/utbot/framework/codegen/model/constructor/tree/CgMethodConstructor.kt:1365
  • (?) org/utbot/framework/codegen/model/constructor/tree/CgMethodConstructor.kt:1312

Looks like these uses of "case" are correct.

@sergeypospelov
sergeypospelovforce-pushed the sergey/ut-test-case-refactoring branch from c6ab919 to e8b13dbCompareJuly 13, 2022 14:07
@sergeypospelov
sergeypospelovforce-pushed the sergey/ut-test-case-refactoring branch from e8b13db to 861990cCompareJuly 13, 2022 14:09
@sergeypospelov
sergeypospelov merged commit 08ded2a into mainJul 14, 2022
@sergeypospelov
sergeypospelov deleted the sergey/ut-test-case-refactoring branch July 14, 2022 09:13
Sign up for freeto 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.

Rename UtTestCase

2 participants

@sergeypospelov@mmvpm