-
Notifications
You must be signed in to change notification settings - Fork 107
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Fix UI tests on Azure #946
Fix UI tests on Azure #946
Conversation
Codecov Report
@@ Coverage Diff @@
## master #946 +/- ##
=========================================
Coverage ? 52.89%
Complexity ? 1
=========================================
Files ? 329
Lines ? 8892
Branches ? 561
=========================================
Hits ? 4703
Misses ? 3985
Partials ? 204 |
@JLLeitschuh Thoughts on why the code coverage dropped so much? |
No idea. This doesn't seem like the kind of thing that would cause it. Perhaps there is binary jacoco comparison going on? |
I think it might be because there was no coverage report for the latest commit on master. I'm not sure, though |
Oh, the UI tests aren't running because Java 11 is being used instead of Java 8 🤦♂️ |
Update TestFX to 4.0.7-alpha
7e38d52
to
e3170c9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is Zulu? How can we know we trust it?
Other than that, this LGTM!
ZuluFX is an OpenJDK + OpenJFX build provided by Azul Systems (https://www.azul.com/). We used the embedded Zulu JRE on the RoboRIO for the 2018 FRC season |
Switch to Linux for running the UI tests
Install OpenJFX and openjdk through
apt