Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@tnm
Copy link
Contributor

@tnm tnm commented Jul 26, 2025

  • Fix Tool object access (use attributes instead of dictionary access)
  • Fix mock signatures for path_mapping_consistency test
  • Fix get_code_summary tests by mocking analyzer instead of non-existent Repository method
  • Fix get_prompt method signatures to use correct parameter order
  • Fix test_get_code_summary_invalid_type to test path validation
  • Fix test_review_help to be more robust with formatting
  • Add explicit main branch creation in test fixtures to prevent 'ref not found' errors

- Fix Tool object access (use attributes instead of dictionary access)
- Fix mock signatures for path_mapping_consistency test
- Fix get_code_summary tests by mocking analyzer instead of non-existent Repository method
- Fix get_prompt method signatures to use correct parameter order
- Fix test_get_code_summary_invalid_type to test path validation
- Fix test_review_help to be more robust with formatting
- Add explicit main branch creation in test fixtures to prevent 'ref not found' errors
@tnm tnm marked this pull request as ready for review July 26, 2025 01:27
@tnm tnm merged commit e34f70d into main Jul 26, 2025
1 check passed
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.

1 participant