Emit transaction.data in contexts.trace.data (#2284)

* add

* set data to null if empty

* update testn ame

* Add CHANGELOG entry

Author: buenaflor

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 ### Features
 
+- Emit `transaction.data` inside `contexts.trace.data` ([#2284](https://github.com/getsentry/sentry-dart/pull/2284))
 - Blocking app starts if "appLaunchedInForeground" is false. (Android only) ([#2291](https://github.com/getsentry/sentry-dart/pull/2291)
 
 ### Enhancements

dart/lib/src/protocol/sentry_trace_context.dart

@@ -41,6 +41,8 @@ class SentryTraceContext {
   /// @see <https://develop.sentry.dev/sdk/performance/trace-origin>
   final String? origin;
 
+  final Map<String, dynamic>? data;
+
   @internal
   final Map<String, dynamic>? unknown;
 
@@ -62,6 +64,7 @@ class SentryTraceContext {
           : SpanStatus.fromString(json['status'] as String),
       sampled: true,
       origin: json['origin'] == null ? null : json['origin'] as String?,
+      data: json['data'] == null ? null : json['data'] as Map<String, dynamic>,
       unknown: json.notAccessed(),
     );
   }
@@ -78,6 +81,7 @@ class SentryTraceContext {
       if (description != null) 'description': description,
       if (status != null) 'status': status!.toString(),
       if (origin != null) 'origin': origin,
+      if (data != null) 'data': data,
     };
   }
 
@@ -92,6 +96,7 @@ class SentryTraceContext {
         origin: origin,
         unknown: unknown,
         replayId: replayId,
+        data: data,
       );
 
   SentryTraceContext({
@@ -105,6 +110,7 @@ class SentryTraceContext {
     this.origin,
     this.unknown,
     this.replayId,
+    this.data,
   })  : traceId = traceId ?? SentryId.newId(),
         spanId = spanId ?? SpanId.newId();
 

dart/lib/src/protocol/sentry_transaction.dart

@@ -57,9 +57,11 @@ class SentryTransaction extends SentryEvent {
     this.metricSummaries =
         metricSummaries ?? tracer.localMetricsAggregator?.getSummaries();
 
+    final data = extra ?? tracer.data;
     contexts.trace = spanContext.toTraceContext(
       sampled: tracer.samplingDecision?.sampled,
       status: tracer.status,
+      data: data.isEmpty ? null : data,
     );
 
     this.transactionInfo = transactionInfo ??

dart/lib/src/sentry_span_context.dart

@@ -53,6 +53,7 @@ class SentrySpanContext {
   SentryTraceContext toTraceContext({
     bool? sampled,
     SpanStatus? status,
+    Map<String, dynamic>? data,
   }) {
     return SentryTraceContext(
       operation: operation,
@@ -63,6 +64,7 @@ class SentrySpanContext {
       sampled: sampled,
       status: status,
       origin: origin,
+      data: data,
     );
   }
 }

dart/test/sentry_trace_context_test.dart

@@ -19,6 +19,7 @@ void main() {
     expect(map['status'], 'aborted');
     expect(map['origin'], 'auto.ui');
     expect(map['replay_id'], isNotNull);
+    expect(map['data'], {'key': 'value'});
   });
 
   test('fromJson deserializes', () {
@@ -30,7 +31,8 @@ void main() {
       'description': 'desc',
       'status': 'aborted',
       'origin': 'auto.ui',
-      'replay_id': '00000000000000000000000000000004'
+      'replay_id': '00000000000000000000000000000004',
+      'data': {'key': 'value'},
     };
     map.addAll(testUnknown);
     final traceContext = SentryTraceContext.fromJson(map);
@@ -44,6 +46,7 @@ void main() {
     expect(traceContext.sampled, true);
     expect(
         traceContext.replayId.toString(), '00000000000000000000000000000004');
+    expect(traceContext.data, {'key': 'value'});
   });
 }
 
@@ -57,6 +60,7 @@ class Fixture {
       status: SpanStatus.aborted(),
       origin: 'auto.ui',
       replayId: SentryId.newId(),
+      data: {'key': 'value'},
       unknown: testUnknown,
     );
   }

dart/test/sentry_transaction_test.dart

@@ -65,6 +65,29 @@ void main() {
     expect(sut.sampled, true);
   });
 
+  test('returns contexts.trace.data if data is set', () async {
+    final tracer = _createTracer(sampled: true);
+    tracer.setData('key', 'value');
+    final child = tracer.startChild('child');
+    await child.finish();
+    await tracer.finish();
+
+    final sut = fixture.getSut(tracer);
+
+    expect(sut.contexts.trace!.data, {'key': 'value'});
+  });
+
+  test('returns null contexts.trace.data if data is not set', () async {
+    final tracer = _createTracer(sampled: true);
+    final child = tracer.startChild('child');
+    await child.finish();
+    await tracer.finish();
+
+    final sut = fixture.getSut(tracer);
+
+    expect(sut.contexts.trace!.data, isNull);
+  });
+
   test('returns sampled false if not sampled', () async {
     final tracer = _createTracer(sampled: false);
     final child = tracer.startChild('child');