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

testsuite: withShorterPathForNewBuildStore and findDependencyInStore should be more abstract wrt store directory #9455

Open
mpickering opened this issue Nov 16, 2023 · 0 comments

Comments

@mpickering
Copy link
Collaborator

withShorterPathForNewBuildStore and findDependencyInStore should live in the TestM monad and the author of the test should be unaware of how the --store-dir argument is passed to cabal.

This would remove the need to explicitly pass the --store-dir option in tests which need withShorterPathForNewBuildStore and also remove the need to explicitly pass the store dir argument to findDependencyInStore. Not passing these arguments is a potential source of bugs.

I have a patch, which I will put up after #9454 is merged.

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

No branches or pull requests

1 participant