jeans/core
jeans/core
1
0
Fork 0
mirror of https://github.com/home-assistant/core.git synced 2025-07-27 15:17:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

Is_allowed_path raise for None path (#8953)

Browse Source 
* is_allowed_path

* Fix #8948

* assert path is not None

* Update test_core.py

* Update test_core.py

* Update test_core.py
This commit is contained in:
Daniel Høyer Iversen 2017-08-15 15:41:37 +02:00 committed by Pascal Vizeli
parent 5309006494
commit 1892eb654f
2 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
homeassistant/core.py
View File

@ -1079,6 +1079,8 @@ class Config(object):
def is_allowed_path(self, path: str) -> bool: def is_allowed_path(self, path: str) -> bool:
"""Check if the path is valid for access from outside.""" """Check if the path is valid for access from outside."""
assert path is not None
parent = pathlib.Path(path).parent parent = pathlib.Path(path).parent
try: try:
parent = parent.resolve() # pylint: disable=no-member parent = parent.resolve() # pylint: disable=no-member

3
tests/test_core.py
View File

@ -833,6 +833,9 @@ class TestConfig(unittest.TestCase):
for path in unvalid: for path in unvalid:
assert not self.config.is_allowed_path(path) assert not self.config.is_allowed_path(path)
with self.assertRaises(AssertionError):
self.config.is_allowed_path(None)
@patch('homeassistant.core.monotonic') @patch('homeassistant.core.monotonic')
def test_create_timer(mock_monotonic, loop): def test_create_timer(mock_monotonic, loop):