Merge pull request #2027 from Orange-OpenSource/add_config_test_Segme…

…ntBaseLoader

SegmentBaseLoader tests

src/dash/SegmentBaseLoader.js

@@ -84,7 +84,14 @@ function SegmentBaseLoader() {
         }
     }
 
+    function checkSetConfigCall() {
+        if (!baseURLController || !baseURLController.hasOwnProperty('resolve')) {
+            throw new Error('setConfig function has to be called previously');
+        }
+    }
+
     function loadInitialization(representation, loadingInfo) {
+        checkSetConfigCall();
         let initRange = null;
         let isoFile = null;
         let baseUrl = baseURLController.resolve(representation.path);
@@ -132,6 +139,7 @@ function SegmentBaseLoader() {
     }
 
     function loadSegments(representation, type, range, loadingInfo, callback) {
+        checkSetConfigCall();
         if (range && (range.start === undefined || range.end === undefined)) {
             let parts = range ? range.toString().split('-') : null;
             range = parts ? {start: parseFloat(parts[0]), end: parseFloat(parts[1])} : null;

test/unit/dash.SegmentBaseLoader.js

@@ -0,0 +1,16 @@
+import SegmentBaseLoader from '../../src/dash/SegmentBaseLoader';
+
+const expect = require('chai').expect;
+
+const context = {};
+const segmentBaseLoader = SegmentBaseLoader(context).getInstance();
+
+describe('SegmentBaseLoader', function () {
+    it("should throw an exception when attempting to call loadInitialization While the setConfig function was not called, and parameters are undefined", function () {
+        expect(segmentBaseLoader.loadInitialization.bind(segmentBaseLoader)).to.throw('setConfig function has to be called previously');
+    });
+
+    it("should throw an exception when attempting to call loadSegments While the setConfig function was not called, and parameters are undefined", function () {
+        expect(segmentBaseLoader.loadSegments.bind(segmentBaseLoader)).to.throw('setConfig function has to be called previously');
+    });
+});