fix(tests): improve test reliability — FileLoader errors, state cleanup, debug logs

P1: FileLoader now throws on HTTP errors instead of logging and
returning undefined. This makes fixture-loading failures immediately
visible as test failures.

P2: AbrController afterEach now rebuilds streamProcessor and resets
modified representation properties to prevent order-dependent tests.
StreamController test removes its EventBus listener after assertion.

P3: Remove leftover console.log calls from VideoModelMock,
ExtUrlQueryInfoController tests, and ServiceDescriptionController tests.

Refs Dash-Industry-Forum#5011

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: PascalThuet

test/unit/helpers/FileLoader.js

@@ -27,29 +27,24 @@ class FileLoader {
     }
 
     static async _loadFile(url, options, returnType) {
-        try {
-            url = HTTP_SERVER + url;
-            const response = await fetch(url, options);
-
-            if (!response.ok) {
-                console.log('Network response was not OK');
-            }
-
-            switch (returnType) {
-                case TYPES.TEXT:
-                    return response.text();
-                case TYPES.JSON:
-                    return response.json();
-                case TYPES.ARRAY_BUFFER:
-                    return response.arrayBuffer();
-                case TYPES.BLOB:
-                    return response.blob();
-                default:
-                    return response.text();
-
-            }
-        } catch (e) {
-            console.log(e);
+        url = HTTP_SERVER + url;
+        const response = await fetch(url, options);
+
+        if (!response.ok) {
+            throw new Error(`FileLoader: ${url} returned ${response.status}`);
+        }
+
+        switch (returnType) {
+            case TYPES.TEXT:
+                return response.text();
+            case TYPES.JSON:
+                return response.json();
+            case TYPES.ARRAY_BUFFER:
+                return response.arrayBuffer();
+            case TYPES.BLOB:
+                return response.blob();
+            default:
+                return response.text();
         }
     }
 }

test/unit/mocks/VideoModelMock.js

@@ -169,7 +169,6 @@ class VideoModelMock {
     getCurrentCue(textTrack) {
         const textTrackList = this.element.textTracks;
         const track = textTrackList.getTrackById(textTrack.id);
-        console.log('track', track);
         if (!track) {
             return null;
         }

test/unit/test/streaming/streaming.controllers.AbrController.js

@@ -82,6 +82,7 @@ describe('AbrController', function () {
         abrCtrl.reset();
         settings.reset();
         eventBus.reset();
+        dummyRepresentations[0].segmentSequenceProperties = undefined;
     });
 
     describe('getOptimalRepresentationForBitrate()', function () {

test/unit/test/streaming/streaming.controllers.ExtUrlQueryInfoController.js

@@ -134,7 +134,7 @@ describe('ExtUrlQueryInfoController', () => {
 
             const expectedResult = [{key: 'urlParam1' , value: 'urlValue1'}, {key: 'urlParam2' , value: 'urlValue2'}];
             const result = extUrlQueryInfoController.getFinalQueryString(request);
-            console.log(result)
+
             expect(result).to.have.deep.members(expectedResult);
         });
 
@@ -269,7 +269,7 @@ describe('ExtUrlQueryInfoController', () => {
             }; 
 
             const result = extUrlQueryInfoController.getFinalQueryString(request);
-            console.log(result)
+
             expect(result).to.be.undefined;
         });
 

test/unit/test/streaming/streaming.controllers.ServiceDescriptionController.js

@@ -86,7 +86,6 @@ describe('ServiceDescriptionController', () => {
             serviceDescriptionController.applyServiceDescription(dummyManifestInfo);
 
             const currentSettings = serviceDescriptionController.getServiceDescriptionSettings();
-            console.log('currentSettings', currentSettings)
             expect(currentSettings.liveDelay).to.be.NaN;
             expect(currentSettings.liveCatchup.maxDrift).to.be.NaN;
             expect(currentSettings.liveCatchup.playbackRate.min).to.be.NaN;