Exclude tests that are Java 9 incompatible.

This commit is contained in:
Tyler Benson 2018-01-23 14:56:40 -05:00
parent 28336a7b4e
commit 6fbe494934
9 changed files with 14 additions and 13 deletions

View File

@ -5,11 +5,11 @@ description = 'dd-java-agent-ittests'
evaluationDependsOn(':dd-java-agent:tooling') evaluationDependsOn(':dd-java-agent:tooling')
compileTestJava.dependsOn tasks.getByPath(':dd-java-agent:tooling:testClasses') compileTestJava.dependsOn tasks.getByPath(':dd-java-agent:tooling:testClasses')
if (!JavaVersion.current().isJava8Compatible()) { if (JavaVersion.current() != JavaVersion.VERSION_1_8) {
sourceSets { sourceSets {
test { test {
groovy { groovy {
// These classes use Ratpack which requires Java 8. // These classes use Ratpack which requires Java 8. (Currently also incompatible with Java 9.)
exclude '**/TestHttpServer.groovy', '**/ApacheHttpClientTest.groovy' exclude '**/TestHttpServer.groovy', '**/ApacheHttpClientTest.groovy'
} }
} }

View File

@ -31,11 +31,11 @@ versionScan {
apply from: "${rootDir}/gradle/java.gradle" apply from: "${rootDir}/gradle/java.gradle"
if (!JavaVersion.current().isJava8Compatible()) { if (JavaVersion.current() != JavaVersion.VERSION_1_8) {
sourceSets { sourceSets {
test { test {
groovy { groovy {
// These classes use Cassandra 3 which requires Java 8. // These classes use Cassandra 3 which requires Java 8. (Currently incompatible with Java 9.)
exclude '**/CassandraClientTest.groovy' exclude '**/CassandraClientTest.groovy'
} }
} }

View File

@ -9,7 +9,7 @@ import org.cassandraunit.utils.EmbeddedCassandraServerHelper
class CassandraClientTest extends AgentTestRunner { class CassandraClientTest extends AgentTestRunner {
def setupSpec() { def setupSpec() {
EmbeddedCassandraServerHelper.startEmbeddedCassandra(40000L) EmbeddedCassandraServerHelper.startEmbeddedCassandra(EmbeddedCassandraServerHelper.DEFAULT_CASSANDRA_YML_FILE, "build/tmp/embeddedCassandra")
} }
def cleanupSpec() { def cleanupSpec() {

View File

@ -14,11 +14,11 @@ versionScan {
apply from: "${rootDir}/gradle/java.gradle" apply from: "${rootDir}/gradle/java.gradle"
if (!JavaVersion.current().isJava8Compatible()) { if (JavaVersion.current() != JavaVersion.VERSION_1_8) {
sourceSets { sourceSets {
test { test {
groovy { groovy {
// These classes use Ratpack which requires Java 8. // These classes use Ratpack which requires Java 8. (Currently also incompatible with Java 9.)
exclude '**/OkHttp3Test.groovy' exclude '**/OkHttp3Test.groovy'
} }
} }

View File

@ -50,7 +50,7 @@ public final class FilterChain3Instrumentation implements Instrumenter {
"io.opentracing.contrib.web.servlet.filter.ServletFilterSpanDecorator$1", "io.opentracing.contrib.web.servlet.filter.ServletFilterSpanDecorator$1",
"io.opentracing.contrib.web.servlet.filter.TracingFilter", "io.opentracing.contrib.web.servlet.filter.TracingFilter",
"io.opentracing.contrib.web.servlet.filter.TracingFilter$1", "io.opentracing.contrib.web.servlet.filter.TracingFilter$1",
getClass().getName() + "$FilterChain3Advice$TagSettingAsyncListener")) FilterChain3Advice.class.getName() + "$TagSettingAsyncListener"))
.transform( .transform(
DDAdvice.create() DDAdvice.create()
.advice( .advice(

View File

@ -48,7 +48,7 @@ public final class HttpServlet3Instrumentation implements Instrumenter {
"io.opentracing.contrib.web.servlet.filter.ServletFilterSpanDecorator$1", "io.opentracing.contrib.web.servlet.filter.ServletFilterSpanDecorator$1",
"io.opentracing.contrib.web.servlet.filter.TracingFilter", "io.opentracing.contrib.web.servlet.filter.TracingFilter",
"io.opentracing.contrib.web.servlet.filter.TracingFilter$1", "io.opentracing.contrib.web.servlet.filter.TracingFilter$1",
getClass().getName() + "$HttpServlet3Advice$TagSettingAsyncListener")) HttpServlet3Advice.class.getName() + "$TagSettingAsyncListener"))
.transform( .transform(
DDAdvice.create() DDAdvice.create()
.advice( .advice(

View File

@ -2,6 +2,7 @@ apply from: "${rootDir}/gradle/java.gradle"
dependencies { dependencies {
compile deps.bytebuddy compile deps.bytebuddy
compile deps.bytebuddyagent
compile deps.slf4j compile deps.slf4j
testCompile deps.opentracing testCompile deps.opentracing

View File

@ -17,11 +17,11 @@ whitelistedInstructionClasses += whitelistedBranchClasses += [
'datadog.trace.common.sampling.PrioritySampling' 'datadog.trace.common.sampling.PrioritySampling'
] ]
if (!JavaVersion.current().isJava8Compatible()) { if (JavaVersion.current() != JavaVersion.VERSION_1_8) {
sourceSets { sourceSets {
test { test {
groovy { groovy {
// These classes use Ratpack which requires Java 8. // These classes use Ratpack which requires Java 8. (Currently also incompatible with Java 9.)
exclude 'datadog/trace/api/writer/DDApiTest.groovy' exclude 'datadog/trace/api/writer/DDApiTest.groovy'
} }
} }

View File

@ -35,9 +35,9 @@ ext {
], ],
bytebuddy : dependencies.create(group: 'net.bytebuddy', name: 'byte-buddy', version: "${version.bytebuddy}"), bytebuddy : dependencies.create(group: 'net.bytebuddy', name: 'byte-buddy', version: "${version.bytebuddy}"),
autoservice: [ autoservice: [
dependencies.create(group: 'com.google.auto.service', name: 'auto-service', version: '1.0-rc4'), dependencies.create(group: 'com.google.auto.service', name: 'auto-service', version: '1.0-rc3'),
dependencies.create(group: 'com.google.auto', name: 'auto-common', version: '0.3'), dependencies.create(group: 'com.google.auto', name: 'auto-common', version: '0.3'),
dependencies.create(group: 'com.google.guava', name: 'guava', version: '19.0'), dependencies.create(group: 'com.google.guava', name: 'guava', version: "${version.guava}"),
], ],
// Testing // Testing