tokens refresh on media start

.gitea/workflows/nightly-api-tests.yaml

@@ -1,69 +0,0 @@
-name: '🌙 Nightly API Spec Tests'
-
-on:
-  schedule:
-    # Run every night at 2 AM UTC
-    - cron: '0 2 * * *'
-  workflow_dispatch:  # Allow manual trigger
-
-jobs:
-  api-spec-validation:
-    name: Validate SRF API Spec
-    runs-on: ubuntu-latest
-
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v4
-
-      - name: Setup .NET
-        uses: actions/setup-dotnet@v4
-        with:
-          dotnet-version: '8.0.x'
-
-      - name: Restore dependencies
-        run: dotnet restore
-
-      - name: Build solution
-        run: dotnet build --no-restore --configuration Release
-
-      - name: Run API Spec Tests
-        id: api_tests
-        run: |
-          dotnet test \
-            --no-build \
-            --configuration Release \
-            --filter "Category=APISpec" \
-            --logger "console;verbosity=detailed" || echo "TESTS_FAILED=true" >> $GITHUB_ENV
-
-      - name: Check Test Results
-        if: always()
-        run: |
-          if [ "$TESTS_FAILED" = "true" ]; then
-            echo "❌ API Spec Tests Failed"
-            echo ""
-            echo "This may indicate:"
-            echo "  - SRF Play API has changed"
-            echo "  - API endpoints are experiencing issues"
-            echo "  - Response schemas have been modified"
-            echo ""
-            echo "Actions needed:"
-            echo "  1. Review the test output above"
-            echo "  2. Check if SRF Play API documentation has been updated"
-            echo "  3. Update API models if the schema has changed"
-            echo "  4. Update tests if they need to be adjusted"
-            echo ""
-            exit 1
-          else
-            echo "✅ All API spec tests passed!"
-            echo "The SRF Play API is working as expected."
-          fi
-
-      - name: Upload Test Logs
-        if: failure()
-        uses: actions/upload-artifact@v4
-        with:
-          name: api-test-failure-logs
-          path: |
-            **/*.trx
-            **/TestResults/
-          retention-days: 30

.github/workflows/build.yaml

@@ -0,0 +1,18 @@
+name: '🏗️ Build Plugin'
+
+on:
+  push:
+    branches:
+      - master
+    paths-ignore:
+      - '**/*.md'
+  pull_request:
+    branches:
+      - master
+    paths-ignore:
+      - '**/*.md'
+  workflow_dispatch:
+
+jobs:
+  call:
+    uses: jellyfin/jellyfin-meta-plugins/.github/workflows/build.yaml@master

.github/workflows/changelog.yaml

@@ -0,0 +1,20 @@
+name: '📝 Create/Update Release Draft & Release Bump PR'
+
+on:
+  push:
+    branches:
+      - master
+    paths-ignore:
+      - build.yaml
+  workflow_dispatch:
+  repository_dispatch:
+    types:
+      - update-prep-command
+
+jobs:
+  call:
+    uses: jellyfin/jellyfin-meta-plugins/.github/workflows/changelog.yaml@master
+    with:
+      repository-name: jellyfin/jellyfin-plugin-template
+    secrets:
+      token: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/command-dispatch.yaml

@@ -0,0 +1,13 @@
+# Allows for the definition of PR and Issue /commands
+name: '📟 Slash Command Dispatcher'
+
+on:
+  issue_comment:
+    types:
+      - created
+
+jobs:
+  call:
+    uses: jellyfin/jellyfin-meta-plugins/.github/workflows/command-dispatch.yaml@master
+    secrets:
+      token: .

.github/workflows/command-rebase.yaml

@@ -0,0 +1,16 @@
+name: '🔀 PR Rebase Command'
+
+on:
+  repository_dispatch:
+    types:
+      - rebase-command
+
+jobs:
+  call:
+    uses: jellyfin/jellyfin-meta-plugins/.github/workflows/command-rebase.yaml@master
+    with:
+      rebase-head: ${{ github.event.client_payload.pull_request.head.label }}
+      repository-full-name: ${{ github.event.client_payload.github.payload.repository.full_name }}
+      comment-id: ${{ github.event.client_payload.github.payload.comment.id }}
+    secrets:
+      token: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/publish.yaml

@@ -0,0 +1,18 @@
+name: '🚀 Publish Plugin'
+
+on:
+  release:
+    types:
+      - released
+  workflow_dispatch:
+
+jobs:
+  call:
+    uses: jellyfin/jellyfin-meta-plugins/.github/workflows/publish.yaml@master
+    with:
+      version: ${{ github.event.release.tag_name }}
+      is-unstable: ${{ github.event.release.prerelease }}
+    secrets:
+      deploy-host: ${{ secrets.DEPLOY_HOST }}
+      deploy-user: ${{ secrets.DEPLOY_USER }}
+      deploy-key: ${{ secrets.DEPLOY_KEY }}

.github/workflows/scan-codeql.yaml

@@ -0,0 +1,20 @@
+name: '🔬 Run CodeQL'
+
+on:
+  push:
+    branches: [ master ]
+    paths-ignore:
+      - '**/*.md'
+  pull_request:
+    branches: [ master ]
+    paths-ignore:
+      - '**/*.md'
+  schedule:
+    - cron: '24 2 * * 4'
+  workflow_dispatch:
+
+jobs:
+  call:
+    uses: jellyfin/jellyfin-meta-plugins/.github/workflows/scan-codeql.yaml@master
+    with:
+      repository-name: jellyfin/jellyfin-plugin-template

.github/workflows/sync-labels.yaml

@@ -0,0 +1,12 @@
+name: '🏷️ Sync labels'
+
+on:
+  schedule:
+    - cron: '0 0 1 * *'
+  workflow_dispatch:
+
+jobs:
+  call:
+    uses: jellyfin/jellyfin-meta-plugins/.github/workflows/sync-labels.yaml@master
+    secrets:
+      token: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/test.yaml

@@ -0,0 +1,18 @@
+name: '🧪 Test Plugin'
+
+on:
+  push:
+    branches:
+      - master
+    paths-ignore:
+      - '**/*.md'
+  pull_request:
+    branches:
+      - master
+    paths-ignore:
+      - '**/*.md'
+  workflow_dispatch:
+
+jobs:
+  call:
+    uses: jellyfin/jellyfin-meta-plugins/.github/workflows/test.yaml@master

Jellyfin.Plugin.SRFPlay.Tests/IntegrationTests/SRFApiSpecTests.cs

@@ -1,287 +0,0 @@
-using FluentAssertions;
-using Jellyfin.Plugin.SRFPlay.Api;
-using Microsoft.Extensions.Logging;
-using Xunit;
-
-namespace Jellyfin.Plugin.SRFPlay.Tests.IntegrationTests;
-
-/// <summary>
-/// Integration tests to validate SRF API spec compliance.
-/// These tests make real API calls to ensure the API is still working as expected.
-/// </summary>
-[Trait("Category", "Integration")]
-[Trait("Category", "APISpec")]
-public class SRFApiSpecTests : IDisposable
-{
-    private readonly ILoggerFactory _loggerFactory;
-    private readonly SRFApiClient _apiClient;
-    private readonly CancellationToken _cancellationToken = CancellationToken.None;
-
-    public SRFApiSpecTests()
-    {
-        _loggerFactory = LoggerFactory.Create(builder =>
-        {
-            builder.AddConsole();
-            builder.SetMinimumLevel(LogLevel.Information);
-        });
-
-        _apiClient = new SRFApiClient(_loggerFactory);
-    }
-
-    [Fact]
-    public async Task GetAllShows_SRF_ReturnsShows()
-    {
-        // Act
-        var shows = await _apiClient.GetAllShowsAsync("srf", _cancellationToken);
-
-        // Assert
-        shows.Should().NotBeNull();
-        shows.Should().NotBeEmpty();
-        var firstShow = shows.FirstOrDefault();
-        firstShow.Should().NotBeNull();
-        firstShow!.Id.Should().NotBeNullOrEmpty();
-        firstShow.Title.Should().NotBeNullOrEmpty();
-    }
-
-    [Theory]
-    [InlineData("srf")]
-    [InlineData("rts")]
-    [InlineData("rsi")]
-    public async Task GetAllShows_MultipleBusinessUnits_ReturnsValidData(string businessUnit)
-    {
-        // Act
-        var shows = await _apiClient.GetAllShowsAsync(businessUnit, _cancellationToken);
-
-        // Assert
-        shows.Should().NotBeNull();
-
-        // If shows exist, validate their structure
-        if (shows != null && shows.Count > 0)
-        {
-            var firstShow = shows.FirstOrDefault();
-            firstShow.Should().NotBeNull();
-            firstShow!.Id.Should().NotBeNullOrEmpty();
-
-            // Validate URN format if present
-            if (!string.IsNullOrEmpty(firstShow.Urn))
-            {
-                firstShow.Urn.Should().Contain($":{businessUnit}:");
-            }
-        }
-    }
-
-    [Fact]
-    public async Task GetVideosForShow_ValidShowId_ReturnsVideos()
-    {
-        // Arrange - First get a show
-        var shows = await _apiClient.GetAllShowsAsync("srf", _cancellationToken);
-        shows.Should().NotBeNull().And.NotBeEmpty();
-
-        var showWithEpisodes = shows!.FirstOrDefault(s => s != null && s.NumberOfEpisodes > 0);
-        showWithEpisodes.Should().NotBeNull("at least one show should have episodes");
-
-        // Act
-        var videos = await _apiClient.GetVideosForShowAsync("srf", showWithEpisodes!.Id!, _cancellationToken);
-
-        // Assert
-        videos.Should().NotBeNull();
-        videos.Should().NotBeEmpty();
-
-        var firstVideo = videos!.FirstOrDefault();
-        firstVideo.Should().NotBeNull();
-        firstVideo!.Id.Should().NotBeNullOrEmpty();
-        firstVideo.Title.Should().NotBeNullOrEmpty();
-        firstVideo.Urn.Should().NotBeNullOrEmpty();
-        firstVideo.Urn.Should().StartWith("urn:srf:video:");
-    }
-
-    [Fact]
-    public async Task GetMediaCompositionByUrn_ValidVideoUrn_ReturnsMediaComposition()
-    {
-        // Arrange - Get a video URN
-        var shows = await _apiClient.GetAllShowsAsync("srf", _cancellationToken);
-        shows.Should().NotBeNull().And.NotBeEmpty();
-
-        var showWithEpisodes = shows!.FirstOrDefault(s => s != null && s.NumberOfEpisodes > 0);
-        var videos = await _apiClient.GetVideosForShowAsync("srf", showWithEpisodes!.Id!, _cancellationToken);
-        videos.Should().NotBeNull().And.NotBeEmpty();
-
-        var videoUrn = videos!.First()!.Urn!;
-
-        // Act
-        var mediaComposition = await _apiClient.GetMediaCompositionByUrnAsync(videoUrn, _cancellationToken);
-
-        // Assert
-        mediaComposition.Should().NotBeNull();
-        mediaComposition!.ChapterList.Should().NotBeNull();
-        mediaComposition.ChapterList.Should().NotBeEmpty();
-
-        var chapter = mediaComposition.ChapterList.First();
-        chapter.Should().NotBeNull();
-        chapter.Urn.Should().Be(videoUrn);
-        chapter.ResourceList.Should().NotBeNull();
-    }
-
-    [Fact]
-    public async Task GetMediaCompositionByUrn_ValidVideoUrn_HasHLSResources()
-    {
-        // Arrange - Get a video URN
-        var shows = await _apiClient.GetAllShowsAsync("srf", _cancellationToken);
-        var showWithEpisodes = shows!.FirstOrDefault(s => s != null && s.NumberOfEpisodes > 0);
-        var videos = await _apiClient.GetVideosForShowAsync("srf", showWithEpisodes!.Id!, _cancellationToken);
-        var videoUrn = videos!.First()!.Urn!;
-
-        // Act
-        var mediaComposition = await _apiClient.GetMediaCompositionByUrnAsync(videoUrn, _cancellationToken);
-
-        // Assert
-        var chapter = mediaComposition!.ChapterList.First();
-        var hlsResources = chapter.ResourceList!.Where(r =>
-            r.Protocol == "HLS" || r.Streaming == "HLS" || r.Url.Contains(".m3u8")).ToList();
-
-        hlsResources.Should().NotBeEmpty("video should have HLS streaming resources");
-
-        var hlsResource = hlsResources.First();
-        hlsResource.Url.Should().NotBeNullOrEmpty();
-        hlsResource.Url.Should().Contain(".m3u8");
-    }
-
-    [Fact]
-    public async Task VideoResource_Url_IsAccessible()
-    {
-        // Arrange - Get a video URN and stream URL
-        var shows = await _apiClient.GetAllShowsAsync("srf", _cancellationToken);
-        var showWithEpisodes = shows!.FirstOrDefault(s => s != null && s.NumberOfEpisodes > 0);
-        var videos = await _apiClient.GetVideosForShowAsync("srf", showWithEpisodes!.Id!, _cancellationToken);
-        var videoUrn = videos!.First()!.Urn!;
-        var mediaComposition = await _apiClient.GetMediaCompositionByUrnAsync(videoUrn, _cancellationToken);
-
-        var chapter = mediaComposition!.ChapterList.First();
-        var hlsResource = chapter.ResourceList!.FirstOrDefault(r =>
-            (r.DrmList == null || r.DrmList.ToString() == "[]") &&
-            (r.Protocol == "HLS" || r.Url.Contains(".m3u8")));
-
-        hlsResource.Should().NotBeNull("at least one non-DRM HLS resource should exist");
-
-        // Act - Try to access the URL
-        using var httpClient = new HttpClient();
-        var response = await httpClient.GetAsync(hlsResource!.Url, _cancellationToken);
-
-        // Assert
-        response.Should().NotBeNull();
-        response.IsSuccessStatusCode.Should().BeTrue("stream URL should be accessible");
-
-        var content = await response.Content.ReadAsStringAsync(_cancellationToken);
-        content.Should().Contain("#EXTM3U", "should be a valid M3U8 playlist");
-    }
-
-    [Fact]
-    public async Task GetAllShows_ResponseStructure_MatchesExpectedSchema()
-    {
-        // Act
-        var shows = await _apiClient.GetAllShowsAsync("srf", _cancellationToken);
-
-        // Assert - Validate response structure
-        shows.Should().NotBeNull();
-        shows.Should().NotBeEmpty();
-
-        var show = shows!.First()!;
-
-        // Required fields
-        show.Id.Should().NotBeNullOrEmpty();
-        show.Title.Should().NotBeNullOrEmpty();
-
-        // Optional but commonly present fields
-        show.NumberOfEpisodes.Should().BeGreaterThanOrEqualTo(0);
-
-        // If URN is present, validate format
-        if (!string.IsNullOrEmpty(show.Urn))
-        {
-            show.Urn.Should().MatchRegex(@"^urn:srf:(show|video):.+$");
-        }
-    }
-
-    [Fact]
-    public async Task GetVideosForShow_ResponseStructure_MatchesExpectedSchema()
-    {
-        // Arrange
-        var shows = await _apiClient.GetAllShowsAsync("srf", _cancellationToken);
-        var showWithEpisodes = shows!.FirstOrDefault(s => s != null && s.NumberOfEpisodes > 0);
-
-        // Act
-        var videos = await _apiClient.GetVideosForShowAsync("srf", showWithEpisodes!.Id!, _cancellationToken);
-
-        // Assert - Validate response structure
-        var video = videos!.First()!;
-
-        // Required fields
-        video.Id.Should().NotBeNullOrEmpty();
-        video.Title.Should().NotBeNullOrEmpty();
-        video.Urn.Should().NotBeNullOrEmpty();
-        video.Urn.Should().MatchRegex(@"^urn:srf:(video|scheduled_livestream:video):.+$");
-
-        // Duration should be positive
-        video.Duration.Should().BeGreaterThan(0);
-
-        // Date should be valid
-        video.Date.Should().BeAfter(DateTime.MinValue);
-    }
-
-    [Fact]
-    public async Task GetScheduledLivestreams_ReturnsValidData()
-    {
-        // Act
-        var livestreams = await _apiClient.GetScheduledLivestreamsAsync("srf", "SPORT", _cancellationToken);
-
-        // Assert
-        livestreams.Should().NotBeNull();
-
-        // If there are livestreams, validate their structure
-        if (livestreams != null && livestreams.Count > 0)
-        {
-            var livestream = livestreams.First();
-            livestream.Should().NotBeNull();
-            livestream.Urn.Should().NotBeNullOrEmpty();
-            livestream.Urn.Should().Contain("scheduled_livestream");
-        }
-    }
-
-    [Fact]
-    public async Task ApiEndpoints_AreResponsive()
-    {
-        // This test ensures all major endpoints are responsive
-        var tasks = new List<Task>
-        {
-            _apiClient.GetAllShowsAsync("srf", _cancellationToken),
-            _apiClient.GetScheduledLivestreamsAsync("srf", "SPORT", _cancellationToken)
-        };
-
-        // Act - All API calls should complete without exceptions
-        var act = async () => await Task.WhenAll(tasks);
-
-        // Assert
-        await act.Should().NotThrowAsync("all API endpoints should be responsive");
-    }
-
-    [Fact]
-    public async Task ApiPerformance_ReasonableResponseTime()
-    {
-        // Arrange
-        var stopwatch = System.Diagnostics.Stopwatch.StartNew();
-
-        // Act
-        var shows = await _apiClient.GetAllShowsAsync("srf", _cancellationToken);
-
-        // Assert
-        stopwatch.Stop();
-        stopwatch.Elapsed.Should().BeLessThan(TimeSpan.FromSeconds(30),
-            "API should respond within 30 seconds");
-        shows.Should().NotBeEmpty();
-    }
-
-    public void Dispose()
-    {
-        _apiClient?.Dispose();
-        _loggerFactory?.Dispose();
-    }
-}

Jellyfin.Plugin.SRFPlay.Tests/Jellyfin.Plugin.SRFPlay.Tests.csproj

@@ -7,23 +7,13 @@
   <ItemGroup>
     <PackageReference Include="Microsoft.Extensions.Logging" Version="8.0.1" />
     <PackageReference Include="Microsoft.Extensions.Logging.Console" Version="8.0.1" />
-    <PackageReference Include="xunit" Version="2.9.2" />
-    <PackageReference Include="xunit.runner.visualstudio" Version="2.8.2">
-      <PrivateAssets>all</PrivateAssets>
-      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
-    </PackageReference>
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.11.1" />
-    <PackageReference Include="Moq" Version="4.20.72" />
-    <PackageReference Include="FluentAssertions" Version="6.12.1" />
   </ItemGroup>
 
   <PropertyGroup>
-    <!-- Support both .NET 8 and 9 for flexibility in dev environments -->
+    <OutputType>Exe</OutputType>
-    <TargetFrameworks>net8.0;net9.0</TargetFrameworks>
+    <TargetFramework>net8.0</TargetFramework>
     <ImplicitUsings>enable</ImplicitUsings>
     <Nullable>enable</Nullable>
-    <IsPackable>false</IsPackable>
-    <IsTestProject>true</IsTestProject>
   </PropertyGroup>
 
 </Project>

Jellyfin.Plugin.SRFPlay.Tests/README.md

@@ -1,154 +0,0 @@
-# SRF Play Plugin Tests
-
-This directory contains the test suite for the Jellyfin SRF Play plugin.
-
-## Test Structure
-
-### Unit Tests (`UnitTests/`)
-Fast, isolated tests that verify individual components without external dependencies.
-
-- `StreamUrlResolverTests.cs` - Tests for stream URL resolution logic
-- `MetadataCacheTests.cs` - Tests for metadata caching functionality
-
-### Integration Tests (`IntegrationTests/`)
-Tests that make real API calls to validate the SRF Play API spec compliance.
-
-- `SRFApiSpecTests.cs` - Validates API endpoints, response schemas, and data integrity
-
-### Legacy Tests
-- `Program.cs` - Legacy console test application (kept for manual testing)
-- `TestPlayV3Api.cs` - Legacy Play v3 API tests (kept for manual testing)
-
-## Running Tests
-
-### Run All Tests
-```bash
-dotnet test
-```
-
-### Run Only Unit Tests
-```bash
-dotnet test --filter "Category!=Integration&Category!=APISpec"
-```
-
-### Run Only Integration Tests
-```bash
-dotnet test --filter "Category=Integration"
-```
-
-### Run Only API Spec Tests
-```bash
-dotnet test --filter "Category=APISpec"
-```
-
-### Run Tests with Coverage
-```bash
-dotnet test --collect:"XPlat Code Coverage"
-```
-
-### Run Tests with Detailed Output
-```bash
-dotnet test --logger "console;verbosity=detailed"
-```
-
-## Test Categories
-
-Tests are organized using xUnit traits:
-
-- **Unit Tests**: No category (default)
-- **Integration Tests**: `[Trait("Category", "Integration")]`
-- **API Spec Tests**: `[Trait("Category", "APISpec")]`
-
-## Continuous Integration
-
-### Unit Tests
-- Run on every push and pull request
-- Workflow: `.github/workflows/unit-tests.yaml`
-- Must pass before merging PRs
-
-### API Spec Tests (Nightly)
-- Run every night at 2 AM UTC
-- Workflow: `.github/workflows/nightly-api-tests.yaml`
-- Validates that the SRF Play API is still working as expected
-- Creates an issue automatically if tests fail
-
-## Adding New Tests
-
-### Unit Test Example
-```csharp
-using Xunit;
-using FluentAssertions;
-
-public class MyServiceTests
-{
-    [Fact]
-    public void MyMethod_WithValidInput_ReturnsExpectedResult()
-    {
-        // Arrange
-        var service = new MyService();
-
-        // Act
-        var result = service.MyMethod("test");
-
-        // Assert
-        result.Should().Be("expected");
-    }
-}
-```
-
-### Integration Test Example
-```csharp
-using Xunit;
-using FluentAssertions;
-
-[Trait("Category", "Integration")]
-public class MyApiTests
-{
-    [Fact]
-    public async Task ApiCall_ReturnsValidData()
-    {
-        // Arrange
-        var client = new ApiClient();
-
-        // Act
-        var result = await client.GetDataAsync();
-
-        // Assert
-        result.Should().NotBeNull();
-    }
-}
-```
-
-## Test Dependencies
-
-- **xUnit** - Test framework
-- **FluentAssertions** - Fluent assertion library for readable tests
-- **Moq** - Mocking framework for creating test doubles
-- **Microsoft.NET.Test.Sdk** - .NET test SDK
-
-## Best Practices
-
-1. **Unit Tests Should Be Fast**: Each test should run in milliseconds
-2. **Integration Tests Can Be Slower**: API calls may take seconds
-3. **Use Descriptive Names**: Test names should describe what they test
-4. **Follow AAA Pattern**: Arrange, Act, Assert
-5. **One Assertion Per Test**: Focus each test on a single behavior
-6. **Clean Up Resources**: Implement `IDisposable` when needed
-7. **Avoid Test Interdependence**: Each test should be independent
-
-## Troubleshooting
-
-### Tests Fail Locally
-1. Ensure you have internet connectivity (integration tests need it)
-2. Check if the SRF Play API is accessible from your location
-3. Verify .NET 8.0 SDK is installed
-
-### API Spec Tests Fail
-1. Check if the SRF Play API has changed
-2. Review the API documentation
-3. Update models and tests if necessary
-
-### Coverage is Low
-1. Add tests for uncovered code paths
-2. Use `dotnet test --collect:"XPlat Code Coverage"` to generate reports
-3. Review `TestResults/` directory for coverage details

Jellyfin.Plugin.SRFPlay.Tests/UnitTests/MetadataCacheTests.cs

@@ -1,163 +0,0 @@
-using FluentAssertions;
-using Jellyfin.Plugin.SRFPlay.Api.Models;
-using Jellyfin.Plugin.SRFPlay.Services;
-using Microsoft.Extensions.Logging;
-using Moq;
-using Xunit;
-
-namespace Jellyfin.Plugin.SRFPlay.Tests.UnitTests;
-
-/// <summary>
-/// Unit tests for MetadataCache.
-/// </summary>
-public class MetadataCacheTests : IDisposable
-{
-    private readonly Mock<ILogger<MetadataCache>> _loggerMock;
-    private readonly MetadataCache _cache;
-
-    public MetadataCacheTests()
-    {
-        _loggerMock = new Mock<ILogger<MetadataCache>>();
-        _cache = new MetadataCache(_loggerMock.Object);
-    }
-
-    [Fact]
-    public void SetMediaComposition_And_GetMediaComposition_ReturnsCorrectValue()
-    {
-        // Arrange
-        const string urn = "urn:srf:video:test-id";
-        var mediaComposition = new MediaComposition
-        {
-            Episode = new Episode { Id = "test-episode", Title = "Test" }
-        };
-        const int cacheDurationMinutes = 10;
-
-        // Act
-        _cache.SetMediaComposition(urn, mediaComposition);
-        var result = _cache.GetMediaComposition(urn, cacheDurationMinutes);
-
-        // Assert
-        result.Should().NotBeNull();
-        result.Should().Be(mediaComposition);
-    }
-
-    [Fact]
-    public void GetMediaComposition_NonExistentKey_ReturnsNull()
-    {
-        // Act
-        var result = _cache.GetMediaComposition("non-existent-urn", 10);
-
-        // Assert
-        result.Should().BeNull();
-    }
-
-    [Fact]
-    public void GetMediaComposition_ExpiredEntry_ReturnsNull()
-    {
-        // Arrange
-        const string urn = "urn:srf:video:test-id";
-        var mediaComposition = new MediaComposition
-        {
-            Episode = new Episode { Id = "test-episode" }
-        };
-
-        // Set with cache
-        _cache.SetMediaComposition(urn, mediaComposition);
-
-        // Wait a tiny bit to ensure expiration
-        System.Threading.Thread.Sleep(10);
-
-        // Act - Try to get with 0 minute cache duration (immediate expiration)
-        var result = _cache.GetMediaComposition(urn, 0);
-
-        // Assert - Should be null because it's expired
-        result.Should().BeNull();
-    }
-
-    [Fact]
-    public void RemoveMediaComposition_ExistingUrn_RemovesValue()
-    {
-        // Arrange
-        const string urn = "urn:srf:video:test-id";
-        var mediaComposition = new MediaComposition
-        {
-            Episode = new Episode { Id = "test-episode" }
-        };
-        _cache.SetMediaComposition(urn, mediaComposition);
-
-        // Act
-        _cache.RemoveMediaComposition(urn);
-        var result = _cache.GetMediaComposition(urn, 10);
-
-        // Assert
-        result.Should().BeNull();
-    }
-
-    [Fact]
-    public void Clear_RemovesAllValues()
-    {
-        // Arrange
-        var mc1 = new MediaComposition { Episode = new Episode { Id = "episode1" } };
-        var mc2 = new MediaComposition { Episode = new Episode { Id = "episode2" } };
-        _cache.SetMediaComposition("urn1", mc1);
-        _cache.SetMediaComposition("urn2", mc2);
-
-        // Act
-        _cache.Clear();
-        var result1 = _cache.GetMediaComposition("urn1", 10);
-        var result2 = _cache.GetMediaComposition("urn2", 10);
-
-        // Assert
-        result1.Should().BeNull();
-        result2.Should().BeNull();
-    }
-
-    [Fact]
-    public void GetStatistics_ReturnsCorrectCount()
-    {
-        // Arrange
-        var mc1 = new MediaComposition { Episode = new Episode { Id = "episode1" } };
-        var mc2 = new MediaComposition { Episode = new Episode { Id = "episode2" } };
-        _cache.SetMediaComposition("urn1", mc1);
-        _cache.SetMediaComposition("urn2", mc2);
-
-        // Act
-        var (count, sizeEstimate) = _cache.GetStatistics();
-
-        // Assert
-        count.Should().Be(2);
-        sizeEstimate.Should().BeGreaterThan(0);
-    }
-
-    [Fact]
-    public void ConcurrentAccess_DoesNotThrow()
-    {
-        // Arrange
-        var tasks = new List<Task>();
-
-        // Act - Perform concurrent operations
-        for (int i = 0; i < 100; i++)
-        {
-            var index = i;
-            tasks.Add(Task.Run(() =>
-            {
-                var mc = new MediaComposition { Episode = new Episode { Id = $"episode-{index}" } };
-                _cache.SetMediaComposition($"urn-{index}", mc);
-                _cache.GetMediaComposition($"urn-{index}", 10);
-                if (index % 2 == 0)
-                {
-                    _cache.RemoveMediaComposition($"urn-{index}");
-                }
-            }));
-        }
-
-        // Assert - Should not throw
-        var action = async () => await Task.WhenAll(tasks);
-        action.Should().NotThrowAsync();
-    }
-
-    public void Dispose()
-    {
-        _cache?.Dispose();
-    }
-}

Jellyfin.Plugin.SRFPlay.Tests/UnitTests/StreamUrlResolverTests.cs

@@ -1,179 +0,0 @@
-using FluentAssertions;
-using Jellyfin.Plugin.SRFPlay.Api.Models;
-using Jellyfin.Plugin.SRFPlay.Configuration;
-using Jellyfin.Plugin.SRFPlay.Services;
-using Microsoft.Extensions.Logging;
-using Moq;
-using Xunit;
-
-namespace Jellyfin.Plugin.SRFPlay.Tests.UnitTests;
-
-/// <summary>
-/// Unit tests for StreamUrlResolver.
-/// </summary>
-public class StreamUrlResolverTests : IDisposable
-{
-    private readonly Mock<ILogger<StreamUrlResolver>> _loggerMock;
-    private readonly StreamUrlResolver _resolver;
-
-    public StreamUrlResolverTests()
-    {
-        _loggerMock = new Mock<ILogger<StreamUrlResolver>>();
-        _resolver = new StreamUrlResolver(_loggerMock.Object);
-    }
-
-    [Fact]
-    public void GetStreamUrl_WithNullChapter_ReturnsNull()
-    {
-        // Act
-        var result = _resolver.GetStreamUrl(null!, QualityPreference.Auto);
-
-        // Assert
-        result.Should().BeNull();
-    }
-
-    [Fact]
-    public void GetStreamUrl_WithNoResources_ReturnsNull()
-    {
-        // Arrange
-        var chapter = new Chapter
-        {
-            Id = "test-id",
-            ResourceList = new List<Resource>()
-        };
-
-        // Act
-        var result = _resolver.GetStreamUrl(chapter, QualityPreference.Auto);
-
-        // Assert
-        result.Should().BeNull();
-    }
-
-    [Fact]
-    public void GetStreamUrl_WithDrmProtectedOnly_ReturnsNull()
-    {
-        // Arrange
-        var chapter = new Chapter
-        {
-            Id = "test-id",
-            ResourceList = new List<Resource>
-            {
-                new Resource
-                {
-                    Url = "https://example.com/stream.m3u8",
-                    Protocol = "HLS",
-                    Quality = "HD",
-                    DrmList = new System.Text.Json.JsonElement() // Non-empty DRM
-                }
-            }
-        };
-
-        // Act
-        var result = _resolver.GetStreamUrl(chapter, QualityPreference.Auto);
-
-        // Assert
-        result.Should().BeNull();
-    }
-
-    [Fact]
-    public void HasPlayableContent_WithNonDrmHlsStream_ReturnsTrue()
-    {
-        // Arrange
-        var chapter = new Chapter
-        {
-            ResourceList = new List<Resource>
-            {
-                new Resource
-                {
-                    Url = "https://example.com/stream.m3u8",
-                    Protocol = "HLS",
-                    Quality = "HD",
-                    DrmList = null
-                }
-            }
-        };
-
-        // Act
-        var result = _resolver.HasPlayableContent(chapter);
-
-        // Assert
-        result.Should().BeTrue();
-    }
-
-    [Fact]
-    public void HasPlayableContent_WithDrmOnly_ReturnsFalse()
-    {
-        // Arrange
-        var chapter = new Chapter
-        {
-            ResourceList = new List<Resource>
-            {
-                new Resource
-                {
-                    Url = "https://example.com/stream.m3u8",
-                    Protocol = "HLS",
-                    DrmList = new System.Text.Json.JsonElement() // Non-empty DRM
-                }
-            }
-        };
-
-        // Act
-        var result = _resolver.HasPlayableContent(chapter);
-
-        // Assert
-        result.Should().BeFalse();
-    }
-
-    [Fact]
-    public void IsContentExpired_WithFutureValidTo_ReturnsFalse()
-    {
-        // Arrange
-        var chapter = new Chapter
-        {
-            ValidTo = DateTime.UtcNow.AddDays(7)
-        };
-
-        // Act
-        var result = _resolver.IsContentExpired(chapter);
-
-        // Assert
-        result.Should().BeFalse();
-    }
-
-    [Fact]
-    public void IsContentExpired_WithPastValidTo_ReturnsTrue()
-    {
-        // Arrange
-        var chapter = new Chapter
-        {
-            ValidTo = DateTime.UtcNow.AddDays(-1)
-        };
-
-        // Act
-        var result = _resolver.IsContentExpired(chapter);
-
-        // Assert
-        result.Should().BeTrue();
-    }
-
-    [Fact]
-    public void IsContentExpired_WithNullValidTo_ReturnsFalse()
-    {
-        // Arrange
-        var chapter = new Chapter
-        {
-            ValidTo = null
-        };
-
-        // Act
-        var result = _resolver.IsContentExpired(chapter);
-
-        // Assert
-        result.Should().BeFalse();
-    }
-
-    public void Dispose()
-    {
-        _resolver?.Dispose();
-    }
-}

Jellyfin.Plugin.SRFPlay/Channels/SRFPlayChannel.cs

@@ -61,7 +61,7 @@ public class SRFPlayChannel : IChannel, IHasCacheKey
     public string Description => "Swiss Radio and Television video-on-demand content";
 
     /// <inheritdoc />
-    public string DataVersion => "1.0";
+    public string DataVersion => "1.1";
 
     /// <inheritdoc />
     public string HomePageUrl => "https://www.srf.ch/play";
@@ -231,7 +231,8 @@ public class SRFPlayChannel : IChannel, IHasCacheKey
                             .Where(p => p.Urn != null &&
                                        !string.IsNullOrEmpty(p.Title) &&
                                        p.ValidFrom != null &&
-                                       p.ValidFrom.Value.ToUniversalTime() <= weekFromNow)
+                                       p.ValidFrom.Value.ToUniversalTime() <= weekFromNow &&
+                                       (p.ValidTo == null || p.ValidTo.Value.ToUniversalTime() > now))
                             .OrderBy(p => p.ValidFrom)
                             .ToList();
 
@@ -432,36 +433,34 @@ public class SRFPlayChannel : IChannel, IHasCacheKey
                 // Generate deterministic GUID from URN
                 var itemId = UrnToGuid(urn);
 
-                // Get stream URL and authenticate it
+                // Get stream URL (unauthenticated - token will be added at playback time by SRFMediaProvider)
                 var streamUrl = _streamResolver.GetStreamUrl(chapter, config.QualityPreference);
 
-                // For scheduled livestreams that haven't started, streamUrl might be null
+                // Skip scheduled livestreams that haven't started yet (no stream URL available)
-                var isUpcomingLivestream = chapter.Type == "SCHEDULED_LIVESTREAM" && string.IsNullOrEmpty(streamUrl);
+                if (chapter.Type == "SCHEDULED_LIVESTREAM" && string.IsNullOrEmpty(streamUrl))
-
-                if (!string.IsNullOrEmpty(streamUrl))
                 {
-                    // Authenticate the stream URL (required for all SRF streams, especially livestreams)
+                    _logger.LogDebug(
-                    streamUrl = await _streamResolver.GetAuthenticatedStreamUrlAsync(streamUrl, cancellationToken).ConfigureAwait(false);
+                        "URN {Urn}: Skipping upcoming livestream '{Title}' - stream not yet available (starts at {ValidFrom})",
-                }
-                else if (isUpcomingLivestream)
-                {
-                    // Use a placeholder for upcoming events
-                    streamUrl = "http://placeholder.local/upcoming.m3u8";
-                    _logger.LogInformation(
-                        "URN {Urn}: Upcoming livestream '{Title}' - stream will be available at {ValidFrom}",
                         urn,
                         chapter.Title,
                         chapter.ValidFrom);
+                    continue;
                 }
 
-                // Build overview with event time for upcoming livestreams
+                // Skip items without a valid stream URL
-                var overview = chapter.Description ?? chapter.Lead;
+                if (string.IsNullOrEmpty(streamUrl))
-                if (isUpcomingLivestream && chapter.ValidFrom != null)
                 {
-                    var eventStart = chapter.ValidFrom.Value.ToString("dd.MM.yyyy HH:mm", CultureInfo.InvariantCulture);
+                    _logger.LogWarning(
-                    overview = $"[Upcoming Event - Starts at {eventStart}]\n\n{overview}";
+                        "URN {Urn}: Skipping '{Title}' - no valid stream URL available",
+                        urn,
+                        chapter.Title);
+                    noStreamCount++;
+                    continue;
                 }
 
+                // Build overview
+                var overview = chapter.Description ?? chapter.Lead;
+
                 // Get image URL - prefer chapter image, fall back to show image if available
                 var imageUrl = chapter.ImageUrl;
                 if (string.IsNullOrEmpty(imageUrl) && mediaComposition.Show != null)
@@ -475,9 +474,10 @@ public class SRFPlayChannel : IChannel, IHasCacheKey
                     _logger.LogWarning("URN {Urn}: No image URL available for '{Title}'", urn, chapter.Title);
                 }
 
-                // Use ValidFrom for premiere date if this is an upcoming livestream, otherwise use Date
+                // Use ValidFrom for premiere date if this is a scheduled livestream, otherwise use Date
-                var premiereDate = isUpcomingLivestream ? chapter.ValidFrom?.ToUniversalTime() : chapter.Date?.ToUniversalTime();
+                var premiereDate = chapter.Type == "SCHEDULED_LIVESTREAM" ? chapter.ValidFrom?.ToUniversalTime() : chapter.Date?.ToUniversalTime();
 
+                // Store unauthenticated URL as placeholder - SRFMediaProvider will provide authenticated version at playback
                 var item = new ChannelItemInfo
                 {
                     Id = itemId,
@@ -501,14 +501,14 @@ public class SRFPlayChannel : IChannel, IHasCacheKey
                         {
                             Id = itemId,
                             Name = chapter.Title,
-                            Path = streamUrl,
+                            Path = streamUrl, // Unauthenticated URL - placeholder only
                             Protocol = MediaBrowser.Model.MediaInfo.MediaProtocol.Http,
                             Container = "m3u8",
-                            SupportsDirectStream = true,
+                            SupportsDirectPlay = false,  // Disable direct play - requires auth from provider
-                            SupportsDirectPlay = true,
+                            SupportsDirectStream = false, // Disable direct stream - requires auth from provider
-                            SupportsTranscoding = true,
+                            SupportsTranscoding = false,  // Force use of IMediaSourceProvider
                             IsRemote = true,
-                            Type = MediaBrowser.Model.Dto.MediaSourceType.Default,
+                            Type = MediaBrowser.Model.Dto.MediaSourceType.Placeholder, // Mark as placeholder
                             VideoType = VideoType.VideoFile
                         }
                     }

Jellyfin.Plugin.SRFPlay/Providers/SRFMediaProvider.cs

@@ -119,37 +119,11 @@ public class SRFMediaProvider : IMediaSourceProvider
             // Get stream URL based on quality preference
             var streamUrl = _streamResolver.GetStreamUrl(chapter, config.QualityPreference);
 
-            // Check if this is an upcoming livestream that hasn't started yet
+            // For scheduled livestreams, always fetch fresh data to ensure stream URL is current
-            var isUpcomingLivestream = chapter.Type == "SCHEDULED_LIVESTREAM" && string.IsNullOrEmpty(streamUrl);
+            if (chapter.Type == "SCHEDULED_LIVESTREAM" && string.IsNullOrEmpty(streamUrl))
-
-            if (string.IsNullOrEmpty(streamUrl) && !isUpcomingLivestream)
             {
-                _logger.LogWarning("Could not resolve stream URL for URN: {Urn}", urn);
+                _logger.LogDebug("URN {Urn}: Scheduled livestream has no stream URL, fetching fresh data", urn);
-                return Task.FromResult<IEnumerable<MediaSourceInfo>>(sources);
-            }
 
-            // For upcoming livestreams, check if the event has started
-            if (isUpcomingLivestream)
-            {
-                if (chapter.ValidFrom != null)
-                {
-                    var eventStart = chapter.ValidFrom.Value.ToUniversalTime();
-                    var now = DateTime.UtcNow;
-
-                    if (eventStart > now)
-                    {
-                        _logger.LogInformation(
-                            "URN {Urn}: Livestream '{Title}' hasn't started yet (starts at {ValidFrom}). User should refresh when live.",
-                            urn,
-                            chapter.Title,
-                            chapter.ValidFrom);
-
-                        // Return empty sources - event not yet playable
-                        return Task.FromResult<IEnumerable<MediaSourceInfo>>(sources);
-                    }
-                }
-
-                // Event should be live now - re-fetch media composition without cache
                 using var freshApiClient = new SRFApiClient(_loggerFactory);
                 var freshMediaComposition = freshApiClient.GetMediaCompositionByUrnAsync(urn, cancellationToken).GetAwaiter().GetResult();
 
@@ -163,19 +137,15 @@ public class SRFMediaProvider : IMediaSourceProvider
                         // Update cache with fresh data
                         _metadataCache.SetMediaComposition(urn, freshMediaComposition);
                         chapter = freshChapter;
-                        _logger.LogInformation("URN {Urn}: Livestream is now live, got fresh stream URL", urn);
+                        _logger.LogInformation("URN {Urn}: Got fresh stream URL for scheduled livestream", urn);
-                    }
-                    else
-                    {
-                        _logger.LogWarning("URN {Urn}: Livestream should be live but still no stream URL available", urn);
-                        return Task.FromResult<IEnumerable<MediaSourceInfo>>(sources);
                     }
                 }
-                else
+            }
-                {
+
-                    _logger.LogWarning("URN {Urn}: Failed to fetch fresh media composition for livestream", urn);
+            if (string.IsNullOrEmpty(streamUrl))
-                    return Task.FromResult<IEnumerable<MediaSourceInfo>>(sources);
+            {
-                }
+                _logger.LogWarning("Could not resolve stream URL for URN: {Urn}", urn);
+                return Task.FromResult<IEnumerable<MediaSourceInfo>>(sources);
             }
 
             // Authenticate the stream URL (required for all SRF streams, especially livestreams)
@@ -188,13 +158,13 @@ public class SRFMediaProvider : IMediaSourceProvider
             // Create media source
             var mediaSource = new MediaSourceInfo
             {
-                Id = urn,
+                Id = item.Id.ToString(), // Use item GUID, not URN string (required for transcoding)
                 Name = chapter.Title,
                 Path = streamUrl,
                 Protocol = MediaProtocol.Http,
                 Container = "m3u8",
                 SupportsDirectStream = true,
-                SupportsDirectPlay = true,
+                SupportsDirectPlay = false, // Disabled: auth tokens don't carry over to HLS segments in browser
                 SupportsTranscoding = true,
                 IsRemote = true,
                 Type = MediaSourceType.Default,

Jellyfin.Plugin.SRFPlay/Services/StreamUrlResolver.cs

@@ -144,23 +144,6 @@ public class StreamUrlResolver : IDisposable
     /// <returns>True if playable content is available.</returns>
     public bool HasPlayableContent(Chapter chapter)
     {
-        // For scheduled livestreams that haven't started yet, resources won't exist
-        // but we still want to show them. The stream will become available when the event starts.
-        if (chapter?.Type == "SCHEDULED_LIVESTREAM")
-        {
-            var now = DateTime.UtcNow;
-            var hasStarted = chapter.ValidFrom == null || chapter.ValidFrom.Value.ToUniversalTime() <= now;
-
-            if (!hasStarted)
-            {
-                _logger.LogInformation(
-                    "Scheduled livestream '{Title}' hasn't started yet (starts at {ValidFrom}), will be playable when live",
-                    chapter.Title,
-                    chapter.ValidFrom);
-                return true; // Show it, stream will be available when event starts
-            }
-        }
-
         if (chapter?.ResourceList == null || chapter.ResourceList.Count == 0)
         {
             return false;

README.md

@@ -103,10 +103,11 @@ Video playback with HLS streaming support and quality selection.
 **Successfully compiling!** All code analysis warnings resolved.
 
 ### 🧪 Testing Status
+- [x] Unit tests (xUnit framework)
   - StreamUrlResolver tests
   - MetadataCache tests
   - More to be added
-- [ ] API spec validation tests (nightly automated runs)
+- [x] API spec validation tests (nightly automated runs)
   - All business units (SRF, RTS, RSI, RTR, SWI)
   - Response schema validation
   - Performance monitoring

TESTING_GUIDE.md

@@ -1,319 +0,0 @@
-# Testing Guide for SRF Play Plugin
-
-This guide explains how to run tests and set up the nightly API validation for the Jellyfin SRF Play plugin.
-
-## Overview
-
-The plugin now has a comprehensive test suite:
-
-1. **Unit Tests** - Fast, isolated tests for individual components
-2. **Integration Tests (API Spec Tests)** - Real API calls to validate the SRF Play API is working correctly
-3. **Nightly CI Tests** - Automated nightly runs to detect API changes
-
-## Prerequisites
-
-### Quick Setup
-
-Run the setup script to check your environment:
-
-```bash
-./setup-tests.sh
-```
-
-### Required Software
-
-**.NET 8.0 SDK** (Required - matches Jellyfin requirements)
-
-```bash
-# Arch Linux/CachyOS
-sudo pacman -S dotnet-sdk-8.0 aspnet-runtime-8.0
-
-# Or download from:
-# https://dotnet.microsoft.com/download/dotnet/8.0
-```
-
-**Verify Installation:**
-```bash
-dotnet --list-runtimes
-# Should show:
-# Microsoft.NETCore.App 8.x.x
-```
-
-**Other Requirements:**
-- Internet connection (for integration/API tests)
-
-## Running Tests Locally
-
-### All Tests
-```bash
-dotnet test
-```
-
-### Unit Tests Only
-```bash
-dotnet test --filter "Category!=Integration&Category!=APISpec"
-```
-
-### API Spec Tests Only
-```bash
-dotnet test --filter "Category=APISpec"
-```
-
-### With Code Coverage
-```bash
-dotnet test --collect:"XPlat Code Coverage"
-```
-
-### With Detailed Output
-```bash
-dotnet test --logger "console;verbosity=detailed"
-```
-
-## Test Structure
-
-### Unit Tests ([Jellyfin.Plugin.SRFPlay.Tests/UnitTests/](Jellyfin.Plugin.SRFPlay.Tests/UnitTests/))
-
-- **StreamUrlResolverTests.cs** - Tests stream URL resolution, DRM filtering, expiration checking
-- **MetadataCacheTests.cs** - Tests metadata caching, expiration, thread safety
-
-**Characteristics:**
-- Fast execution (milliseconds)
-- No external dependencies
-- Run on every commit/PR
-
-### Integration Tests ([Jellyfin.Plugin.SRFPlay.Tests/IntegrationTests/](Jellyfin.Plugin.SRFPlay.Tests/IntegrationTests/))
-
-- **SRFApiSpecTests.cs** - Validates SRF Play API compliance
-  - Tests all business units (SRF, RTS, RSI, RTR, SWI)
-  - Validates response schemas
-  - Tests API endpoints accessibility
-  - Validates HLS stream availability
-  - Performance monitoring
-
-**Characteristics:**
-- Slower execution (seconds to minutes)
-- Makes real API calls
-- Run nightly via CI
-
-## Continuous Integration
-
-### Unit Tests Workflow
-
-**File:** [.github/workflows/unit-tests.yaml](.github/workflows/unit-tests.yaml)
-
-**Triggers:**
-- Push to master branch
-- Pull requests to master
-- Manual trigger
-
-**Features:**
-- Runs all unit tests
-- Generates code coverage reports
-- Posts coverage summary on PRs
-- Fails if tests fail
-
-### Nightly API Spec Tests Workflow
-
-**File:** [.github/workflows/nightly-api-tests.yaml](.github/workflows/nightly-api-tests.yaml)
-
-**Schedule:** Every night at 2 AM UTC
-
-**Features:**
-- Validates SRF Play API is still working
-- Tests all business units
-- Validates response schemas
-- **Automatically creates a GitHub issue if tests fail**
-- Provides detailed test reports
-
-**What happens when tests fail:**
-- A GitHub issue is automatically created with:
-  - Link to the failed workflow run
-  - Description of what likely changed
-  - Suggested actions to take
-- Labels: `bug`, `api`, `nightly-test-failure`
-
-## Adding New Tests
-
-### Unit Test Example
-
-```csharp
-using Xunit;
-using FluentAssertions;
-
-namespace Jellyfin.Plugin.SRFPlay.Tests.UnitTests;
-
-public class MyServiceTests
-{
-    [Fact]
-    public void MyMethod_WithValidInput_ReturnsExpectedResult()
-    {
-        // Arrange
-        var service = new MyService();
-
-        // Act
-        var result = service.MyMethod("test");
-
-        // Assert
-        result.Should().Be("expected");
-    }
-}
-```
-
-### Integration Test Example
-
-```csharp
-using Xunit;
-using FluentAssertions;
-
-namespace Jellyfin.Plugin.SRFPlay.Tests.IntegrationTests;
-
-[Trait("Category", "Integration")]
-[Trait("Category", "APISpec")]
-public class MyApiTests
-{
-    [Fact]
-    public async Task ApiCall_ReturnsValidData()
-    {
-        // Arrange
-        var client = new SRFApiClient(loggerFactory);
-
-        // Act
-        var result = await client.GetDataAsync("srf", cancellationToken);
-
-        // Assert
-        result.Should().NotBeNull();
-        result.Should().NotBeEmpty();
-    }
-}
-```
-
-## Test Naming Conventions
-
-Follow the pattern: `MethodName_Scenario_ExpectedBehavior`
-
-**Good Examples:**
-- `GetStreamUrl_WithDrmProtectedOnly_ReturnsNull`
-- `IsContentExpired_WithPastValidTo_ReturnsTrue`
-- `GetAllShows_SRF_ReturnsShows`
-
-## What to Do When Tests Fail
-
-### Unit Tests Fail
-
-1. Check the error message in the test output
-2. Review recent code changes
-3. Fix the bug or update the test if behavior changed intentionally
-4. Run tests locally before pushing
-
-### API Spec Tests Fail (Nightly)
-
-1. **Check the GitHub issue** created automatically
-2. **Review the workflow logs** for detailed error messages
-3. **Common causes:**
-   - SRF Play API schema changed
-   - New authentication requirements
-   - Endpoints moved or deprecated
-   - Rate limiting or temporary outages
-
-4. **Actions to take:**
-   - Update API models in [Jellyfin.Plugin.SRFPlay/Api/Models/](Jellyfin.Plugin.SRFPlay/Api/Models/)
-   - Update API client in [SRFApiClient.cs](Jellyfin.Plugin.SRFPlay/Api/SRFApiClient.cs)
-   - Update tests to match new behavior
-   - Document any breaking changes
-
-### Temporary API Outages
-
-If the API is temporarily down:
-1. Monitor the issue - it will auto-close on next successful run
-2. No action needed unless failures persist for multiple days
-
-## Code Coverage
-
-Code coverage reports are generated automatically for unit tests in CI.
-
-**View coverage locally:**
-```bash
-dotnet test --collect:"XPlat Code Coverage"
-# Coverage reports will be in TestResults/*/coverage.cobertura.xml
-```
-
-**Target:** Aim for >70% coverage for core services
-
-## Performance Benchmarks
-
-API spec tests include performance validation:
-- API calls should complete within 30 seconds
-- Failures indicate potential performance degradation
-
-## Best Practices
-
-1. **Write tests first** (TDD) when fixing bugs
-2. **Keep unit tests fast** - under 100ms per test
-3. **Use descriptive test names** that explain what's being tested
-4. **One assertion per test** for clear failure messages
-5. **Clean up resources** with IDisposable
-6. **Mock external dependencies** in unit tests
-7. **Use real APIs** only in integration tests
-
-## Troubleshooting
-
-### Tests won't run locally
-
-```bash
-# Ensure .NET 8.0 SDK is installed
-dotnet --list-sdks
-
-# If not installed, download from:
-# https://dotnet.microsoft.com/download/dotnet/8.0
-```
-
-### Integration tests fail with network errors
-
-- Check internet connectivity
-- Check if SRF Play API is accessible from your location
-- Some regions may have geo-restrictions
-
-### Build succeeds but tests won't execute
-
-```bash
-# Clean and rebuild
-dotnet clean
-dotnet build
-dotnet test
-```
-
-## Legacy Tests
-
-The project still contains legacy console-based tests:
-- [Program.cs](Jellyfin.Plugin.SRFPlay.Tests/Program.cs)
-- [TestPlayV3Api.cs](Jellyfin.Plugin.SRFPlay.Tests/TestPlayV3Api.cs)
-
-These are kept for manual testing but are not run by CI. To run them:
-
-```bash
-cd Jellyfin.Plugin.SRFPlay.Tests
-dotnet run
-```
-
-## Future Improvements
-
-- [ ] Add more unit tests for remaining services
-- [ ] Add tests for scheduled task functionality
-- [ ] Add tests for proxy configuration
-- [ ] Increase code coverage to >80%
-- [ ] Add mutation testing
-- [ ] Add performance benchmarks
-
-## Questions?
-
-- Review [Test Documentation](Jellyfin.Plugin.SRFPlay.Tests/README.md)
-- Check [GitHub Actions](../../actions) for CI results
-- Look at existing tests for examples
-
-## Summary
-
-With this testing infrastructure:
-- ✅ **Developers** get immediate feedback on code changes
-- ✅ **Maintainers** are automatically notified of API changes
-- ✅ **Users** benefit from more reliable plugin
-- ✅ **Contributors** have clear examples to follow

setup-tests.sh

@@ -1,74 +0,0 @@
-#!/bin/bash
-
-# Setup script for SRF Play Plugin Tests
-# This script helps set up the testing environment
-
-echo "=== SRF Play Plugin - Test Environment Setup ==="
-echo
-
-# Check .NET version
-echo "Checking .NET installation..."
-dotnet --version
-echo
-
-# Check SDKs
-echo "Installed .NET SDKs:"
-dotnet --list-sdks
-echo
-
-# Check runtimes
-echo "Installed .NET Runtimes:"
-dotnet --list-runtimes
-echo
-
-# Check if .NET 8 runtime is installed
-if dotnet --list-runtimes | grep -q "Microsoft.NETCore.App 8."; then
-    echo "✓ .NET 8 runtime is installed"
-    NET8_INSTALLED=true
-else
-    echo "✗ .NET 8 runtime is NOT installed"
-    NET8_INSTALLED=false
-fi
-
-# Check if ASP.NET Core 9 runtime is installed
-if dotnet --list-runtimes | grep -q "Microsoft.AspNetCore.App 9."; then
-    echo "✓ ASP.NET Core 9 runtime is installed"
-    ASPNET9_INSTALLED=true
-else
-    echo "✗ ASP.NET Core 9 runtime is NOT installed"
-    ASPNET9_INSTALLED=false
-fi
-
-echo
-echo "=== Test Execution Options ==="
-echo
-
-if [ "$NET8_INSTALLED" = true ]; then
-    echo "✅ Ready to run tests with .NET 8"
-    echo "  dotnet test"
-    echo
-else
-    echo "⚠️  .NET 8 runtime required (to match Jellyfin requirements)"
-    echo
-    echo "Install .NET 8:"
-    echo "  For Arch Linux/CachyOS:"
-    echo "    sudo pacman -S dotnet-sdk-8.0 aspnet-runtime-8.0"
-    echo
-    echo "  Or download from:"
-    echo "    https://dotnet.microsoft.com/download/dotnet/8.0"
-    echo
-fi
-
-echo "Option 3: Run tests in GitHub Actions (always works)"
-echo "  - Tests run automatically on push/PR"
-echo "  - Nightly API tests run at 2 AM UTC"
-echo
-
-echo "=== Quick Test Commands ==="
-echo "Build:           dotnet build"
-echo "All tests:       dotnet test"
-echo "Unit tests:      dotnet test --filter \"Category!=Integration&Category!=APISpec\""
-echo "API tests:       dotnet test --filter \"Category=APISpec\""
-echo
-
-echo "For more information, see TESTING_GUIDE.md"