Skip to content

Add gated tool tests for CrewAI and LlamaIndex - #14

Merged
inboxpraveen merged 1 commit into
inboxpraveen:mainfrom
tamerbak:test/crewai-llamaindex-gated-tools
Sep 29, 2026
Merged

inboxpraveen merged 1 commit into
inboxpraveen:mainfrom
tamerbak:test/crewai-llamaindex-gated-tools

Conversation

@tamerbak

Copy link
Copy Markdown
Contributor

Summary

CrewAI and LlamaIndex were documented as compatible with gate(), but lacked integration tests.

This PR adds coverage for both frameworks to verify that:

  • Gated tools preserve argument properties and required fields.
  • Allowed calls execute the underlying function.
  • Blocked calls return the configured on_block message without executing the function.

The tests use pytest.importorskip for optional dependencies. Documentation includes usage examples and the tested versions: CrewAI 1.15.22 and llama-index-core 0.14.25.

Validation

  • New integration tests: 2 passed.
  • Integration test file: 6 passed, 7 skipped because unrelated optional frameworks were not installed.
  • Ruff lint and formatting checks passed.
  • Prose check passed.

Closes #11

@inboxpraveen
inboxpraveen merged commit a52f73a into inboxpraveen:main Sep 29, 2026
18 checks 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.

Test gate() under CrewAI and LlamaIndex tool decorators

2 participants