ExtensionTracer should set traceId on spans (#157)

* Failing test for extension not preserving traceId

* ExtensionTracer correctly sets traceId on returned spans

Author: chingor13

src/Trace/Tracer/ExtensionTracer.php

@@ -93,6 +93,7 @@ public function withSpan(Span $span)
             ? (float)($spanData->startTime()->format('U.u'))
             : microtime(true);
         $info = [
+            'traceId' => $spanData->traceId(),
             'spanId' => $spanData->spanId(),
             'parentSpanId' => $spanData->parentSpanId(),
             'startTime' => $startTime,
@@ -117,7 +118,10 @@ public function spans()
     {
         // each span returned from opencensus_trace_list should be a
         // OpenCensus\Span object
-        return array_map([$this, 'mapSpan'], opencensus_trace_list());
+        $traceId = $this->spanContext()->traceId();
+        return array_map(function ($span) use ($traceId) {
+            return $this->mapSpan($span, $traceId);
+        }, opencensus_trace_list());
     }
 
     /**
@@ -246,9 +250,10 @@ private function generateSpanName()
         return uniqid('span');
     }
 
-    private function mapSpan($span)
+    private function mapSpan($span, $traceId)
     {
         return new Span([
+            'traceId' => $traceId,
             'name' => $span->name(),
             'spanId' => $span->spanId(),
             'parentSpanId' => $span->parentSpanId(),

tests/unit/Trace/Tracer/AbstractTracerTest.php

@@ -48,6 +48,7 @@ public function testMaintainsContext()
         $spans = $tracer->spans();
         $this->assertCount(1, $spans);
         $spanData = $spans[0]->spanData();
+        $this->assertEquals('traceid', $spanData->traceId());
         $this->assertEquals('test', $spanData->name());
         $this->assertEquals($parentSpanId, $spanData->parentSpanId());
     }