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

[QA] Snapshot testing #763

Closed
2 of 4 tasks
prabhu opened this issue Dec 8, 2023 · 0 comments · Fixed by #1090
Closed
2 of 4 tasks

[QA] Snapshot testing #763

prabhu opened this issue Dec 8, 2023 · 0 comments · Fixed by #1090
Assignees

Comments

@prabhu
Copy link
Contributor

prabhu commented Dec 8, 2023

We have many repotests that invoke cdxgen with a range of repos but don't validate the results :)

This issue is to seek community help to setup a snapshot testing workflow.

Requirements

  • We use jest currently. Can we use jest snapshot testing as well?
  • How do we deal with attributes such as timestamps that can change, thus breaking such tests?
  • How do we deal with ordering (of files and components), which relies on the filesystem and the output from tools and can change based on the runner, version of the tools, weather, etc?
  • How do we deal with snapshot updates where the changes are genuine improvements?

Bugs

Some of the bugs that escaped our current batch of repotests.

#937

@prabhu prabhu added good first issue Good for newcomers help wanted Extra attention is needed Consider Funding labels Dec 8, 2023
@prabhu prabhu added sponsored and removed good first issue Good for newcomers help wanted Extra attention is needed Consider Funding labels Mar 11, 2024
@cerrussell cerrussell mentioned this issue May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants