Merge pull request #5 from gensyn/copilot/add-tests-and-workflows

gensyn · web-flow · commit b29eb73e6405 · 2026-03-10T08:59:41.000+01:00
Add tests, CI workflows, and homeassistant mock for task_tracker
diff --git a/.github/workflows/pylint.yml b/.github/workflows/pylint.yml
@@ -0,0 +1,30 @@
+name: Pylint
+
+on: [push]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+
+    strategy:
+      matrix:
+        python-version: ["3.14"]
+
+    steps:
+    - uses: actions/checkout@v4
+
+    - name: Set up Python ${{ matrix.python-version }}
+      uses: actions/setup-python@v3
+      with:
+        python-version: ${{ matrix.python-version }}
+
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install pylint
+
+    - name: Analysing the code with pylint
+      run: |
+        pylint --fail-under=9 --max-line-length=120 --disable=import-error,wrong-import-position,wrong-import-order,too-many-instance-attributes,too-many-locals,too-many-arguments,too-few-public-methods --ignore-paths=^test/.*$ $(git ls-files '*.py')
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -0,0 +1,30 @@
+name: CI
+
+on: [push]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+
+    strategy:
+      matrix:
+        python-version: ["3.14"]
+
+    steps:
+    - uses: actions/checkout@v4
+
+    - name: Set up Python ${{ matrix.python-version }}
+      uses: actions/setup-python@v3
+      with:
+        python-version: ${{ matrix.python-version }}
+
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install -r requirements_unittests.txt
+
+    - name: Run tests
+      run: |
+        python -m unittest discover -s test -v
diff --git a/.gitignore b/.gitignore
@@ -0,0 +1,4 @@
+__pycache__/
+*.pyc
+htmlcov/
+.coverage
diff --git a/frontend/__init__.py b/frontend/__init__.py
@@ -1,14 +1,12 @@
-"""View Assist Javascript module registration."""
+"""Task Tracker JavaScript module registration."""
 
 import logging
 import os
 from pathlib import Path
 
 from homeassistant.components.http import StaticPathConfig
 from homeassistant.components.lovelace import MODE_STORAGE, LovelaceData
-from homeassistant.const import MAJOR_VERSION, MINOR_VERSION
 from homeassistant.core import HomeAssistant
-
 from ..const import URL_BASE, TASK_TRACKER_CARDS  # noqa: TID252
 
 _LOGGER = logging.getLogger(__name__)
@@ -21,10 +19,10 @@ def __init__(self, hass: HomeAssistant) -> None:
         """Initialise."""
         self.hass = hass
         self.lovelace: LovelaceData = self.hass.data.get("lovelace")
-        # Backwards compatability after/before 2026.2
-        if MAJOR_VERSION > 2026  or (MAJOR_VERSION == 2026 and MINOR_VERSION >= 2):
+        if hasattr(self.lovelace, "resource_mode"):
             self.resource_mode = self.lovelace.resource_mode
         else:
+            # Backwards compatibility before 2026.2
             self.resource_mode = self.lovelace.mode
 
     async def async_register(self):
@@ -38,7 +36,7 @@ async def _async_register_path(self):
         """Register resource path if not already registered."""
         try:
             await self.hass.http.async_register_static_paths(
-                [StaticPathConfig(URL_BASE, Path(__file__).parent, False)]
+                [StaticPathConfig(URL_BASE, str(Path(__file__).parent), False)]
             )
             _LOGGER.debug("Registered resource path from %s", Path(__file__).parent)
         except RuntimeError:
@@ -66,7 +64,7 @@ async def _async_register_modules(self):
                     card_registered = True
                     # check version
                     if self._get_resource_version(resource["url"]) != module.get(
-                        "version"
+                            "version"
                     ):
                         # Update card version
                         _LOGGER.debug(
@@ -136,10 +134,10 @@ def remove_gzip_files(self):
         for file in gzip_files:
             try:
                 if (
-                    Path.stat(f"{path}/{file}").st_mtime
-                    < Path.stat(f"{path}/{file.replace('.gz', '')}").st_mtime
+                        Path.stat(Path(f"{path}/{file}")).st_mtime
+                        < Path.stat(Path(f"{path}/{file.replace('.gz', '')}")).st_mtime
                 ):
                     _LOGGER.debug("Removing older gzip file - %s", file)
-                    Path.unlink(f"{path}/{file}")
+                    Path.unlink(Path(f"{path}/{file}"))
             except OSError:
                 pass
diff --git a/options_flow.py b/options_flow.py
@@ -95,7 +95,7 @@ async def validate_options(user_input: dict[str, Any]) -> dict[str, Any]:
     if user_input.get(CONF_NOTIFICATION_INTERVAL, 0) < 1:
         user_input[CONF_NOTIFICATION_INTERVAL] = 1
 
-    options = {
+    return {
         CONF_ACTIVE: user_input[CONF_ACTIVE],
         CONF_TASK_INTERVAL_VALUE: user_input[CONF_TASK_INTERVAL_VALUE],
         CONF_TASK_INTERVAL_TYPE: user_input[CONF_TASK_INTERVAL_TYPE],
@@ -105,5 +105,3 @@ async def validate_options(user_input: dict[str, Any]) -> dict[str, Any]:
         CONF_TODO_OFFSET_DAYS: user_input[CONF_TODO_OFFSET_DAYS],
         CONF_NOTIFICATION_INTERVAL: user_input[CONF_NOTIFICATION_INTERVAL],
     }
-
-    return options
diff --git a/requirements_unittests.txt b/requirements_unittests.txt
@@ -0,0 +1,2 @@
+python-dateutil
+voluptuous
diff --git a/run_tests.sh b/run_tests.sh
@@ -0,0 +1,3 @@
+#!/bin/bash
+
+coverage run --omit='test/*' -m unittest discover -s test; coverage html
diff --git a/test/__init__.py b/test/__init__.py
diff --git a/test/homeassistant_mock/homeassistant/__init__.py b/test/homeassistant_mock/homeassistant/__init__.py
diff --git a/test/homeassistant_mock/homeassistant/components/__init__.py b/test/homeassistant_mock/homeassistant/components/__init__.py
diff --git a/test/homeassistant_mock/homeassistant/components/button.py b/test/homeassistant_mock/homeassistant/components/button.py
@@ -0,0 +1,11 @@
+class ButtonEntity:
+    hass = None
+    device_entry = None
+    _attr_has_entity_name = False
+    _attr_should_poll = False
+    _attr_translation_key = None
+    _attr_unique_id = None
+    _attr_device_info = None
+
+    async def async_press(self):
+        pass
diff --git a/test/homeassistant_mock/homeassistant/components/http.py b/test/homeassistant_mock/homeassistant/components/http.py
@@ -0,0 +1,5 @@
+class StaticPathConfig:
+    def __init__(self, url_path, path, cache_headers):
+        self.url_path = url_path
+        self.path = path
+        self.cache_headers = cache_headers
diff --git a/test/homeassistant_mock/homeassistant/components/local_todo/__init__.py b/test/homeassistant_mock/homeassistant/components/local_todo/__init__.py
diff --git a/test/homeassistant_mock/homeassistant/components/local_todo/const.py b/test/homeassistant_mock/homeassistant/components/local_todo/const.py
@@ -0,0 +1 @@
+DOMAIN = "local_todo"
diff --git a/test/homeassistant_mock/homeassistant/components/lovelace.py b/test/homeassistant_mock/homeassistant/components/lovelace.py
@@ -0,0 +1,6 @@
+MODE_STORAGE = "storage"
+MODE_YAML = "yaml"
+
+
+class LovelaceData:
+    pass
diff --git a/test/homeassistant_mock/homeassistant/components/sensor.py b/test/homeassistant_mock/homeassistant/components/sensor.py
@@ -0,0 +1,28 @@
+class SensorEntity:
+    _attr_has_entity_name = False
+    _attr_should_poll = False
+    _attr_translation_key = None
+    _attr_native_value = None
+    _attr_extra_state_attributes = None
+    _attr_device_info = None
+    _attr_name = None
+    _attr_unique_id = None
+    hass = None
+    entity_id = None
+
+    def async_write_ha_state(self):
+        pass
+
+    def async_on_remove(self, cb):
+        pass
+
+
+class RestoreSensor:
+    async def async_added_to_hass(self):
+        pass
+
+    async def async_get_last_sensor_data(self):
+        return None
+
+    async def async_get_last_state(self):
+        return None
diff --git a/test/homeassistant_mock/homeassistant/config_entries.py b/test/homeassistant_mock/homeassistant/config_entries.py
@@ -0,0 +1,41 @@
+ConfigFlowResult = dict
+
+
+class ConfigEntry:
+    def __init__(self, entry_id="test_id", version=1, minor_version=2, data=None, options=None, title=""):
+        self.entry_id = entry_id
+        self.version = version
+        self.minor_version = minor_version
+        self.data = data or {}
+        self.options = options or {}
+        self.title = title
+
+
+class ConfigFlow:
+    def __init_subclass__(cls, domain=None, **kwargs):
+        super().__init_subclass__(**kwargs)
+        if domain is not None:
+            cls.DOMAIN = domain
+
+    def async_show_form(self, step_id=None, data_schema=None, errors=None):
+        return {"type": "form", "step_id": step_id}
+
+    def async_create_entry(self, title=None, data=None, options=None):
+        result = {"type": "create_entry", "title": title, "data": data or {}}
+        if options is not None:
+            result["options"] = options
+        return result
+
+
+class OptionsFlowWithReload:
+    hass = None
+    config_entry = None
+
+    def async_show_form(self, **kwargs):
+        return {"type": "form", **kwargs}
+
+    def async_create_entry(self, data=None):
+        return {"type": "create_entry", "data": data or {}}
+
+    def add_suggested_values_to_schema(self, schema, values):
+        return schema
diff --git a/test/homeassistant_mock/homeassistant/const.py b/test/homeassistant_mock/homeassistant/const.py
@@ -0,0 +1,14 @@
+class Platform:
+    SENSOR = "sensor"
+    BUTTON = "button"
+
+
+CONF_NAME = "name"
+CONF_ICON = "icon"
+CONF_ENTITY_ID = "entity_id"
+CONF_OPTIONS = "options"
+CONF_MODE = "mode"
+EVENT_HOMEASSISTANT_STARTED = "homeassistant_started"
+EVENT_STATE_CHANGED = "state_changed"
+MAJOR_VERSION = 2025
+MINOR_VERSION = 1
diff --git a/test/homeassistant_mock/homeassistant/core.py b/test/homeassistant_mock/homeassistant/core.py
@@ -0,0 +1,25 @@
+class CoreState:
+    running = "running"
+    not_running = "not_running"
+
+
+class HomeAssistant:
+    def __init__(self):
+        self.state = CoreState.running
+        self.services = None
+        self.data = {}
+        self.bus = None
+        self.config_entries = None
+        self.states = None
+
+
+class ServiceCall:
+    def __init__(self, data=None):
+        self.data = data or {}
+
+
+EventStateChangedData = dict
+
+
+def callback(func):
+    return func
diff --git a/test/homeassistant_mock/homeassistant/exceptions.py b/test/homeassistant_mock/homeassistant/exceptions.py
@@ -0,0 +1,9 @@
+class ServiceValidationError(Exception):
+    def __init__(self, message="", translation_domain="", translation_key=""):
+        super().__init__(message)
+        self.translation_domain = translation_domain
+        self.translation_key = translation_key
+
+
+class HomeAssistantError(Exception):
+    pass
diff --git a/test/homeassistant_mock/homeassistant/helpers/__init__.py b/test/homeassistant_mock/homeassistant/helpers/__init__.py
diff --git a/test/homeassistant_mock/homeassistant/helpers/config_validation.py b/test/homeassistant_mock/homeassistant/helpers/config_validation.py
@@ -0,0 +1,13 @@
+def empty_config_schema(domain):
+    return lambda config: config
+
+
+def entity_id(value):
+    return value
+
+
+def date(value):
+    if isinstance(value, str):
+        from datetime import date as _date  # pylint: disable=import-outside-toplevel
+        return _date.fromisoformat(value)
+    return value
diff --git a/test/homeassistant_mock/homeassistant/helpers/device_registry.py b/test/homeassistant_mock/homeassistant/helpers/device_registry.py
@@ -0,0 +1,3 @@
+class DeviceInfo(dict):
+    def __init__(self, **kwargs):
+        super().__init__(**kwargs)
diff --git a/test/homeassistant_mock/homeassistant/helpers/entity.py b/test/homeassistant_mock/homeassistant/helpers/entity.py
@@ -0,0 +1,2 @@
+def generate_entity_id(format_str, value, hass=None):
+    return format_str.replace("{}", value)
diff --git a/test/homeassistant_mock/homeassistant/helpers/entity_platform.py b/test/homeassistant_mock/homeassistant/helpers/entity_platform.py
@@ -0,0 +1 @@
+AddEntitiesCallback = None
diff --git a/test/homeassistant_mock/homeassistant/helpers/entity_registry.py b/test/homeassistant_mock/homeassistant/helpers/entity_registry.py
@@ -0,0 +1,10 @@
+def async_get(hass):
+    return getattr(hass, "entity_registry", None)
+
+
+def async_entries_for_device(registry, device_id):
+    return []
+
+
+def async_entries_for_config_entry(registry, entry_id):
+    return []
diff --git a/test/homeassistant_mock/homeassistant/helpers/event.py b/test/homeassistant_mock/homeassistant/helpers/event.py
@@ -0,0 +1,6 @@
+def async_call_later(hass, delay, action):
+    return lambda: None
+
+
+def async_track_time_change(hass, action, **kwargs):
+    return lambda: None
diff --git a/test/homeassistant_mock/homeassistant/helpers/selector.py b/test/homeassistant_mock/homeassistant/helpers/selector.py
@@ -0,0 +1,2 @@
+def selector(config):
+    return lambda x: x
diff --git a/test/homeassistant_mock/homeassistant/helpers/typing.py b/test/homeassistant_mock/homeassistant/helpers/typing.py
@@ -0,0 +1 @@
+ConfigType = dict
diff --git a/test/homeassistant_mock/homeassistant/util/__init__.py b/test/homeassistant_mock/homeassistant/util/__init__.py
@@ -0,0 +1,8 @@
+import re
+
+
+def slugify(text: str) -> str:
+    """Convert text to a slug."""
+    text = text.lower()
+    text = re.sub(r"[^a-z0-9]+", "_", text)
+    return text.strip("_")
diff --git a/test/homeassistant_mock/homeassistant/util/dt.py b/test/homeassistant_mock/homeassistant/util/dt.py
@@ -0,0 +1,3 @@
+from datetime import timezone
+
+UTC = timezone.utc
diff --git a/test/test_button.py b/test/test_button.py
diff --git a/test/test_config_flow.py b/test/test_config_flow.py
diff --git a/test/test_init.py b/test/test_init.py
diff --git a/test/test_sensor.py b/test/test_sensor.py
diff --git a/test/test_validate_options.py b/test/test_validate_options.py

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+#!/bin/bash`
	`2`	`+`
	`3`	`+coverage run --omit='test/*' -m unittest discover -s test; coverage html`