Skip to content

test: add unit tests for ActiveStreamingTool#461

Closed
nuthalapativarun wants to merge 1 commit into
google:mainfrom
nuthalapativarun:test/active-streaming-tool
Closed

test: add unit tests for ActiveStreamingTool#461
nuthalapativarun wants to merge 1 commit into
google:mainfrom
nuthalapativarun:test/active-streaming-tool

Conversation

@nuthalapativarun

Copy link
Copy Markdown
Contributor

Problem

ActiveStreamingTool in core/src/agents/active_streaming_tool.ts has no unit test coverage.

Solution

Add unit tests covering constructor defaults, constructor with params, and field access.

Testing

  • All new tests pass: npm test
  • No existing tests broken

Checklist

  • I have performed a self-review of my code
  • Tests added and passing
  • Manual E2E verified (N/A — test-only change)

@nuthalapativarun

Copy link
Copy Markdown
Contributor Author

Superseded by #464 which combines all three test files into a single PR.

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