Flight Log / dhar174/tiny_village #446

Fix over-mocking antipatterns in memory tests - replace MagicMock with real test objects — a GitHub Copilot (GitHub) bot flight

The GitHub Copilot bot (GitHub) opened this pull request in dhar174/tiny_village on Jul 20, 2025; merged in 55 min.

All agent flights in dhar174/tiny_village · GitHub Copilot flights

Not yet classifiedGitHub Copilot flightsby GitHub ↗ attribution: AIDev dataset

Flown by @CopilotGitHub ↗— unclaimed

Lines
—
Files
—
Commits
—
Reviews
—
To merge
55 min

From the pull request description

## Problem The memory tests in `tests/test_tiny_memories.py` were using MagicMock with predefined attributes, which provided false confidence in test validation. These tests always passed regardless of whether the memory processing logic was working correctly. ```python # PROBLEMATIC PATTERN (befo…

Excerpt (first 300 characters) — the full description is on GitHub.

This is your flight? Claim it — sign in with GitHub. Ownership is proven by the sign-in: the account that opened the PR is the only one that can claim it.

Cite this flight: AgentRiders Flight Log, dhar174/tiny_village#446, record updated Nov 3, 2026, https://agentriders.com/logs/dhar174/tiny_village/446/

Data from public GitHub activity; this page is not affiliated with GitHub or the repository.