Skip to content

Conversation

bitsandfoxes
Copy link
Contributor

No description provided.

@@ -210,7 +210,7 @@ public IEnumerator DebugLogException_IsMarkedUnhandled()
{
yield return SetupSceneCoroutine("1_BugFarm");

var expectedMechanism = "\"mechanism\":{\"type\":\"Unity.LogException\",\"handled\":false}}]}";
var expectedMechanism = "\"mechanism\":{\"type\":\"Unity.LogException\",\"handled\":false";
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The mechanism now also contains the ,"data":{"HResult":"magicnumber"}. So we're trimming the check because that's not what we're testing here.

Flash0ver
Flash0ver previously approved these changes Apr 2, 2025
@@ -81,6 +81,9 @@ public void SetUser(SentryUser? user)
}
}

public void SetTrace(SentryId traceId, SpanId parentSpanId)
{ }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note: will be implemented in follow-up

@bitsandfoxes bitsandfoxes requested a review from Flash0ver April 2, 2025 15:38
@bitsandfoxes bitsandfoxes requested review from bruno-garcia and vaind and removed request for Flash0ver April 2, 2025 15:49
@bitsandfoxes bitsandfoxes merged commit 3e02634 into main Apr 3, 2025
6 checks passed
@bitsandfoxes bitsandfoxes deleted the chore/bump-net-5.5 branch April 3, 2025 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants