test(tools): move log_compaction_test.py into tests/ tree#964
Conversation
Greptile SummaryThis PR is a pure rename — Confidence Score: 5/5Safe to merge — pure file rename with no logic changes. No code changes whatsoever; the move targets the correct directory, follows naming conventions (test_*.py), and the destination already has init.py and conftest.py in place for pytest discovery. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["app/tools/utils/log_compaction_test.py\n(deleted)"] -->|"git rename\n(100% similarity)"| B["tests/tools/test_log_compaction.py\n(new location)"]
B --> C["pytest auto-discovery\nvia tests/tools/__init__.py"]
B --> D["Imports unchanged:\napp.tools.utils.log_compaction"]
Reviews (1): Last reviewed commit: "test(tools): move log_compaction_test.py..." | Re-trigger Greptile |
VaibhavUpreti
left a comment
There was a problem hiding this comment.
awesome, thanks a lot for taking this up @Lozsku ! Welcome to the OpenSRE community
Summary
app/tools/utils/log_compaction_test.pyintotests/tools/test_log_compaction.py, following the project convention that all tests live undertests/(AGENTS.md: "If adding new tests → always place them intests/, never inapp/").Test plan
pytest tests/tools/test_log_compaction.pypasses (all cases green locally)app/tools/utils/log_compaction_test.pyno longer existsCloses #904