Conversation
abhisek
commented
Oct 25, 2025
- test: Add callgraph signature tests
- test: Add Java arg matching tests
SafeDep Report SummaryPackage Details
This report is generated by SafeDep Github App |
There was a problem hiding this comment.
Pull Request Overview
This PR adds comprehensive test coverage for callgraph signature matching across multiple programming languages (Python, Go, and Java). The tests validate that the signature matcher correctly identifies function/method calls and supports advanced features like argument matching by index and type resolution.
Key Changes:
- Added signature matching test cases for Python, Go, and Java to verify detection of common function calls
- Added Java argument matching tests to validate literal value matching, multi-argument matching, and type resolution
- Added test fixture file for Java argument matching scenarios
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| plugin/callgraph/signatures_test.go | Added test cases for Python, Go, and Java signature matching, including argument matching scenarios |
| plugin/callgraph/imports.go | Added blank lines for consistent formatting |
| plugin/callgraph/fixtures/testJavaArguments.java | Added new test fixture with Java code demonstrating various argument matching scenarios |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #45 +/- ##
==========================================
+ Coverage 60.23% 61.31% +1.07%
==========================================
Files 47 47
Lines 5289 4123 -1166
==========================================
- Hits 3186 2528 -658
+ Misses 1825 1316 -509
- Partials 278 279 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|