Skip to content

test: fix two broken main tests - #2982

Merged
chaynabors merged 1 commit into
strands-agents:mainfrom
chaynabors:fix/bedrock-3gp-test-method-typo
Jun 26, 2026
Merged

chaynabors merged 1 commit into
strands-agents:mainfrom
chaynabors:fix/bedrock-3gp-test-method-typo

Conversation

@chaynabors

@chaynabors chaynabors commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Fix tests on main.

Both surfaced on the Python 3.14 linux job after a sequence of merges today:

1. test_format_request_maps_3gp_video_formats (from strands-agents#2306) called a non-existent
   _format_request; the public method is format_request.

2. test_ingests_extracted_facts_through_add (from strands-agents#2859) called the async
   MemoryManager.init_agent without awaiting it, so the extractor was never
   wired up and assert_called_once failed.
@github-actions github-actions Bot added size/xs area-community Related to community and contributor health python Pull requests that update python code chore Maintenance tasks, dependency updates, CI changes, refactoring with no user-facing impact labels Jun 26, 2026
@chaynabors
chaynabors enabled auto-merge (squash) June 26, 2026 15:19
@chaynabors
chaynabors merged commit b1c6008 into strands-agents:main Jun 26, 2026
29 of 30 checks passed
@chaynabors
chaynabors deleted the fix/bedrock-3gp-test-method-typo branch June 26, 2026 15:21
@codecov

codecov Bot commented Jun 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-community Related to community and contributor health chore Maintenance tasks, dependency updates, CI changes, refactoring with no user-facing impact python Pull requests that update python code size/xs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants