-
Notifications
You must be signed in to change notification settings - Fork 205
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.
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
Fix: serialise breadcrumb message correctly #306
Conversation
the breadcrumb message should be serialised as part of the metadata, not as the name attribute
Partially addresses #291 |
assertEquals("Hello World", breadcrumb.getMetadata().get("message")); | ||
} | ||
} | ||
assertEquals(1, count); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the count necessary for this test? And if so, is the for loop the best way to test for it? Would testing the length of the queue be more clear?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The size and order of the breadcrumb queue cannot be guaranteed when calling client#leaveBreadcrumb
, as automatic breadcrumbs may occur at any time (around 6-7 tend to be recorded in practice).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, fair enough.
Add uploadNdkUnityLibraryMappings flag to bugsnag extension
fix(ANR): allow other error classes to be reported
The breadcrumb message should be serialised as part of the metadata, not as the name attribute.