ci: Add integration test timeouts and download progress monitoring

[tablackburn]

Summary

• Add timeout-minutes: 90 to the integration tests job and timeout-minutes: 75 to the "Run Integration Tests" step to prevent indefinite hangs when Plex server downloads stall
• Add Start-SyncProgressMonitor / Stop-SyncProgressMonitor helper functions that poll the download destination every 30 seconds and emit [SYNC PROGRESS] lines to CI logs with elapsed time, file count, and total MB
• Wrap the Sync-PatMedia download and idempotency tests with the progress monitor for CI visibility

Test Plan

• CI logs show [SYNC PROGRESS] lines during the media download test
• CI logs show [SYNC COMPLETE] line with total time after download finishes
• If the download hangs, the step times out at 75 minutes instead of 6 hours
• All other integration tests remain unaffected

🤖 Generated with Claude Code

Summary by CodeRabbit

• New Features

  • Added a background sync progress monitor with configurable polling intervals to report ongoing download progress and completion.

• Tests

  • Updated integration tests to start/stop the progress monitor around media sync runs, ensuring progress reporting and proper cleanup even on failure.

• Chores

  • Added timeout limits to integration test workflows to improve test run reliability.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: fc38e56f-47b4-4aab-afda-1c3e33b92d44

📥 Commits

Reviewing files that changed from the base of the PR and between e9a1d72 and c492616.

📒 Files selected for processing (1)

• .github/workflows/CI.yaml

🚧 Files skipped from review as they are similar to previous changes (1)

• .github/workflows/CI.yaml

---

📝 Walkthrough

Walkthrough

Adds job- and step-level timeout settings to the integration-tests CI job and introduces Start-SyncProgressMonitor / Stop-SyncProgressMonitor functions used by integration tests to log periodic file-sync progress and final completion time.

Changes

Cohort / File(s)	Summary
Workflow timeout constraints ​.github/workflows/CI.yaml	Added timeout-minutes: 90 at the integration-tests job level and timeout-minutes: 75 to the Run Integration Tests step (when steps.check-secrets.outputs.secrets_configured == 'true').
Sync progress monitoring utilities tests/Integration/IntegrationTestHelpers.psm1	Added exported Start-SyncProgressMonitor and Stop-SyncProgressMonitor. They create/manage a System.Timers.Timer, poll a destination path at a configurable interval, log [SYNC PROGRESS] with aggregated bytes/MB, and emit [SYNC COMPLETE] on stop.
Integration test instrumentation tests/Integration/Public/MediaSync.Integration.tests.ps1	Wrapped Sync-PatMedia invocations in try/finally blocks that start and stop the sync progress monitor; one test passes -IntervalSeconds 10 for more frequent polling.

Sequence Diagram

sequenceDiagram
    actor TestRunner as Test Runner
    participant Monitor as SyncProgressMonitor
    participant Timer as System.Timers.Timer
    participant Sync as Sync-PatMedia
    participant FS as File System

    TestRunner->>Monitor: Start-SyncProgressMonitor(path, interval)
    Monitor->>Timer: Create & Start (interval)
    Note right of Timer: Timer ticks every interval

    TestRunner->>Sync: Invoke Sync-PatMedia(...)
    Sync->>FS: Download/Create files
    activate Sync

    loop every interval
        Timer->>FS: Scan destination path
        FS-->>Timer: Return files & sizes
        Timer->>Monitor: Log "[SYNC PROGRESS]" with totals
    end

    Sync->>FS: Finish downloads
    deactivate Sync

    TestRunner->>Monitor: Stop-SyncProgressMonitor(monitor)
    Monitor->>Timer: Stop & Dispose
    Monitor->>TestRunner: Log "[SYNC COMPLETE]" with elapsed time

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 I watch the bytes hop, tick by tick,
Timers tap, the progress ticks quick—
Start the monitor, count each file,
Logs that cheer across the mile,
Stop and sip the sync-time tea.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately and concisely summarizes the main changes: adding timeouts and download progress monitoring to integration tests, matching the core objectives.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch ci/integration-test-timeouts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[Copilot]

Pull request overview

This PR improves CI reliability and debuggability for the media sync integration tests by adding workflow timeouts and emitting periodic download progress to the job logs.

Changes:

• Add job- and step-level timeouts to the GitHub Actions integration-tests workflow to prevent indefinite hangs.
• Introduce Start-SyncProgressMonitor / Stop-SyncProgressMonitor helpers to periodically log file count and downloaded size while syncing media.
• Wrap the Sync-PatMedia download and idempotency integration tests with the progress monitor.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
tests/Integration/Public/MediaSync.Integration.tests.ps1	Wraps long-running sync tests with a progress monitor to improve CI log visibility.
tests/Integration/IntegrationTestHelpers.psm1	Adds and exports progress monitor helpers used by integration tests.
.github/workflows/CI.yaml	Adds timeouts to integration test job/step to prevent long hangs.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.