fix: add missing __init__.py to tests/integrations and tests/nodes packages#1000
Conversation
Greptile SummaryThis PR adds empty Confidence Score: 5/5Safe to merge — two empty files with no functional risk. Both files are empty and only affect Python package discovery. The sole finding is a P2 style/consistency note about sibling subdirectories that are still missing No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[tests/] --> B[tests/integrations/]
A --> C[tests/nodes/]
B --> B1["__init__.py ✅ (new)"]
B --> B2[llm_cli/ ⚠️ missing __init__.py]
C --> C1["__init__.py ✅ (new)"]
C --> C2[investigate/ ✅]
C --> C3[plan_actions/ ✅]
C --> C4[resolve_integrations/ ✅]
C --> C5[root_cause_diagnosis/ ✅]
C --> C6[evaluate_opensre/ ⚠️ missing __init__.py]
C --> C7[extract_alert/ ⚠️ missing __init__.py]
C --> C8[publish_findings/ ⚠️ missing __init__.py]
|
|
Verified locally: Pre-PR: 859 tests collected from tests/integrations/ + tests/nodes/ |
VaibhavUpreti
left a comment
There was a problem hiding this comment.
awesome @Ghraven , welcome to the OpenSRE community!
Fixes #998
What
Adds missing
__init__.pyfiles to two test directories that were not proper Python packages:tests/integrations/__init__.pytests/nodes/__init__.pyWhy
All other test directories under
tests/(e.g.tests/tools/,tests/services/,tests/utils/,tests/cli/) already have__init__.py. Without these files:test_config.pyfiles can collide)Verification
Both files are empty, matching the pattern of
tests/tools/__init__.py.