Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
347b8f4
Prepare breadcrumb tests
denrase Sep 19, 2023
12f0b6d
Merge branch 'main' into feat/database-breadcrumbs
denrase Sep 25, 2023
6de44bb
create breadcrumbs for batch operations
denrase Sep 25, 2023
bd9a135
add breadcrumbs to sentry database & executor
denrase Sep 25, 2023
2e6d025
create breadcrumb when opening db
denrase Sep 25, 2023
b7a2fbc
create breadcrumb in SentrySqfliteDatabaseFactory
denrase Sep 25, 2023
86d796e
choose correct breadcrumb in batch tests
denrase Sep 26, 2023
cfc7715
set additional error info in batch
denrase Sep 26, 2023
98ee0af
set query and error breadcrumb type
denrase Sep 26, 2023
a38cfd5
update in sqflite and database factory
denrase Sep 26, 2023
932ca1c
Merge branch 'main' into feat/database-breadcrumbs
denrase Sep 26, 2023
aa66f4a
run format
denrase Sep 26, 2023
0382383
add changelog entries
denrase Sep 26, 2023
2778594
bump ios im runtime
denrase Sep 26, 2023
9978a74
manually set deployemtn target for pods
denrase Oct 3, 2023
26a77fc
Merge branch 'main' into feat/database-breadcrumbs
denrase Oct 17, 2023
788364d
Change status value
denrase Oct 17, 2023
e58a4d8
Merge branch 'main' into feat/database-breadcrumbs
denrase Oct 24, 2023
af3d295
change breadcrumb type and level
denrase Oct 24, 2023
1a5d4e3
remove duplicated lines
denrase Oct 24, 2023
aaf22b4
set type query to success breadcrumbs
denrase Oct 24, 2023
24f79d4
Merge branch 'main' into feat/database-breadcrumbs
denrase Oct 30, 2023
73d740a
fix changelog
denrase Oct 30, 2023
2c353b2
remove remaining error level
denrase Oct 31, 2023
6830e50
Merge branch 'main' into feat/database-breadcrumbs
denrase Oct 31, 2023
a20d194
fix changelog
denrase Oct 31, 2023
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
change breadcrumb type and level
  • Loading branch information
denrase committed Oct 24, 2023
commit af3d295d769b27916f338aca7e610d1b6a9afd70
8 changes: 4 additions & 4 deletions sqflite/lib/src/sentry_batch.dart
Original file line number Diff line number Diff line change
Expand Up @@ -79,8 +79,8 @@ class SentryBatch implements Batch {

breadcrumb.data?['status'] = 'internal_error';
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);

rethrow;
Expand Down Expand Up @@ -133,8 +133,8 @@ class SentryBatch implements Batch {

breadcrumb.data?['status'] = 'internal_error';
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);

rethrow;
Expand Down
8 changes: 4 additions & 4 deletions sqflite/lib/src/sentry_database.dart
Original file line number Diff line number Diff line change
Expand Up @@ -99,8 +99,8 @@ class SentryDatabase extends SentryDatabaseExecutor implements Database {
span?.status = SpanStatus.internalError();
breadcrumb.data?['status'] = 'internal_error';
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);

rethrow;
Expand Down Expand Up @@ -183,8 +183,8 @@ class SentryDatabase extends SentryDatabaseExecutor implements Database {
span?.status = SpanStatus.internalError();
breadcrumb.data?['status'] = 'internal_error';
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);

rethrow;
Expand Down
48 changes: 24 additions & 24 deletions sqflite/lib/src/sentry_database_executor.dart
Original file line number Diff line number Diff line change
Expand Up @@ -67,8 +67,8 @@ class SentryDatabaseExecutor implements DatabaseExecutor {
span?.status = SpanStatus.internalError();
breadcrumb.data?['status'] = 'internal_error';
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);

rethrow;
Expand Down Expand Up @@ -110,12 +110,12 @@ class SentryDatabaseExecutor implements DatabaseExecutor {
span?.status = SpanStatus.internalError();
breadcrumb.data?['status'] = 'internal_error';
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);

rethrow;
Expand Down Expand Up @@ -174,8 +174,8 @@ class SentryDatabaseExecutor implements DatabaseExecutor {
span?.status = SpanStatus.internalError();
breadcrumb.data?['status'] = 'internal_error';
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);

rethrow;
Expand Down Expand Up @@ -253,8 +253,8 @@ class SentryDatabaseExecutor implements DatabaseExecutor {
span?.status = SpanStatus.internalError();
breadcrumb.data?['status'] = 'internal_error';
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);

rethrow;
Expand Down Expand Up @@ -334,8 +334,8 @@ class SentryDatabaseExecutor implements DatabaseExecutor {
span?.status = SpanStatus.internalError();
breadcrumb.data?['status'] = 'internal_error';
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);

rethrow;
Expand Down Expand Up @@ -378,8 +378,8 @@ class SentryDatabaseExecutor implements DatabaseExecutor {
span?.status = SpanStatus.internalError();
breadcrumb.data?['status'] = 'internal_error';
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);

rethrow;
Expand Down Expand Up @@ -422,8 +422,8 @@ class SentryDatabaseExecutor implements DatabaseExecutor {
span?.status = SpanStatus.internalError();
breadcrumb.data?['status'] = 'internal_error';
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);

rethrow;
Expand Down Expand Up @@ -470,8 +470,8 @@ class SentryDatabaseExecutor implements DatabaseExecutor {
span?.status = SpanStatus.internalError();
breadcrumb.data?['status'] = 'internal_error';
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);

rethrow;
Expand Down Expand Up @@ -523,8 +523,8 @@ class SentryDatabaseExecutor implements DatabaseExecutor {
span?.status = SpanStatus.internalError();
breadcrumb.data?['status'] = 'internal_error';
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);

rethrow;
Expand Down Expand Up @@ -567,8 +567,8 @@ class SentryDatabaseExecutor implements DatabaseExecutor {
span?.status = SpanStatus.internalError();
breadcrumb.data?['status'] = 'internal_error';
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);

rethrow;
Expand Down Expand Up @@ -630,8 +630,8 @@ class SentryDatabaseExecutor implements DatabaseExecutor {
span?.status = SpanStatus.internalError();
breadcrumb.data?['status'] = 'internal_error';
breadcrumb = breadcrumb.copyWith(
type: 'error',
level: SentryLevel.error,
type: 'query',
level: SentryLevel.warning,
);

rethrow;
Expand Down
8 changes: 4 additions & 4 deletions sqflite/test/sentry_batch_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -615,8 +615,8 @@ SELECT * FROM Product''';

final breadcrumb = fixture.hub.scope.breadcrumbs.last;
expect(breadcrumb.data?['status'], 'internal_error');
expect(breadcrumb.type, 'error');
expect(breadcrumb.level, SentryLevel.error);
expect(breadcrumb.type, 'query');
expect(breadcrumb.level, SentryLevel.warning);
});

test('apply sets span to internal error if its thrown', () async {
Expand Down Expand Up @@ -646,8 +646,8 @@ SELECT * FROM Product''';

final breadcrumb = fixture.hub.scope.breadcrumbs.last;
expect(breadcrumb.data?['status'], 'internal_error');
expect(breadcrumb.type, 'error');
expect(breadcrumb.level, SentryLevel.error);
expect(breadcrumb.type, 'query');
expect(breadcrumb.level, SentryLevel.warning);
});
});
}
Expand Down
52 changes: 26 additions & 26 deletions sqflite/test/sentry_database_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -242,8 +242,8 @@ void main() {

final breadcrumb = fixture.hub.scope.breadcrumbs.first;
expect(breadcrumb.data?['status'], 'internal_error');
expect(breadcrumb.type, 'error');
expect(breadcrumb.level, SentryLevel.error);
expect(breadcrumb.type, 'query');
expect(breadcrumb.level, SentryLevel.warning);
});

test('transaction sets span to internal error', () async {
Expand Down Expand Up @@ -281,8 +281,8 @@ void main() {

final breadcrumb = fixture.hub.scope.breadcrumbs.first;
expect(breadcrumb.data?['status'], 'internal_error');
expect(breadcrumb.type, 'error');
expect(breadcrumb.level, SentryLevel.error);
expect(breadcrumb.type, 'query');
expect(breadcrumb.level, SentryLevel.warning);
});
});

Expand Down Expand Up @@ -1007,8 +1007,8 @@ void main() {

final breadcrumb = fixture.hub.scope.breadcrumbs.first;
expect(breadcrumb.data?['status'], 'internal_error');
expect(breadcrumb.type, 'error');
expect(breadcrumb.level, SentryLevel.error);
expect(breadcrumb.type, 'query');
expect(breadcrumb.level, SentryLevel.warning);
});

test('execute sets breadcrumb to internal error', () async {
Expand All @@ -1023,8 +1023,8 @@ void main() {

final breadcrumb = fixture.hub.scope.breadcrumbs.first;
expect(breadcrumb.data?['status'], 'internal_error');
expect(breadcrumb.type, 'error');
expect(breadcrumb.level, SentryLevel.error);
expect(breadcrumb.type, 'query');
expect(breadcrumb.level, SentryLevel.warning);
});

test('insert sets breadcrumb to internal error', () async {
Expand All @@ -1040,8 +1040,8 @@ void main() {

final breadcrumb = fixture.hub.scope.breadcrumbs.first;
expect(breadcrumb.data?['status'], 'internal_error');
expect(breadcrumb.type, 'error');
expect(breadcrumb.level, SentryLevel.error);
expect(breadcrumb.type, 'query');
expect(breadcrumb.level, SentryLevel.warning);
});

test('query sets breadcrumb to internal error', () async {
Expand All @@ -1056,8 +1056,8 @@ void main() {

final breadcrumb = fixture.hub.scope.breadcrumbs.first;
expect(breadcrumb.data?['status'], 'internal_error');
expect(breadcrumb.type, 'error');
expect(breadcrumb.level, SentryLevel.error);
expect(breadcrumb.type, 'query');
expect(breadcrumb.level, SentryLevel.warning);
});

test('query cursor sets breadcrumb to internal error', () async {
Expand All @@ -1072,8 +1072,8 @@ void main() {

final breadcrumb = fixture.hub.scope.breadcrumbs.first;
expect(breadcrumb.data?['status'], 'internal_error');
expect(breadcrumb.type, 'error');
expect(breadcrumb.level, SentryLevel.error);
expect(breadcrumb.type, 'query');
expect(breadcrumb.level, SentryLevel.warning);
});

test('raw delete sets breadcrumb to internal error', () async {
Expand All @@ -1088,8 +1088,8 @@ void main() {

final breadcrumb = fixture.hub.scope.breadcrumbs.first;
expect(breadcrumb.data?['status'], 'internal_error');
expect(breadcrumb.type, 'error');
expect(breadcrumb.level, SentryLevel.error);
expect(breadcrumb.type, 'query');
expect(breadcrumb.level, SentryLevel.warning);
});

test('raw insert sets breadcrumb to internal error', () async {
Expand All @@ -1104,8 +1104,8 @@ void main() {

final breadcrumb = fixture.hub.scope.breadcrumbs.first;
expect(breadcrumb.data?['status'], 'internal_error');
expect(breadcrumb.type, 'error');
expect(breadcrumb.level, SentryLevel.error);
expect(breadcrumb.type, 'query');
expect(breadcrumb.level, SentryLevel.warning);
});

test('raw query sets breadcrumb to internal error', () async {
Expand All @@ -1120,8 +1120,8 @@ void main() {

final breadcrumb = fixture.hub.scope.breadcrumbs.first;
expect(breadcrumb.data?['status'], 'internal_error');
expect(breadcrumb.type, 'error');
expect(breadcrumb.level, SentryLevel.error);
expect(breadcrumb.type, 'query');
expect(breadcrumb.level, SentryLevel.warning);
});

test('raw query cursor sets breadcrumb to internal error', () async {
Expand All @@ -1137,8 +1137,8 @@ void main() {

final breadcrumb = fixture.hub.scope.breadcrumbs.first;
expect(breadcrumb.data?['status'], 'internal_error');
expect(breadcrumb.type, 'error');
expect(breadcrumb.level, SentryLevel.error);
expect(breadcrumb.type, 'query');
expect(breadcrumb.level, SentryLevel.warning);
});

test('raw update sets breadcrumb to internal error', () async {
Expand All @@ -1153,8 +1153,8 @@ void main() {

final breadcrumb = fixture.hub.scope.breadcrumbs.first;
expect(breadcrumb.data?['status'], 'internal_error');
expect(breadcrumb.type, 'error');
expect(breadcrumb.level, SentryLevel.error);
expect(breadcrumb.type, 'query');
expect(breadcrumb.level, SentryLevel.warning);
});

test('update sets breadcrumb to internal error', () async {
Expand All @@ -1170,8 +1170,8 @@ void main() {

final breadcrumb = fixture.hub.scope.breadcrumbs.first;
expect(breadcrumb.data?['status'], 'internal_error');
expect(breadcrumb.type, 'error');
expect(breadcrumb.level, SentryLevel.error);
expect(breadcrumb.type, 'query');
expect(breadcrumb.level, SentryLevel.warning);
});
});
}
Expand Down