Fixes parse error when date is only spaces (#6)

* Fixes parse error when date is only spaces

* Test action fix

* Set timezone in test

.github/workflows/test.yml

@@ -0,0 +1,23 @@
+name: Unit Testing
+
+on:
+  pull_request:
+    branches:
+      - master
+
+jobs:
+  unti_test:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Python
+      uses: actions/setup-python@v1
+      with:
+        python-version: '3.x'
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install -r requirements_test.txt -r requirements.txt
+        pip install -e .
+    - name: Run unit tests
+      run: python -m pytest --import-mode=append tests/

pyplaato/models/keg.py

@@ -42,7 +42,7 @@ def __repr__(self):
 
     @property
     def date(self) -> float:
-        if self.__date is not None and self.__date:
+        if self.__date is not None and self.__date and not self.__date.isspace():
             date = dateutil.parser.parse(self.__date)
             return date.timestamp()
         return super().date

requirements.txt

@@ -1,2 +1,2 @@
-aiohttp==3.7.4
+aiohttp==3.8.1
 python-dateutil==2.8.1

requirements_test.txt

@@ -1 +1 @@
-pytest==7.0.1
+pytest==7.0.1

setup.py

@@ -12,7 +12,7 @@
     long_description=long_description,
     long_description_content_type="text/markdown",
     url="https://github.com/JohNan/pyplaato",
-    packages=setuptools.find_packages(exclude=["tests","tests.*"]),
+    packages=setuptools.find_packages(exclude=["tests", "tests.*"]),
     classifiers=[
         "Framework :: AsyncIO",
         "Programming Language :: Python :: 3",

tests/models/test_keg.py

@@ -1,13 +1,19 @@
+import os, time
 from datetime import datetime
 from unittest import mock
 
 from pyplaato.models.keg import PlaatoKeg
 
 
+def setup_module(module):
+    os.environ['TZ'] = 'Europe/London'
+    time.tzset()
+
+
 def test_date_prop_with_value_returns_its_timestamp():
     pins = PlaatoKeg.Pins
-    keg = PlaatoKeg({pins.DATE: "1/1/2022"})
-    assert 1641024000.0 == keg.date
+    keg = PlaatoKeg({pins.DATE: "10/1/2022"})
+    assert 1664578800.0 == keg.date
 
 
 @mock.patch("pyplaato.models.device.datetime")
@@ -17,3 +23,20 @@ def test_date_prop_with_no_value_returns_current_timestamp(m_datetime):
     keg = PlaatoKeg({})
     assert now.timestamp() == keg.date
 
+
+@mock.patch("pyplaato.models.device.datetime")
+def test_date_prop_with_empty_string(m_datetime):
+    now = datetime(2022, 10, 1)
+    m_datetime.now.return_value = now
+    pins = PlaatoKeg.Pins
+    keg = PlaatoKeg({pins.DATE: ""})
+    assert now.timestamp() == keg.date
+
+
+@mock.patch("pyplaato.models.device.datetime")
+def test_date_prop_with_space_string(m_datetime):
+    now = datetime(2022, 10, 1)
+    m_datetime.now.return_value = now
+    pins = PlaatoKeg.Pins
+    keg = PlaatoKeg({pins.DATE: " "})
+    assert now.timestamp() == keg.date