Skip to content

/ utPLSQL-SQLDeveloper

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

Jump to bottom

oddgen "Run test" view under "All Suitepaths": show package icon only for parents of test procedures #54

Closed
PhilippSalvisberg opened this issue Nov 25, 2018 · 0 comments
Closed

oddgen "Run test" view under "All Suitepaths": show package icon only for parents of test procedures #54

PhilippSalvisberg opened this issue Nov 25, 2018 · 0 comments
Assignees
@PhilippSalvisberg
Labels
bug
Milestone
v0.7.1

Comments

@PhilippSalvisberg
Copy link
Member

@PhilippSalvisberg PhilippSalvisberg commented Nov 25, 2018

The following tree shows Core and Reporters with a package icon. This is not really wrong, since in fact a package suite annotation exists. However these packages do not have tests (procedures annotated with test). In such cases just folder instead of a package icon should be used.

run_test_oddgen

This change should not affect the behavior of the generator.
@PhilippSalvisberg PhilippSalvisberg mentioned this issue Nov 25, 2018
Closed
@PhilippSalvisberg PhilippSalvisberg self-assigned this Nov 25, 2018
@PhilippSalvisberg PhilippSalvisberg added this to the v0.7.1 milestone Nov 25, 2018
PhilippSalvisberg added a commit that referenced this issue Nov 25, 2018
@PhilippSalvisberg
Merge pull request #57 from PhilippSalvisberg/develop
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
260f55a 
Fixes #54, #55, #56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PhilippSalvisberg PhilippSalvisberg

Labels
bug
Projects
None yet
Milestone
v0.7.1
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
@PhilippSalvisberg
You can’t perform that action at this time.