[PR #2444] [MERGED] converted tests to inlinedata #9121

Closed
opened 2026-02-07 05:56:31 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/2444
Author: @Narfinger
Created: 2/23/2020
Status: Merged
Merged: 2/23/2020
Merged by: @dkanada

Base: masterHead: tests-fix


📝 Commits (5)

  • 21f11c6 converted tests to inlinedata
  • c2fe628 removed failing tests
  • 4dabc50 fixes last tests and cleanup
  • b306b8b add todos and fixes some todo tests
  • fd5f0c5 fixes formatting and enabling another test

📊 Changes

1 file changed (+59 additions, -382 deletions)

View changed files

📝 tests/Jellyfin.Naming.Tests/TV/EpisodeNumberTests.cs (+59 -382)

📄 Description

This converts separate tests to inlinedata.
Additionally, some of the tests did not seem to run and hence fail now.
Failing tests are probably better to see what can be improved in parsing.


🔄 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/2444 **Author:** [@Narfinger](https://github.com/Narfinger) **Created:** 2/23/2020 **Status:** ✅ Merged **Merged:** 2/23/2020 **Merged by:** [@dkanada](https://github.com/dkanada) **Base:** `master` ← **Head:** `tests-fix` --- ### 📝 Commits (5) - [`21f11c6`](https://github.com/jellyfin/jellyfin/commit/21f11c600a51ec15810ae052fa85abf6bafadabf) converted tests to inlinedata - [`c2fe628`](https://github.com/jellyfin/jellyfin/commit/c2fe628c796e7c2525629b39d7eaceaaf4d07ae9) removed failing tests - [`4dabc50`](https://github.com/jellyfin/jellyfin/commit/4dabc50f09425fe8011288ee356509cb3095ed59) fixes last tests and cleanup - [`b306b8b`](https://github.com/jellyfin/jellyfin/commit/b306b8b8815cb4b3e9f6c8f7987b281a31fc0455) add todos and fixes some todo tests - [`fd5f0c5`](https://github.com/jellyfin/jellyfin/commit/fd5f0c54a645d024d1decca7bf4b7da302882d67) fixes formatting and enabling another test ### 📊 Changes **1 file changed** (+59 additions, -382 deletions) <details> <summary>View changed files</summary> 📝 `tests/Jellyfin.Naming.Tests/TV/EpisodeNumberTests.cs` (+59 -382) </details> ### 📄 Description This converts separate tests to inlinedata. Additionally, some of the tests did not seem to run and hence fail now. Failing tests are probably better to see what can be improved in parsing. --- <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:31 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#9121