51 lines
1.6 KiB
Go
51 lines
1.6 KiB
Go
|
package jellycompat
|
||
|
|
|
||
|
|
import (
|
||
|
|
"errors"
|
||
|
|
"fmt"
|
||
|
|
"net/http"
|
||
|
|
"testing"
|
||
|
|
|
||
|
|
"github.com/Silo-Server/silo-server/internal/playback"
|
||
|
|
)
|
||
|
|
|
||
|
|
// TestHLSSegmentErrorResponse pins the never-500 contract for the HLS segment
|
||
|
|
// handler: a segment that will never materialize (absent, or whose transcode
|
||
|
|
// process started then died) maps to 404 like Jellyfin, and only genuinely
|
||
|
|
// unexpected errors keep the 500.
|
||
|
|
func TestHLSSegmentErrorResponse(t *testing.T) {
|
||
|
|
tests := []struct {
|
||
|
|
name string
|
||
|
|
err error
|
||
|
|
wantStatus int
|
||
|
|
wantCode string
|
||
|
|
}{
|
||
|
|
{"segment not found", playback.ErrSegmentNotFound, http.StatusNotFound, "NotFound"},
|
||
|
|
{"transcode failed", playback.ErrTranscodeFailed, http.StatusNotFound, "NotFound"},
|
||
|
|
{
|
||
|
|
// WaitForSegment wraps the ffmpeg exit error as
|
||
|
|
// fmt.Errorf("%w: %v", ErrTranscodeFailed, waitErr) — this is exactly the
|
||
|
|
// error that hit the catch-all 500 in production (Fire TV, seg_00000.ts).
|
||
|
|
name: "wrapped transcode failed",
|
||
|
|
err: fmt.Errorf("%w: exit status 1", playback.ErrTranscodeFailed),
|
||
|
|
wantStatus: http.StatusNotFound,
|
||
|
|
wantCode: "NotFound",
|
||
|
|
},
|
||
|
|
{
|
||
|
|
name: "unexpected error stays 500",
|
||
|
|
err: errors.New("stat segment: permission denied"),
|
||
|
|
wantStatus: http.StatusInternalServerError,
|
||
|
|
wantCode: "ServerError",
|
||
|
|
},
|
||
|
|
}
|
||
|
|
for _, tt := range tests {
|
||
|
|
t.Run(tt.name, func(t *testing.T) {
|
||
|
|
status, code, _ := hlsSegmentErrorResponse(tt.err)
|
||
|
|
if status != tt.wantStatus || code != tt.wantCode {
|
||
|
|
t.Fatalf("hlsSegmentErrorResponse(%v) = (%d, %q), want (%d, %q)",
|
||
|
|
tt.err, status, code, tt.wantStatus, tt.wantCode)
|
||
|
|
}
|
||
|
|
})
|
||
|
|
}
|
||
|
|
}
|