Skip to content

Create a custom MediaRouter for screenshot tests and add new screenshot tests#65

Merged
MGaetan89 merged 4 commits intomainfrom
RouteInfo_screenshots
Apr 14, 2025
Merged

Create a custom MediaRouter for screenshot tests and add new screenshot tests#65
MGaetan89 merged 4 commits intomainfrom
RouteInfo_screenshots

Conversation

@MGaetan89
Copy link
Collaborator

Description

This PR creates ScreenshotMediaRouteProvider and ScreenshotMediaRouter to help generate screenshots without relying on the platform MediaRouter system service.
It also adds new screenshot tests to help work on #58.

Changes made

  • Create ScreenshotMediaRouteProvider to provide some sample routes in screenshots.
  • Create ScreenshotMediaRouter as an alternative to MediaRouter in screenshots.
  • Update MediaRouteChooserDialogScreenshot to use the new ScreenshotMediaRouter.
  • Create some screenshot tests for MediaRouteControllerDialog.
  • Small visual adjustments to MediaRouteControllerDialog.

@MGaetan89 MGaetan89 self-assigned this Apr 4, 2025
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Pillarbox Apr 4, 2025
@MGaetan89 MGaetan89 moved this from 📋 Backlog to 🚧 In Progress in Pillarbox Apr 4, 2025
@github-actions
Copy link

github-actions bot commented Apr 4, 2025

Code Coverage

Overall Project 30.28% -0.38% 🟢
Files changed 0% 🟢

File Coverage
MediaRouteControllerDialog.kt 0% -2.2% 🟢

@MGaetan89 MGaetan89 removed the status in Pillarbox Apr 4, 2025
@MGaetan89 MGaetan89 moved this to 🍿 Code Review in Pillarbox Apr 4, 2025
@MGaetan89 MGaetan89 requested a review from StaehliJ April 4, 2025 09:37
@MGaetan89 MGaetan89 force-pushed the RouteInfo_screenshots branch from 043be6c to a10ab27 Compare April 8, 2025 15:19
@MGaetan89 MGaetan89 added this pull request to the merge queue Apr 14, 2025
Merged via the queue into main with commit 1413c31 Apr 14, 2025
11 checks passed
@MGaetan89 MGaetan89 deleted the RouteInfo_screenshots branch April 14, 2025 11:24
@github-project-automation github-project-automation bot moved this from 🍿 Code Review to ✅ Done in Pillarbox Apr 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants