Skip to content

Comments

test: add unit and integration tests for annotatedast#612

Open
Sneha41sb wants to merge 1 commit intostackql:mainfrom
Sneha41sb:test/annotatedast-coverage-85
Open

test: add unit and integration tests for annotatedast#612
Sneha41sb wants to merge 1 commit intostackql:mainfrom
Sneha41sb:test/annotatedast-coverage-85

Conversation

@Sneha41sb
Copy link

Summary

This PR improves test coverage for the annotatedast package by adding unit and integration tests.

Details

  • Added new tests without modifying production code
  • Improved coverage for reachable logic paths
  • Avoided mocking non-exported internal types

Coverage Notes

Some functions depend on planner-injected AST state or internal types and cannot be safely unit-tested. Coverage has been improved as far as possible without violating package boundaries.

@Sneha41sb
Copy link
Author

Hi maintainers
This PR adds unit and integration tests to improve coverage for annotatedast.
Some branches depend on planner-injected AST state and cannot be unit-tested safely, so coverage was improved as far as possible without modifying production code.

Happy to adjust if you’d like changes — thanks!

@general-kroll-4-life
Copy link
Contributor

@Sneha41sb thanks for the contribution. Please remove all files other that the *_test.go files

@general-kroll-4-life
Copy link
Contributor

@Sneha41sb can you please fix the issues picked up by the linter

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants