-
Notifications
You must be signed in to change notification settings - Fork 201
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
Add some documentation to make the unit tests easier to follow #556
Conversation
@raboof IMO: I think we should note that this testing framework originated when we used JUnit 3 and that we want to update it to JUnit 5 which would make some of this complex and awkward test set up no longer necessary. |
26f30e9
to
6123271
Compare
Makes sense, filed https://issues.apache.org/jira/browse/VFS-855 and linked to that |
I canceled and restarted the failing builds. Maybe one of the SSH tests is hung:
We are on an antique version of Mina, which I've tried to update in the past, but didn't manage to get working... |
suspicious that it's |
Yeah, pretty suspicious 🤔 |
Since this is only documentation, I'm going to merge it, and see if the random-ish failure goes away with the develop-Java-macos-13 combination. |
See also https://issues.apache.org/jira/browse/VFS-855