[P] Add database update after marker deletion.

.gitea/workflows/reset-docker.yaml

@@ -7,13 +7,10 @@ jobs:
     name: Reset docker state
     steps:
       - name: Stop all containers
-        if: ${{always()}}
         run: docker stop $(docker ps -a | cut -d " " -f 1 | tail -n +2)
       
       - name: Remove all containers
-        if: ${{always()}}
         run: docker rm $(docker ps -a | cut -d " " -f 1 | tail -n +2)
 
       - name: Remove extra volumes
-        if: ${{always()}}
         run: docker volume rm road_vision_postgres_data road_vision_pgadmin-data

.gitea/workflows/tests.yaml

@@ -28,49 +28,24 @@ jobs:
       - name: Clone repository
         run: git clone --revision ${{ gitea.sha }} --depth 1 ${{ gitea.server_url }}/${{ gitea.repository }}
 
-      - name: Start postgres_db container for testing
-        working-directory: IoT-Systems
-        run: docker-compose up -d postgres_db
-
       - name: Build Store testing container
         working-directory: IoT-Systems
         run: docker build -t local/store/${{gitea.sha}} -f store/Dockerfile-test .
 
       - name: Run Store tests
         working-directory: IoT-Systems
-        run: docker run --network host --rm -it local/store/${{gitea.sha}}
+        run: docker run --rm -it local/store/${{gitea.sha}}
 
       - name: Clean up containers
         if: ${{always()}}
         run: docker image rm local/store/${{gitea.sha}}
 
-      - name: Clean up docker-compose
-        if: ${{always()}}
-        working-directory: IoT-Systems
-        run: docker-compose down -v --remove-orphans
-
-  mapview-test:
-    name: MapView testing
-    runs-on: host-arch-x86_64
-    steps:
-      - name: Clone repository
-        run: git clone --revision ${{ gitea.sha }} --depth 1 ${{ gitea.server_url }}/${{ gitea.repository }}
-
-      - name: Build MapView testing container
-        working-directory: IoT-Systems
-        run: docker build -t local/mapview -f MapView/Dockerfile-test .
-
-      - name: Run MapView tests
-        working-directory: IoT-Systems
-        run: docker run --rm -it -v $PWD/MapView:/app:ro local/mapview
-
   integration-smoke-test:
     name: Integration smoke testing
     runs-on: host-arch-x86_64
     needs:
       - hub-test
       - store-test
-      - mapview-test
     steps:
       - name: Clone repository
         run: git clone --revision ${{ gitea.sha }} --depth 1 ${{ gitea.server_url }}/${{ gitea.repository }}

MapView/Dockerfile-test

@@ -1,14 +0,0 @@
-# Use the official Python image as the base image
-FROM python:latest
-# Set the working directory inside the container
-WORKDIR /app
-# Copy the requirements.txt file and install dependencies
-COPY MapView/requirements.txt .
-RUN sed -i 's/==.*//' requirements.txt
-
-RUN apt update && apt install -y libgl-dev libsdl1.2-dev
-
-RUN pip install --no-cache-dir -r requirements.txt
-RUN pip install --root-user-action ignore --no-cache-dir python-unittest kivy[base] pygame
-
-CMD ["./test-entry.sh"]

MapView/datasource.py

@@ -4,12 +4,13 @@ from datetime import datetime
 import websockets
 from kivy import Logger
 from pydantic import BaseModel, field_validator
+from websockets.asyncio.connection import Connection
+
 from config import STORE_HOST, STORE_PORT
 
 
 # Pydantic models
 class ProcessedAgentData(BaseModel):
-    id: int
     road_state: str
     user_id: int
     x: float
@@ -33,17 +34,14 @@ class ProcessedAgentData(BaseModel):
 
 
 class Datasource:
-    def __init__(self):
+    def __init__(self, user_id: int):
         self.websocket: Connection | None = None
         self.index = 0
+        self.user_id = user_id
         self.connection_status = None
         self._new_points = []
         self._active_markers = []
-
-        try:
-            asyncio.ensure_future(self.connect_to_server())
-        except RuntimeError:
-            Logger.info("No event loop detected, running in offline mode")
+        asyncio.ensure_future(self.connect_to_server())
 
     def get_new_points(self):
         Logger.debug(self._new_points)
@@ -52,12 +50,12 @@ class Datasource:
         return points
 
     async def connect_to_server(self):
-        uri = f"ws://{STORE_HOST}:{STORE_PORT}/ws"
+        uri = f"ws://{STORE_HOST}:{STORE_PORT}/ws/{self.user_id}"
         while True:
             Logger.debug("CONNECT TO SERVER")
             async with websockets.connect(uri) as websocket:
                 self.connection_status = "Connected"
-                self.websocket = websocket
+                self.websocket = websocket 
                 try:
                     while True:
                         data = await websocket.recv()
@@ -67,10 +65,10 @@ class Datasource:
                     self.connection_status = "Disconnected"
                     Logger.debug("SERVER DISCONNECT")
 
-    def update_db_record_visibility(self, record_id):
+    async def update_db_records(self):
         if self.websocket:
-            data = json.dumps({"id": record_id})
-            asyncio.ensure_future(self.websocket.send(data))
+            data = json.dumps({"id": self.user_id})
+            await self.websocket.send(data, True)
 
     def map_lat_lon_to_processed_agent_data(self, lat: float, lon: float) -> ProcessedAgentData | None:
         distances = tuple((abs(lon - marker.latitude) ** 2 + abs(lat - marker.longitude) ** 2) ** 0.5 for marker in

MapView/main.py

@@ -33,7 +33,7 @@ class MapViewApp(App):
         super().__init__(**kwargs)
 
         self.mapview: MapView | None = None
-        self.datasource = Datasource()
+        self.datasource = Datasource(user_id=1)
         self.line_layers = dict()
         self.car_markers = dict()
 
@@ -162,7 +162,7 @@ class MapViewApp(App):
         elif clicked_marker in self.bump_markers:
             self.bump_markers.pop(self.bump_markers.index(clicked_marker))
 
-        self.datasource.update_db_record_visibility(clicked_marker_data.id)
+        self.datasource.update_db_records()
 
     def on_touch_down(self, widget, touch):
         if touch.button == "right":

MapView/test-entry.sh

@@ -1,3 +0,0 @@
-#!/bin/sh
-
-PYTHONPATH=$PWD python3 -m unittest tests.test_mp

MapView/tests/test_mp.py

@@ -1,137 +0,0 @@
-import unittest
-import sys
-from unittest.mock import Mock
-
-sys.modules['lineMapLayer'] = Mock()
-sys.modules['config'] = Mock()
-
-from main import get_lat_lon, MapViewApp
-from datasource import Datasource as DataSourceClass
-
-
-# -----------------------
-# get_lat_lon
-# -----------------------
-class TestGetLatLon(unittest.TestCase):
-
-    def test_dict(self):
-        point = {"lat": 50.45, "lon": 30.52}
-        self.assertEqual(get_lat_lon(point), (50.45, 30.52))
-
-    def test_tuple(self):
-        point = (50.45, 30.52)
-        self.assertEqual(get_lat_lon(point), (50.45, 30.52))
-
-    def test_list(self):
-        point = [50.45, 30.52]
-        self.assertEqual(get_lat_lon(point), (50.45, 30.52))
-
-    def test_invalid(self):
-        point = {"lat": None, "lon": 30.52}
-        self.assertIsNone(get_lat_lon(point))
-
-
-# -----------------------
-# map_lat_lon_to_marker
-# -----------------------
-class TestMapLatLonToMarker(unittest.TestCase):
-
-    def setUp(self):
-        self.instance = MapViewApp()
-
-    def test_marker_found(self):
-        marker = Mock()
-        marker.lat = 30.52
-        marker.lon = 50.45
-
-        self.instance.pothole_markers = [marker]
-        self.instance.bump_markers = []
-
-        result = self.instance.map_lat_lon_to_marker(50.45, 30.52)
-
-        self.assertEqual(result, marker)
-
-    def test_marker_not_found(self):
-        self.instance.pothole_markers = []
-        self.instance.bump_markers = []
-
-        result = self.instance.map_lat_lon_to_marker(50.45, 30.52)
-
-        self.assertIsNone(result)
-
-
-# -----------------------
-# map_lat_lon_to_processed_agent_data
-# -----------------------
-class TestMapLatLonToProcessedAgentData(unittest.TestCase):
-
-    def setUp(self):
-        self.instance = DataSourceClass()
-
-    def test_returns_marker(self):
-        marker = Mock()
-        marker.latitude = 30.52
-        marker.longitude = 50.45
-
-        self.instance._active_markers = [marker]
-
-        result = self.instance.map_lat_lon_to_processed_agent_data(50.45, 30.52)
-
-        self.assertIsNotNone(result)
-        self.assertEqual(result, marker)
-
-    def test_marker_is_close_enough(self):
-        marker = Mock()
-        marker.latitude = 30.521
-        marker.longitude = 50.452
-
-        self.instance._active_markers = [marker]
-
-        result = self.instance.map_lat_lon_to_processed_agent_data(50.45, 30.52)
-
-        self.assertIsNotNone(result)
-        self.assertEqual(result, marker)
-
-    def test_marker_is_too_far(self):
-        marker = Mock()
-        marker.latitude = 30.524
-        marker.longitude = 50.454
-
-        self.instance._active_markers = [marker]
-
-        result = self.instance.map_lat_lon_to_processed_agent_data(50.45, 30.52)
-
-        self.assertIsNone(result)
-
-    def test_return_closer_marker(self):
-        marker1 = Mock()
-        marker1.latitude = 30.521
-        marker1.longitude = 50.451
-
-        marker2 = Mock()
-        marker2.latitude = 30.524
-        marker2.longitude = 50.454
-
-        self.instance._active_markers = [marker1, marker2]
-
-        result = self.instance.map_lat_lon_to_processed_agent_data(50.452, 30.522)
-        self.assertIsNotNone(result)
-        self.assertEqual(result, marker1)
-
-        result = self.instance.map_lat_lon_to_processed_agent_data(50.453, 30.523)
-        self.assertIsNotNone(result)
-        self.assertEqual(result, marker2)
-
-        result = self.instance.map_lat_lon_to_processed_agent_data(50.459, 30.529)
-        self.assertIsNone(result)
-
-    def test_empty_markers(self):
-        self.instance._active_markers = []
-
-        result = self.instance.map_lat_lon_to_processed_agent_data(50.45, 30.52)
-
-        self.assertIsNone(result)
-
-
-if __name__ == "__main__":
-    unittest.main()

diagrams/architecture-diagram.puml

@@ -1,81 +0,0 @@
-@startuml
-interface MQTT as mqtt1
-interface MQTT as mqtt2
-interface HTTP as http1
-interface HTTP as http2
-interface WebSocket
-
-node Agent {
-  component Accelerometer
-  component GPS
-  component CarParkingSlots
-  component "Data Aggregator" as agent_agg
-}
-
-node Edge {
-  component AgentMQTTAdapter
-  component AgentDataProcessor
-  component HubMQTTAdapter
-  component HubHTTPAdapter
-}
-
-node Hub {
-  component EdgeHTTPAdapter
-  component EdgeMQTTAdapter
-  component Aggregator
-  component StoreHTTPAdapter
-  component Redis
-}
-
-node Store {
-  component "HTTP Rest API" as store_http
-  component "SQL Adapter" as store_sql
-  component "Websocket API" as store_ws
-}
-
-node MapView {
-  component "Store Processor" as mv_sp
-  component "Input Handler" as mv_ih
-  component "Map"
-}
-
-component PostgreSQL
-
-agent_agg --> Accelerometer
-agent_agg --> GPS
-agent_agg --> CarParkingSlots
-
-agent_agg -> mqtt1
-
-mqtt1 <- AgentMQTTAdapter
-AgentMQTTAdapter -> AgentDataProcessor
-AgentDataProcessor --> HubMQTTAdapter
-AgentDataProcessor --> HubHTTPAdapter
-HubMQTTAdapter --> mqtt2
-HubHTTPAdapter --> http1
-
-mqtt2 <-- EdgeMQTTAdapter
-http1 <-- EdgeHTTPAdapter
-EdgeMQTTAdapter --> Aggregator
-EdgeHTTPAdapter --> Aggregator
-Aggregator <--> Redis
-StoreHTTPAdapter <-- Aggregator
-
-http2 <-- StoreHTTPAdapter
-store_http --> http2
-store_http -> store_sql
-store_sql <- store_ws
-store_ws --> WebSocket
-
-store_sql <--> PostgreSQL
-
-WebSocket <-- mv_sp
-mv_sp <-- mv_ih
-mv_sp --> Map
-
-actor Spectator as user
-
-Map --> user
-mv_ih <-- user
-
-@enduml

diagrams/general-sequence.puml

@@ -1,22 +0,0 @@
-@startuml
-actor Worker as worker
-participant Agent as agent
-participant "Edge" as edge
-participant "Hub" as hub
-participant "Store" as store
-participant "MapView" as mapview
-actor Spectator as spectator
-
-worker -> agent : Start agent
-agent -> edge : Send sensor data
-edge -> edge : Classify road state
-edge -> hub : Send processed data
-hub -> hub : Save processed data
-hub -> store : Send processed data batch
-store -> store : Save processed data batch
-store -> mapview : Send new rows
-mapview -> spectator : Show updated paths and markers
-spectator -> mapview : Click on markers
-mapview -> spectator : Hide clicked markers
-mapview -> store : Save display preference
-@enduml

diagrams/main-activity-diagram.puml

@@ -1,15 +0,0 @@
-@startuml
-|Користувач|
-start
-:Натискає на іконку ями;
-|Система|
-:Запитує підтвердження видалення ями;
-|Користувач|
-:Підтверджує видалення ями;
-|Система|
-:Модифікує запис про яму;
-note right #ff9999: Можлива виключна ситуація 1
-:Видаляє яму;
-|Користувач|
-stop
-@enduml

store/Dockerfile-test

@@ -6,7 +6,6 @@ WORKDIR /app
 COPY store/requirements.txt .
 
 RUN pip install --no-cache-dir -r requirements.txt
-RUN pip install --root-user-action ignore --no-cache-dir pytest httpx
 # Copy the entire application into the container
 COPY store/. .
 # Run the main.py script inside the container when it starts

store/main.py

@@ -39,18 +39,15 @@ subscriptions: Set[WebSocket] = set()
 
 
 # FastAPI WebSocket endpoint
-@app.websocket("/ws")
-async def websocket_endpoint(websocket: WebSocket):
+@app.websocket("/ws/{user_id}")
+async def websocket_endpoint(websocket: WebSocket, user_id: int):
     await websocket.accept()
 
     subscriptions.add(websocket)
 
     try:
         # send already available data
-        r = processed_agent_data.select() \
-                                .where(processed_agent_data.c.visible == True) \
-                                .order_by(processed_agent_data.c.timestamp)
-
+        r = processed_agent_data.select().order_by(processed_agent_data.c.timestamp)
         session = SessionLocal()
         stored_data = session.execute(r).fetchall()
         session.close() 

store/test-entry.sh

@@ -1,4 +1,3 @@
 #!/bin/sh
 
 PYTHONPATH=$PWD python3 test/main_test.py
-PYTHONPATH=$PWD python3 -m pytest test_db.py