bug: ``make test` aborts during collection: orphaned tests for deleted `diy_agent` and `pydanticai_agent`

Start with tests/unit/agent/diy_agent/ and tests/unit/agent/pydanticai_agent/, then run make test on a clean checkout. Remove the two test directories for the deleted agent packages and confirm pytest completes collection; the four unrelated test failures noted in the issue are separate work.

Summary

make test cannot run on a clean checkout of main. Four test modules import agent packages that were deleted from src/, so pytest aborts during collection and the remaining 149 tests never execute.

Version

• Commit: e41eca1 (main, 2026-06-17)
• Python 3.12.8, pytest 8.3.5, macOS arm64
• No local modifications

Reproduction

git clone https://github.com/aws-samples/sample-agentic-platform.git
cd sample-agentic-platform
make install
make test

Actual result

collected 163 items / 4 errors

ERROR tests/unit/agent/diy_agent/test_diy_agent.py
ERROR tests/unit/agent/diy_agent/test_diy_agent_controller.py
ERROR tests/unit/agent/pydanticai_agent/test_pyai_agent.py
ERROR tests/unit/agent/pydanticai_agent/test_pyai_agent_controller.py

E ModuleNotFoundError: No module named 'agentic_platform.agent.diy_agent'
E ModuleNotFoundError: No module named 'agentic_platform.agent.pydanticai_agent'

!!!! Interrupted: 4 errors during collection !!!!
make: *** [test] Error 2

Cause

Commit 75ba436 (Feat: Built an agentic chat agent that can be used as the interface to our frontend (separate push) deleted these source packages:

• src/agentic_platform/agent/diy_agent/
• src/agentic_platform/agent/pydanticai_agent/

Their test directories were left in place:

• tests/unit/agent/diy_agent/
• tests/unit/agent/pydanticai_agent/

src/agentic_platform/agent/ now contains only agentic_chat, agentic_rag, coding_agent, jira_agent, langgraph_chat, and strands_glue_athena. The orphaned tests reference implementations that no longer exist and cannot pass in any configuration.

Because these are collection errors rather than test failures, pytest exits before running anything, so the failure is not contained to the affected modules.

Suggested fix

Delete both orphaned test directories. Excluding them locally yields 4 failed, 145 passed; the 4 remaining failures are a separate concern, filed as its own issue.

Impact

Contributors have no green baseline to validate changes against, and make test is listed in AGENTS.md as the expected check after code changes.