Account for varying mlr locations (#1086)

The expected error output for the "not-a-valid-timezone" test assumes that mlr is invoked as mlr. Use "${MLR}" instead to cope with any invocation path. This also adds a regression test to catch future instances before they get merged. Signed-off-by: Stephen Kitt <[email protected]> Signed-off-by: Stephen Kitt <[email protected]>
johnkerl · Aug 23, 2022 · cdbe8b8 · cdbe8b8
1 parent abf58f4
commit cdbe8b8
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.github/workflows/go.yml b/.github/workflows/go.yml
@@ -28,6 +28,11 @@ jobs:
     - name: Test
       run: make check
 
+    - name: Regression tests
+      # We run these with a convoluted path to ensure the tests don't
+      # rely on a specific invocation
+      run: test/../mlr regtest -S
+
     - name: PrepareArtifactNonWindows
       if: matrix.os != 'windows-latest'
       run: mkdir -p bin/${{matrix.os}} && cp mlr bin/${{matrix.os}}

diff --git a/test/cases/dsl-local-date-time-functions/not-a-valid-timezone/experr b/test/cases/dsl-local-date-time-functions/not-a-valid-timezone/experr
@@ -1 +1 @@
-mlr :  unknown time zone this-is-not-a-valid-timezone-name
+${MLR} :  unknown time zone this-is-not-a-valid-timezone-name