Merge pull request #2141 from Orange-OpenSource/AddEventBusUnitTest

add EventBus unit test

test/unit/core.EventBus.js

@@ -0,0 +1,43 @@
+import EventBus from '../../src/core/EventBus';
+
+const chai = require('chai');
+const sinon = require('sinon');
+const expect = chai.expect;
+const assert = chai.assert;
+
+const context = {};
+const eventBus = EventBus(context).getInstance();
+
+describe('EventBus', function () {
+
+    it("should throw an exception when attempting to register the on callback with an undefined type", function () {
+        expect(eventBus.on.bind(eventBus)).to.throw('event type cannot be null or undefined');
+    });
+
+    it("should throw an exception when attempting to register the on callback with an undefined listener", function () {
+        expect(eventBus.on.bind(eventBus, 'EVENT_TEST')).to.throw('listener must be a function: undefined');
+    });
+
+    it("should throw an exception when attempting to trigger a 'type' payload parameter", function () {
+        const spy = sinon.spy();
+
+        eventBus.on('EVENT_TEST', spy);
+
+        expect(eventBus.trigger.bind(eventBus, 'EVENT_TEST', { type:{}})).to.throw('\'type\' is a reserved word for event dispatching');
+    });
+
+    it("should respect priority parameter in order to notify the different listeners", function () {
+        const spy = sinon.spy();
+        const spy2 = sinon.spy();
+
+        eventBus.on('EVENT_TEST', spy);
+        eventBus.on('EVENT_TEST', spy2, this, EventBus.EVENT_PRIORITY_HIGH);
+
+        eventBus.trigger('EVENT_TEST', {});
+
+        assert.equal(spy.calledOnce, true);
+        assert.equal(spy2.calledOnce, true);
+        assert.equal(spy2.calledBefore(spy), true);
+    });
+
+});

test/unit/mocks/StreamProcessorMock.js

@@ -90,7 +90,7 @@ class StreamProcessorMock {
     }
 
     getCurrentRepresentationInfo() {
-        return {};
+        return {mediaInfo: {type : this.type}};
     }
 
     isBufferingCompleted() {

test/unit/streaming.controllers.ScheduleController.js

@@ -6,6 +6,10 @@ import PlaybackControllerMock from './mocks/PlaybackControllerMock';
 import StreamProcessorMock from './mocks/StreamProcessorMock';
 import MediaPlayerModelMock from './mocks/MediaPlayerModelMock';
 import DashManifestModelMock from './mocks/DashManifestModelMock';
+import AbrControllerMock from './mocks/AbrControllerMock';
+import StreamControllerMock from './mocks/StreamControllerMock';
+import DashMetricsMock from './mocks/DashMetricsMock';
+import MetricsModelMock from './mocks/MetricsModelMock';
 
 const expect = require('chai').expect;
 const context = {};
@@ -24,19 +28,31 @@ describe('ScheduleController', function () {
     let streamProcessorMock;
     let dashManifestModelMock;
     let playbackControllerMock;
+    let abrControllerMock;
+    let streamControllerMock;
+    let dashMetricsMock;
+    let metricsModelMock;
 
     beforeEach(function () {
         mediaPlayerModelMock = new MediaPlayerModelMock();
         streamProcessorMock = new StreamProcessorMock(testType, streamInfo);
         dashManifestModelMock = new DashManifestModelMock();
         playbackControllerMock = new PlaybackControllerMock();
+        abrControllerMock = new AbrControllerMock();
+        streamControllerMock = new StreamControllerMock();
+        dashMetricsMock = new DashMetricsMock();
+        metricsModelMock = new MetricsModelMock();
 
         scheduleController = ScheduleController(context).create({
             type: testType,
             mediaPlayerModel: mediaPlayerModelMock,
             streamProcessor: streamProcessorMock,
             dashManifestModel: dashManifestModelMock,
-            playbackController: playbackControllerMock
+            playbackController: playbackControllerMock,
+            abrController: abrControllerMock,
+            streamController: streamControllerMock,
+            dashMetrics: dashMetricsMock,
+            metricsModel: metricsModelMock
         });
 
         scheduleController.initialize();