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

Added ForceStaticMockListener to run "configure mockito-inline" action.#438

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
nikitavlaev merged 1 commit intomainfromnikitavlaev/force-static-mock-npe
Jul 7, 2022

Conversation

@nikitavlaev
Copy link
Member

@nikitavlaevnikitavlaev commentedJul 6, 2022
edited
Loading

Description

Added ForceStaticMockListener to run "configure mockito-inline" action.
Refactored url listener in notifications to support callbacks (to encapsulate GenerateTestsDialogWindow).

Fixes#447

Type of Change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Automated Testing

Not applicable.

Manual Scenario

  1. Create setup like in the issue.
  2. Run tests without static mocking.
  3. No tests with static mocking should be generated.
    Warning with link to an action with "configure mockito-inline" must be available.
  4. Click on the link, OK.
    Mockito-inline must be installed.
  5. Generate tests once again.
    Tests with static mocks should be generated.
    Run tests, no errors must be shown.

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

@nikitavlaevnikitavlaev self-assigned thisJul 6, 2022
Copy link
Collaborator

@EgorkaKulikovEgorkaKulikov left a comment

Choose a reason for hiding this comment

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

Have several minor points to discussion, sent them to Nikita.
Approve changes anyway as we should merge this changes tomain as soon as possible.

@dtimdtim self-requested a reviewJuly 7, 2022 07:14
@nikitavlaevnikitavlaevforce-pushed thenikitavlaev/force-static-mock-npe branch from7911545 to916623cCompareJuly 7, 2022 07:51
Copy link
Collaborator

@dtimdtim left a comment

Choose a reason for hiding this comment

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

LGTM

@nikitavlaevnikitavlaevforce-pushed thenikitavlaev/force-static-mock-npe branch from916623c to0ee36a1CompareJuly 7, 2022 08:34
@nikitavlaevnikitavlaevenabled auto-merge (rebase)July 7, 2022 08:35
@nikitavlaevnikitavlaev merged commitacc2fff intomainJul 7, 2022
@nikitavlaevnikitavlaev deleted the nikitavlaev/force-static-mock-npe branchJuly 7, 2022 10:02
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@dtimdtimdtim approved these changes

@EgorkaKulikovEgorkaKulikovEgorkaKulikov approved these changes

Assignees

@nikitavlaevnikitavlaev

Labels

None yet

Projects

Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

NullPointerException in runtime for Random without static mocking

4 participants

@nikitavlaev@dtim@EgorkaKulikov

[8]ページ先頭

©2009-2025 Movatter.jp