test(pmoves-yt): add endpoint tests

[POWERFULMOVES]

Summary

• add FastAPI endpoint tests for pmoves-yt service
• mock external services (yt_dlp, S3, ffmpeg-whisper, Hi-RAG)
• assert Supabase inserts/updates and event emissions

Testing

• pytest pmoves/services/pmoves-yt/tests/test_endpoints.py -q

---

https://chatgpt.com/codex/tasks/task_b_68c613e88aac8324a699a258d8626aa4

Summary by CodeRabbit

• Tests
  • Added comprehensive end-to-end tests for YouTube-related endpoints (info, download, transcript, summarize, emit).
  • Validates HTTP responses and key side effects while isolating external integrations via mocks.
  • Improves reliability, prevents regressions, and documents expected behaviors across common workflows.
  • Enhances confidence in critical paths without performing real network or file operations.

[coderabbitai]

Walkthrough

Adds a new test module for pmoves-yt endpoints using FastAPI TestClient and monkeypatching. It tests /yt/info, /yt/download, /yt/transcript, /yt/summarize, and /yt/emit, validating HTTP responses and mocked side effects for external services (YoutubeDL, S3 upload, Supabase ops, event publishing, and external POSTs).

Changes

Cohort / File(s)	Summary
Endpoint tests for pmoves-yt pmoves/services/pmoves-yt/tests/test_endpoints.py	New test suite covering info, download, transcript, summarize, and emit endpoints; uses TestClient and monkeypatching for YoutubeDL, S3 upload, Supabase insert/update/get, event publishing, and external HTTP calls; asserts HTTP 200 responses and specific side-effect counts/arguments.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

I thump my paw on passing tests,
Hop-hop through endpoints’ subtle quests.
Mocked winds blow, no I/O stress—
S3, Supa, events coalesce.
One carrot raised for green-lit nights,
The burrow hums with tidy bytes.
☁️🥕✅

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Free

📥 Commits

Reviewing files that changed from the base of the PR and between f3607a3 and ddb366e.

📒 Files selected for processing (1)

• pmoves/services/pmoves-yt/tests/test_endpoints.py (1 hunks)

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

• Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
• Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.

---

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}