From cfd6773070ce63e5dbbde7a644d3bf374848d251 Mon Sep 17 00:00:00 2001 From: Joey Parrish Date: Fri, 13 Jan 2023 09:34:31 -0800 Subject: [PATCH] test: Fix surround sound test case on Tizen (#4893) This test started failing on Tizen after PR #4858, which updates codecs in manifest metadata any time they are rewritten. Since ac-3 is rewritten as ec-3 on Tizen, that caused this test case covering ac-3 to start failing (because the codec says ec-3 by the end of the test). --- test/player_unit.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/test/player_unit.js b/test/player_unit.js index 2650bf7937..d6e8a58c6d 100644 --- a/test/player_unit.js +++ b/test/player_unit.js @@ -3185,7 +3185,9 @@ describe('Player', () => { variant.addAudio(0, (stream) => { stream.channelsCount = 6; stream.audioSamplingRate = 48000; - stream.codecs = 'ac-3'; + // ac-3 is rewritten as ec-3 on Tizen, so for the stability of this + // test case, use ec-3. + stream.codecs = 'ec-3'; }); }); @@ -3210,7 +3212,7 @@ describe('Player', () => { expect(abrManager.variants.length).toBe(1); // It should be the 6-channel variant, based on our preference. expect(abrManager.variants[0].audio.channelsCount).toBe(6); - expect(abrManager.variants[0].audio.codecs).toBe('ac-3'); + expect(abrManager.variants[0].audio.codecs).toBe('ec-3'); }); });