Fixed Groovy ambiguity. Tweaked

Author: AlexeyKuznetsov-DD

dd-java-agent/agent-bootstrap/src/test/groovy/datadog/trace/bootstrap/instrumentation/decorator/BaseDecoratorTest.groovy

@@ -80,7 +80,7 @@ class BaseDecoratorTest extends DDSpecification {
     decorator.beforeFinish(span)
 
     then:
-    (0..1) * span.localRootSpan
+    (0..1) * span.getLocalRootSpan()
     0 * _
   }
 

dd-java-agent/agent-bootstrap/src/test/groovy/datadog/trace/bootstrap/instrumentation/decorator/ServerDecoratorTest.groovy

@@ -39,7 +39,7 @@ class ServerDecoratorTest extends BaseDecoratorTest {
     newDecorator().beforeFinish(span)
 
     then:
-    (0..1) * span.localRootSpan
+    (0..1) * span.getLocalRootSpan()
   }
 
   @Override

dd-java-agent/instrumentation-testing/src/main/groovy/datadog/trace/agent/test/ValidatingRequestContextDecorator.groovy

@@ -24,7 +24,7 @@ class ValidatingRequestContextDecorator implements RequestContext {
     this.traceSegment = new PreconditionCheckTraceSegment(
       segment, {
         ->
-        if (useStrictTraceWrites && spiedAgentSpan.localRootSpan.durationNano != 0) {
+        if (useStrictTraceWrites && spiedAgentSpan.getLocalRootSpan().durationNano != 0) {
           throw new AssertionError("Interaction with TraceSegment after root span has already finished: $spiedAgentSpan")
         }
       }

dd-java-agent/instrumentation-testing/src/main/groovy/datadog/trace/agent/test/base/HttpServerTest.groovy

@@ -2536,7 +2536,7 @@ abstract class HttpServerTest<SERVER> extends WithHttpServer<SERVER> {
       if (!context.requestBodySupplier.is(supplier)) {
         throw new RuntimeException("Expected same instance: ${context.requestBodySupplier} and $supplier")
       }
-      activeSpan().localRootSpan.setTag('request.body', supplier.get() as String)
+      activeSpan().getLocalRootSpan().setTag('request.body', supplier.get() as String)
       if (context.bodyEndBlock) {
         new RbaFlow(
         new Flow.Action.RequestBlockingAction(413, BlockingContentType.JSON)

dd-java-agent/instrumentation/google-pubsub/src/test/groovy/PubSubTest.groovy

@@ -171,7 +171,7 @@ abstract class PubSubTest extends VersionedNamingTestBase {
     assertTraces(shadowGrpcSpans() ? 2 : 3, [
       compare            : { List<DDSpan> o1, List<DDSpan> o2 ->
         // trace will never be empty
-        o1[0].localRootSpan.getTag(Tags.SPAN_KIND) <=> o2[0].localRootSpan.getTag(Tags.SPAN_KIND)
+        o1[0].getLocalRootSpan().getTag(Tags.SPAN_KIND) <=> o2[0].getLocalRootSpan().getTag(Tags.SPAN_KIND)
       },
     ] as Comparator) {
       trace(shadowGrpcSpans() ? 2 : 3) {

dd-java-agent/instrumentation/kafka/kafka-clients-0.11/src/latestDepTest/groovy/KafkaClientTestBase.groovy

@@ -75,7 +75,7 @@ abstract class KafkaClientTestBase extends VersionedNamingTestBase {
 
     int rootSpanTrace(List<DDSpan> trace) {
       assert !trace.isEmpty()
-      def rootSpan = trace.get(0).localRootSpan
+      def rootSpan = trace.get(0).getLocalRootSpan()
       switch (rootSpan.operationName.toString()) {
         case "parent":
           return 3

dd-java-agent/instrumentation/kafka/kafka-clients-0.11/src/test/groovy/KafkaClientTestBase.groovy

@@ -85,7 +85,7 @@ abstract class KafkaClientTestBase extends VersionedNamingTestBase {
 
     int rootSpanTrace(List<DDSpan> trace) {
       assert !trace.isEmpty()
-      def rootSpan = trace.get(0).localRootSpan
+      def rootSpan = trace.get(0).getLocalRootSpan()
       switch (rootSpan.operationName.toString()) {
         case "parent":
           return 3

dd-java-agent/instrumentation/kafka/kafka-clients-3.8/src/test/groovy/KafkaClientTestBase.groovy

@@ -98,7 +98,7 @@ abstract class KafkaClientTestBase extends VersionedNamingTestBase {
 
     int rootSpanTrace(List<DDSpan> trace) {
       assert !trace.isEmpty()
-      def rootSpan = trace.get(0).localRootSpan
+      def rootSpan = trace.get(0).getLocalRootSpan()
       switch (rootSpan.operationName.toString()) {
         case "parent":
           return 3

dd-java-agent/instrumentation/opentelemetry/opentelemetry-0.3/src/test/groovy/OpenTelemetryTest.groovy

@@ -50,7 +50,7 @@ class OpenTelemetryTest extends InstrumentationSpecification {
 
     expect:
     result instanceof MutableSpan
-    (result as MutableSpan).localRootSpan == result.delegate
+    (result as MutableSpan).getLocalRootSpan() == result.delegate
     tracer.currentSpan == null
 
     when:
@@ -108,7 +108,7 @@ class OpenTelemetryTest extends InstrumentationSpecification {
 
     expect:
     result instanceof MutableSpan
-    (result as MutableSpan).localRootSpan == result.delegate
+    (result as MutableSpan).getLocalRootSpan() == result.delegate
     (result as MutableSpan).isError() == (exception != null)
     tracer.currentSpan == null
 
@@ -150,7 +150,7 @@ class OpenTelemetryTest extends InstrumentationSpecification {
 
     expect:
     result instanceof MutableSpan
-    (result as MutableSpan).localRootSpan == result.delegate
+    (result as MutableSpan).getLocalRootSpan() == result.delegate
     tracer.currentSpan == null
 
     when:

dd-java-agent/instrumentation/opentracing/api-0.31/src/test/groovy/OpenTracing31Test.groovy

@@ -64,7 +64,7 @@ class OpenTracing31Test extends InstrumentationSpecification {
 
     expect:
     result instanceof MutableSpan
-    (result as MutableSpan).localRootSpan.delegate == result.delegate
+    (result as MutableSpan).getLocalRootSpan().delegate == result.delegate
     (result as MutableSpan).isError() == (exception != null)
     tracer.activeSpan() == null
     result.context().baggageItems().isEmpty()