Merge branch 'main' of github.com:mxstack/mxmake

mxstack · Feb 9, 2024 · 43d572d · 43d572d
2 parents 511affd + f0f2d43
commit 43d572d
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 1 deletion.
diff --git a/CHANGES.md b/CHANGES.md
@@ -10,6 +10,7 @@
 - Remove ruff cache when running `make ruff-clean` target.
 
 - Add `wtr` (Web test runner) domain to `js` topic.
+- Fix #20: make VENV_ENABLED=false test does not work.
 
 
 ## 1.0a3 (2024-02-06)

diff --git a/src/mxmake/tests/test_utils.py b/src/mxmake/tests/test_utils.py
@@ -12,6 +12,8 @@ def test_mxenv_path(self):
         self.assertEqual(utils.mxenv_path(), os.path.join("venv", "bin") + os.path.sep)
         os.environ["MXMAKE_MXENV_PATH"] = "other"
         self.assertEqual(utils.mxenv_path(), "other" + os.path.sep)
+        os.environ["MXMAKE_MXENV_PATH"] = ""
+        self.assertEqual(utils.mxenv_path(), "")
         del os.environ["MXMAKE_MXENV_PATH"]
 
     def test_mxmake_files(self):

diff --git a/src/mxmake/utils.py b/src/mxmake/utils.py
@@ -8,7 +8,7 @@
 def mxenv_path() -> str:
     """MX environment path."""
     path = os.environ.get("MXMAKE_MXENV_PATH", os.path.join("venv", "bin"))
-    if not path.endswith(os.path.sep):
+    if path and not path.endswith(os.path.sep):
         path = f"{path}{os.path.sep}"
     return path