A small test cleanup (#3014)
This commit is contained in:
parent
8be4148dea
commit
32399ea7a9
|
@ -15,7 +15,7 @@ class JaxWsAnnotationsTest extends AgentInstrumentationSpecification {
|
|||
new SoapProvider().invoke(null)
|
||||
|
||||
then:
|
||||
assertTraces(1, {
|
||||
assertTraces(1) {
|
||||
trace(0, 1) {
|
||||
span(0) {
|
||||
name 'SoapProvider.invoke'
|
||||
|
@ -25,6 +25,6 @@ class JaxWsAnnotationsTest extends AgentInstrumentationSpecification {
|
|||
}
|
||||
}
|
||||
}
|
||||
})
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
@ -20,7 +20,7 @@ class JwsAnnotationsTest extends AgentInstrumentationSpecification {
|
|||
new WebServiceClass().doSomethingProtected()
|
||||
|
||||
then:
|
||||
assertTraces(1, {
|
||||
assertTraces(1) {
|
||||
trace(0, 1) {
|
||||
span(0) {
|
||||
name "WebServiceClass.doSomethingPublic"
|
||||
|
@ -30,7 +30,7 @@ class JwsAnnotationsTest extends AgentInstrumentationSpecification {
|
|||
}
|
||||
}
|
||||
}
|
||||
})
|
||||
}
|
||||
}
|
||||
|
||||
def "WebService via interface generates spans only for methods of the interface"() {
|
||||
|
@ -40,7 +40,7 @@ class JwsAnnotationsTest extends AgentInstrumentationSpecification {
|
|||
new WebServiceFromInterface().notPartOfAnything()
|
||||
|
||||
then:
|
||||
assertTraces(1, {
|
||||
assertTraces(1) {
|
||||
trace(0, 1) {
|
||||
span(0) {
|
||||
name "WebServiceFromInterface.partOfPublicInterface"
|
||||
|
@ -50,7 +50,7 @@ class JwsAnnotationsTest extends AgentInstrumentationSpecification {
|
|||
}
|
||||
}
|
||||
}
|
||||
})
|
||||
}
|
||||
}
|
||||
|
||||
def "WebService via proxy must have span attributes from actual implementation"() {
|
||||
|
@ -64,7 +64,7 @@ class JwsAnnotationsTest extends AgentInstrumentationSpecification {
|
|||
|
||||
then:
|
||||
proxy.getClass() != WebServiceFromInterface
|
||||
assertTraces(1, {
|
||||
assertTraces(1) {
|
||||
trace(0, 1) {
|
||||
span(0) {
|
||||
name "WebServiceFromInterface.partOfPublicInterface"
|
||||
|
@ -74,6 +74,6 @@ class JwsAnnotationsTest extends AgentInstrumentationSpecification {
|
|||
}
|
||||
}
|
||||
}
|
||||
})
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
@ -162,7 +162,7 @@ class Jms2Test extends AgentInstrumentationSpecification {
|
|||
expect:
|
||||
receivedMessage == null
|
||||
// span is not created if no message is received
|
||||
assertTraces(0, {})
|
||||
assertTraces(0) {}
|
||||
|
||||
cleanup:
|
||||
consumer.close()
|
||||
|
@ -183,7 +183,7 @@ class Jms2Test extends AgentInstrumentationSpecification {
|
|||
expect:
|
||||
receivedMessage == null
|
||||
// span is not created if no message is received
|
||||
assertTraces(0, {})
|
||||
assertTraces(0) {}
|
||||
|
||||
cleanup:
|
||||
consumer.close()
|
||||
|
|
|
@ -137,7 +137,7 @@ class Jms1Test extends AgentInstrumentationSpecification {
|
|||
expect:
|
||||
receivedMessage == null
|
||||
// span is not created if no message is received
|
||||
assertTraces(0, {})
|
||||
assertTraces(0) {}
|
||||
|
||||
cleanup:
|
||||
consumer.close()
|
||||
|
@ -158,7 +158,7 @@ class Jms1Test extends AgentInstrumentationSpecification {
|
|||
expect:
|
||||
receivedMessage == null
|
||||
// span is not created if no message is received
|
||||
assertTraces(0, {})
|
||||
assertTraces(0) {}
|
||||
|
||||
cleanup:
|
||||
consumer.close()
|
||||
|
|
Loading…
Reference in New Issue