Skip to content
This repository was archived by the owner on May 3, 2026. It is now read-only.

Refactor emergency controllers to share SQLAlchemy mixin#203

Merged
brothercorvo merged 1 commit intomainfrom
corvo/create-mixin-for-async-crud-operations
Nov 11, 2025
Merged

Refactor emergency controllers to share SQLAlchemy mixin#203
brothercorvo merged 1 commit intomainfrom
corvo/create-mixin-for-async-crud-operations

Conversation

@brothercorvo
Copy link
Copy Markdown
Contributor

Summary

  • add a reusable SQLAlchemyControllerMixin with session factory resolution, identifier coercion, and async CRUD helpers
  • refactor the emergency example controllers to inherit from the mixin for concise method bodies
  • add dedicated mixin unit tests, update example controller tests, and record the completed work in TASK.md

Testing

  • ./venv_linux/bin/flake8 reticulum_openapi/sqlalchemy_controller.py examples/EmergencyManagement/Server/controllers_emergency.py tests/test_sqlalchemy_controller.py
  • ./venv_linux/bin/pytest tests/test_sqlalchemy_controller.py tests/test_example_emergency_management.py::test_event_controller_list_without_session_factory tests/test_example_emergency_management.py::test_emergency_action_message_crud tests/test_example_emergency_management.py::test_event_controller_crud

Codex Task

@brothercorvo brothercorvo merged commit cbb6124 into main Nov 11, 2025
0 of 2 checks passed
@brothercorvo brothercorvo deleted the corvo/create-mixin-for-async-crud-operations branch November 11, 2025 17:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant