From 5d97b7247332ddfc662fe27eb9e125c51e676675 Mon Sep 17 00:00:00 2001 From: 4gray Date: Mon, 6 Jul 2026 19:33:03 +0200 Subject: [PATCH] test(tmdb): cover combined overview+trailer fallback fill MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Adds the missing case where the primary payload lacks both overview and trailer and the fallback supplies both — the two fill branches are independent (greptile). Co-Authored-By: Claude Fable 5 --- .../services/src/lib/tmdb/tmdb-language-fallback.spec.ts | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/libs/services/src/lib/tmdb/tmdb-language-fallback.spec.ts b/libs/services/src/lib/tmdb/tmdb-language-fallback.spec.ts index ea20177f9..ed7690380 100644 --- a/libs/services/src/lib/tmdb/tmdb-language-fallback.spec.ts +++ b/libs/services/src/lib/tmdb/tmdb-language-fallback.spec.ts @@ -70,6 +70,15 @@ describe('fillDetailsFromFallback', () => { expect(merged.overview).toBe('English plot'); }); + it('fills both overview and trailer when the primary payload has neither', () => { + const merged = fillDetailsFromFallback( + { id: 1, overview: '', original_language: 'ru' }, + { id: 1, overview: 'Русское описание', videos } + ); + expect(merged.overview).toBe('Русское описание'); + expect(merged.videos?.results?.[0].key).toBe('abc123'); + }); + it('fills the trailer when the primary payload has none', () => { const merged = fillDetailsFromFallback( { id: 1, overview: 'English plot' },