Why this change is needed:
The test file contained assert statements using tuple syntax `assert (condition, message)`,
which Python interprets as asserting a non-empty tuple (always True). This meant the tests
were passing even when the actual conditions failed, creating a false sense of test coverage.
Additionally, there were unused imports (pytest, patch, MagicMock) that needed cleanup.
How it solves it:
- Fixed assert statements on lines 61-63 and 105-109 to use correct syntax:
`assert condition, message` instead of `assert (condition, message)`
- Removed unused imports to satisfy linter requirements
- Applied automatic formatting via ruff-format and ruff
Impact:
- Tests now correctly validate the empty model suffix behavior
- Prevents false positive test results that could hide bugs
- Passes all pre-commit hooks (F631 error resolved)
Testing:
- Verified with `uv run pre-commit run --all-files` - all checks pass
- Assert statements now properly fail when conditions are not met
This change ensures that when the model_suffix is empty, the final_namespace falls back to the legacy_namespace, preventing potential naming issues. A warning is logged to inform users about the missing model suffix and the fallback to the legacy naming scheme.
Additionally, comprehensive tests have been added to verify the behavior of both PostgreSQL and Qdrant storage when model_suffix is empty, ensuring that the naming conventions are correctly applied and that no trailing underscores are present.
Impact:
- Prevents crashes due to empty model_suffix
- Provides clear feedback to users regarding configuration issues
- Maintains backward compatibility with existing setups
Testing:
All new tests pass, validating the handling of empty model_suffix scenarios.
