Skip to content

Avoid incorrect warning about the Destination package #1977

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

Conversation

Vassiliy-Kudryashov
Copy link
Member

Description

Actually we don't show textfield to specify result package for tests. Thus in model is't just an empty string. The fix don't procude the warning in case of empty string.

Fixes #1915

Manual tests

See the steps from #1915, actually the issue should appear always when you generate tests.

Self-check list

  • I've set the proper labels for my PR (at least, for category and component).
  • PR title and description are clear and intelligible.
  • I've added enough comments to my code, particularly in hard-to-understand areas.
  • The functionality I've repaired, changed or added is covered with automated tests.
  • Manual tests have been provided optionally.
  • The documentation for the functionality I've been working on is up-to-date.

@Vassiliy-Kudryashov Vassiliy-Kudryashov added comp-ui Improvements of plugin UI appearence and functionality comp-ide Issue is related to IDE support ctg-bug-fix PR is fixing a bug labels Mar 17, 2023
@Vassiliy-Kudryashov Vassiliy-Kudryashov linked an issue Mar 17, 2023 that may be closed by this pull request
@Vassiliy-Kudryashov Vassiliy-Kudryashov merged commit 1389967 into main Mar 18, 2023
@Vassiliy-Kudryashov Vassiliy-Kudryashov deleted the Vassiliy-Kudryashov/1915-incorrect-warning-about-the-destination-package branch March 18, 2023 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp-ide Issue is related to IDE support comp-ui Improvements of plugin UI appearence and functionality ctg-bug-fix PR is fixing a bug
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Incorrect warning about the Destination package
2 participants