Skip to content

Commit ba98146

Browse files
Ignore tests.
1 parent 27c46bc commit ba98146

File tree

4 files changed

+11
-13
lines changed

4 files changed

+11
-13
lines changed

dd-java-agent/instrumentation/jackson-core/jackson-core-1/src/test/groovy/Json1ParserInstrumentationTest.groovy

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -33,13 +33,12 @@ class Json1ParserInstrumentationTest extends InstrumentationSpecification {
3333
JsonOutput.toJson(taintedResult) == JSON_STRING
3434
_ * module.taintObjectIfTainted(_, _)
3535
_ * module.findSource(_) >> source
36-
// TODO: Groovy 4 issues
37-
// 1 * module.taintString(_, 'root', source.origin, 'root', JSON_STRING)
38-
// 1 * module.taintString(_, 'root_value', source.origin, 'root', JSON_STRING)
39-
// 1 * module.taintString(_, 'nested', source.origin, 'nested', JSON_STRING)
40-
// 1 * module.taintString(_, 'nested_array', source.origin, 'nested_array', JSON_STRING)
41-
// 1 * module.taintString(_, 'array_0', source.origin, 'nested_array', JSON_STRING)
42-
// 1 * module.taintString(_, 'array_1', source.origin, 'nested_array', JSON_STRING)
36+
1 * module.taintString(_, 'root', source.origin, 'root', JSON_STRING)
37+
1 * module.taintString(_, 'root_value', source.origin, 'root', JSON_STRING)
38+
1 * module.taintString(_, 'nested', source.origin, 'nested', JSON_STRING)
39+
1 * module.taintString(_, 'nested_array', source.origin, 'nested_array', JSON_STRING)
40+
1 * module.taintString(_, 'array_0', source.origin, 'nested_array', JSON_STRING)
41+
1 * module.taintString(_, 'array_1', source.origin, 'nested_array', JSON_STRING)
4342
0 * _
4443

4544
where:

dd-java-agent/instrumentation/jackson-core/jackson-core-2.16/src/test/groovy/datadog/trace/instrumentation/jackson216/core/JsonParserInstrumentationTest.groovy

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,9 @@ class JsonParserInstrumentationTest extends InstrumentationSpecification {
3636
JsonOutput.toJson(taintedResult) == JSON_STRING
3737
_ * module.taintObjectIfTainted(_, _)
3838
_ * module.findSource(_) >> source
39-
// TODO: Groovy 4 issues
40-
// 1 * module.taintString(_, 'root', source.origin, 'root', JSON_STRING)
41-
// 1 * module.taintString(_, 'nested', source.origin, 'nested', JSON_STRING)
42-
// 1 * module.taintString(_, 'nested_array', source.origin, 'nested_array', JSON_STRING)
39+
1 * module.taintString(_, 'root', source.origin, 'root', JSON_STRING)
40+
1 * module.taintString(_, 'nested', source.origin, 'nested', JSON_STRING)
41+
1 * module.taintString(_, 'nested_array', source.origin, 'nested_array', JSON_STRING)
4342
0 * _
4443
}
4544

dd-smoke-tests/maven/src/test/groovy/datadog/smoketest/MavenSmokeTest.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,12 @@ import okhttp3.OkHttpClient
88
import okhttp3.Request
99
import okhttp3.Response
1010
import org.apache.maven.wrapper.MavenWrapperMain
11-
import org.junit.Ignore
1211
import org.slf4j.Logger
1312
import org.slf4j.LoggerFactory
1413
import org.w3c.dom.Document
1514
import org.w3c.dom.NodeList
1615
import spock.lang.AutoCleanup
16+
import spock.lang.Ignore
1717
import spock.lang.Shared
1818
import spock.lang.TempDir
1919
import spock.util.environment.Jvm

settings.gradle.kts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -435,7 +435,7 @@ include(
435435
":dd-java-agent:instrumentation:junit:junit-5.3",
436436
":dd-java-agent:instrumentation:junit:junit-5.3:junit-5.8",
437437
":dd-java-agent:instrumentation:junit:junit-5.3:cucumber-junit-5",
438-
// ":dd-java-agent:instrumentation:junit:junit-5.3:spock-junit-5", TODO fix Groovy 4.x compatibility probably this module needs Groovy 3
438+
":dd-java-agent:instrumentation:junit:junit-5.3:spock-junit-5",
439439
":dd-java-agent:instrumentation:kafka:kafka-clients-0.11",
440440
":dd-java-agent:instrumentation:kafka:kafka-clients-3.8",
441441
":dd-java-agent:instrumentation:kafka:kafka-common",

0 commit comments

Comments
 (0)