Skip to content

Add unit test for LLM call_tool method#21

Merged
williamFalcon merged 1 commit into
mainfrom
aniket/add-tests
Jul 30, 2025
Merged

Add unit test for LLM call_tool method#21
williamFalcon merged 1 commit into
mainfrom
aniket/add-tests

Conversation

@aniketmaurya
Copy link
Copy Markdown
Collaborator

Before submitting
  • Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
  • Did you read the contributor guideline, Pull Request section?
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?

What does this PR do?

users will get a tested code as the LLM.call_tool method was not covered in test

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in GitHub issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

- Introduced a new test, `test_llm_call_tool`, to validate the functionality of the LLM's call_tool method using a decorator-based tool.
- The test checks the response from the tool when invoked with a specific message.
@williamFalcon williamFalcon merged commit f861ae8 into main Jul 30, 2025
29 checks passed
@williamFalcon williamFalcon deleted the aniket/add-tests branch July 30, 2025 23:06
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 30, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81%. Comparing base (ed2bb48) to head (7edc731).
⚠️ Report is 90 commits behind head on main.

Additional details and impacted files
@@        Coverage Diff         @@
##           main   #21   +/-   ##
==================================
+ Coverage    73%   81%   +7%     
==================================
  Files         6     6           
  Lines       294   329   +35     
==================================
+ Hits        215   265   +50     
+ Misses       79    64   -15     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

3 participants