Skip to content

Commit

Permalink
Update guppy to 3.1.5 (home-assistant#134646)
Browse files Browse the repository at this point in the history
  • Loading branch information
frenck authored Jan 4, 2025
1 parent baa13de commit 584439c
Show file tree
Hide file tree
Showing 5 changed files with 3 additions and 29 deletions.
4 changes: 0 additions & 4 deletions homeassistant/components/profiler/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -436,10 +436,6 @@ async def _async_generate_memory_profile(hass: HomeAssistant, call: ServiceCall)
# Imports deferred to avoid loading modules
# in memory since usually only one part of this
# integration is used at a time
if sys.version_info >= (3, 13):
raise HomeAssistantError(
"Memory profiling is not supported on Python 3.13. Please use Python 3.12."
)
from guppy import hpy # pylint: disable=import-outside-toplevel

start_time = int(time.time() * 1000000)
Expand Down
2 changes: 1 addition & 1 deletion homeassistant/components/profiler/manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
"quality_scale": "internal",
"requirements": [
"pyprof2calltree==1.4.5",
"guppy3==3.1.4.post1;python_version<'3.13'",
"guppy3==3.1.5",
"objgraph==3.5.0"
],
"single_config_entry": true
Expand Down
2 changes: 1 addition & 1 deletion requirements_all.txt
Original file line number Diff line number Diff line change
Expand Up @@ -1079,7 +1079,7 @@ gspread==5.5.0
gstreamer-player==1.1.2

# homeassistant.components.profiler
guppy3==3.1.4.post1;python_version<'3.13'
guppy3==3.1.5

# homeassistant.components.iaqualink
h2==4.1.0
Expand Down
2 changes: 1 addition & 1 deletion requirements_test_all.txt
Original file line number Diff line number Diff line change
Expand Up @@ -920,7 +920,7 @@ growattServer==1.5.0
gspread==5.5.0

# homeassistant.components.profiler
guppy3==3.1.4.post1;python_version<'3.13'
guppy3==3.1.5

# homeassistant.components.iaqualink
h2==4.1.0
Expand Down
22 changes: 0 additions & 22 deletions tests/components/profiler/test_init.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
import logging
import os
from pathlib import Path
import sys
from unittest.mock import patch

from freezegun.api import FrozenDateTimeFactory
Expand Down Expand Up @@ -71,9 +70,6 @@ def _mock_path(filename: str) -> str:
await hass.async_block_till_done()


@pytest.mark.skipif(
sys.version_info >= (3, 13), reason="not yet available on Python 3.13"
)
async def test_memory_usage(hass: HomeAssistant, tmp_path: Path) -> None:
"""Test we can setup and the service is registered."""
test_dir = tmp_path / "profiles"
Expand Down Expand Up @@ -105,24 +101,6 @@ def _mock_path(filename: str) -> str:
await hass.async_block_till_done()


@pytest.mark.skipif(sys.version_info < (3, 13), reason="still works on python 3.12")
async def test_memory_usage_py313(hass: HomeAssistant, tmp_path: Path) -> None:
"""Test raise an error on python3.13."""
entry = MockConfigEntry(domain=DOMAIN)
entry.add_to_hass(hass)

assert await hass.config_entries.async_setup(entry.entry_id)
await hass.async_block_till_done()
assert hass.services.has_service(DOMAIN, SERVICE_MEMORY)
with pytest.raises(
HomeAssistantError,
match="Memory profiling is not supported on Python 3.13. Please use Python 3.12.",
):
await hass.services.async_call(
DOMAIN, SERVICE_MEMORY, {CONF_SECONDS: 0.000001}, blocking=True
)


async def test_object_growth_logging(
hass: HomeAssistant,
caplog: pytest.LogCaptureFixture,
Expand Down

0 comments on commit 584439c

Please sign in to comment.