diff options
| author | John Ericson <John.Ericson@Obsidian.Systems> | 2023-10-05 12:12:18 -0400 |
|---|---|---|
| committer | John Ericson <John.Ericson@Obsidian.Systems> | 2023-12-01 12:06:43 -0500 |
| commit | 30dcc19d1f30fc203be460134c4578509cce704f (patch) | |
| tree | 6cc32609b9984a2c4d5ecc0cac5cf30609e208b9 /tests/lang/eval-okay-pathexists.nix | |
| parent | 72425212657d795dc215b334b7c8c8cd36d06b72 (diff) | |
Put functional tests in `tests/functional`
I think it is bad for these reasons when `tests/` contains a mix of
functional and integration tests
- Concepts is harder to understand, the documentation makes a good
unit vs functional vs integration distinction, but when the
integration tests are just two subdirs within `tests/` this is not
clear.
- Source filtering in the `flake.nix` is more complex. We need to
filter out some of the dirs from `tests/`, rather than simply pick
the dirs we want and take all of them. This is a good sign the
structure of what we are trying to do is not matching the structure
of the files.
With this change we have a clean:
```shell-session
$ git show 'HEAD:tests'
tree HEAD:tests
functional/
installer/
nixos/
```
(cherry picked from commit 68c81c737571794f7246db53fb4774e94fcf4b7e)
Diffstat (limited to 'tests/lang/eval-okay-pathexists.nix')
| -rw-r--r-- | tests/lang/eval-okay-pathexists.nix | 29 |
1 files changed, 0 insertions, 29 deletions
diff --git a/tests/lang/eval-okay-pathexists.nix b/tests/lang/eval-okay-pathexists.nix deleted file mode 100644 index c5e7a62de..000000000 --- a/tests/lang/eval-okay-pathexists.nix +++ /dev/null @@ -1,29 +0,0 @@ -builtins.pathExists (./lib.nix) -&& builtins.pathExists (builtins.toPath ./lib.nix) -&& builtins.pathExists (builtins.toString ./lib.nix) -&& !builtins.pathExists (builtins.toString ./lib.nix + "/") -&& !builtins.pathExists (builtins.toString ./lib.nix + "/.") -# FIXME -# && !builtins.pathExists (builtins.toString ./lib.nix + "/..") -# && !builtins.pathExists (builtins.toString ./lib.nix + "/a/..") -# && !builtins.pathExists (builtins.toString ./lib.nix + "/../lib.nix") -&& !builtins.pathExists (builtins.toString ./lib.nix + "/./") -&& !builtins.pathExists (builtins.toString ./lib.nix + "/./.") -&& builtins.pathExists (builtins.toString ./.. + "/lang/lib.nix") -&& !builtins.pathExists (builtins.toString ./.. + "lang/lib.nix") -&& builtins.pathExists (builtins.toString ./. + "/../lang/lib.nix") -&& builtins.pathExists (builtins.toString ./. + "/../lang/./lib.nix") -&& builtins.pathExists (builtins.toString ./.) -&& builtins.pathExists (builtins.toString ./. + "/") -&& builtins.pathExists (builtins.toString ./. + "/../lang") -&& builtins.pathExists (builtins.toString ./. + "/../lang/") -&& builtins.pathExists (builtins.toString ./. + "/../lang/.") -&& builtins.pathExists (builtins.toString ./. + "/../lang/./") -&& builtins.pathExists (builtins.toString ./. + "/../lang//./") -&& builtins.pathExists (builtins.toString ./. + "/../lang/..") -&& builtins.pathExists (builtins.toString ./. + "/../lang/../") -&& builtins.pathExists (builtins.toString ./. + "/../lang/..//") -&& builtins.pathExists (builtins.toPath (builtins.toString ./lib.nix)) -&& !builtins.pathExists (builtins.toPath (builtins.toString ./bla.nix)) -&& builtins.pathExists ./lib.nix -&& !builtins.pathExists ./bla.nix |