Skip to content

Add integration tests for tools#834

Draft
codegen-sh[bot] wants to merge 2 commits intodevelopfrom
gen/6fe69672-f2cc-4c5a-b814-bbf096096662
Draft

Add integration tests for tools#834
codegen-sh[bot] wants to merge 2 commits intodevelopfrom
gen/6fe69672-f2cc-4c5a-b814-bbf096096662

Conversation

@codegen-sh
Copy link
Contributor

@codegen-sh codegen-sh bot commented Mar 13, 2025

This PR adds integration tests for various tools in the codebase, addressing the requirements in CG-11038.

Changes

  • Added test_workspace_tools.py with tests for:

    • view_file
    • list_directory
    • create_file
    • edit_file
    • delete_file
    • rename_file
    • search
    • replacement_edit
  • Added test_slack.py with tests for:

    • add_links_to_message
    • send_slack_message
  • Added test_reflection.py with tests for:

    • perform_reflection
  • Added test_relace_edit.py with tests for:

    • relace_edit with real API (skipped if API key not available)
    • relace_edit with mocked API
    • relace_edit error handling

Testing

The tests can be run with pytest:

pytest tests/integration/extension/test_workspace_tools.py
pytest tests/integration/extension/test_slack.py
pytest tests/integration/extension/test_reflection.py
pytest tests/integration/extension/test_relace_edit.py

Some tests require environment variables to be set:

  • test_relace_edit.py requires RELACE_API to be set for the real API test (otherwise it's skipped)

These tests complement the existing integration tests for Linear, GitHub, and Bash tools.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


codegen-bot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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