Fixed issue when env var explicitly set to empty string.

Fixes sloria#71
hvtuananh · Apr 30, 2019 · d4ed648 · d4ed648
1 parent cf0b5e8
commit d4ed648
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/environs.py b/environs.py
@@ -46,7 +46,10 @@ def method(self, name, default=ma.missing, subcast=None, **kwargs):
             raise EnvError(
                 'Environment variable "{}" not set'.format(proxied_key or parsed_key)
             )
-        value = raw_value or field.missing
+        if raw_value or raw_value == "":
+            value = raw_value
+        else:
+            value = field.missing
         if preprocess:
             value = preprocess(value, subcast=subcast, **kwargs)
         try:

diff --git a/tests/test_environs.py b/tests/test_environs.py
@@ -47,6 +47,10 @@ def test_basic(self, set_env, env):
         set_env({"STR": "foo"})
         assert env.str("STR") == "foo"
 
+    def test_empty_str(self, set_env, env):
+        set_env({"STR": ""})
+        assert env.str("STR") == ""
+
     def test_int_cast(self, set_env, env):
         set_env({"INT": "42"})
         assert env.int("INT") == 42