silo-server/internal/api/handlers/progress_access_test.go

package handlers

import (
	"context"
	"slices"
	"testing"

	"github.com/Silo-Server/silo-server/internal/access"
	"github.com/Silo-Server/silo-server/internal/userstore"
)

// fakeProgressLookup records the arguments it was called with and returns a
// fixed accessibility map.
type fakeProgressLookup struct {
	accessible map[string]bool

	gotContentIDs []string
	gotAllowed    []int
	gotDisabled   []int
	gotRating     string
}

func (f *fakeProgressLookup) GetItemsInFolder(context.Context, []string, int) (map[string]bool, error) {
	return nil, nil
}

func (f *fakeProgressLookup) FilterAccessibleContentIDs(
	_ context.Context, contentIDs []string, allowedFolderIDs, disabledFolderIDs []int, maxContentRating string,
) (map[string]bool, error) {
	f.gotContentIDs = contentIDs
	f.gotAllowed = allowedFolderIDs
	f.gotDisabled = disabledFolderIDs
	f.gotRating = maxContentRating
	return f.accessible, nil
}

func entries(ids ...string) []userstore.WatchProgress {
	out := make([]userstore.WatchProgress, 0, len(ids))
	for _, id := range ids {
		out = append(out, userstore.WatchProgress{MediaItemID: id})
	}
	return out
}

func ids(entries []userstore.WatchProgress) []string {
	out := make([]string, 0, len(entries))
	for _, e := range entries {
		out = append(out, e.MediaItemID)
	}
	return out
}

func TestFilterProgressEntriesByAccess(t *testing.T) {
	// access.Scope sets DisabledLibraryIDs only when AllowedLibraryIDs is nil
	// (see access.Scope docs), so the two restriction shapes are tested
	// separately rather than as one impossible combined scope.
	cases := []struct {
		name  string
		scope access.Scope
	}{
		{
			name:  "allowed libraries + rating",
			scope: access.Scope{AllowedLibraryIDs: []int{1, 2}, MaxContentRating: "PG-13"},
		},
		{
			name:  "disabled libraries + rating",
			scope: access.Scope{DisabledLibraryIDs: []int{9}, MaxContentRating: "PG-13"},
		},
	}

	for _, tc := range cases {
		t.Run(tc.name, func(t *testing.T) {
			lookup := &fakeProgressLookup{accessible: map[string]bool{"a": true, "c": true}}

			got, err := filterProgressEntriesByAccess(context.Background(), entries("a", "b", "c"), tc.scope, lookup)
			if err != nil {
				t.Fatalf("unexpected error: %v", err)
			}

			want := []string{"a", "c"}
			if g := ids(got); len(g) != len(want) || g[0] != want[0] || g[1] != want[1] {
				t.Fatalf("filtered entries = %v, want %v", g, want)
			}

			// Scope must be forwarded verbatim to the lookup.
			if !slices.Equal(lookup.gotAllowed, tc.scope.AllowedLibraryIDs) {
				t.Errorf("allowed folders = %v, want %v", lookup.gotAllowed, tc.scope.AllowedLibraryIDs)
			}
			if !slices.Equal(lookup.gotDisabled, tc.scope.DisabledLibraryIDs) {
				t.Errorf("disabled folders = %v, want %v", lookup.gotDisabled, tc.scope.DisabledLibraryIDs)
			}
			if len(lookup.gotContentIDs) != 3 {
				t.Errorf("content ids = %v, want 3 entries", lookup.gotContentIDs)
			}
			if lookup.gotRating != tc.scope.MaxContentRating {
				t.Errorf("max content rating = %q, want %q", lookup.gotRating, tc.scope.MaxContentRating)
			}
		})
	}
}

func TestFilterProgressEntriesByAccessEmpty(t *testing.T) {
	lookup := &fakeProgressLookup{accessible: map[string]bool{}}
	got, err := filterProgressEntriesByAccess(context.Background(), nil, access.Scope{}, lookup)
	if err != nil {
		t.Fatalf("unexpected error: %v", err)
	}
	if len(got) != 0 {
		t.Fatalf("expected no entries, got %v", ids(got))
	}
}