Skip to content

Commit

Permalink
Merge branch 'master' into test-gh-docs
Browse files Browse the repository at this point in the history
* master:
  feat: return False when we do not discover any environment variables (#388)
  • Loading branch information
theskumar committed Jun 5, 2022
2 parents 69ec89c + ee15221 commit 88a41d8
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
5 changes: 5 additions & 0 deletions src/dotenv/main.py
Expand Up @@ -87,6 +87,9 @@ def set_as_environment_variables(self) -> bool:
"""
Load the current dotenv as system environment variable.
"""
if not self.dict():
return False

for k, v in self.dict().items():
if k in os.environ and not self.override:
continue
Expand Down Expand Up @@ -324,6 +327,8 @@ def load_dotenv(
override: Whether to override the system environment variables with the variables
from the `.env` file.
encoding: Encoding to be used to read the file.
Returns:
Bool: True if atleast one environment variable is set elese False
If both `dotenv_path` and `stream` are `None`, `find_dotenv()` is used to find the
.env file.
Expand Down
3 changes: 2 additions & 1 deletion tests/test_main.py
Expand Up @@ -259,8 +259,9 @@ def test_load_dotenv_no_file_verbose():
logger = logging.getLogger("dotenv.main")

with mock.patch.object(logger, "info") as mock_info:
dotenv.load_dotenv('.does_not_exist', verbose=True)
result = dotenv.load_dotenv('.does_not_exist', verbose=True)

assert result is False
mock_info.assert_called_once_with("Python-dotenv could not find configuration file %s.", ".does_not_exist")


Expand Down

0 comments on commit 88a41d8

Please sign in to comment.