Skip to content

Cover TapeAgent.get_action step logic in tests - #22

Open
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1789677945-tapeagent-get-action-tests
Open

devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1789677945-tapeagent-get-action-tests

Conversation

@devin-ai-integration

Copy link
Copy Markdown

Summary

Closes a code-scan coverage gap: TapeAgent.get_action (the tape step loop) had no test — only agent construction and GAIA benchmark wiring were covered.

Adds two tests to tests/agents/test_gaia_agent.py, behind the existing module-level ModuleNotFoundError skip. They drive get_action with a stub tape agent instead of an LLM:

class MockTapeAgent:          # stands in for tapeagents.agent.Agent
    def run(self, tape):
        yield MockEvent()     # step-less event -> skipped
        for step in self.steps:
            yield MockEvent(step)

Asserted behavior:

  • returns the action plus a TapeAgentInfo whose thoughts hold the emitted Thought steps;
  • the tape accumulates observation, thought and action steps across calls;
  • when a run emits two actions, the first one is returned while both still land on the tape;
  • final_tape marks the tape truncated when no StopStep was produced.

tapeagents is commented out in pyproject.toml, so these tests skip locally and in CI just like the existing ones. The step accounting (6 tape steps after two calls) was verified by replaying the exact get_action loop against stub Tape/Thought/Action types.

Devin-Org: engineering

Link to Devin session: https://app.devin.ai/sessions/0a384ae103d94008b71e41a40c6d43de
Open in Devin Desktop: https://app.devin.ai/desktop/session/0a384ae103d94008b71e41a40c6d43de?variant=devin

@devin-ai-integration

Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

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.

0 participants