Удалён лишний __init__ в Pg; API_URL_WEATHER используется напрямую
This commit is contained in:
parent
bb9f291623
commit
0d5d8dbe7b
@ -8,13 +8,10 @@ logger = logging.getLogger(__name__)
|
|||||||
class Pg(commands.Cog):
|
class Pg(commands.Cog):
|
||||||
"""Команда !pg — прогноз погоды для Магнитогорска"""
|
"""Команда !pg — прогноз погоды для Магнитогорска"""
|
||||||
|
|
||||||
def __init__(self):
|
|
||||||
self.api_url = API_URL_WEATHER
|
|
||||||
|
|
||||||
@commands.command(name="pg")
|
@commands.command(name="pg")
|
||||||
async def pg(self, ctx: commands.Context) -> None:
|
async def pg(self, ctx: commands.Context) -> None:
|
||||||
"""Прогноз погоды в Магнитогорске"""
|
"""Прогноз погоды в Магнитогорске"""
|
||||||
data = await fetch_weather(self.api_url)
|
data = await fetch_weather(API_URL_WEATHER)
|
||||||
if data is None:
|
if data is None:
|
||||||
logger.warning(
|
logger.warning(
|
||||||
"%s: !pg — не удалось получить погоду (API вернул None)", ctx.author
|
"%s: !pg — не удалось получить погоду (API вернул None)", ctx.author
|
||||||
|
|||||||
@ -7,10 +7,14 @@ from commands.pg import Pg
|
|||||||
class TestPgInit:
|
class TestPgInit:
|
||||||
"""Тесты инициализации Cog Pg."""
|
"""Тесты инициализации Cog Pg."""
|
||||||
|
|
||||||
def test_init_sets_api_url(self) -> None:
|
def test_uses_api_url_constant(self) -> None:
|
||||||
"""__init__ должен устанавливать api_url."""
|
"""Pg использует API_URL_WEATHER напрямую (без инстанс-переменной)."""
|
||||||
|
from utils.pogoda import API_URL_WEATHER
|
||||||
|
|
||||||
cog = Pg()
|
cog = Pg()
|
||||||
assert cog.api_url == "https://wttr.in/Magnitogorsk?format=j1&lang=ru"
|
# Pg не хранит api_url как инстанс-переменную — использует константу напрямую
|
||||||
|
assert not hasattr(cog, "api_url")
|
||||||
|
assert API_URL_WEATHER == "https://wttr.in/Magnitogorsk?format=j1&lang=ru"
|
||||||
|
|
||||||
|
|
||||||
class TestPgCommand:
|
class TestPgCommand:
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user