Added GUI basic testing implementation#1768
Conversation
…in-view.fxml Added minWidth and minHeigt to main window in order to prevent 0 * 0 window size
|
Seems like the tests fail because a System environment variable is null in the GitHub Actions context, that points to the Project directory (IdeVariables.IDE_ROOT) Current idea is to refactor MainController to allow Dependency Injection, and then supply the controller with a temporary mock project directory that we create in the test, instead of the directory from the System environment variable. |
- AppBaseTest.java now generates mock project directory (in order for the test to work on the CI)
|
Seems like increasing the timeout limits did the trick! |
Pull Request Test Coverage Report for Build 23589507101Details
💛 - Coveralls |
hohwille
left a comment
There was a problem hiding this comment.
@laim2003 thanks for your PR and doing the ground-work for automated GUI testing. Great job 👍
BTW: Actually you revealed a bug in the GUI code that if IDE_ROOT is undefined we end up with a NullPointerException. The entire GUI is still in prototype state and needs rework.
But there is already #1295 open with some remaining sub-tasks addressing this.
Therefore, we can easily ignore this here and cleanup later in upcoming PRs.
Ready for merge.
|
For the record: The general infrastructure code for GUI tests like the |
Good point. I believe opening a seperate Issue would be a bit over the top. Maybe I can push this change today and include it in this PR |
… separate class HeadlessApplicationTest.java
… GUI-testing-implementation
This PR adds a basic UI test for
ide-guiImplemented changes:
Checklist for this PR
Make sure everything is checked before merging this PR. For further info please also see
our DoD.
mvn clean testlocally all tests pass and build is successful#«issue-id»: «brief summary»(e.g.#921: fixed setup.bat). If no issue ID exists, title only.In Progressand assigned to you or there is no issue (might happen for very small PRs)with
internal