Silencing Mock checking

To simplify the code, I pulled CoreTracer.getTagInterceptor() into a member variable of DDSpanContext

Unfortunately, that caused Mock call checks to fail, so needed to update the PendingTraceBufferTest

Author: dougqh

dd-trace-core/src/test/groovy/datadog/trace/core/PendingTraceBufferTest.groovy

@@ -95,6 +95,7 @@ class PendingTraceBufferTest extends DDSpecification {
     1 * bufferSpy.longRunningSpansEnabled()
     1 * bufferSpy.enqueue(trace)
     _ * tracer.getPartialFlushMinSpans() >> 10
+    _ * tracer.getTagInterceptor()
     1 * tracer.getTimeWithNanoTicks(_)
     1 * tracer.onRootSpanPublished(span)
     0 * _
@@ -128,6 +129,7 @@ class PendingTraceBufferTest extends DDSpecification {
     1 * bufferSpy.enqueue(trace)
     _ * bufferSpy.longRunningSpansEnabled()
     _ * tracer.getPartialFlushMinSpans() >> 10
+    _ * tracer.getTagInterceptor()
     1 * tracer.getTimeWithNanoTicks(_)
     1 * tracer.onRootSpanPublished(parent)
     0 * _
@@ -142,6 +144,7 @@ class PendingTraceBufferTest extends DDSpecification {
     1 * tracer.write({ it.size() == 2 })
     1 * tracer.writeTimer() >> Monitoring.DISABLED.newTimer("")
     _ * tracer.getPartialFlushMinSpans() >> 10
+    _ * tracer.getTagInterceptor()
     1 * tracer.getTimeWithNanoTicks(_)
     0 * _
   }
@@ -159,6 +162,7 @@ class PendingTraceBufferTest extends DDSpecification {
 
     then:
     _ * tracer.getPartialFlushMinSpans() >> 10
+    _ * tracer.getTagInterceptor()
     _ * traceConfig.getServiceMapping() >> [:]
     _ * tracer.getTimeWithNanoTicks(_)
     1 * tracer.writeTimer() >> Monitoring.DISABLED.newTimer("")
@@ -185,6 +189,7 @@ class PendingTraceBufferTest extends DDSpecification {
     buffer.queue.capacity() * bufferSpy.enqueue(_)
     _ * bufferSpy.longRunningSpansEnabled()
     _ * tracer.getPartialFlushMinSpans() >> 10
+    _ * tracer.getTagInterceptor()
     _ * traceConfig.getServiceMapping() >> [:]
     _ * tracer.getTimeWithNanoTicks(_)
     buffer.queue.capacity() * tracer.onRootSpanPublished(_)
@@ -201,6 +206,7 @@ class PendingTraceBufferTest extends DDSpecification {
     _ * bufferSpy.longRunningSpansEnabled()
     1 * tracer.write({ it.size() == 1 })
     _ * tracer.getPartialFlushMinSpans() >> 10
+    _ * tracer.getTagInterceptor()
     _ * traceConfig.getServiceMapping() >> [:]
     2 * tracer.getTimeWithNanoTicks(_)
     1 * tracer.onRootSpanPublished(_)
@@ -223,6 +229,7 @@ class PendingTraceBufferTest extends DDSpecification {
     buffer.queue.capacity() * bufferSpy.enqueue(_)
     _ * bufferSpy.longRunningSpansEnabled()
     _ * tracer.getPartialFlushMinSpans() >> 10
+    _ * tracer.getTagInterceptor()
     _ * traceConfig.getServiceMapping() >> [:]
     _ * tracer.getTimeWithNanoTicks(_)
     buffer.queue.capacity() * tracer.onRootSpanPublished(_)
@@ -241,6 +248,7 @@ class PendingTraceBufferTest extends DDSpecification {
     _ * bufferSpy.longRunningSpansEnabled()
     0 * tracer.write({ it.size() == 1 })
     _ * tracer.getPartialFlushMinSpans() >> 10
+    _ * tracer.getTagInterceptor()
     _ * traceConfig.getServiceMapping() >> [:]
     _ * tracer.getTimeWithNanoTicks(_)
     1 * tracer.onRootSpanPublished(_)
@@ -270,6 +278,7 @@ class PendingTraceBufferTest extends DDSpecification {
     _ * bufferSpy.longRunningSpansEnabled()
     1 * bufferSpy.enqueue(trace)
     _ * tracer.getPartialFlushMinSpans() >> 10
+    _ * tracer.getTagInterceptor()
     1 * tracer.getTimeWithNanoTicks(_)
     1 * tracer.onRootSpanPublished(parent)
     0 * _
@@ -298,6 +307,7 @@ class PendingTraceBufferTest extends DDSpecification {
       latch.countDown()
     }
     _ * tracer.getPartialFlushMinSpans() >> 10
+    _ * tracer.getTagInterceptor()
     0 * _
   }
 
@@ -324,6 +334,7 @@ class PendingTraceBufferTest extends DDSpecification {
       parentLatch.countDown()
     }
     _ * tracer.getPartialFlushMinSpans() >> 10
+    _ * tracer.getTagInterceptor()
     1 * tracer.getTimeWithNanoTicks(_)
     1 * tracer.onRootSpanPublished(parent)
     0 * _
@@ -340,6 +351,7 @@ class PendingTraceBufferTest extends DDSpecification {
     1 * bufferSpy.enqueue(trace)
     _ * bufferSpy.longRunningSpansEnabled()
     _ * tracer.getPartialFlushMinSpans() >> 10
+    _ * tracer.getTagInterceptor()
     1 * tracer.writeTimer() >> Monitoring.DISABLED.newTimer("")
     1 * tracer.write({ it.size() == 1 }) >> {
       childLatch.countDown()
@@ -419,6 +431,7 @@ class PendingTraceBufferTest extends DDSpecification {
     1 * tracer.writeTimer() >> Monitoring.DISABLED.newTimer("")
     1 * tracer.write({ it.size() == 1 })
     1 * tracer.getPartialFlushMinSpans() >> 10000
+    _ * tracer.getTagInterceptor()
     1 * traceConfig.getServiceMapping() >> [:]
     2 * tracer.getTimeWithNanoTicks(_)
     1 * tracer.onRootSpanPublished(_)
@@ -435,6 +448,7 @@ class PendingTraceBufferTest extends DDSpecification {
     buffer.queue.capacity() * bufferSpy.enqueue(_)
     _ * bufferSpy.longRunningSpansEnabled()
     _ * tracer.getPartialFlushMinSpans() >> 10000
+    _ * tracer.getTagInterceptor()
     _ * traceConfig.getServiceMapping() >> [:]
     _ * tracer.getTimeWithNanoTicks(_)
     0 * _