reftest: add 'env.unix' & 'env.win32' to test environment variables r…

…ewriting rules
ocaml · Nov 13, 2023 · a9015ff · a9015ff
1 parent 8e65837
commit a9015ff
Show file tree

Hide file tree

Showing 4 changed files with 1,534 additions and 9 deletions.
diff --git a/master_changes.md b/master_changes.md
@@ -130,6 +130,7 @@ users)
   * dot-install: add a test for removal of non specified in .install empty directories [#5701 @rjbou]
   * Add test in assume-built for depends with switch variable filters [#5700 @rjbou]
   * Add undefined variable handling in assume built test [#5701 @rjbou]
+  * Add `env.unix` & `env.win32` to test environment variables rewriting rules [#5636 @rjbou]
 
 ### Engine
   * With real path resolved for all opam temp dir, remove `/private` from mac temp dir regexp [#5654 @rjbou]

diff --git a/tests/reftests/dune.inc b/tests/reftests/dune.inc
@@ -542,6 +542,27 @@
    %{targets}
    (run ./run.exe %{exe:../../src/client/opamMain.exe.exe} %{dep:env.test} %{read-lines:testing-env}))))
 
+(rule
+ (alias reftest-env.unix)
+ (enabled_if (= %{os_type} "Unix"))
+ (action
+  (diff env.unix.test env.unix.out)))
+
+(alias
+ (name reftest)
+ (enabled_if (= %{os_type} "Unix"))
+ (deps (alias reftest-env.unix)))
+
+(rule
+ (targets env.unix.out)
+ (deps root-N0REP0)
+ (enabled_if (= %{os_type} "Unix"))
+ (package opam)
+ (action
+  (with-stdout-to
+   %{targets}
+   (run ./run.exe %{exe:../../src/client/opamMain.exe.exe} %{dep:env.unix.test} %{read-lines:testing-env}))))
+
 (rule
  (alias reftest-env.win32)
  (enabled_if (= %{os_type} "Win32"))