mirror of
https://github.com/pewdiepie-archdaemon/odysseus.git
synced 2026-08-04 18:35:30 +00:00
bb719f217a
Slice 2m of the route-domain reorganization (#4082/#4071, per
specs/architecture-runtime-inventory.md §6.3). Moves document_routes.py
(1810 lines) and document_helpers.py (243 lines) into routes/document/,
leaving backward-compat sys.modules shims at the old paths. Pure file
reorganization, no behavior change.
Both shims use sys.modules replacement so the `import ... as droutes` +
`droutes.SessionLocal = ...` / `monkeypatch.setattr(droutes, ...)` pattern
in multiple tests, and the `sys.modules.pop("routes.document_helpers")` +
re-import pattern in test_security_regressions.py, all reach the canonical
modules.
The canonical document_routes.py imports helpers from the canonical path
(routes.document.document_helpers), not the legacy shim.
Three source-introspection test sites repointed to the new canonical path:
- test_imap_mailbox_quoting.py
- test_model_helper_owner_scope.py
- test_vision_owner_scope.py (shared with other domains; document entry repointed)
Adds tests/test_document_routes_shim.py to pin the sys.modules shim contract
for both modules.
Verified: compileall clean; full suite 4789 passed, 3 skipped.
29 lines
1.2 KiB
Python
29 lines
1.2 KiB
Python
"""Regression test for the document route shim (slice 2m, #4082/#4071).
|
|
|
|
The backward-compat shims at ``routes/document_routes.py`` and
|
|
``routes/document_helpers.py`` use ``sys.modules`` replacement so the legacy
|
|
import paths and the canonical ``routes.document.*`` paths resolve to the
|
|
*same* module objects. This is required because multiple tests do
|
|
``import routes.document_routes as droutes`` followed by
|
|
``droutes.SessionLocal = ...`` / ``monkeypatch.setattr(droutes, ...)`` and
|
|
``sys.modules.pop("routes.document_helpers")`` + re-import — for those to
|
|
take effect at runtime, the legacy and canonical module objects must be
|
|
identical.
|
|
"""
|
|
|
|
import importlib
|
|
|
|
import routes.document_routes as _shim_routes # noqa: F401
|
|
import routes.document_helpers as _shim_helpers # noqa: F401
|
|
|
|
|
|
def test_legacy_and_canonical_routes_are_same_object():
|
|
legacy = importlib.import_module("routes.document_routes")
|
|
canonical = importlib.import_module("routes.document.document_routes")
|
|
assert legacy is canonical
|
|
|
|
|
|
def test_legacy_and_canonical_helpers_are_same_object():
|
|
legacy = importlib.import_module("routes.document_helpers")
|
|
canonical = importlib.import_module("routes.document.document_helpers")
|
|
assert legacy is canonical
|