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

Update reference tests to the latest stable version? #442

Closed
gdalle opened this issue Dec 15, 2023 · 7 comments
Closed

Update reference tests to the latest stable version? #442

gdalle opened this issue Dec 15, 2023 · 7 comments

Comments

@gdalle
Copy link
Collaborator

gdalle commented Dec 15, 2023

People only test locally on the latest stable version, and as a result some PRs can introduce test failures (like #441). This would be easier to avoid if the reference tests were kept up to date with the latest stable version, so 1.9.4 at the time of writing.

On the other hand, it means that once we re-generate the fixtures with a new version, they are basically useless. Indeed, by definition, they will agree with the output of the code.
@oxinabox any idea how to resolve this conundrum?

@gdalle
Copy link
Collaborator Author

gdalle commented Dec 15, 2023

More generally, this raises the question of what reference tests are for

@oxinabox
Copy link
Collaborator

We actually need to keep testing on 1.7 even if we update the fixtures,
because 1.8 changes how IO works in tests,
so we can't run the interactive tests on 1.8

though maybe it would just be ok to stop testing them?

@oxinabox
Copy link
Collaborator

In general the notion of reference tests is that you look at them before merging after regenerating, and confirm that they actually are correct

@gdalle
Copy link
Collaborator Author

gdalle commented Dec 15, 2023

In general the notion of reference tests is that you look at them before merging after regenerating, and confirm that they actually are correct

So if we work on the assumption that the code is correct whenever reference tests pass, we could use that correct code to re-generate new reference tests at each Julia release?

@oxinabox
Copy link
Collaborator

Yes, because we assume that new julia releases are not breaking our stuff.

@gdalle
Copy link
Collaborator Author

gdalle commented Dec 15, 2023

Okay. I'll try a big PR that does only that, re-generate all the fixtures with 1.9.4

@gdalle
Copy link
Collaborator Author

gdalle commented Feb 11, 2024

Solved by #453

@gdalle gdalle closed this as completed Feb 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants