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

AutoTest UI issues fixed #57

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ThomasGoering
Copy link

There was an issue with the AutoTest preferences dialog ("Test Creation Setting"): When no project was open, the dialog could not be closed. Because no clusters are available and displayed in the location tree, the close button was always disabled.

I fixed this by disabling the "Preferences..." drop down menu entry in the AutoTest toolbar if the system is not valid. Then I decided to disable other menu entries as well if they can not be selected (e.g. "Generate tests for open classes..." if no class is open).

Finally there was an issue in the location tree where the cluster for a new test class can be selected. If a folder below a recursive cluster was selected, its path was not stored. I fixed this, but if you think such folders should not be selectable at all, they shouldn't appear in the tree at all, in my opinion.

All fixes are listed in the commit descriptions.

- Test case preferences (in drop down menu) is now disabled when the system is not valid.
- Generate tests for open classes is now disabled when there are no open classes.
- Create test case (left most button in toolbar) and create manual test (in drop down menu) now don't report an exception when no system is available or the compiler is already running.
Folders under recursive clusters are displayed in the location tree for the test classes. But selecting these folders did not store the path correctly.
This is now fixed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant