starred/jellyfin
1
0
Fork 0
mirror of https://github.com/jellyfin/jellyfin.git synced 2026-07-16 05:43:54 +03:00
Code Issues 822 Packages Projects Releases 17 Wiki Activity

[PR #6546] [MERGED] Use TheoryData instead of IEnumerable #11021

New Issue
Closed
opened 2026-02-07 06:30:59 +03:00 by OVERLORD · 0 comments
OVERLORD commented 2026-02-07 06:30:59 +03:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/6546
Author: @iansw246
Created: 9/12/2021
Status: ✅ Merged
Merged: 9/28/2021
Merged by: @Bond-009

Base: master ← Head: theorydata

📝 Commits (4)

  • c5393b2 Use TheoryData in UrlDecodeQueryFeatureTest
  • 3313efc Revert "Use TheoryData in UrlDecodeQueryFeatureTest"
  • 19b8bca Use TheoryData instead of MemberData and ClassData
  • 8858d8e Merge branch 'master' into theorydata

📊 Changes

13 files changed (+348 additions, -427 deletions)

View changed files

📝 tests/Jellyfin.Api.Tests/Controllers/DynamicHlsControllerTests.cs (+13 -18)
📝 tests/Jellyfin.Api.Tests/Helpers/RequestHelpersTests.cs (+13 -13)
📝 tests/Jellyfin.Common.Tests/Cryptography/PasswordHashTests.cs (+19 -35)
📝 tests/Jellyfin.Extensions.Tests/CopyToExtensionsTests.cs (+29 -9)
📝 tests/Jellyfin.MediaEncoding.Tests/EncoderValidatorTests.cs (+12 -14)
📝 tests/Jellyfin.MediaEncoding.Tests/Subtitles/SsaParserTests.cs (+7 -5)
📝 tests/Jellyfin.Model.Tests/Entities/MediaStreamTests.cs (+48 -53)
📝 tests/Jellyfin.Naming.Tests/AudioBook/AudioBookResolverTests.cs (+13 -13)
📝 tests/Jellyfin.Naming.Tests/Video/VideoResolverTests.cs (+51 -65)
📝 tests/Jellyfin.Providers.Tests/MediaInfo/SubtitleResolverTests.cs (+8 -6)
📝 tests/Jellyfin.Server.Implementations.Tests/Data/SqliteItemRepositoryTests.cs (+44 -56)
📝 tests/Jellyfin.Server.Implementations.Tests/LiveTv/RecordingHelperTests.cs (+21 -37)
📝 tests/Jellyfin.Server.Implementations.Tests/Sorting/AiredEpisodeOrderComparerTests.cs (+70 -103)

📄 Description

Changes
Changes Test functions and classes used with MemberData and ClassData to use TheoryData instead of IEnumerable<object[]>

Issues
Completes #6432

🔄 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/6546 **Author:** [@iansw246](https://github.com/iansw246) **Created:** 9/12/2021 **Status:** ✅ Merged **Merged:** 9/28/2021 **Merged by:** [@Bond-009](https://github.com/Bond-009) **Base:** `master` ← **Head:** `theorydata` --- ### 📝 Commits (4) - [`c5393b2`](https://github.com/jellyfin/jellyfin/commit/c5393b25798098aa079c185971b7ac28e5f5b994) Use TheoryData in UrlDecodeQueryFeatureTest - [`3313efc`](https://github.com/jellyfin/jellyfin/commit/3313efce195a66b0363ea7224d70056fdda14ab6) Revert "Use TheoryData in UrlDecodeQueryFeatureTest" - [`19b8bca`](https://github.com/jellyfin/jellyfin/commit/19b8bcaec43835c698a35975a748c2129c1413aa) Use TheoryData instead of MemberData and ClassData - [`8858d8e`](https://github.com/jellyfin/jellyfin/commit/8858d8e59789ee3ad7b55077900d80437ef982b5) Merge branch 'master' into theorydata ### 📊 Changes **13 files changed** (+348 additions, -427 deletions) <details> <summary>View changed files</summary> 📝 `tests/Jellyfin.Api.Tests/Controllers/DynamicHlsControllerTests.cs` (+13 -18) 📝 `tests/Jellyfin.Api.Tests/Helpers/RequestHelpersTests.cs` (+13 -13) 📝 `tests/Jellyfin.Common.Tests/Cryptography/PasswordHashTests.cs` (+19 -35) 📝 `tests/Jellyfin.Extensions.Tests/CopyToExtensionsTests.cs` (+29 -9) 📝 `tests/Jellyfin.MediaEncoding.Tests/EncoderValidatorTests.cs` (+12 -14) 📝 `tests/Jellyfin.MediaEncoding.Tests/Subtitles/SsaParserTests.cs` (+7 -5) 📝 `tests/Jellyfin.Model.Tests/Entities/MediaStreamTests.cs` (+48 -53) 📝 `tests/Jellyfin.Naming.Tests/AudioBook/AudioBookResolverTests.cs` (+13 -13) 📝 `tests/Jellyfin.Naming.Tests/Video/VideoResolverTests.cs` (+51 -65) 📝 `tests/Jellyfin.Providers.Tests/MediaInfo/SubtitleResolverTests.cs` (+8 -6) 📝 `tests/Jellyfin.Server.Implementations.Tests/Data/SqliteItemRepositoryTests.cs` (+44 -56) 📝 `tests/Jellyfin.Server.Implementations.Tests/LiveTv/RecordingHelperTests.cs` (+21 -37) 📝 `tests/Jellyfin.Server.Implementations.Tests/Sorting/AiredEpisodeOrderComparerTests.cs` (+70 -103) </details> ### 📄 Description **Changes** Changes Test functions and classes used with `MemberData` and `ClassData` to use `TheoryData` instead of `IEnumerable<object[]>` **Issues** Completes #6432 --- <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:30:59 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
EFjellyfin.db
area:database
awaiting-feedback
backend
blocked
breaking change: web api
bug
build
ci
confirmed
discussion needed
dotnet future
downstream
duplicate
enhancement
feature
future
github-actions
good first issue
hdr
help wanted
invalid
investigation
librarydb
live-tv
lyrics
media playback
music
needs testing
nuget
performance
platform
pull-request
Mirrored from GitHub Pull Request
 question
regression
release critical
requires-web
roadmap
security
security
stale
support
syncplay
ui & ux
upstream
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#11021
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?