Skip to content

ref: SentryId generates UUID #119

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
marandaneto merged 8 commits into from
Oct 21, 2020
Merged

ref: SentryId generates UUID #119

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
1c663b4
ref: sentryId generates uuid
marandaneto Oct 20, 2020
88436d0
changelog
marandaneto Oct 20, 2020
627f774
ref
marandaneto Oct 20, 2020
45e685e
fix tests
marandaneto Oct 20, 2020
95cacb3
fix tests
marandaneto Oct 20, 2020
3d989d5
fix merge conflicts
marandaneto Oct 20, 2020
667fa87
fix test
marandaneto Oct 20, 2020
0f3c104
fix
marandaneto Oct 20, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix tests
  • Loading branch information
@marandaneto
marandaneto committed Oct 20, 2020
commit 95cacb33e9ebc98d00ae8f82e402f391339c238c
4 changes: 4 additions & 0 deletions dart/test/event_test.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,13 +26,15 @@ void main() {
});
test('$Sdk serializes', () {
final event = Event(
eventId: SentryId.empty(),
sdk: Sdk(
name: 'sentry.dart.flutter',
version: '4.3.2',
integrations: <String>['integration'],
packages: <Package>[Package('npm:@sentry/javascript', '1.3.4')]));
expect(event.toJson(), <String, dynamic>{
'platform': 'dart',
'event_id': '00000000000000000000000000000000',
'sdk': {
'name': 'sentry.dart.flutter',
'version': '4.3.2',
Expand Down Expand Up @@ -62,6 +64,7 @@ void main() {

expect(
Event(
eventId: SentryId.empty(),
message: Message(
'test-message 1 2',
template: 'test-message %d %d',
Expand All @@ -85,6 +88,7 @@ void main() {
).toJson(),
<String, dynamic>{
'platform': 'dart',
'event_id': '00000000000000000000000000000000',
'sdk': {'version': sdkVersion, 'name': 'sentry.dart'},
'message': {
'formatted': 'test-message 1 2',
Expand Down
15 changes: 10 additions & 5 deletions dart/test/test_utils.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -350,12 +350,17 @@ void runTest({Codec<List<int>, List<int>> gzip, bool isWeb = false}) {
try {
throw ArgumentError('Test error');
} catch (error, stackTrace) {
final eventWithoutContext =
Event(exception: error, stackTrace: stackTrace);
final eventWithoutContext = Event(
eventId: SentryId.empty(),
exception: error,
stackTrace: stackTrace,
);
final eventWithContext = Event(
exception: error,
stackTrace: stackTrace,
userContext: eventUserContext);
eventId: SentryId.empty(),
exception: error,
stackTrace: stackTrace,
userContext: eventUserContext,
);
await client.captureEvent(eventWithoutContext);
expect(loggedUserId, clientUserContext.id);
await client.captureEvent(eventWithContext);
Expand Down