Skip to content

feat: Add multi-server player toggle and resume helpers#52

Merged
justinhartman merged 7 commits intomainfrom
develop
Oct 15, 2025
Merged

feat: Add multi-server player toggle and resume helpers#52
justinhartman merged 7 commits intomainfrom
develop

Conversation

@justinhartman
Copy link
Owner

  • helpers/appHelper.ts:30 introduces buildSources, buildCanonical, and history upsert helpers so view routing can share embed selection, resume redirects, and watched state logic in one place.
  • controllers/appController.ts:9 replaces inline history handling with the new helpers, adding resume redirects, multi-server source wiring, and clearer canonical URL composition for both movies and series.
  • .env.example:17 and config/app.ts:94 document the new default VIDSRC domain plus optional MULTI_DOMAIN flag, keeping config/tests (config/app.spec.ts:12) aligned.
  • views/view.ejs:1, views/partials/series-buttons.ejs:28, and views/partials/footer.ejs:2 surface a toggle-able second server in the UI with a small client script to live-switch iframe sources.
  • tests/utils/routerAgent.ts:1 adds a reusable Express router harness, enabling broader coverage across controller specs and new branches (controllers/appController.multidomain.spec.ts:1, controllers/appController.resume.spec.ts:1, controllers/watchlistController.branch.spec.ts:1, plus updated e2e/integration suites).

justinhartman and others added 7 commits August 19, 2025 11:06
- Add a reusable router agent test utility and replace all
supertest-dependent suites with in-memory request execution so tests
run without socket permissions.
- Expand helper and controller specs (including multi-domain and
watchlist error paths)
- Build new targeted suites, and rework mocks to exercise every branch
required for full coverage.
…for-server-selection

feat: add multi server support
@vercel
Copy link

vercel bot commented Oct 15, 2025

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

Project Deployment Preview Comments Updated (UTC)
imdb-app Ready Ready Preview Comment Oct 15, 2025 2:56pm

@codecov
Copy link

codecov bot commented Oct 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (00db910) to head (ea2b956).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##              main       #52    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files           18        18            
  Lines         1497      1627   +130     
  Branches       117       130    +13     
==========================================
+ Hits          1497      1627   +130     
Flag Coverage Δ
tests 100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@justinhartman justinhartman merged commit 1638528 into main Oct 15, 2025
6 checks passed
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.

1 participant