getsentry / getsentry/sentry-dotnet

Integration tests should validate the DebugID matches

Open
#4,481 2 comments 0 reactions 0 assignees View on GitHub
.NET Tests
Dominant language
C#
Stars
770
Forks
248
Avg merge
3d 4h
Merged PRs (30d)
49

Description

### Description

We discovered somewhat randomly (and not in a very timely way) that [DebugID's are broken on iOS]().

We also already have a couple of tests that ensure symbols and source get uploaded:
[https://github.com/getsentry/sentry-dotnet/blob/main/integration-test/cli.Tests.ps1#L149]()
[https://github.com/getsentry/sentry-dotnet/blob/main/integration-test/cli.Tests.ps1#L162]()

Ideally once the symbols were uploaded, these tests could also validate that DebugID in the PE header for one of the assemblies matched the the DebugID of [the PDB uploaded to Sentry]().

It might not be trivial/possible to do this with the PWSH tests. Might be a good use case for the Microsoft Test Platform.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.