Skip to content
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

Improve error popup for test failures- go to assertion and don't duplicate test code #24512

Open
chrmarti opened this issue Dec 3, 2024 · 1 comment
Assignees
Labels
area-testing feature-request Request for new features or functionality

Comments

@chrmarti
Copy link

chrmarti commented Dec 3, 2024

Testing #24505

I clicked on a test failure in the test explorer and it navigated to the test function telling me about the failed test assertion. It should instead directly go to the failed assertion. When directly going to the failed assertion there is no need for duplicating the test's code in the zone widget (without syntax highlighting).
Image

@github-actions github-actions bot added the triage-needed Needs assignment to the proper sub-team label Dec 3, 2024
@eleanorjboyd
Copy link
Member

Agree with this as there are feature improvements possible in the error pop-up. Right now we take the information output by pytest for the specific test and return it directly in this section. Would be great if there was more sophistication to not duplicate information etc

@eleanorjboyd eleanorjboyd added feature-request Request for new features or functionality and removed triage-needed Needs assignment to the proper sub-team labels Dec 4, 2024
@eleanorjboyd eleanorjboyd changed the title Click on test failure should go to failed assertion Improve error popup for test failures- go to assertion and don't duplicate test code Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-testing feature-request Request for new features or functionality
Projects
None yet
Development

No branches or pull requests

2 participants