fix(tests): update correlation patch path after refactor#2758
Conversation
Greptile code reviewThis repo uses Greptile for automated review. Before merge, aim for Confidence Score: 5/5 with zero unresolved review threads — see CONTRIBUTING.md. Run a review — add a PR comment with: Give it ~5-10 minutes (sometimes longer) for results, then fix feedback and re-trigger until you reach Confidence Score: 5/5. Optional: automate with the greploop skill. |
Greptile SummaryThis PR fixes three
Confidence Score: 5/5Safe to merge — the change only updates mock patch targets to reflect an already-landed module move, with no logic changes. All three patch paths now correctly point to app.agent.correlation.node.node_correlate_upstream, where the function is confirmed to exist. The change is mechanical, test-only, and fully consistent across every occurrence in the file. No files require special attention. Important Files Changed
Reviews (1): Last reviewed commit: "fix(tests): update correlation patch pat..." | Re-trigger Greptile |
|
🤖 CI passed. Linter didn't scream. Reviewer typed LGTM. @cerencamkiran, every machine in this pipeline just slow-clapped. 🖥️✨ 👋 Join us on Discord - OpenSRE : hang out, contribute, or hunt for features and issues. Everyone's welcome. |

Fixes failing pipeline tests after the correlation package move from
app.correlationtoapp.agent.correlation.The tests in
tests/pipeline/test_runners_agent_class.pywere still patching the oldnode_correlate_upstreamimport path, which caused:AttributeError: module 'app' has no attribute 'correlation'This PR updates the patch targets to match the new package location introduced by the correlation refactor.
Tested locally:
pytest tests/pipeline/test_runners_agent_class.py -v