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

Identifying unused snapshots #899

Open
epenet opened this issue Oct 14, 2024 · 2 comments
Open

Identifying unused snapshots #899

epenet opened this issue Oct 14, 2024 · 2 comments
Labels

Comments

@epenet
Copy link
Contributor

epenet commented Oct 14, 2024

Is your feature request related to a problem? Please describe.

We know there are unused snapshots, but it is very hard to identify where they are.

Describe the solution you'd like

It would be nice to have a way to identify the individual unused snapshots

Describe alternatives you've considered

Additional context

See home-assistant/core#128162
Due to xdist incompatility, we are unable to identify the unused snapshots as soon as they enter the codebase.

@epenet epenet changed the title Identifying unusued snapshots Identifying unused snapshots Oct 14, 2024
@noahnu
Copy link
Collaborator

noahnu commented Oct 17, 2024

There's a --snapshot-details flag you can pass to pytest. See https://github.com/syrupy-project/syrupy#cli-options. Let me know if that's sufficient for your use case.

@epenet
Copy link
Contributor Author

epenet commented Oct 17, 2024

There's a --snapshot-details flag you can pass to pytest. See https://github.com/syrupy-project/syrupy#cli-options. Let me know if that's sufficient for your use case.

It would be sufficient if it worked with xdist... but that doesn't work.
I've started work on #901, and it kind of works on my machine but I can't get it inside the Home Assistant CI

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

No branches or pull requests

2 participants