Skip to content

chore-agentcore-backend-error-path-tests-2026-06-10: AgentCoreBackend error-path tests#71

Merged
emp3thy merged 1 commit into
mainfrom
ralph/chore-agentcore-backend-error-path-tests-2026-06-10
Jun 11, 2026
Merged

chore-agentcore-backend-error-path-tests-2026-06-10: AgentCoreBackend error-path tests#71
emp3thy merged 1 commit into
mainfrom
ralph/chore-agentcore-backend-error-path-tests-2026-06-10

Conversation

@emp3thy

@emp3thy emp3thy commented Jun 10, 2026

Copy link
Copy Markdown
Owner

Adds the missing error-path unit tests for AgentCoreBackend flagged by the tech-debt scan. tests/storage/test_agentcore_unit.py previously covered only happy paths of the boto3-backed backend; this adds 8 tests covering ClientError and transport-timeout propagation from observe(), single-polarity / all-polarity / worker-timeout failure propagation in retrieve(), and the malformed-JSON, missing-content, and non-dict-JSON graceful-degradation shape in _parse_reflection_record. No production code changed.

Closes chore-agentcore-backend-error-path-tests-2026-06-10.

Test plan:

  • uv run python -m pytest tests/storage/test_agentcore_unit.py -> 60 passed (52 pre-existing + 8 new)
  • Full suite: 25 unrelated failures caused by the ralph executor's BETTER_MEMORY_PROJECT env var overriding project-name detection; verified those files pass with the var unset (88 passed)

PBI: ralph-queue .ralph/current/chore-agentcore-backend-error-path-tests-2026-06-10

…add AgentCoreBackend error-path tests

Cover the gaps flagged by the tech-debt finding: ClientError and
transport-timeout propagation from observe(), single/total polarity
fetch failures and worker timeout in retrieve(), and the malformed-JSON
/ missing-content / non-dict-body graceful-degradation shape in
_parse_reflection_record.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Claude BugBot Analysis

No bugs found in the changes. All new tests are logically correct: assertions match the implementation, async tests are handled by the project-wide asyncio_mode=auto setting, mock side-effect stubs correctly guard against false filter matches, and exception propagation paths through run_in_executor and ThreadPoolExecutor.Future.result() are sound.

No bugs were detected in this PR.

@emp3thy
emp3thy merged commit d00489a into main Jun 11, 2026
2 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.

1 participant