Skip to content

Add e2e test #4174

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

Open
bruno-garcia opened this issue May 7, 2025 · 3 comments
Open

Add e2e test #4174

bruno-garcia opened this issue May 7, 2025 · 3 comments
Assignees

Comments

@bruno-garcia
Copy link
Member

We recently had symbolication related issues that would have been caught had we tested the SDK through Sentry itself.
Other repos have this, like React Native.

@jamescrosswell
Copy link
Collaborator

@bruno-garcia is there a reference to the issue describing the symbolication difficulties? What kind of end to end tests are we talking about? Different to the integration tests?

@vaind
Copy link
Collaborator

vaind commented May 11, 2025

@jamescrosswell this came up in an offline meeting. The issue with current integration tests in the dotnet repo is that they only run locally, against a fake sentry server (a python script that prints out what it has received). This is fast and provides baseline checks what has been sent to the server by the CLI during build.
However, because the dotnet SDK depends on server-side symbolication, it would be preferable to verify that it continues to work and doesn't break between .net versions, as it did in the past. This can be achieved by building a sample app, uploading it's debug files to sentry.io, running it, and checking sentry.io APIs to fetch the issue. There, we can verify the issue is symbolicated properly. I imagine this would be useful for native AOT, portable PDBs support, etc.
Similar tests have been done for other things than symbolication, for example replay. Maybe this would be useful to test profile capture, which also got broken by a past .net release.

I'm assigned here because I've volunteered to prepare the scaffolding and some basic test as an example, because I have experience with these in other repos. There can be more tests added in the future, ideally by sentry-dotnet SDK maintainers.

@jamescrosswell
Copy link
Collaborator

Cool, thanks @vaind - that sounds a bit step up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

3 participants