Skip to content

Test list videos responsiveness during in-flight scan - #1756

Draft
groupthinking with Copilot wants to merge 2 commits into
mainfrom
copilot/test-fix-responsiveness-test
Draft

Test list videos responsiveness during in-flight scan#1756
groupthinking with Copilot wants to merge 2 commits into
mainfrom
copilot/test-fix-responsiveness-test

Conversation

Copilot AI commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Canonical issue

Issue link handled separately.

Outcome

The list-videos responsiveness test now proves loop progress happens while the blocking scan is still running, not merely after a bounded timeout expires.

Scope

  • Included:
    • Timestamp scan completion with time.monotonic().
    • Timestamp loop ticks before releasing the mocked scan.
    • Assert qualifying ticks occurred before scan completion.
    • Failure message names inline event-loop execution.
  • Explicitly excluded:
    • No /api/v1/videos behavior changes.
    • No src/ changes.
concurrent = [t for t in tick_times if t < finished_at["scan"]]
assert len(concurrent) >= 3

Risk

  • Risk level: low
  • Failure mode: test timing sensitivity on unusually constrained runners
  • Rollback: revert the test-only change in tests/unit/test_v1_router_extended.py

Verification

Current head: be11b94

  • Focused tests: python -m pytest tests/unit/test_v1_router_extended.py::TestListVideosOffloading --no-cov -q
  • Required CI
  • Review threads resolved: no validation comments

Production evidence

Not applicable; test-only change.

Agent handoff

  • One canonical issue is linked
  • No competing PR implements the same issue
  • Acceptance criteria are satisfied
  • Required checks pass on the current head
  • Human decision is requested only for product, security, irreversible infrastructure, or production approval

@vercel

vercel Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
v0-uvai Ready Ready Preview, v0 Sep 8, 2026 11:39pm UTC

Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix TestListVideosOffloading responsiveness test Test list videos responsiveness during in-flight scan Sep 8, 2026
Copilot AI requested a review from groupthinking September 8, 2026 23:38
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.

test: TestListVideosOffloading responsiveness test passes against an inline implementation

2 participants