[PR #2472] [MERGED] moves shows tests to Theory and InlineData format #9130

New Issue

OVERLORD · 2026-02-07T05:56:42+03:00

OVERLORD commented

2026-02-07 05:56:42 +03:00

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/2472
Author: @Narfinger
Created: 2/27/2020
Status: ✅ Merged
Merged: 2/27/2020
Merged by: @Bond-009

Base: master ← Head: tests-fix3

📝 Commits (1)

107974e moves shows tests to Theory and InlineData format

📊 Changes

9 files changed (+191 additions, -807 deletions)

View changed files

📝 tests/Jellyfin.Naming.Tests/TV/AbsoluteEpisodeNumberTests.cs (+10 -44)
📝 tests/Jellyfin.Naming.Tests/TV/DailyEpisodeTests.cs (+9 -44)
📝 tests/Jellyfin.Naming.Tests/TV/EpisodeNumberWithoutSeasonTests.cs (+18 -109)
📝 tests/Jellyfin.Naming.Tests/TV/EpisodePathParserTest.cs (+2 -18)
📝 tests/Jellyfin.Naming.Tests/TV/EpisodeWithoutSeasonTests.cs (+7 -36)
📝 tests/Jellyfin.Naming.Tests/TV/MultiEpisodeTests.cs (+64 -89)
📝 tests/Jellyfin.Naming.Tests/TV/SeasonFolderTests.cs (+18 -98)
📝 tests/Jellyfin.Naming.Tests/TV/SeasonNumberTests.cs (+44 -294)
📝 tests/Jellyfin.Naming.Tests/TV/SimpleEpisodeTests.cs (+19 -75)

📄 Description

This moves all current tests in tests/Jellyfin.Naming.Tests/TV to the [Theory] and [InlineData] format.
This makes the files much shorter and clearer imho. I also added a couple more testcases from issues I saw around my files and github issues.

All testcases included are passing, tests that would not pass are commented out and with a // TODO:

_{🔄 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/2472 **Author:** [@Narfinger](https://github.com/Narfinger) **Created:** 2/27/2020 **Status:** ✅ Merged **Merged:** 2/27/2020 **Merged by:** [@Bond-009](https://github.com/Bond-009) **Base:** `master` ← **Head:** `tests-fix3` --- ### 📝 Commits (1) - [`107974e`](https://github.com/jellyfin/jellyfin/commit/107974e3f8ad1c4461bd289ef9cf4d5fb1e619df) moves shows tests to Theory and InlineData format ### 📊 Changes **9 files changed** (+191 additions, -807 deletions) <details> <summary>View changed files</summary> 📝 `tests/Jellyfin.Naming.Tests/TV/AbsoluteEpisodeNumberTests.cs` (+10 -44) 📝 `tests/Jellyfin.Naming.Tests/TV/DailyEpisodeTests.cs` (+9 -44) 📝 `tests/Jellyfin.Naming.Tests/TV/EpisodeNumberWithoutSeasonTests.cs` (+18 -109) 📝 `tests/Jellyfin.Naming.Tests/TV/EpisodePathParserTest.cs` (+2 -18) 📝 `tests/Jellyfin.Naming.Tests/TV/EpisodeWithoutSeasonTests.cs` (+7 -36) 📝 `tests/Jellyfin.Naming.Tests/TV/MultiEpisodeTests.cs` (+64 -89) 📝 `tests/Jellyfin.Naming.Tests/TV/SeasonFolderTests.cs` (+18 -98) 📝 `tests/Jellyfin.Naming.Tests/TV/SeasonNumberTests.cs` (+44 -294) 📝 `tests/Jellyfin.Naming.Tests/TV/SimpleEpisodeTests.cs` (+19 -75) </details> ### 📄 Description This moves all current tests in tests/Jellyfin.Naming.Tests/TV to the [Theory] and [InlineData] format. This makes the files much shorter and clearer imho. I also added a couple more testcases from issues I saw around my files and github issues. All testcases included are passing, tests that would not pass are commented out and with a // TODO: --- <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 05:56:42 +03:00

OVERLORD closed this issue

2026-02-07 05:56:43 +03:00

OVERLORD referenced this issue

2026-02-07 06:48:10 +03:00

[PR #9130] Fix API access permissions for LiveTV and downloads #11995

OVERLORD referenced this issue

2026-02-07 06:48:23 +03:00

[PR #9160] Backport #9130 #12006

Sign in to join this conversation.