Merge pull request #91 from jwillemsen/jwi-tests

First step for unit tests, enable gzip compression on data

Author: jwillemsen

.github/workflows/tests.yaml

@@ -0,0 +1,32 @@
+name: Run unit tests with pytest
+
+on:
+  push:
+  pull_request:
+  schedule:
+    - cron: "0 1 * * SUN"
+  workflow_dispatch:
+
+jobs:
+  tests:
+    runs-on: "ubuntu-latest"
+    name: Run tests
+    steps:
+      - name: Check out code from GitHub
+        uses: "actions/checkout@v4"
+      - name: Setup Python
+        uses: "actions/setup-python@v5"
+        with:
+          python-version: "3.11"
+      - name: Install requirements
+        run: |
+          pip install --constraint=.github/workflows/constraints.txt pip
+          pip install -r requirements_test.txt
+      - name: Tests suite
+        run: |
+          pytest \
+            --timeout=9 \
+            --durations=10 \
+            -n auto \
+            -p no:sugar \
+            tests

custom_components/daikin_onecta/daikin_api.py

@@ -64,6 +64,7 @@ async def doBearerRequest(self, resourceUrl, options=None):
                 resourceUrl = "https://api.onecta.daikineurope.com" + resourceUrl
 
             headers = {
+                "Accept-Encoding": "gzip",
                 "Authorization": "Bearer " + token,
                 "Content-Type": "application/json",
             }

requirements_dev.txt

@@ -0,0 +1,6 @@
+pre-commit==3.6.2
+black==24.2.0
+flake8==7.0.0
+reorder-python-imports==3.12.0
+homeassistant
+pytest

requirements_test.txt

@@ -0,0 +1,10 @@
+-r requirements_dev.txt
+pytest-homeassistant-custom-component
+anyio
+pytest-asyncio
+pytest-tornasync
+pytest-trio
+pytest-twisted
+aiohttp_cors
+twisted
+service_identity

setup.cfg

@@ -0,0 +1,47 @@
+[flake8]
+exclude = .venv,.git,.tox,docs,venv,bin,lib,deps,build
+doctests = True
+# To work with Black
+max-line-length = 88
+# E501: line too long
+# W503: Line break occurred before a binary operator
+# E203: Whitespace before ':'
+# D202 No blank lines allowed after function docstring
+# W504 line break after binary operator
+ignore =
+    E501,
+    W503,
+    E203,
+    D202,
+    W504
+
+[isort]
+# https://github.com/timothycrosley/isort
+# https://github.com/timothycrosley/isort/wiki/isort-Settings
+# splits long import on multiple lines indented by 4 spaces
+multi_line_output = 3
+include_trailing_comma=True
+force_grid_wrap=0
+use_parentheses=True
+line_length=88
+indent = "    "
+# by default isort don't check module indexes
+not_skip = __init__.py
+# will group `import x` and `from x import` of the same module.
+force_sort_within_sections = true
+sections = FUTURE,STDLIB,INBETWEENS,THIRDPARTY,FIRSTPARTY,LOCALFOLDER
+default_section = THIRDPARTY
+known_first_party = custom_components.daikin_onecta, tests
+combine_as_imports = true
+
+[tool:pytest]
+addopts = -qq --cov=custom_components.daikin_onecta
+console_output_style = count
+asyncio_mode = auto
+
+[coverage:run]
+branch = False
+
+[coverage:report]
+show_missing = true
+fail_under = 0

tests/__init__.py

@@ -0,0 +1 @@
+"""Tests for the daikin_onecta integration."""

tests/test_config_flow.py

@@ -0,0 +1,82 @@
+"""Test the daikin_onecta config flow."""
+
+from unittest.mock import patch
+
+import pytest
+
+from homeassistant import config_entries
+from homeassistant.components.application_credentials import (
+    ClientCredential,
+    async_import_client_credential,
+)
+from custom_components.daikin_onecta.const import (
+    DOMAIN,
+    OAUTH2_AUTHORIZE,
+    OAUTH2_TOKEN,
+)
+from custom_components.daikin_onecta.device import DaikinOnectaDevice
+from homeassistant.core import HomeAssistant
+from homeassistant.helpers import config_entry_oauth2_flow
+from homeassistant.setup import async_setup_component
+
+CLIENT_ID = "1234"
+CLIENT_SECRET = "5678"
+
+
+@pytest.fixture(autouse=True)
+async def setup_credentials(hass: HomeAssistant) -> None:
+    """Fixture to setup credentials."""
+    assert await async_setup_component(hass, "application_credentials", {})
+    # await async_import_client_credential(
+    #     hass,
+    #     DOMAIN,
+    #     ClientCredential(CLIENT_ID, CLIENT_SECRET),
+    #     DOMAIN,
+    # )
+
+
+async def test_full_flow(
+    hass: HomeAssistant,
+    hass_client_no_auth,
+    aioclient_mock,
+    current_request_with_host,
+    setup_credentials,
+) -> None:
+    """Check full flow."""
+    # assert await async_setup_component(hass, "daikin_onecta", {})
+    #
+    # await async_import_client_credential(hass, DOMAIN, ClientCredential(CLIENT_ID, CLIENT_SECRET))
+    #
+    # result = await hass.config_entries.flow.async_init("daikin_onecta", context={"source": config_entries.SOURCE_USER})
+    # state = config_entry_oauth2_flow._encode_jwt(
+    #     hass,
+    #     {
+    #         "flow_id": result["flow_id"],
+    #         "redirect_uri": "https://example.com/auth/external/callback",
+    #     },
+    # )
+    #
+    # assert result["url"] == (
+    #     f"{OAUTH2_AUTHORIZE}?response_type=code&client_id={CLIENT_ID}" "&redirect_uri=https://example.com/auth/external/callback" f"&state={state}"
+    # )
+    #
+    # client = await hass_client_no_auth()
+    # resp = await client.get(f"/auth/external/callback?code=abcd&state={state}")
+    # assert resp.status == 200
+    # assert resp.headers["content-type"] == "text/html; charset=utf-8"
+    #
+    # aioclient_mock.post(
+    #     OAUTH2_TOKEN,
+    #     json={
+    #         "refresh_token": "mock-refresh-token",
+    #         "access_token": "mock-access-token",
+    #         "type": "Bearer",
+    #         "expires_in": 60,
+    #     },
+    # )
+    #
+    # with patch("homeassistant.components.daikin_onecta.async_setup_entry", return_value=True) as mock_setup:
+    #     await hass.config_entries.flow.async_configure(result["flow_id"])
+    #
+    # assert len(hass.config_entries.async_entries(DOMAIN)) == 1
+    # assert len(mock_setup.mock_calls) == 1