refactor(weather): add app_build method

Dockerfile

@@ -21,4 +21,4 @@ COPY --from=builder /app ./
 COPY gismeteo gismeteo/
 COPY weather weather/
 
-CMD ["uvicorn", "weather.app:app", "--host", "0.0.0.0", "--port", "80"]
+CMD ["uvicorn", "weather.main:app", "--host", "0.0.0.0", "--port", "80"]

gismeteo/datehelp.py

@@ -1,9 +1,18 @@
 import datetime
+from enum import Enum
 
 import dateparser
 import dateparser.date_parser
 
 
+class Day(str, Enum):
+    TODAY = "today"
+    TOMORROW = "tomorrow"
+    DAY_3 = "3-day"
+    DAY_4 = "4-day"
+    DAY_5 = "5-day"
+
+
 def parse(value: str) -> datetime.date:
     if value == "today" or value == "mock":
         return datetime.date.today()

pyproject.toml

@@ -31,7 +31,7 @@ requires = ["poetry-core"]
 build-backend = "poetry.core.masonry.api"
 
 [tool.poetry.scripts]
-app = 'weather.app:run'
+app = 'weather.main:run'
 
 [tool.pytest.ini_options]
 addopts = "-p no:warnings"

weather/api.py

@@ -6,10 +6,3 @@ from .model import WeatherResponse
 class WeatherApi:
     async def get_day(self, location_id: str, date: datetime.date) -> WeatherResponse:
         raise NotImplementedError
-
-
-DEFAULT_API: WeatherApi = None
-
-
-def get_api() -> WeatherApi:
-    return DEFAULT_API

weather/app.py

@@ -1,27 +1,22 @@
 import locale
-from os import environ
 
-import uvicorn
 from fastapi import FastAPI
 
-from gismeteo.api import GismeteoApi
+from weather.api import WeatherApi
 
-from . import api as _api
 from .route import api, doc, view
 
-_api.DEFAULT_API = GismeteoApi()
 
-locale.setlocale(locale.LC_TIME, "ru_RU.UTF-8")
+def build_app(weather_api: WeatherApi) -> FastAPI:
+    locale.setlocale(locale.LC_TIME, "ru_RU.UTF-8")
 
-app = FastAPI(
+    app = FastAPI(
         title="Weather",
         docs_url=None,
         redoc_url=None,
-)
+    )
-doc.mount(app)
+    app.state.weather_api = weather_api
-api.mount(app)
+    doc.mount(app)
-view.mount(app)
+    api.mount(app)
-
+    view.mount(app)
-
+    return app
-def run():
-    uvicorn.run("weather.app:app", host="0.0.0.0", port=8000, reload="DEBUG" in environ)

weather/main.py

@@ -0,0 +1,12 @@
+from os import environ
+
+import uvicorn
+
+from gismeteo.api import GismeteoApi
+from weather.app import build_app
+
+app = build_app(GismeteoApi())
+
+
+def run():
+    uvicorn.run("weather.main:app", host="0.0.0.0", port=8000, reload="DEBUG" in environ)

weather/route/api.py

@@ -1,12 +1,13 @@
 import datetime
 
-from fastapi import FastAPI
+from fastapi import FastAPI, Request
 
-from weather.api import get_api
 from weather.model import WeatherResponse
 
 
 def mount(app: FastAPI):
     @app.get("/api/weather/{location}/{date}")
-    async def get_weather(location: str, date: datetime.date) -> WeatherResponse:
+    async def get_weather_api(
-        return await get_api().get_day(location, date)
+        request: Request, location: str, date: datetime.date
+    ) -> WeatherResponse:
+        return await request.app.state.weather_api.get_day(location, date)

weather/route/view/__init__.py

@@ -6,8 +6,10 @@ from fastapi.responses import HTMLResponse, RedirectResponse
 from fastapi.staticfiles import StaticFiles
 from fastapi.templating import Jinja2Templates
 
+from gismeteo import datehelp
+from gismeteo.location import LOCATION_BUNDLE
 from gismeteo.mock import MOCK_DATA
-from weather.api import get_api
+from weather.model import WeatherResponse
 
 from .filters import cloudness_icon, wind_direction_icon
 
@@ -19,6 +21,30 @@ def mount(app: FastAPI):
     templates.env.filters["wind_direction_icon"] = wind_direction_icon
     templates.env.filters["cloudness_icon"] = cloudness_icon
 
+    def build_weather_response(request: Request, response: WeatherResponse):
+        return templates.TemplateResponse(
+            request=request,
+            name="weather.html",
+            context={
+                "datetime": datetime,
+                "response": response,
+            },
+        )
+
+    @app.get("/", response_class=RedirectResponse)
+    async def get_weather_root():
+        return RedirectResponse("weather")
+
+    @app.get("/weather", response_class=HTMLResponse)
+    async def get_weather_list(request: Request):
+        return templates.TemplateResponse(
+            request=request,
+            name="index.html",
+            context={
+                "locations": LOCATION_BUNDLE._values,
+            },
+        )
+
     @app.get("/weather/{location}", response_class=RedirectResponse)
     async def get_weather_default(location: str):
         return RedirectResponse(f"{location}/{datetime.date.today()}")
@@ -26,23 +52,15 @@ def mount(app: FastAPI):
     @app.get("/weather/{location}/mock", response_class=HTMLResponse)
     async def get_weather_mock(request: Request):
         response = MOCK_DATA.response
-        return templates.TemplateResponse(
+        return build_weather_response(request, response)
-            request=request,
+
-            name="weather.html",
+    # @app.get("/weather/{location}/{day}", response_class=HTMLResponse)
-            context={
+    async def get_weather_day(request: Request, location: str, day: datehelp.Day):
-                "datetime": datetime,
+        date = datehelp.parse(day)
-                "response": response,
+        response = await request.app.state.weather_api.get_day(location, date)
-            },
+        return build_weather_response(request, response)
-        )
 
     @app.get("/weather/{location}/{date}", response_class=HTMLResponse)
     async def get_weather(request: Request, location: str, date: datetime.date):
-        response = await get_api().get_day(location, date)
+        response = await request.app.state.weather_api.get_day(location, date)
-        return templates.TemplateResponse(
+        return build_weather_response(request, response)
-            request=request,
-            name="weather.html",
-            context={
-                "datetime": datetime,
-                "response": response,
-            },
-        )

weather/route/view/filters.py

@@ -3,14 +3,14 @@ from weather.model import Cloudness, Precipitation, Sky, WindDirection
 
 def wind_direction_icon(wind_direction: WindDirection) -> str:
     return {
-        WindDirection.N: "🡫",
+        WindDirection.N: "⬇️",
-        WindDirection.NO: "🡯",
+        WindDirection.NO: "↙️",
-        WindDirection.O: "🡨",
+        WindDirection.O: "⬅️",
-        WindDirection.SO: "🡬",
+        WindDirection.SO: "↖️",
-        WindDirection.S: "🡡",
+        WindDirection.S: "⬆️",
-        WindDirection.SW: "🡭",
+        WindDirection.SW: "↗️",
-        WindDirection.W: "🡪",
+        WindDirection.W: "➡️",
-        WindDirection.NW: "🡦",
+        WindDirection.NW: "↘️",
         WindDirection.CALM: "",
     }.get(wind_direction, wind_direction)
 

weather/route/view/static/style.css

@@ -15,6 +15,7 @@ a.button {
   pointer-events: none;
   cursor: default;
   color: gray;
+  filter: grayscale(100%);
 }
 
 table {

weather/route/view/templates/index.html

@@ -0,0 +1,26 @@
+<!DOCTYPE html>
+<html lang="en">
+
+<head>
+    <meta charset="UTF-8">
+    <meta name="viewport"
+          content="width=device-width, initial-scale=1.0">
+    <meta http-equiv="X-UA-Compatible"
+          content="ie=edge">
+    <title>Погода</title>
+    <link rel="stylesheet"
+          href="/static/style.css">
+    <link rel="icon"
+          href="/static/favicon.ico"
+          type="image/x-icon">
+</head>
+
+<body>
+    <ul>
+        {% for location in locations %}
+        <li><a href="weather/{{location.name}}-{{location.location_id}}">{{location.name}}</a></li>
+        {% endfor %}
+    </ul>
+</body>
+
+</html>

weather/route/view/templates/weather.html

@@ -18,10 +18,10 @@
 <body>
     <h3>
         <a class="button {{'disabled' if response.date == datetime.date.today() else ''}}"
-               href="{{response.date - datetime.timedelta(days=1)}}">🡨</a>
+           href="{{response.date - datetime.timedelta(days=1)}}">⬅️</a>
         <span>{{response.location}} | {{response.date.strftime('%a, %d %B %Y')}}</span>
         <a class="button"
-               href="{{response.date + datetime.timedelta(days=1)}}">🡪</a>
+           href="{{response.date + datetime.timedelta(days=1)}}">➡️</a>
     </h3>
     <table>
         <tbody>