1
0
mirror of https://github.com/home-assistant/supervisor.git synced 2026-05-20 06:38:53 +01:00
Files
supervisor/tests/api/test_store.py
Mike Degatano 56abe94d74 Add versioned v2 API with apps terminology (#6741)
* Add versioned v2 API with apps terminology

Introduce a v2 API sub-app mounted at /v2 that uses 'apps' terminology
throughout, while keeping v1 fully backward-compatible.

Key changes:
- Add ATTR_ADDONS = 'addons' constant alongside ATTR_APPS = 'apps' so
  backup file data (which must remain 'addons' for backward compat) and
  v2 API responses can use distinct constants
- Add FeatureFlag.SUPERVISOR_V2_API to gate v2 route registration
- Mount aiohttp sub-app at /v2 in RestAPI.load() when flag is enabled
- Add _AppSecurityPatterns frozen dataclass and _V1_PATTERNS/_V2_PATTERNS
  with strict per-version regex sets (no cross-version matching)
- Add _register_v2_apps, _register_v2_backups, _register_v2_store route
  registration methods
- Add v1 thin wrapper methods (*_v1) for all affected endpoints so
  business logic lives in the canonical v2 methods
- Extract _info_data() helper in APIApps so v1 closure can bypass
  @api_process and still catch APIAppNotInstalled for store routing
- Add _rename_apps_to_addons_in_backups(), _process_location_in_body(),
  _all_store_apps_info() shared helpers to eliminate duplication
- Add api_client_v2, api_client_with_prefix, app_api_client_with_root,
  store_app_api_client_with_root parameterized test fixtures
- Add test_v2_api_disabled_without_feature_flag
- Parameterize backup, addons, and store tests to cover both v1 and v2
  paths

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Fix pylint false positive for re.Pattern C extension methods

re.Pattern methods (match, search, etc.) are C extension methods.
Pylint cannot detect them via static analysis when re.Pattern is used
as a type annotation in a dataclass field, producing false E1101
no-member errors. Add generated-members to inform pylint these members
exist.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* pylint and feedback fixes

* Copilot suggested fixes

* Minor feedback fixes

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-04-20 21:19:27 +02:00

1018 lines
35 KiB
Python

"""Test Store API."""
import asyncio
from pathlib import Path
from unittest.mock import AsyncMock, PropertyMock, patch
from aiodocker.containers import DockerContainer
from aiohttp.test_utils import TestClient
from awesomeversion import AwesomeVersion
import pytest
from supervisor.addons.addon import App
from supervisor.arch import CpuArchManager
from supervisor.backups.manager import BackupManager
from supervisor.config import CoreConfig
from supervisor.const import AppState, CoreState
from supervisor.coresys import CoreSys
from supervisor.docker.addon import DockerApp
from supervisor.docker.const import ContainerState
from supervisor.docker.interface import DockerInterface
from supervisor.docker.monitor import DockerContainerStateEvent
from supervisor.exceptions import StoreGitError
from supervisor.homeassistant.const import WSEvent
from supervisor.homeassistant.module import HomeAssistant
from supervisor.resolution.const import ContextType, IssueType, SuggestionType
from supervisor.resolution.data import Issue, Suggestion
from supervisor.store.addon import AppStore
from supervisor.store.repository import Repository
from tests.common import AsyncIterator, load_json_fixture
from tests.const import TEST_ADDON_SLUG
REPO_URL = "https://github.com/awesome-developer/awesome-repo"
@pytest.mark.asyncio
async def test_api_store(
api_client: TestClient,
store_app: AppStore,
test_repository: Repository,
caplog: pytest.LogCaptureFixture,
):
"""Test /store REST API."""
resp = await api_client.get("/store")
result = await resp.json()
assert result["data"]["addons"][-1]["slug"] == store_app.slug
assert result["data"]["repositories"][-1]["slug"] == test_repository.slug
assert f"App {store_app.slug} not supported on this platform" not in caplog.text
@pytest.mark.asyncio
async def test_api_store_apps(api_client: TestClient, store_app: AppStore):
"""Test /store/apps REST API."""
resp = await api_client.get("/store/addons")
result = await resp.json()
assert result["data"]["addons"][-1]["slug"] == store_app.slug
@pytest.mark.asyncio
async def test_api_store_apps_app(
store_app_api_client_with_root: tuple[TestClient, str], store_app: AppStore
):
"""Test /store/apps/{app} REST API."""
client, root = store_app_api_client_with_root
resp = await client.get(f"/{root}/{store_app.slug}")
result = await resp.json()
assert result["data"]["slug"] == store_app.slug
@pytest.mark.asyncio
async def test_api_store_apps_app_version(
store_app_api_client_with_root: tuple[TestClient, str], store_app: AppStore
):
"""Test /store/apps/{app}/{version} REST API."""
client, root = store_app_api_client_with_root
resp = await client.get(f"/{root}/{store_app.slug}/1.0.0")
result = await resp.json()
assert result["data"]["slug"] == store_app.slug
@pytest.mark.asyncio
async def test_api_store_repositories(
api_client: TestClient, test_repository: Repository
):
"""Test /store/repositories REST API."""
resp = await api_client.get("/store/repositories")
result = await resp.json()
assert result["data"][-1]["slug"] == test_repository.slug
@pytest.mark.asyncio
async def test_api_store_repositories_repository(
api_client: TestClient, test_repository: Repository
):
"""Test /store/repositories/{repository} REST API."""
resp = await api_client.get(f"/store/repositories/{test_repository.slug}")
result = await resp.json()
assert result["data"]["slug"] == test_repository.slug
async def test_api_store_add_repository(
api_client: TestClient, coresys: CoreSys, supervisor_internet: AsyncMock
) -> None:
"""Test POST /store/repositories REST API."""
with (
patch("supervisor.store.repository.RepositoryGit.load", return_value=None),
patch("supervisor.store.repository.RepositoryGit.validate", return_value=True),
):
response = await api_client.post(
"/store/repositories", json={"repository": REPO_URL}
)
assert response.status == 200
assert REPO_URL in coresys.store.repository_urls
async def test_api_store_remove_repository(
api_client: TestClient, coresys: CoreSys, test_repository: Repository
):
"""Test DELETE /store/repositories/{repository} REST API."""
response = await api_client.delete(f"/store/repositories/{test_repository.slug}")
assert response.status == 200
assert test_repository.source not in coresys.store.repository_urls
assert test_repository.slug not in coresys.store.repositories
@pytest.mark.parametrize("repo", ["core", "a474bbd1"])
@pytest.mark.usefixtures("test_repository")
async def test_api_store_repair_repository(api_client: TestClient, repo: str):
"""Test POST /store/repositories/{repository}/repair REST API."""
with patch("supervisor.store.repository.RepositoryGit.reset") as mock_reset:
response = await api_client.post(f"/store/repositories/{repo}/repair")
assert response.status == 200
mock_reset.assert_called_once()
@pytest.mark.parametrize(
"issue_type", [IssueType.CORRUPT_REPOSITORY, IssueType.FATAL_ERROR]
)
@pytest.mark.usefixtures("test_repository")
async def test_api_store_repair_repository_removes_suggestion(
api_client: TestClient,
coresys: CoreSys,
test_repository: Repository,
issue_type: IssueType,
):
"""Test POST /store/repositories/core/repair REST API removes EXECUTE_RESET suggestions."""
issue = Issue(issue_type, ContextType.STORE, reference=test_repository.slug)
suggestion = Suggestion(
SuggestionType.EXECUTE_RESET, ContextType.STORE, reference=test_repository.slug
)
coresys.resolution.add_issue(issue, suggestions=[SuggestionType.EXECUTE_RESET])
with patch("supervisor.store.repository.RepositoryGit.reset") as mock_reset:
response = await api_client.post(
f"/store/repositories/{test_repository.slug}/repair"
)
assert response.status == 200
mock_reset.assert_called_once()
assert issue not in coresys.resolution.issues
assert suggestion not in coresys.resolution.suggestions
@pytest.mark.usefixtures("test_repository")
async def test_api_store_repair_repository_local_fail(api_client: TestClient):
"""Test POST /store/repositories/local/repair REST API fails."""
response = await api_client.post("/store/repositories/local/repair")
assert response.status == 400
result = await response.json()
assert result["error_key"] == "store_repository_local_cannot_reset"
assert result["extra_fields"] == {"local_repo": "local"}
assert result["message"] == "Can't reset repository local as it is not git based!"
async def test_api_store_repair_repository_git_error(
api_client: TestClient, test_repository: Repository
):
"""Test POST /store/repositories/{repository}/repair REST API git error."""
with patch(
"supervisor.store.git.GitRepo.reset",
side_effect=StoreGitError("Git error"),
):
response = await api_client.post(
f"/store/repositories/{test_repository.slug}/repair"
)
assert response.status == 500
result = await response.json()
assert result["error_key"] == "store_repository_unknown_error"
assert result["extra_fields"] == {
"repo": test_repository.slug,
}
assert (
result["message"]
== f"An unknown error occurred with app repository {test_repository.slug}. Check Supervisor logs for details"
)
@pytest.mark.usefixtures("tmp_supervisor_data", "path_extern")
async def test_api_store_update_healthcheck(
api_client: TestClient,
coresys: CoreSys,
install_app_ssh: App,
container: DockerContainer,
):
"""Test updating an app with healthcheck waits for health status."""
coresys.hardware.disk.get_disk_free_space = lambda x: 5000
container.show.return_value["State"]["Status"] = "running"
container.show.return_value["State"]["Running"] = True
container.show.return_value["Config"] = {"Healthcheck": "exists"}
install_app_ssh.path_data.mkdir()
await install_app_ssh.load()
with patch(
"supervisor.store.data.read_json_or_yaml_file",
return_value=load_json_fixture("addon-config-add-image.json"),
):
await coresys.store.data.update()
assert install_app_ssh.need_update is True
state_changes: list[AppState] = []
_container_events_task: asyncio.Task | None = None
async def container_events():
nonlocal state_changes
await asyncio.sleep(0.01)
await install_app_ssh.container_state_changed(
DockerContainerStateEvent(
name=f"addon_{TEST_ADDON_SLUG}",
state=ContainerState.STOPPED,
id="abc123",
time=1,
)
)
state_changes.append(install_app_ssh.state)
await install_app_ssh.container_state_changed(
DockerContainerStateEvent(
name=f"addon_{TEST_ADDON_SLUG}",
state=ContainerState.RUNNING,
id="abc123",
time=1,
)
)
state_changes.append(install_app_ssh.state)
await install_app_ssh.container_state_changed(
DockerContainerStateEvent(
name=f"addon_{TEST_ADDON_SLUG}",
state=ContainerState.HEALTHY,
id="abc123",
time=1,
)
)
async def container_events_task(*args, **kwargs):
nonlocal _container_events_task
_container_events_task = asyncio.create_task(container_events())
with (
patch.object(DockerApp, "run", new=container_events_task),
patch.object(DockerInterface, "install"),
patch.object(DockerApp, "is_running", return_value=False),
patch.object(
CpuArchManager, "supported", new=PropertyMock(return_value=["amd64"])
),
):
resp = await api_client.post(f"/store/addons/{TEST_ADDON_SLUG}/update")
assert state_changes == [AppState.STOPPED, AppState.STARTUP]
assert install_app_ssh.state == AppState.STARTED
assert resp.status == 200
await _container_events_task
@pytest.mark.parametrize("resource", ["store/addons", "addons"])
async def test_api_store_apps_no_changelog(
api_client: TestClient, coresys: CoreSys, store_app: AppStore, resource: str
):
"""Test /store/apps/{app}/changelog REST API (v1 paths).
Currently the frontend expects a valid body even in the error case. Make sure that is
what the API returns.
"""
assert store_app.with_changelog is False
resp = await api_client.get(f"/{resource}/{store_app.slug}/changelog")
assert resp.status == 200
result = await resp.text()
assert result == "No changelog found for app test_store_addon!"
async def test_api_store_apps_no_changelog_v2(
store_app_api_client_with_root: tuple[TestClient, str],
coresys: CoreSys,
store_app: AppStore,
):
"""Test /store/apps/{app}/changelog REST API for both v1 and v2 store paths."""
client, root = store_app_api_client_with_root
assert store_app.with_changelog is False
resp = await client.get(f"/{root}/{store_app.slug}/changelog")
assert resp.status == 200
result = await resp.text()
assert result == "No changelog found for app test_store_addon!"
@pytest.mark.parametrize("resource", ["store/addons", "addons"])
async def test_api_detached_app_changelog(
api_client: TestClient,
coresys: CoreSys,
install_app_ssh: App,
tmp_supervisor_data: Path,
resource: str,
):
"""Test /store/apps/{app}/changelog for a detached app (v1 paths).
Currently the frontend expects a valid body even in the error case. Make sure that is
what the API returns.
"""
(apps_dir := tmp_supervisor_data / "addons" / "local").mkdir()
with patch.object(
CoreConfig, "path_apps_local", new=PropertyMock(return_value=apps_dir)
):
await coresys.store.load()
assert install_app_ssh.is_detached is True
assert install_app_ssh.with_changelog is False
resp = await api_client.get(f"/{resource}/{install_app_ssh.slug}/changelog")
assert resp.status == 200
result = await resp.text()
assert result == "App local_ssh does not exist in the store"
async def test_api_detached_app_changelog_v2(
store_app_api_client_with_root: tuple[TestClient, str],
coresys: CoreSys,
install_app_ssh: App,
tmp_supervisor_data: Path,
):
"""Test /store/apps/{app}/changelog for a detached app for both v1 and v2 store paths."""
client, root = store_app_api_client_with_root
(apps_dir := tmp_supervisor_data / "addons" / "local").mkdir()
with patch.object(
CoreConfig, "path_apps_local", new=PropertyMock(return_value=apps_dir)
):
await coresys.store.load()
assert install_app_ssh.is_detached is True
assert install_app_ssh.with_changelog is False
resp = await client.get(f"/{root}/{install_app_ssh.slug}/changelog")
assert resp.status == 200
result = await resp.text()
assert result == "App local_ssh does not exist in the store"
@pytest.mark.parametrize("resource", ["store/addons", "addons"])
async def test_api_store_apps_no_documentation(
api_client: TestClient, coresys: CoreSys, store_app: AppStore, resource: str
):
"""Test /store/apps/{app}/documentation REST API (v1 paths).
Currently the frontend expects a valid body even in the error case. Make sure that is
what the API returns.
"""
assert store_app.with_documentation is False
resp = await api_client.get(f"/{resource}/{store_app.slug}/documentation")
assert resp.status == 200
result = await resp.text()
assert result == "No documentation found for app test_store_addon!"
async def test_api_store_apps_no_documentation_v2(
store_app_api_client_with_root: tuple[TestClient, str],
coresys: CoreSys,
store_app: AppStore,
):
"""Test /store/apps/{app}/documentation REST API for both v1 and v2 store paths."""
client, root = store_app_api_client_with_root
assert store_app.with_documentation is False
resp = await client.get(f"/{root}/{store_app.slug}/documentation")
assert resp.status == 200
result = await resp.text()
assert result == "No documentation found for app test_store_addon!"
@pytest.mark.parametrize("resource", ["store/addons", "addons"])
async def test_api_detached_app_documentation(
api_client: TestClient,
coresys: CoreSys,
install_app_ssh: App,
tmp_supervisor_data: Path,
resource: str,
):
"""Test /store/apps/{app}/documentation for a detached app (v1 paths).
Currently the frontend expects a valid body even in the error case. Make sure that is
what the API returns.
"""
(apps_dir := tmp_supervisor_data / "addons" / "local").mkdir()
with patch.object(
CoreConfig, "path_apps_local", new=PropertyMock(return_value=apps_dir)
):
await coresys.store.load()
assert install_app_ssh.is_detached is True
assert install_app_ssh.with_documentation is False
resp = await api_client.get(f"/{resource}/{install_app_ssh.slug}/documentation")
assert resp.status == 200
result = await resp.text()
assert result == "App local_ssh does not exist in the store"
async def test_api_detached_app_documentation_v2(
store_app_api_client_with_root: tuple[TestClient, str],
coresys: CoreSys,
install_app_ssh: App,
tmp_supervisor_data: Path,
):
"""Test /store/apps/{app}/documentation for a detached app for both v1 and v2 store paths."""
client, root = store_app_api_client_with_root
(apps_dir := tmp_supervisor_data / "addons" / "local").mkdir()
with patch.object(
CoreConfig, "path_apps_local", new=PropertyMock(return_value=apps_dir)
):
await coresys.store.load()
assert install_app_ssh.is_detached is True
assert install_app_ssh.with_documentation is False
resp = await client.get(f"/{root}/{install_app_ssh.slug}/documentation")
assert resp.status == 200
result = await resp.text()
assert result == "App local_ssh does not exist in the store"
@pytest.mark.parametrize(
("method", "action", "json_expected"),
[
("get", "bad", True),
("get", "bad/1", True),
("get", "bad/icon", False),
("get", "bad/logo", False),
("post", "bad/install", True),
("post", "bad/install/1", True),
("post", "bad/update", True),
("post", "bad/update/1", True),
("get", "bad/availability", True),
],
)
async def test_store_app_not_found(
store_app_api_client_with_root: tuple[TestClient, str],
method: str,
action: str,
json_expected: bool,
):
"""Test store app not found error for both v1 and v2 store paths."""
client, root = store_app_api_client_with_root
resp = await client.request(method, f"/{root}/{action}")
assert resp.status == 404
if json_expected:
body = await resp.json()
assert body["message"] == "App bad does not exist in the store"
assert body["error_key"] == "store_addon_not_found_error"
assert body["extra_fields"] == {"addon": "bad"}
else:
assert await resp.text() == "App bad does not exist in the store"
@pytest.mark.parametrize(
("method", "url", "json_expected"),
[
("get", "/addons/bad/icon", False),
("get", "/addons/bad/logo", False),
("post", "/addons/bad/install", True),
("post", "/addons/bad/update", True),
],
)
async def test_store_app_not_found_legacy_paths(
api_client: TestClient, method: str, url: str, json_expected: bool
):
"""Test store app not found error for legacy /addons/ store paths."""
resp = await api_client.request(method, url)
assert resp.status == 404
if json_expected:
body = await resp.json()
assert body["message"] == "App bad does not exist in the store"
assert body["error_key"] == "store_addon_not_found_error"
assert body["extra_fields"] == {"addon": "bad"}
else:
assert await resp.text() == "App bad does not exist in the store"
@pytest.mark.parametrize(
("method", "url"),
[
("post", "/store/addons/local_ssh/update"),
("post", "/store/addons/local_ssh/update/1"),
# Legacy paths
("post", "/addons/local_ssh/update"),
],
)
@pytest.mark.usefixtures("test_repository")
async def test_store_app_not_installed(api_client: TestClient, method: str, url: str):
"""Test store app not installed error."""
resp = await api_client.request(method, url)
assert resp.status == 400
body = await resp.json()
assert body["message"] == "App local_ssh is not installed"
@pytest.mark.parametrize(
("method", "url"),
[
("get", "/store/repositories/bad"),
("delete", "/store/repositories/bad"),
],
)
async def test_repository_not_found(api_client: TestClient, method: str, url: str):
"""Test repository not found error."""
resp = await api_client.request(method, url)
assert resp.status == 404
body = await resp.json()
assert body["message"] == "Repository bad does not exist in the store"
@pytest.mark.parametrize("resource", ["store/addons", "addons"])
async def test_api_store_apps_documentation_corrupted(
api_client: TestClient, coresys: CoreSys, store_app: AppStore, resource: str
):
"""Test /store/apps/{app}/documentation REST API.
Test app with documentation file with byte sequences which cannot be decoded
using UTF-8.
"""
store_app.path_documentation.write_bytes(b"Text with an invalid UTF-8 char: \xff")
await store_app.refresh_path_cache()
assert store_app.with_documentation is True
resp = await api_client.get(f"/{resource}/{store_app.slug}/documentation")
assert resp.status == 200
result = await resp.text()
assert result == "Text with an invalid UTF-8 char: "
@pytest.mark.parametrize("resource", ["store/addons", "addons"])
async def test_api_store_apps_changelog_corrupted(
api_client: TestClient, coresys: CoreSys, store_app: AppStore, resource: str
):
"""Test /store/apps/{app}/changelog REST API.
Test app with changelog file with byte sequences which cannot be decoded
using UTF-8.
"""
store_app.path_changelog.write_bytes(b"Text with an invalid UTF-8 char: \xff")
await store_app.refresh_path_cache()
assert store_app.with_changelog is True
resp = await api_client.get(f"/{resource}/{store_app.slug}/changelog")
assert resp.status == 200
result = await resp.text()
assert result == "Text with an invalid UTF-8 char: "
@pytest.mark.usefixtures("test_repository", "tmp_supervisor_data")
async def test_app_install_in_background(api_client: TestClient, coresys: CoreSys):
"""Test installing an app in the background."""
coresys.hardware.disk.get_disk_free_space = lambda x: 5000
event = asyncio.Event()
# Mock a long-running install task
async def mock_app_install(*args, **kwargs):
await event.wait()
with patch.object(App, "install", new=mock_app_install):
resp = await api_client.post(
"/store/addons/local_ssh/install", json={"background": True}
)
assert resp.status == 200
body = await resp.json()
assert (job := coresys.jobs.get_job(body["data"]["job_id"]))
assert job.name == "addon_manager_install"
event.set()
@pytest.mark.usefixtures("install_app_ssh")
async def test_background_app_install_fails_fast(
api_client: TestClient, coresys: CoreSys
):
"""Test background app install returns error not job if validation fails."""
coresys.hardware.disk.get_disk_free_space = lambda x: 5000
resp = await api_client.post(
"/store/addons/local_ssh/install", json={"background": True}
)
assert resp.status == 400
body = await resp.json()
assert body["message"] == "App local_ssh is already installed"
@pytest.mark.parametrize(
("make_backup", "backup_called", "update_called"),
[(True, True, False), (False, False, True)],
)
@pytest.mark.usefixtures("test_repository", "tmp_supervisor_data")
async def test_app_update_in_background(
api_client: TestClient,
coresys: CoreSys,
install_app_ssh: App,
make_backup: bool,
backup_called: bool,
update_called: bool,
):
"""Test updating an app in the background."""
coresys.hardware.disk.get_disk_free_space = lambda x: 5000
install_app_ssh.data_store["version"] = "10.0.0"
event = asyncio.Event()
mock_update_called = mock_backup_called = False
# Mock backup/update as long-running tasks
async def mock_app_update(*args, **kwargs):
nonlocal mock_update_called
mock_update_called = True
await event.wait()
async def mock_partial_backup(*args, **kwargs):
nonlocal mock_backup_called
mock_backup_called = True
await event.wait()
with (
patch.object(App, "update", new=mock_app_update),
patch.object(BackupManager, "do_backup_partial", new=mock_partial_backup),
):
resp = await api_client.post(
"/store/addons/local_ssh/update",
json={"background": True, "backup": make_backup},
)
assert mock_backup_called is backup_called
assert mock_update_called is update_called
assert resp.status == 200
body = await resp.json()
assert (job := coresys.jobs.get_job(body["data"]["job_id"]))
assert job.name == "addon_manager_update"
event.set()
@pytest.mark.usefixtures("install_app_ssh")
async def test_background_app_update_fails_fast(
api_client: TestClient, coresys: CoreSys
):
"""Test background app update returns error not job if validation doesn't succeed."""
coresys.hardware.disk.get_disk_free_space = lambda x: 5000
resp = await api_client.post(
"/store/addons/local_ssh/update", json={"background": True}
)
assert resp.status == 400
body = await resp.json()
assert body["message"] == "No update available for app local_ssh"
async def test_api_store_apps_app_availability_success(
api_client: TestClient, store_app: AppStore
):
"""Test /store/apps/{app}/availability REST API - success case."""
resp = await api_client.get(f"/store/addons/{store_app.slug}/availability")
assert resp.status == 200
@pytest.mark.parametrize(
("supported_architectures", "api_action", "api_method", "installed"),
[
(["aarch64"], "availability", "get", False),
(["aarch64", "fooarch"], "availability", "get", False),
(["aarch64"], "install", "post", False),
(["aarch64", "fooarch"], "install", "post", False),
(["aarch64"], "update", "post", True),
(["aarch64", "fooarch"], "update", "post", True),
],
)
async def test_api_store_apps_app_availability_arch_not_supported(
api_client: TestClient,
coresys: CoreSys,
supported_architectures: list[str],
api_action: str,
api_method: str,
installed: bool,
):
"""Test availability errors for /store/apps/{app}/* REST APIs - architecture not supported."""
coresys.hardware.disk.get_disk_free_space = lambda x: 5000
# Create an app with unsupported architecture
app_obj = AppStore(coresys, "test_arch_addon")
coresys.apps.store[app_obj.slug] = app_obj
# Set app config with unsupported architecture
app_config = {
"advanced": False,
"arch": supported_architectures,
"slug": "test_arch_addon",
"description": "Test arch add-on",
"name": "Test Arch Add-on",
"repository": "test",
"stage": "stable",
"version": "1.0.0",
}
coresys.store.data.apps[app_obj.slug] = app_config
if installed:
coresys.apps.local[app_obj.slug] = App(coresys, app_obj.slug)
coresys.apps.data.user[app_obj.slug] = {"version": AwesomeVersion("0.0.1")}
# Mock the system architecture to be different
with patch.object(
CpuArchManager, "supported", new=PropertyMock(return_value=["amd64"])
):
resp = await api_client.request(
api_method, f"/store/addons/{app_obj.slug}/{api_action}"
)
assert resp.status == 400
result = await resp.json()
assert result["error_key"] == "addon_not_supported_architecture_error"
assert result["extra_fields"] == {
"slug": "test_arch_addon",
"architectures": (architectures := ", ".join(supported_architectures)),
}
assert (
result["message"]
== f"App test_arch_addon not supported on this platform, supported architectures: {architectures}"
)
@pytest.mark.parametrize(
("supported_machines", "api_action", "api_method", "installed"),
[
(["odroid-n2"], "availability", "get", False),
(["!qemux86-64"], "availability", "get", False),
(["a", "b"], "availability", "get", False),
(["odroid-n2"], "install", "post", False),
(["!qemux86-64"], "install", "post", False),
(["a", "b"], "install", "post", False),
(["odroid-n2"], "update", "post", True),
(["!qemux86-64"], "update", "post", True),
(["a", "b"], "update", "post", True),
],
)
async def test_api_store_apps_app_availability_machine_not_supported(
api_client: TestClient,
coresys: CoreSys,
supported_machines: list[str],
api_action: str,
api_method: str,
installed: bool,
):
"""Test availability errors for /store/apps/{app}/* REST APIs - machine not supported."""
coresys.hardware.disk.get_disk_free_space = lambda x: 5000
# Create an app with unsupported machine type
app_obj = AppStore(coresys, "test_machine_addon")
coresys.apps.store[app_obj.slug] = app_obj
# Set app config with unsupported machine
app_config = {
"advanced": False,
"arch": ["amd64"],
"machine": supported_machines,
"slug": "test_machine_addon",
"description": "Test machine add-on",
"name": "Test Machine Add-on",
"repository": "test",
"stage": "stable",
"version": "1.0.0",
}
coresys.store.data.apps[app_obj.slug] = app_config
if installed:
coresys.apps.local[app_obj.slug] = App(coresys, app_obj.slug)
coresys.apps.data.user[app_obj.slug] = {"version": AwesomeVersion("0.0.1")}
# Mock the system machine to be different
with patch.object(CoreSys, "machine", new=PropertyMock(return_value="qemux86-64")):
resp = await api_client.request(
api_method, f"/store/addons/{app_obj.slug}/{api_action}"
)
assert resp.status == 400
result = await resp.json()
assert result["error_key"] == "addon_not_supported_machine_type_error"
assert result["extra_fields"] == {
"slug": "test_machine_addon",
"machine_types": (machine_types := ", ".join(supported_machines)),
}
assert (
result["message"]
== f"App test_machine_addon not supported on this machine, supported machine types: {machine_types}"
)
@pytest.mark.parametrize(
("api_action", "api_method", "installed"),
[
("availability", "get", False),
("install", "post", False),
("update", "post", True),
],
)
async def test_api_store_apps_app_availability_homeassistant_version_too_old(
api_client: TestClient,
coresys: CoreSys,
api_action: str,
api_method: str,
installed: bool,
):
"""Test availability errors for /store/apps/{app}/* REST APIs - Home Assistant version too old."""
coresys.hardware.disk.get_disk_free_space = lambda x: 5000
# Create an app that requires newer Home Assistant version
app_obj = AppStore(coresys, "test_version_addon")
coresys.apps.store[app_obj.slug] = app_obj
# Set app config with minimum Home Assistant version requirement
app_config = {
"advanced": False,
"arch": ["amd64"],
"homeassistant": "2023.1.1", # Requires newer version than current
"slug": "test_version_addon",
"description": "Test version add-on",
"name": "Test Version Add-on",
"repository": "test",
"stage": "stable",
"version": "1.0.0",
}
coresys.store.data.apps[app_obj.slug] = app_config
if installed:
coresys.apps.local[app_obj.slug] = App(coresys, app_obj.slug)
coresys.apps.data.user[app_obj.slug] = {"version": AwesomeVersion("0.0.1")}
# Mock the Home Assistant version to be older
with patch.object(
HomeAssistant,
"version",
new=PropertyMock(return_value=AwesomeVersion("2022.1.1")),
):
resp = await api_client.request(
api_method, f"/store/addons/{app_obj.slug}/{api_action}"
)
assert resp.status == 400
result = await resp.json()
assert result["error_key"] == "addon_not_supported_home_assistant_version_error"
assert result["extra_fields"] == {
"slug": "test_version_addon",
"version": "2023.1.1",
}
assert (
result["message"]
== "App test_version_addon not supported on this system, requires Home Assistant version 2023.1.1 or greater"
)
async def test_api_store_apps_app_availability_installed_app(
api_client: TestClient, install_app_ssh: App
):
"""Test /store/apps/{app}/availability REST API - installed app checks against latest version."""
resp = await api_client.get("/store/addons/local_ssh/availability")
assert resp.status == 200
install_app_ssh.data_store["version"] = AwesomeVersion("10.0.0")
install_app_ssh.data_store["homeassistant"] = AwesomeVersion("2023.1.1")
# Mock the Home Assistant version to be older
with patch.object(
HomeAssistant,
"version",
new=PropertyMock(return_value=AwesomeVersion("2022.1.1")),
):
resp = await api_client.get("/store/addons/local_ssh/availability")
assert resp.status == 400
result = await resp.json()
assert (
"requires Home Assistant version 2023.1.1 or greater" in result["message"]
)
@pytest.mark.parametrize(
("action", "job_name", "app_slug"),
[
("install", "addon_manager_install", "local_ssh"),
("update", "addon_manager_update", "local_example"),
],
)
@pytest.mark.usefixtures("tmp_supervisor_data")
async def test_api_progress_updates_app_install_update(
api_client: TestClient,
coresys: CoreSys,
ha_ws_client: AsyncMock,
install_app_example: App,
action: str,
job_name: str,
app_slug: str,
):
"""Test progress updates sent to Home Assistant for installs/updates."""
coresys.hardware.disk.get_disk_free_space = lambda x: 5000
coresys.core.set_state(CoreState.RUNNING)
logs = load_json_fixture("docker_pull_image_log.json")
coresys.docker.images.pull.return_value = AsyncIterator(logs)
coresys.arch._supported_arch = ["amd64"] # pylint: disable=protected-access
install_app_example.data_store["version"] = AwesomeVersion("2.0.0")
with (
patch.object(App, "load"),
patch.object(App, "need_build", new=PropertyMock(return_value=False)),
patch.object(App, "latest_need_build", new=PropertyMock(return_value=False)),
):
resp = await api_client.post(f"/store/addons/{app_slug}/{action}")
assert resp.status == 200
events = [
{
"stage": evt.args[0]["data"]["data"]["stage"],
"progress": evt.args[0]["data"]["data"]["progress"],
"done": evt.args[0]["data"]["data"]["done"],
}
for evt in ha_ws_client.async_send_command.call_args_list
if "data" in evt.args[0]
and evt.args[0]["data"]["event"] == WSEvent.JOB
and evt.args[0]["data"]["data"]["name"] == job_name
and evt.args[0]["data"]["data"]["reference"] == app_slug
]
# Count-based progress: 2 layers need pulling (each worth 50%)
# Layers that already exist are excluded from progress calculation
assert events[:4] == [
{
"stage": None,
"progress": 0,
"done": False,
},
{
"stage": None,
"progress": 9.2,
"done": False,
},
{
"stage": None,
"progress": 25.6,
"done": False,
},
{
"stage": None,
"progress": 35.4,
"done": False,
},
]
assert events[-5:] == [
{
"stage": None,
"progress": 95.5,
"done": False,
},
{
"stage": None,
"progress": 96.9,
"done": False,
},
{
"stage": None,
"progress": 98.2,
"done": False,
},
{
"stage": None,
"progress": 100,
"done": False,
},
{
"stage": None,
"progress": 100,
"done": True,
},
]
# ── V2 API tests ──────────────────────────────────────────────────────────────
async def test_v2_store_info_uses_apps_key(
api_client_v2: TestClient, coresys: CoreSys, store_app: App
):
"""V2 GET /v2/store returns 'apps' key (not 'addons')."""
resp = await api_client_v2.get("/v2/store")
assert resp.status == 200
result = await resp.json()
assert "apps" in result["data"]
assert "addons" not in result["data"]
async def test_v2_store_apps_list_uses_apps_key(
api_client_v2: TestClient, coresys: CoreSys, store_app: App
):
"""V2 GET /v2/store/apps returns 'apps' key (not 'addons')."""
resp = await api_client_v2.get("/v2/store/apps")
assert resp.status == 200
result = await resp.json()
assert "apps" in result["data"]
assert "addons" not in result["data"]
assert result["data"]["apps"][-1]["slug"] == store_app.slug