-
Notifications
You must be signed in to change notification settings - Fork 33
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
feat: upgrade to Jakarta XML Binding 3 and Java 8 #69
Conversation
pom.xml
Outdated
<dependency> | ||
<groupId>org.junit</groupId> | ||
<artifactId>junit-bom</artifactId> | ||
<version>5.8.1</version> |
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.
upgraded to JUnit 5, XmlUnit 2 and AssertJ. The added benefits of AssertJ are much better assertion messages, especially more detailed file content comparisons
@Test | ||
@Disabled("This test works reliably on Java7 but produces different results from run to run on Java8.") |
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.
Disabled this test because it is not deterministic due to the mentioned behavior
The CI job isn't running. I guess that's because you are still using travis-ci.org which has sunset in June 2021, see https://blog.travis-ci.com/2021-05-07-orgshutdown. You will have to sign in to travis-ci.com. The disadvantage over the old service is that it is necessary to provide a payment method - even for their free tier. |
this file is not generated anymore
Was handled in issue #66. |
Fixes #66
Closes #65, #67, #64, #63