fix (build): Add (initial) missing CI test coverage for DevEnv 🫙

enola-dev · Feb 20, 2024 · c511fb0 · c511fb0
1 parent 3be6a6c
commit c511fb0
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -121,7 +121,12 @@ jobs:
         shell: bash
         run: asdf reshim
 
-      - run: ./tools/test-ci/test.bash
+      - name: Run CI Tests
+        run: ./tools/test-ci/test.bash
+
+      # TODO https://github.com/enola-dev/enola/issues/458 turn this "upside down"...
+      - name: Test Development Environment (DevEnv)
+        run: ./devenv.bash
 
       # Build docs/ into site/ (not just on main branch but also for pull requests, as test)
       - name: Build Docs Site

diff --git a/devenv.bash b/devenv.bash
@@ -21,4 +21,8 @@ set -euo pipefail
 
 docker build -t enola.dev-devenv --rm -f Dockerfile-DevEnv .
 
-docker run -v .:/workspace/:Z -it --rm enola.dev-devenv
+set +u
+if [ -z "$CI" ]; then
+  docker run -v "$PWD":/workspace/:Z -it --rm enola.dev-devenv
+fi
+set -u