Test --working-dir in the testsuite
When testing `./Setup` only, when `withDirectory` is used, instead of changing into that directory when invoking processes, we now use the `--working-dir` flag and keep a fixed CWD. This will therefore passively test that `--working-dir` is working In addition, it makes it possible to test things easily such as `--working-dir` with a relative path as an argument. `cabal-install` will only invoke `--working-dir` with an absolute path and hence is isolated from any double interpretation issues. Testing against these double interpretation issues is very important as it also prevents over-interpretation of relative paths into absolute paths. Passing absolute paths to tools such as hsc2hs can lead to the build directory leaking into an interface file which leads to non-reproducible results.
Showing
- cabal-testsuite/PackageTests/Backpack/Includes2/setup-external.cabal.out 6 additions, 6 deletions.../PackageTests/Backpack/Includes2/setup-external.cabal.out
- cabal-testsuite/PackageTests/Backpack/Includes2/setup-external.out 6 additions, 6 deletions...tsuite/PackageTests/Backpack/Includes2/setup-external.out
- cabal-testsuite/PackageTests/Backpack/Includes2/setup-per-component.out 6 additions, 6 deletions...e/PackageTests/Backpack/Includes2/setup-per-component.out
- cabal-testsuite/PackageTests/Backpack/Includes3/setup-external-explicit.out 2 additions, 2 deletions...ckageTests/Backpack/Includes3/setup-external-explicit.out
- cabal-testsuite/PackageTests/Backpack/Includes3/setup-external-ok.cabal.out 8 additions, 6 deletions...ckageTests/Backpack/Includes3/setup-external-ok.cabal.out
- cabal-testsuite/PackageTests/Backpack/Includes3/setup-external-ok.out 8 additions, 6 deletions...ite/PackageTests/Backpack/Includes3/setup-external-ok.out
- cabal-testsuite/PackageTests/Backpack/Reexport1/setup.cabal.out 2 additions, 2 deletions...testsuite/PackageTests/Backpack/Reexport1/setup.cabal.out
- cabal-testsuite/PackageTests/Backpack/Reexport1/setup.out 2 additions, 2 deletionscabal-testsuite/PackageTests/Backpack/Reexport1/setup.out
- cabal-testsuite/PackageTests/TestNameCollision/setup.cabal.out 1 addition, 1 deletion...-testsuite/PackageTests/TestNameCollision/setup.cabal.out
- cabal-testsuite/PackageTests/TestNameCollision/setup.out 1 addition, 1 deletioncabal-testsuite/PackageTests/TestNameCollision/setup.out
- cabal-testsuite/Setup.hs 9 additions, 1 deletioncabal-testsuite/Setup.hs
- cabal-testsuite/src/Test/Cabal/Prelude.hs 26 additions, 13 deletionscabal-testsuite/src/Test/Cabal/Prelude.hs
Loading
Please register or sign in to comment