haskell / haskell/cabal

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

Open
#9,455 0 comments 1 reaction 0 assignees View on GitHub
needs triage type: bug
Dominant language
Haskell
Stars
1.7k
Forks
750
Avg merge
4d 3h
Merged PRs (30d)
28

Description

`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.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.