Skip to content

Commit

Permalink
TraceContext cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
kevmoo committed Sep 20, 2023
1 parent f2b7add commit 7d72fa4
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
10 changes: 6 additions & 4 deletions server/lib/src/trace_context.dart
Expand Up @@ -3,10 +3,10 @@ import 'dart:typed_data';

class TraceContext {
final String traceId;
final String parentId;
final String spaceId;
final String traceFlags;

TraceContext._(this.traceId, this.parentId, this.traceFlags);
TraceContext._(this.traceId, this.spaceId, this.traceFlags);

factory TraceContext.parse(String input) {
final sections = input.split('-');
Expand All @@ -25,7 +25,9 @@ class TraceContext {
);
}

/// Returns a new [TraceContext] with a randomized [parentId].
/// Returns a new [TraceContext] with a randomized [spaceId].
///
/// If [random] is not provided a new instance of [Random] is used.
TraceContext randomize({Random? random}) {
random ??= Random();

Expand All @@ -46,7 +48,7 @@ class TraceContext {
}

@override
String toString() => '00-$traceId-$parentId-$traceFlags';
String toString() => '00-$traceId-$spaceId-$traceFlags';
}

final _hexThing = RegExp(r'^(?:[0-9a-f][0-9a-f])+$');
Expand Down
2 changes: 1 addition & 1 deletion server/test/trace_context_test.dart
Expand Up @@ -76,7 +76,7 @@ void main() {
final randomized = value.randomize();
expect(randomized.traceId, value.traceId);
expect(randomized.traceFlags, value.traceFlags);
expect(randomized.parentId, isNot(value.parentId));
expect(randomized.spaceId, isNot(value.spaceId));
expect(
TraceContext.parse(randomized.toString()).toString(),
randomized.toString(),
Expand Down

0 comments on commit 7d72fa4

Please sign in to comment.