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 jdk9 support #249
Add jdk9 support #249
Conversation
settings.gradle
Outdated
@@ -41,3 +42,5 @@ examples.forEach({ | |||
include "examples/$it" | |||
project(":examples/$it").name = it | |||
}) | |||
include 'allure-model' |
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.
why included twice?
build.gradle
Outdated
|
||
dependency 'ru.yandex.qatools.matchers:nio-matchers:1.3' | ||
dependency 'com.fasterxml.jackson.core:jackson-databind:2.7.0' | ||
dependency 'com.fasterxml.jackson.module:jackson-module-jaxb-annotations:2.7.0' |
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.
no need to have this dependency any more
build.gradle
Outdated
@@ -100,6 +99,13 @@ subprojects { project -> | |||
dependency 'org.springframework:spring-webmvc:4.3.18.RELEASE' | |||
dependency 'org.testng:testng:6.14.3' | |||
dependency 'net.javacrumbs.json-unit:json-unit:2.0.0.RC1' | |||
|
|||
dependency 'ru.yandex.qatools.matchers:nio-matchers:1.3' |
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.
please rewrite tests using Assertj and remove this dependency
build.gradle
Outdated
dependency 'com.fasterxml.jackson.core:jackson-databind:2.7.0' | ||
dependency 'com.fasterxml.jackson.module:jackson-module-jaxb-annotations:2.7.0' | ||
dependency 'org.apache.tika:tika-core:1.14' | ||
dependency 'commons-codec:commons-codec:1.11' |
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.
if this only used for converting bytes to hex string consider removing dependency and using the following util method:
return new BigInteger(1, bytes).toString(16);
* <p>The following schema fragment specifies the expected content contained within this class. | ||
* | ||
* <pre> | ||
* <complexType name="TestResult"> |
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.
update javadocs
allure-model/build.gradle
Outdated
dependencies { | ||
|
||
compile('com.fasterxml.jackson.core:jackson-databind') | ||
compile('com.fasterxml.jackson.module:jackson-module-jaxb-annotations') |
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.
remove this dependency
return DatatypeConverter.printHexBinary(getMessageDigest() | ||
.digest(string.getBytes(StandardCharsets.UTF_8)) | ||
); | ||
return new String(Hex.encodeHex(getMessageDigest() |
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.
return new BigInteger(1, bytes).toString(16);
} | ||
|
||
@Override | ||
public TestResult withName(final String value) { |
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.
add deprecation. Also comment should be scheduled to remove in 3.0 release. Use setter method instead
build.gradle
Outdated
dependency 'ru.yandex.qatools.matchers:nio-matchers:1.3' | ||
dependency 'com.fasterxml.jackson.core:jackson-databind:2.7.0' | ||
dependency 'com.fasterxml.jackson.module:jackson-module-jaxb-annotations:2.7.0' | ||
dependency 'org.apache.tika:tika-core:1.14' |
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.
use latest version since org.apache.tika:tika-core < 1.18
has security vulnerability
3ffb6af
to
f142b04
Compare
cdf3e9d
to
ac6605b
Compare
fixes #208