[PR #5291] [MERGED] Add tests for DashboardController #10470

Closed
opened 2026-02-07 06:20:59 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/5291
Author: @Bond-009
Created: 2/22/2021
Status: Merged
Merged: 2/23/2021
Merged by: @joshuaboniface

Base: masterHead: tests12


📝 Commits (3)

📊 Changes

11 files changed (+224 additions, -45 deletions)

View changed files

📝 Jellyfin.Api/Controllers/DashboardController.cs (+11 -32)
📝 Jellyfin.Api/Models/ConfigurationPageInfo.cs (+8 -1)
📝 Jellyfin.Server/CoreAppHost.cs (+0 -1)
📝 MediaBrowser.Common/Plugins/BasePlugin.cs (+0 -3)
📝 tests/Jellyfin.Api.Tests/Controllers/BrandingControllerTests.cs (+10 -6)
tests/Jellyfin.Api.Tests/Controllers/DashboardControllerTests.cs (+86 -0)
📝 tests/Jellyfin.Api.Tests/Jellyfin.Api.Tests.csproj (+4 -0)
📝 tests/Jellyfin.Api.Tests/JellyfinApplicationFactory.cs (+2 -2)
tests/Jellyfin.Api.Tests/TestAppHost.cs (+51 -0)
tests/Jellyfin.Api.Tests/TestPage.html (+9 -0)
tests/Jellyfin.Api.Tests/TestPlugin.cs (+43 -0)

📄 Description

No description provided


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/5291 **Author:** [@Bond-009](https://github.com/Bond-009) **Created:** 2/22/2021 **Status:** ✅ Merged **Merged:** 2/23/2021 **Merged by:** [@joshuaboniface](https://github.com/joshuaboniface) **Base:** `master` ← **Head:** `tests12` --- ### 📝 Commits (3) - [`5710209`](https://github.com/jellyfin/jellyfin/commit/57102090d307a62b3759177df3ba545cfad6c327) Add tests for DashboardController - [`ec3237b`](https://github.com/jellyfin/jellyfin/commit/ec3237ba55a6c0c6e7a31e2aaa5fbf77c9978ac7) Fix tests - [`baadc48`](https://github.com/jellyfin/jellyfin/commit/baadc48f43ec425659d0d7210f1a97416d7d34c5) Address comments ### 📊 Changes **11 files changed** (+224 additions, -45 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Api/Controllers/DashboardController.cs` (+11 -32) 📝 `Jellyfin.Api/Models/ConfigurationPageInfo.cs` (+8 -1) 📝 `Jellyfin.Server/CoreAppHost.cs` (+0 -1) 📝 `MediaBrowser.Common/Plugins/BasePlugin.cs` (+0 -3) 📝 `tests/Jellyfin.Api.Tests/Controllers/BrandingControllerTests.cs` (+10 -6) ➕ `tests/Jellyfin.Api.Tests/Controllers/DashboardControllerTests.cs` (+86 -0) 📝 `tests/Jellyfin.Api.Tests/Jellyfin.Api.Tests.csproj` (+4 -0) 📝 `tests/Jellyfin.Api.Tests/JellyfinApplicationFactory.cs` (+2 -2) ➕ `tests/Jellyfin.Api.Tests/TestAppHost.cs` (+51 -0) ➕ `tests/Jellyfin.Api.Tests/TestPage.html` (+9 -0) ➕ `tests/Jellyfin.Api.Tests/TestPlugin.cs` (+43 -0) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 06:20:59 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#10470