@webex/plugin-meetings 3.0.0-beta.31 → 3.0.0-beta.310
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +46 -8
- package/dist/annotation/annotation.types.js +7 -0
- package/dist/annotation/annotation.types.js.map +1 -0
- package/dist/annotation/constants.js +49 -0
- package/dist/annotation/constants.js.map +1 -0
- package/dist/annotation/index.js +342 -0
- package/dist/annotation/index.js.map +1 -0
- package/dist/breakouts/breakout.js +94 -15
- package/dist/breakouts/breakout.js.map +1 -1
- package/dist/breakouts/edit-lock-error.js +52 -0
- package/dist/breakouts/edit-lock-error.js.map +1 -0
- package/dist/breakouts/events.js +45 -0
- package/dist/breakouts/events.js.map +1 -0
- package/dist/breakouts/index.js +709 -35
- package/dist/breakouts/index.js.map +1 -1
- package/dist/breakouts/utils.js +45 -1
- package/dist/breakouts/utils.js.map +1 -1
- package/dist/common/errors/no-meeting-info.js +51 -0
- package/dist/common/errors/no-meeting-info.js.map +1 -0
- package/dist/common/errors/reclaim-host-role-errors.js +158 -0
- package/dist/common/errors/reclaim-host-role-errors.js.map +1 -0
- package/dist/common/errors/webex-errors.js +48 -7
- package/dist/common/errors/webex-errors.js.map +1 -1
- package/dist/common/logs/logger-proxy.js +1 -1
- package/dist/common/logs/logger-proxy.js.map +1 -1
- package/dist/common/logs/request.js +5 -1
- package/dist/common/logs/request.js.map +1 -1
- package/dist/common/queue.js +24 -9
- package/dist/common/queue.js.map +1 -1
- package/dist/config.js +5 -11
- package/dist/config.js.map +1 -1
- package/dist/constants.js +233 -29
- package/dist/constants.js.map +1 -1
- package/dist/controls-options-manager/enums.js +14 -2
- package/dist/controls-options-manager/enums.js.map +1 -1
- package/dist/controls-options-manager/index.js +109 -15
- package/dist/controls-options-manager/index.js.map +1 -1
- package/dist/controls-options-manager/types.js +7 -0
- package/dist/controls-options-manager/types.js.map +1 -0
- package/dist/controls-options-manager/util.js +309 -18
- package/dist/controls-options-manager/util.js.map +1 -1
- package/dist/index.js +112 -1
- package/dist/index.js.map +1 -1
- package/dist/interpretation/collection.js +23 -0
- package/dist/interpretation/collection.js.map +1 -0
- package/dist/interpretation/index.js +366 -0
- package/dist/interpretation/index.js.map +1 -0
- package/dist/interpretation/siLanguage.js +25 -0
- package/dist/interpretation/siLanguage.js.map +1 -0
- package/dist/locus-info/controlsUtils.js +91 -2
- package/dist/locus-info/controlsUtils.js.map +1 -1
- package/dist/locus-info/index.js +383 -62
- package/dist/locus-info/index.js.map +1 -1
- package/dist/locus-info/infoUtils.js +7 -1
- package/dist/locus-info/infoUtils.js.map +1 -1
- package/dist/locus-info/mediaSharesUtils.js +57 -1
- package/dist/locus-info/mediaSharesUtils.js.map +1 -1
- package/dist/locus-info/parser.js +249 -72
- package/dist/locus-info/parser.js.map +1 -1
- package/dist/locus-info/selfUtils.js +89 -14
- package/dist/locus-info/selfUtils.js.map +1 -1
- package/dist/media/index.js +61 -116
- package/dist/media/index.js.map +1 -1
- package/dist/media/properties.js +73 -124
- package/dist/media/properties.js.map +1 -1
- package/dist/meeting/in-meeting-actions.js +82 -2
- package/dist/meeting/in-meeting-actions.js.map +1 -1
- package/dist/meeting/index.js +3777 -2929
- package/dist/meeting/index.js.map +1 -1
- package/dist/meeting/locusMediaRequest.js +292 -0
- package/dist/meeting/locusMediaRequest.js.map +1 -0
- package/dist/meeting/muteState.js +230 -124
- package/dist/meeting/muteState.js.map +1 -1
- package/dist/meeting/request.js +260 -196
- package/dist/meeting/request.js.map +1 -1
- package/dist/meeting/util.js +601 -417
- package/dist/meeting/util.js.map +1 -1
- package/dist/meeting-info/index.js +73 -7
- package/dist/meeting-info/index.js.map +1 -1
- package/dist/meeting-info/meeting-info-v2.js +192 -51
- package/dist/meeting-info/meeting-info-v2.js.map +1 -1
- package/dist/meeting-info/util.js +1 -1
- package/dist/meeting-info/util.js.map +1 -1
- package/dist/meeting-info/utilv2.js +36 -36
- package/dist/meeting-info/utilv2.js.map +1 -1
- package/dist/meetings/collection.js +39 -0
- package/dist/meetings/collection.js.map +1 -1
- package/dist/meetings/index.js +415 -115
- package/dist/meetings/index.js.map +1 -1
- package/dist/meetings/meetings.types.js +7 -0
- package/dist/meetings/meetings.types.js.map +1 -0
- package/dist/meetings/request.js +2 -0
- package/dist/meetings/request.js.map +1 -1
- package/dist/meetings/util.js +72 -6
- package/dist/meetings/util.js.map +1 -1
- package/dist/member/index.js +58 -0
- package/dist/member/index.js.map +1 -1
- package/dist/member/types.js +25 -0
- package/dist/member/types.js.map +1 -0
- package/dist/member/util.js +132 -25
- package/dist/member/util.js.map +1 -1
- package/dist/members/collection.js +10 -0
- package/dist/members/collection.js.map +1 -1
- package/dist/members/index.js +102 -6
- package/dist/members/index.js.map +1 -1
- package/dist/members/request.js +106 -38
- package/dist/members/request.js.map +1 -1
- package/dist/members/types.js +15 -0
- package/dist/members/types.js.map +1 -0
- package/dist/members/util.js +326 -232
- package/dist/members/util.js.map +1 -1
- package/dist/metrics/constants.js +13 -5
- package/dist/metrics/constants.js.map +1 -1
- package/dist/metrics/index.js +1 -468
- package/dist/metrics/index.js.map +1 -1
- package/dist/multistream/mediaRequestManager.js +238 -49
- package/dist/multistream/mediaRequestManager.js.map +1 -1
- package/dist/multistream/receiveSlot.js +29 -16
- package/dist/multistream/receiveSlot.js.map +1 -1
- package/dist/multistream/receiveSlotManager.js +39 -36
- package/dist/multistream/receiveSlotManager.js.map +1 -1
- package/dist/multistream/remoteMedia.js +44 -18
- package/dist/multistream/remoteMedia.js.map +1 -1
- package/dist/multistream/remoteMediaGroup.js +60 -3
- package/dist/multistream/remoteMediaGroup.js.map +1 -1
- package/dist/multistream/remoteMediaManager.js +209 -59
- package/dist/multistream/remoteMediaManager.js.map +1 -1
- package/dist/multistream/sendSlotManager.js +233 -0
- package/dist/multistream/sendSlotManager.js.map +1 -0
- package/dist/reachability/index.js +225 -59
- package/dist/reachability/index.js.map +1 -1
- package/dist/reachability/request.js +17 -8
- package/dist/reachability/request.js.map +1 -1
- package/dist/reconnection-manager/index.js +201 -156
- package/dist/reconnection-manager/index.js.map +1 -1
- package/dist/recording-controller/index.js +21 -2
- package/dist/recording-controller/index.js.map +1 -1
- package/dist/recording-controller/util.js +9 -8
- package/dist/recording-controller/util.js.map +1 -1
- package/dist/roap/index.js +62 -32
- package/dist/roap/index.js.map +1 -1
- package/dist/roap/request.js +112 -97
- package/dist/roap/request.js.map +1 -1
- package/dist/roap/turnDiscovery.js +95 -36
- package/dist/roap/turnDiscovery.js.map +1 -1
- package/dist/rtcMetrics/constants.js +12 -0
- package/dist/rtcMetrics/constants.js.map +1 -0
- package/dist/rtcMetrics/index.js +117 -0
- package/dist/rtcMetrics/index.js.map +1 -0
- package/dist/statsAnalyzer/index.js +86 -78
- package/dist/statsAnalyzer/index.js.map +1 -1
- package/dist/statsAnalyzer/mqaUtil.js +11 -10
- package/dist/statsAnalyzer/mqaUtil.js.map +1 -1
- package/dist/types/annotation/annotation.types.d.ts +42 -0
- package/dist/types/annotation/constants.d.ts +31 -0
- package/dist/types/annotation/index.d.ts +117 -0
- package/dist/types/breakouts/edit-lock-error.d.ts +15 -0
- package/dist/types/breakouts/events.d.ts +8 -0
- package/dist/types/breakouts/utils.d.ts +14 -0
- package/dist/types/common/errors/no-meeting-info.d.ts +14 -0
- package/dist/types/common/errors/reclaim-host-role-errors.d.ts +60 -0
- package/dist/types/common/errors/webex-errors.d.ts +25 -1
- package/dist/types/common/logs/request.d.ts +2 -0
- package/dist/types/common/queue.d.ts +9 -7
- package/dist/types/config.d.ts +1 -7
- package/dist/types/constants.d.ts +194 -24
- package/dist/types/controls-options-manager/enums.d.ts +11 -1
- package/dist/types/controls-options-manager/index.d.ts +17 -1
- package/dist/types/controls-options-manager/types.d.ts +43 -0
- package/dist/types/controls-options-manager/util.d.ts +1 -7
- package/dist/types/index.d.ts +6 -4
- package/dist/types/interpretation/collection.d.ts +5 -0
- package/dist/types/interpretation/index.d.ts +5 -0
- package/dist/types/interpretation/siLanguage.d.ts +5 -0
- package/dist/types/locus-info/index.d.ts +57 -4
- package/dist/types/locus-info/parser.d.ts +67 -6
- package/dist/types/media/index.d.ts +2 -0
- package/dist/types/media/properties.d.ts +34 -48
- package/dist/types/meeting/in-meeting-actions.d.ts +82 -2
- package/dist/types/meeting/index.d.ts +463 -510
- package/dist/types/meeting/locusMediaRequest.d.ts +74 -0
- package/dist/types/meeting/muteState.d.ts +99 -23
- package/dist/types/meeting/request.d.ts +72 -43
- package/dist/types/meeting/util.d.ts +101 -1
- package/dist/types/meeting-info/index.d.ts +13 -1
- package/dist/types/meeting-info/meeting-info-v2.d.ts +31 -1
- package/dist/types/meetings/collection.d.ts +17 -0
- package/dist/types/meetings/index.d.ts +98 -20
- package/dist/types/meetings/meetings.types.d.ts +4 -0
- package/dist/types/member/index.d.ts +14 -0
- package/dist/types/member/types.d.ts +32 -0
- package/dist/types/members/collection.d.ts +5 -0
- package/dist/types/members/index.d.ts +35 -2
- package/dist/types/members/request.d.ts +73 -9
- package/dist/types/members/types.d.ts +25 -0
- package/dist/types/members/util.d.ts +214 -1
- package/dist/types/metrics/constants.d.ts +12 -4
- package/dist/types/metrics/index.d.ts +4 -119
- package/dist/types/multistream/mediaRequestManager.d.ts +73 -5
- package/dist/types/multistream/receiveSlot.d.ts +13 -11
- package/dist/types/multistream/receiveSlotManager.d.ts +14 -4
- package/dist/types/multistream/remoteMedia.d.ts +8 -29
- package/dist/types/multistream/remoteMediaGroup.d.ts +0 -9
- package/dist/types/multistream/remoteMediaManager.d.ts +46 -2
- package/dist/types/multistream/sendSlotManager.d.ts +61 -0
- package/dist/types/reachability/index.d.ts +61 -7
- package/dist/types/reachability/request.d.ts +7 -3
- package/dist/types/reconnection-manager/index.d.ts +9 -0
- package/dist/types/recording-controller/index.d.ts +15 -1
- package/dist/types/recording-controller/util.d.ts +5 -4
- package/dist/types/roap/index.d.ts +2 -1
- package/dist/types/roap/request.d.ts +15 -11
- package/dist/types/roap/turnDiscovery.d.ts +21 -3
- package/dist/types/rtcMetrics/constants.d.ts +4 -0
- package/dist/types/rtcMetrics/index.d.ts +47 -0
- package/dist/types/statsAnalyzer/index.d.ts +7 -1
- package/dist/types/webinar/collection.d.ts +16 -0
- package/dist/types/webinar/index.d.ts +5 -0
- package/dist/webinar/collection.js +44 -0
- package/dist/webinar/collection.js.map +1 -0
- package/dist/webinar/index.js +69 -0
- package/dist/webinar/index.js.map +1 -0
- package/package.json +23 -20
- package/src/annotation/annotation.types.ts +50 -0
- package/src/annotation/constants.ts +36 -0
- package/src/annotation/index.ts +328 -0
- package/src/breakouts/README.md +42 -12
- package/src/breakouts/breakout.ts +67 -9
- package/src/breakouts/edit-lock-error.ts +25 -0
- package/src/breakouts/events.ts +56 -0
- package/src/breakouts/index.ts +592 -20
- package/src/breakouts/utils.ts +42 -0
- package/src/common/errors/no-meeting-info.ts +24 -0
- package/src/common/errors/reclaim-host-role-errors.ts +134 -0
- package/src/common/errors/webex-errors.ts +44 -2
- package/src/common/logs/logger-proxy.ts +1 -1
- package/src/common/logs/request.ts +5 -1
- package/src/common/queue.ts +22 -8
- package/src/config.ts +4 -10
- package/src/constants.ts +221 -19
- package/src/controls-options-manager/enums.ts +12 -0
- package/src/controls-options-manager/index.ts +116 -21
- package/src/controls-options-manager/types.ts +59 -0
- package/src/controls-options-manager/util.ts +294 -14
- package/src/index.ts +40 -0
- package/src/interpretation/README.md +60 -0
- package/src/interpretation/collection.ts +19 -0
- package/src/interpretation/index.ts +332 -0
- package/src/interpretation/siLanguage.ts +18 -0
- package/src/locus-info/controlsUtils.ts +108 -0
- package/src/locus-info/index.ts +413 -59
- package/src/locus-info/infoUtils.ts +10 -2
- package/src/locus-info/mediaSharesUtils.ts +64 -0
- package/src/locus-info/parser.ts +258 -47
- package/src/locus-info/selfUtils.ts +81 -5
- package/src/media/index.ts +102 -122
- package/src/media/properties.ts +87 -110
- package/src/meeting/in-meeting-actions.ts +163 -3
- package/src/meeting/index.ts +3132 -2541
- package/src/meeting/locusMediaRequest.ts +313 -0
- package/src/meeting/muteState.ts +229 -131
- package/src/meeting/request.ts +177 -121
- package/src/meeting/util.ts +588 -394
- package/src/meeting-info/index.ts +81 -8
- package/src/meeting-info/meeting-info-v2.ts +170 -14
- package/src/meeting-info/util.ts +1 -1
- package/src/meeting-info/utilv2.ts +23 -23
- package/src/meetings/collection.ts +33 -0
- package/src/meetings/index.ts +445 -123
- package/src/meetings/meetings.types.ts +12 -0
- package/src/meetings/request.ts +2 -0
- package/src/meetings/util.ts +80 -11
- package/src/member/index.ts +58 -0
- package/src/member/types.ts +38 -0
- package/src/member/util.ts +141 -25
- package/src/members/collection.ts +8 -0
- package/src/members/index.ts +134 -8
- package/src/members/request.ts +97 -17
- package/src/members/types.ts +29 -0
- package/src/members/util.ts +333 -240
- package/src/metrics/constants.ts +12 -4
- package/src/metrics/index.ts +1 -490
- package/src/multistream/mediaRequestManager.ts +289 -79
- package/src/multistream/receiveSlot.ts +31 -17
- package/src/multistream/receiveSlotManager.ts +34 -24
- package/src/multistream/remoteMedia.ts +27 -2
- package/src/multistream/remoteMediaGroup.ts +59 -0
- package/src/multistream/remoteMediaManager.ts +148 -30
- package/src/multistream/sendSlotManager.ts +170 -0
- package/src/reachability/index.ts +228 -37
- package/src/reachability/request.ts +17 -8
- package/src/reconnection-manager/index.ts +83 -56
- package/src/recording-controller/index.ts +20 -3
- package/src/recording-controller/util.ts +26 -9
- package/src/roap/index.ts +63 -32
- package/src/roap/request.ts +100 -104
- package/src/roap/turnDiscovery.ts +48 -26
- package/src/rtcMetrics/constants.ts +3 -0
- package/src/rtcMetrics/index.ts +100 -0
- package/src/statsAnalyzer/index.ts +105 -91
- package/src/statsAnalyzer/mqaUtil.ts +13 -14
- package/src/webinar/collection.ts +31 -0
- package/src/webinar/index.ts +62 -0
- package/test/integration/spec/converged-space-meetings.js +60 -3
- package/test/integration/spec/journey.js +320 -261
- package/test/integration/spec/space-meeting.js +76 -3
- package/test/unit/spec/annotation/index.ts +418 -0
- package/test/unit/spec/breakouts/breakout.ts +118 -28
- package/test/unit/spec/breakouts/edit-lock-error.ts +30 -0
- package/test/unit/spec/breakouts/events.ts +89 -0
- package/test/unit/spec/breakouts/index.ts +1395 -69
- package/test/unit/spec/breakouts/utils.js +52 -1
- package/test/unit/spec/common/queue.js +31 -2
- package/test/unit/spec/controls-options-manager/index.js +163 -0
- package/test/unit/spec/controls-options-manager/util.js +576 -60
- package/test/unit/spec/fixture/locus.js +1 -0
- package/test/unit/spec/interpretation/collection.ts +15 -0
- package/test/unit/spec/interpretation/index.ts +589 -0
- package/test/unit/spec/interpretation/siLanguage.ts +28 -0
- package/test/unit/spec/locus-info/controlsUtils.js +316 -43
- package/test/unit/spec/locus-info/index.js +1304 -33
- package/test/unit/spec/locus-info/infoUtils.js +37 -15
- package/test/unit/spec/locus-info/lib/SeqCmp.json +16 -0
- package/test/unit/spec/locus-info/mediaSharesUtils.ts +32 -0
- package/test/unit/spec/locus-info/parser.js +116 -35
- package/test/unit/spec/locus-info/selfConstant.js +27 -4
- package/test/unit/spec/locus-info/selfUtils.js +208 -17
- package/test/unit/spec/media/index.ts +104 -37
- package/test/unit/spec/media/properties.ts +2 -2
- package/test/unit/spec/meeting/in-meeting-actions.ts +81 -3
- package/test/unit/spec/meeting/index.js +5216 -1956
- package/test/unit/spec/meeting/locusMediaRequest.ts +442 -0
- package/test/unit/spec/meeting/muteState.js +408 -208
- package/test/unit/spec/meeting/request.js +483 -49
- package/test/unit/spec/meeting/utils.js +679 -64
- package/test/unit/spec/meeting-info/index.js +300 -0
- package/test/unit/spec/meeting-info/meetinginfov2.js +526 -5
- package/test/unit/spec/meeting-info/utilv2.js +21 -0
- package/test/unit/spec/meetings/collection.js +26 -0
- package/test/unit/spec/meetings/index.js +1011 -205
- package/test/unit/spec/meetings/utils.js +202 -2
- package/test/unit/spec/member/index.js +61 -6
- package/test/unit/spec/member/util.js +510 -34
- package/test/unit/spec/members/index.js +432 -1
- package/test/unit/spec/members/request.js +206 -27
- package/test/unit/spec/members/utils.js +210 -0
- package/test/unit/spec/metrics/index.js +1 -50
- package/test/unit/spec/multistream/mediaRequestManager.ts +803 -162
- package/test/unit/spec/multistream/receiveSlot.ts +28 -20
- package/test/unit/spec/multistream/receiveSlotManager.ts +32 -30
- package/test/unit/spec/multistream/remoteMedia.ts +30 -0
- package/test/unit/spec/multistream/remoteMediaGroup.ts +266 -0
- package/test/unit/spec/multistream/remoteMediaManager.ts +326 -0
- package/test/unit/spec/multistream/sendSlotManager.ts +242 -0
- package/test/unit/spec/reachability/index.ts +549 -9
- package/test/unit/spec/reachability/request.js +68 -0
- package/test/unit/spec/reconnection-manager/index.js +85 -9
- package/test/unit/spec/recording-controller/index.js +294 -218
- package/test/unit/spec/recording-controller/util.js +223 -96
- package/test/unit/spec/roap/index.ts +178 -64
- package/test/unit/spec/roap/request.ts +203 -85
- package/test/unit/spec/roap/turnDiscovery.ts +82 -36
- package/test/unit/spec/rtcMetrics/index.ts +73 -0
- package/test/unit/spec/stats-analyzer/index.js +136 -2
- package/test/unit/spec/webinar/collection.ts +13 -0
- package/test/unit/spec/webinar/index.ts +60 -0
- package/test/utils/integrationTestUtils.js +46 -0
- package/test/utils/testUtils.js +0 -52
- package/dist/meeting/effectsState.js +0 -262
- package/dist/meeting/effectsState.js.map +0 -1
- package/dist/metrics/config.js +0 -299
- package/dist/metrics/config.js.map +0 -1
- package/dist/types/meeting/effectsState.d.ts +0 -42
- package/dist/types/metrics/config.d.ts +0 -178
- package/src/index.js +0 -16
- package/src/meeting/effectsState.ts +0 -211
- package/src/metrics/config.ts +0 -495
- package/test/unit/spec/meeting/effectsState.js +0 -285
|
@@ -1,8 +1,7 @@
|
|
|
1
1
|
import sinon from 'sinon';
|
|
2
2
|
import {assert} from '@webex/test-helper-chai';
|
|
3
3
|
import MeetingUtil from '@webex/plugin-meetings/src/meeting/util';
|
|
4
|
-
import {createMuteState} from '@webex/plugin-meetings/src/meeting/muteState';
|
|
5
|
-
import Media from '@webex/plugin-meetings/src/media/index';
|
|
4
|
+
import {createMuteState, MuteState} from '@webex/plugin-meetings/src/meeting/muteState';
|
|
6
5
|
import PermissionError from '@webex/plugin-meetings/src/common/errors/permission';
|
|
7
6
|
import {AUDIO, VIDEO} from '@webex/plugin-meetings/src/constants';
|
|
8
7
|
|
|
@@ -16,29 +15,44 @@ describe('plugin-meetings', () => {
|
|
|
16
15
|
|
|
17
16
|
const fakeLocus = {info: 'this is a fake locus'};
|
|
18
17
|
|
|
19
|
-
|
|
18
|
+
const createFakeLocalStream = (id, muted) => {
|
|
19
|
+
return {
|
|
20
|
+
id,
|
|
21
|
+
setServerMuted: sinon.stub(),
|
|
22
|
+
setUnmuteAllowed: sinon.stub(),
|
|
23
|
+
setMuted: sinon.stub(),
|
|
24
|
+
muted,
|
|
25
|
+
};
|
|
26
|
+
};
|
|
27
|
+
|
|
28
|
+
beforeEach(async () => {
|
|
20
29
|
meeting = {
|
|
21
30
|
mediaProperties: {
|
|
22
|
-
|
|
23
|
-
|
|
31
|
+
audioStream: createFakeLocalStream('fake audio stream', false),
|
|
32
|
+
videoStream: createFakeLocalStream('fake video stream', false),
|
|
24
33
|
},
|
|
25
34
|
remoteMuted: false,
|
|
26
35
|
unmuteAllowed: true,
|
|
36
|
+
remoteVideoMuted: false,
|
|
37
|
+
unmuteVideoAllowed: true,
|
|
38
|
+
|
|
27
39
|
locusInfo: {
|
|
28
|
-
|
|
40
|
+
handleLocusDelta: sinon.stub(),
|
|
29
41
|
},
|
|
30
42
|
members: {
|
|
31
43
|
selfId: 'fake self id',
|
|
32
44
|
muteMember: sinon.stub().resolves(),
|
|
33
45
|
},
|
|
34
46
|
};
|
|
35
|
-
audio = createMuteState(AUDIO, meeting, {sendAudio: true});
|
|
36
|
-
video = createMuteState(VIDEO, meeting, {sendVideo: true});
|
|
37
47
|
|
|
38
48
|
originalRemoteUpdateAudioVideo = MeetingUtil.remoteUpdateAudioVideo;
|
|
39
49
|
|
|
40
50
|
MeetingUtil.remoteUpdateAudioVideo = sinon.stub().resolves(fakeLocus);
|
|
41
|
-
|
|
51
|
+
|
|
52
|
+
audio = createMuteState(AUDIO, meeting, true);
|
|
53
|
+
video = createMuteState(VIDEO, meeting, true);
|
|
54
|
+
|
|
55
|
+
await testUtils.flushPromises();
|
|
42
56
|
});
|
|
43
57
|
|
|
44
58
|
afterEach(() => {
|
|
@@ -46,239 +60,242 @@ describe('plugin-meetings', () => {
|
|
|
46
60
|
});
|
|
47
61
|
|
|
48
62
|
describe('mute state library', () => {
|
|
49
|
-
it('does not create an audio instance if we are not sending audio', async () => {
|
|
50
|
-
assert.isNull(createMuteState(AUDIO, meeting, {sendAudio: false}));
|
|
51
|
-
assert.isNull(createMuteState(AUDIO, meeting, {}));
|
|
52
|
-
});
|
|
53
|
-
|
|
54
|
-
it('does not create a video instance if we are not sending video', async () => {
|
|
55
|
-
assert.isNull(createMuteState(VIDEO, meeting, {sendVideo: false}));
|
|
56
|
-
assert.isNull(createMuteState(VIDEO, meeting, {}));
|
|
57
|
-
});
|
|
58
|
-
|
|
59
63
|
it('takes into account current remote mute status when instantiated', async () => {
|
|
60
64
|
// simulate being already remote muted
|
|
61
65
|
meeting.remoteMuted = true;
|
|
62
|
-
|
|
63
|
-
|
|
66
|
+
|
|
67
|
+
// create a new MuteState instance
|
|
68
|
+
audio = createMuteState(AUDIO, meeting, true);
|
|
69
|
+
|
|
70
|
+
await testUtils.flushPromises();
|
|
64
71
|
|
|
65
72
|
assert.isTrue(audio.isMuted());
|
|
66
|
-
assert.
|
|
73
|
+
assert.isTrue(audio.isRemotelyMuted());
|
|
67
74
|
|
|
68
75
|
// now check the opposite case
|
|
69
76
|
meeting.remoteMuted = false;
|
|
70
77
|
|
|
71
|
-
// create a new MuteState
|
|
72
|
-
audio = createMuteState(AUDIO, meeting,
|
|
78
|
+
// create a new MuteState instance
|
|
79
|
+
audio = createMuteState(AUDIO, meeting, true);
|
|
73
80
|
|
|
74
|
-
|
|
75
|
-
|
|
81
|
+
await testUtils.flushPromises();
|
|
82
|
+
|
|
83
|
+
assert.isTrue(audio.isMuted()); // because we start with no stream
|
|
84
|
+
assert.isFalse(audio.isRemotelyMuted());
|
|
76
85
|
});
|
|
77
86
|
|
|
78
87
|
it('initialises correctly for video', async () => {
|
|
79
|
-
// setup fields related to
|
|
80
|
-
meeting.
|
|
81
|
-
meeting.
|
|
82
|
-
// create a new video MuteState intance
|
|
83
|
-
video = createMuteState(VIDEO, meeting, {sendVideo: true});
|
|
88
|
+
// setup fields related to video remote state
|
|
89
|
+
meeting.remoteVideoMuted = false;
|
|
90
|
+
meeting.unmuteVideoAllowed = false;
|
|
84
91
|
|
|
85
|
-
|
|
92
|
+
// create a new video MuteState instance
|
|
93
|
+
video = createMuteState(VIDEO, meeting, true);
|
|
94
|
+
|
|
95
|
+
await testUtils.flushPromises();
|
|
96
|
+
|
|
97
|
+
assert.isTrue(video.isMuted()); // because we start with no stream
|
|
98
|
+
assert.isFalse(video.isRemotelyMuted());
|
|
86
99
|
assert.isFalse(video.state.server.remoteMute);
|
|
87
|
-
assert.
|
|
100
|
+
assert.isFalse(video.state.server.unmuteAllowed);
|
|
88
101
|
});
|
|
89
102
|
|
|
90
103
|
it('takes remote mute into account when reporting current state', async () => {
|
|
91
|
-
assert.isFalse(audio.
|
|
104
|
+
assert.isFalse(audio.isRemotelyMuted());
|
|
92
105
|
|
|
93
106
|
// simulate remote mute
|
|
94
|
-
audio.handleServerRemoteMuteUpdate(true, true);
|
|
107
|
+
audio.handleServerRemoteMuteUpdate(meeting, true, true);
|
|
95
108
|
|
|
96
|
-
assert.isTrue(audio.
|
|
97
|
-
assert.isFalse(audio.isSelf());
|
|
109
|
+
assert.isTrue(audio.isRemotelyMuted());
|
|
98
110
|
});
|
|
99
111
|
|
|
100
112
|
it('does local unmute if localAudioUnmuteRequired is received', async () => {
|
|
101
|
-
// first we need to mute
|
|
102
|
-
|
|
113
|
+
// first we need to mute have the local stream muted
|
|
114
|
+
meeting.mediaProperties.audioStream.muted = true;
|
|
115
|
+
audio.handleLocalStreamChange(meeting);
|
|
103
116
|
|
|
104
117
|
assert.isTrue(audio.isMuted());
|
|
105
|
-
assert.isTrue(audio.isSelf());
|
|
106
118
|
|
|
107
119
|
MeetingUtil.remoteUpdateAudioVideo.resetHistory();
|
|
108
120
|
|
|
109
121
|
// now simulate server requiring us to locally unmute
|
|
110
122
|
audio.handleServerLocalUnmuteRequired(meeting);
|
|
123
|
+
|
|
111
124
|
await testUtils.flushPromises();
|
|
112
125
|
|
|
113
|
-
// check that local
|
|
114
|
-
assert.calledWith(
|
|
126
|
+
// check that local stream was unmuted
|
|
127
|
+
assert.calledWith(
|
|
128
|
+
meeting.mediaProperties.audioStream.setServerMuted,
|
|
129
|
+
false,
|
|
130
|
+
'localUnmuteRequired'
|
|
131
|
+
);
|
|
115
132
|
|
|
116
133
|
// and local unmute was sent to server
|
|
117
134
|
assert.calledOnce(MeetingUtil.remoteUpdateAudioVideo);
|
|
118
|
-
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, false, undefined
|
|
135
|
+
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, meeting, false, undefined);
|
|
119
136
|
|
|
120
137
|
assert.isFalse(audio.isMuted());
|
|
121
|
-
assert.isFalse(audio.isSelf());
|
|
122
138
|
});
|
|
123
139
|
|
|
124
|
-
it('
|
|
125
|
-
|
|
126
|
-
|
|
127
|
-
|
|
128
|
-
// first we need to mute and make that request last forever
|
|
129
|
-
let serverResponseResolve;
|
|
130
|
-
|
|
131
|
-
MeetingUtil.remoteUpdateAudioVideo = sinon.stub().returns(
|
|
132
|
-
new Promise((resolve) => {
|
|
133
|
-
serverResponseResolve = resolve;
|
|
134
|
-
})
|
|
135
|
-
);
|
|
140
|
+
it('does local video unmute if localVideoUnmuteRequired is received', async () => {
|
|
141
|
+
// first we need to mute
|
|
142
|
+
meeting.mediaProperties.videoStream.muted = true;
|
|
143
|
+
video.handleLocalStreamChange(meeting);
|
|
136
144
|
|
|
137
|
-
|
|
138
|
-
.handleClientRequest(meeting, true)
|
|
139
|
-
.then(() => {
|
|
140
|
-
clientPromiseResolved = true;
|
|
141
|
-
})
|
|
142
|
-
.catch(() => {
|
|
143
|
-
clientPromiseRejected = true;
|
|
144
|
-
});
|
|
145
|
+
assert.isTrue(video.isMuted());
|
|
145
146
|
|
|
146
147
|
MeetingUtil.remoteUpdateAudioVideo.resetHistory();
|
|
147
148
|
|
|
148
149
|
// now simulate server requiring us to locally unmute
|
|
149
|
-
|
|
150
|
+
video.handleServerLocalUnmuteRequired(meeting);
|
|
150
151
|
await testUtils.flushPromises();
|
|
151
152
|
|
|
152
|
-
//
|
|
153
|
-
assert.
|
|
154
|
-
|
|
155
|
-
|
|
156
|
-
|
|
157
|
-
|
|
158
|
-
await testUtils.flushPromises();
|
|
153
|
+
// check that local stream was unmuted
|
|
154
|
+
assert.calledWith(
|
|
155
|
+
meeting.mediaProperties.videoStream.setServerMuted,
|
|
156
|
+
false,
|
|
157
|
+
'localUnmuteRequired'
|
|
158
|
+
);
|
|
159
159
|
|
|
160
|
-
// local unmute
|
|
160
|
+
// and local unmute was sent to server
|
|
161
161
|
assert.calledOnce(MeetingUtil.remoteUpdateAudioVideo);
|
|
162
|
-
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo,
|
|
163
|
-
|
|
164
|
-
// and local track should be enabled
|
|
165
|
-
assert.calledWith(Media.setLocalTrack, true, meeting.mediaProperties.audioTrack);
|
|
162
|
+
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, meeting, undefined, false);
|
|
166
163
|
|
|
167
|
-
assert.isFalse(
|
|
168
|
-
assert.isFalse(audio.isSelf());
|
|
164
|
+
assert.isFalse(video.isMuted());
|
|
169
165
|
});
|
|
170
166
|
|
|
171
|
-
describe('#
|
|
172
|
-
it('
|
|
173
|
-
//
|
|
174
|
-
|
|
175
|
-
|
|
167
|
+
describe('#isLocallyMuted()', () => {
|
|
168
|
+
it('does not consider remote mute status for audio', async () => {
|
|
169
|
+
// simulate being already remote muted and locally unmuted
|
|
170
|
+
meeting.remoteMuted = true;
|
|
171
|
+
meeting.mediaProperties.audioStream.muted = false;
|
|
176
172
|
|
|
177
|
-
//
|
|
178
|
-
audio
|
|
179
|
-
|
|
173
|
+
// create a new MuteState instance
|
|
174
|
+
audio = createMuteState(AUDIO, meeting, true);
|
|
175
|
+
audio.handleLocalStreamChange(meeting);
|
|
180
176
|
|
|
181
|
-
|
|
182
|
-
audio.handleClientRequest(meeting, false);
|
|
183
|
-
assert.calledWith(Media.setLocalTrack, true, meeting.mediaProperties.audioTrack);
|
|
177
|
+
await testUtils.flushPromises();
|
|
184
178
|
|
|
185
|
-
|
|
186
|
-
audio.handleClientRequest(meeting, false);
|
|
187
|
-
assert.calledWith(Media.setLocalTrack, true, meeting.mediaProperties.audioTrack);
|
|
179
|
+
assert.isFalse(audio.isLocallyMuted());
|
|
188
180
|
});
|
|
189
181
|
|
|
190
|
-
it('
|
|
191
|
-
//
|
|
192
|
-
|
|
193
|
-
|
|
182
|
+
it('does not consider remote mute status for video', async () => {
|
|
183
|
+
// simulate being already remote muted
|
|
184
|
+
meeting.remoteVideoMuted = true;
|
|
185
|
+
meeting.mediaProperties.videoStream.muted = false;
|
|
194
186
|
|
|
195
|
-
//
|
|
196
|
-
video
|
|
197
|
-
|
|
187
|
+
// create a new MuteState instance
|
|
188
|
+
video = createMuteState(VIDEO, meeting, true);
|
|
189
|
+
video.handleLocalStreamChange(meeting);
|
|
198
190
|
|
|
199
|
-
|
|
200
|
-
video.handleClientRequest(meeting, false);
|
|
201
|
-
assert.calledWith(Media.setLocalTrack, true, meeting.mediaProperties.videoTrack);
|
|
191
|
+
await testUtils.flushPromises();
|
|
202
192
|
|
|
203
|
-
|
|
204
|
-
video.handleClientRequest(meeting, false);
|
|
205
|
-
assert.calledWith(Media.setLocalTrack, true, meeting.mediaProperties.videoTrack);
|
|
193
|
+
assert.isFalse(video.isLocallyMuted());
|
|
206
194
|
});
|
|
195
|
+
});
|
|
207
196
|
|
|
208
|
-
|
|
209
|
-
|
|
210
|
-
audio.
|
|
197
|
+
describe('handling local stream mute events', () => {
|
|
198
|
+
beforeEach(async () => {
|
|
199
|
+
audio.handleLocalStreamChange(meeting);
|
|
200
|
+
video.handleLocalStreamChange(meeting);
|
|
201
|
+
|
|
202
|
+
await testUtils.flushPromises();
|
|
203
|
+
});
|
|
211
204
|
|
|
205
|
+
const simulateAudioMuteChange = async (muteValue) => {
|
|
206
|
+
meeting.mediaProperties.audioStream.muted = muteValue;
|
|
207
|
+
audio.handleLocalStreamMuteStateChange(meeting, muteValue);
|
|
208
|
+
|
|
209
|
+
await testUtils.flushPromises();
|
|
210
|
+
};
|
|
211
|
+
|
|
212
|
+
const simulateVideoMuteChange = async (muteValue) => {
|
|
213
|
+
meeting.mediaProperties.videoStream.muted = muteValue;
|
|
214
|
+
video.handleLocalStreamMuteStateChange(meeting, muteValue);
|
|
215
|
+
|
|
216
|
+
await testUtils.flushPromises();
|
|
217
|
+
};
|
|
218
|
+
|
|
219
|
+
it('returns correct value in isMuted() methods after local stream is muted/unmuted', async () => {
|
|
220
|
+
// mute
|
|
221
|
+
await simulateAudioMuteChange(true);
|
|
212
222
|
assert.isTrue(audio.isMuted());
|
|
213
|
-
assert.isTrue(audio.isSelf());
|
|
214
223
|
|
|
215
224
|
// unmute
|
|
216
|
-
|
|
217
|
-
|
|
225
|
+
await simulateAudioMuteChange(false);
|
|
218
226
|
assert.isFalse(audio.isMuted());
|
|
219
|
-
assert.isFalse(audio.isSelf());
|
|
220
227
|
});
|
|
221
228
|
|
|
222
229
|
it('does remote unmute when unmuting and remote mute is on', async () => {
|
|
223
230
|
// simulate remote mute
|
|
224
|
-
audio.handleServerRemoteMuteUpdate(true, true);
|
|
231
|
+
audio.handleServerRemoteMuteUpdate(meeting, true, true);
|
|
225
232
|
|
|
226
233
|
// unmute
|
|
227
|
-
await
|
|
234
|
+
await simulateAudioMuteChange(false);
|
|
228
235
|
|
|
229
236
|
// check that remote unmute was sent to server
|
|
230
237
|
assert.calledOnce(meeting.members.muteMember);
|
|
231
|
-
assert.calledWith(meeting.members.muteMember, meeting.members.selfId, false);
|
|
238
|
+
assert.calledWith(meeting.members.muteMember, meeting.members.selfId, false, true);
|
|
232
239
|
|
|
233
240
|
assert.isFalse(audio.isMuted());
|
|
234
|
-
assert.isFalse(audio.isSelf());
|
|
235
241
|
});
|
|
236
242
|
|
|
237
|
-
it('
|
|
238
|
-
|
|
243
|
+
it('does video remote unmute when unmuting and remote mute is on', async () => {
|
|
244
|
+
// simulate remote mute
|
|
245
|
+
video.handleServerRemoteMuteUpdate(meeting, true, true);
|
|
239
246
|
|
|
240
|
-
|
|
247
|
+
// unmute
|
|
248
|
+
await simulateVideoMuteChange(false);
|
|
241
249
|
|
|
242
|
-
|
|
243
|
-
|
|
244
|
-
|
|
245
|
-
})
|
|
246
|
-
);
|
|
250
|
+
// check that remote unmute was sent to server
|
|
251
|
+
assert.calledOnce(meeting.members.muteMember);
|
|
252
|
+
assert.calledWith(meeting.members.muteMember, meeting.members.selfId, false, false);
|
|
247
253
|
|
|
248
|
-
|
|
249
|
-
|
|
250
|
-
});
|
|
254
|
+
assert.isFalse(video.isMuted());
|
|
255
|
+
});
|
|
251
256
|
|
|
252
|
-
|
|
253
|
-
|
|
254
|
-
|
|
257
|
+
it('does not video remote unmute when unmuting and remote mute is off', async () => {
|
|
258
|
+
// simulate remote mute
|
|
259
|
+
video.handleServerRemoteMuteUpdate(meeting, false, true);
|
|
255
260
|
|
|
256
|
-
//
|
|
257
|
-
|
|
258
|
-
|
|
261
|
+
// unmute
|
|
262
|
+
await simulateVideoMuteChange(false);
|
|
263
|
+
|
|
264
|
+
// check that remote unmute was not sent to server
|
|
265
|
+
assert.notCalled(meeting.members.muteMember);
|
|
259
266
|
|
|
260
|
-
assert.
|
|
267
|
+
assert.isFalse(video.isMuted());
|
|
261
268
|
});
|
|
262
269
|
|
|
263
|
-
it('
|
|
264
|
-
MeetingUtil.remoteUpdateAudioVideo = sinon.stub().
|
|
265
|
-
|
|
266
|
-
|
|
267
|
-
})
|
|
268
|
-
);
|
|
270
|
+
it('calls setServerMuted with "clientRequestFailed" when server request for local mute fails', async () => {
|
|
271
|
+
MeetingUtil.remoteUpdateAudioVideo = sinon.stub().rejects(new Error('fake error'));
|
|
272
|
+
|
|
273
|
+
await simulateAudioMuteChange(true);
|
|
269
274
|
|
|
270
|
-
assert.
|
|
275
|
+
assert.calledOnceWithExactly(
|
|
276
|
+
meeting.mediaProperties.audioStream.setServerMuted,
|
|
277
|
+
false,
|
|
278
|
+
'clientRequestFailed'
|
|
279
|
+
);
|
|
271
280
|
});
|
|
272
281
|
|
|
273
|
-
it('
|
|
282
|
+
it('calls setServerMuted with "clientRequestFailed" if server request for remote mute fails', async () => {
|
|
274
283
|
// we only send remote mute requests when we're unmuting, so first we need to do a remote mute
|
|
275
|
-
audio.handleServerRemoteMuteUpdate(true, true);
|
|
284
|
+
audio.handleServerRemoteMuteUpdate(meeting, true, true);
|
|
285
|
+
|
|
286
|
+
await testUtils.flushPromises();
|
|
276
287
|
|
|
277
288
|
// setup the stub to simulate server error response
|
|
278
289
|
meeting.members.muteMember = sinon.stub().rejects();
|
|
290
|
+
meeting.mediaProperties.audioStream.setServerMuted.resetHistory();
|
|
291
|
+
|
|
292
|
+
await simulateAudioMuteChange(false);
|
|
279
293
|
|
|
280
|
-
|
|
281
|
-
|
|
294
|
+
assert.calledOnceWithExactly(
|
|
295
|
+
meeting.mediaProperties.audioStream.setServerMuted,
|
|
296
|
+
true,
|
|
297
|
+
'clientRequestFailed'
|
|
298
|
+
);
|
|
282
299
|
|
|
283
300
|
// even though remote mute update in the server failed, isMuted() should still return true,
|
|
284
301
|
// because of local mute
|
|
@@ -294,12 +311,13 @@ describe('plugin-meetings', () => {
|
|
|
294
311
|
})
|
|
295
312
|
);
|
|
296
313
|
|
|
297
|
-
//
|
|
298
|
-
|
|
299
|
-
|
|
300
|
-
|
|
301
|
-
|
|
302
|
-
|
|
314
|
+
// the stream is initially unmuted
|
|
315
|
+
// simulate many mute changes with the last one matching the first one
|
|
316
|
+
await simulateAudioMuteChange(true);
|
|
317
|
+
await simulateAudioMuteChange(false);
|
|
318
|
+
await simulateAudioMuteChange(true);
|
|
319
|
+
await simulateAudioMuteChange(false);
|
|
320
|
+
await simulateAudioMuteChange(true);
|
|
303
321
|
|
|
304
322
|
// so far there should have been only 1 request to server (because our stub hasn't resolved yet
|
|
305
323
|
// and MuteState sends only 1 server request at a time)
|
|
@@ -314,7 +332,7 @@ describe('plugin-meetings', () => {
|
|
|
314
332
|
assert.notCalled(MeetingUtil.remoteUpdateAudioVideo);
|
|
315
333
|
});
|
|
316
334
|
|
|
317
|
-
it('queues up server requests when multiple
|
|
335
|
+
it('queues up server requests when multiple mute changes happen to local stream', async () => {
|
|
318
336
|
let serverResponseResolve;
|
|
319
337
|
|
|
320
338
|
MeetingUtil.remoteUpdateAudioVideo = sinon.stub().returns(
|
|
@@ -323,104 +341,286 @@ describe('plugin-meetings', () => {
|
|
|
323
341
|
})
|
|
324
342
|
);
|
|
325
343
|
|
|
326
|
-
let firstClientPromiseResolved = false;
|
|
327
|
-
let secondClientPromiseResolved = false;
|
|
328
|
-
|
|
329
344
|
// 2 client requests, one after another without waiting for first one to resolve
|
|
330
|
-
|
|
331
|
-
|
|
332
|
-
});
|
|
333
|
-
audio.handleClientRequest(meeting, false).then(() => {
|
|
334
|
-
secondClientPromiseResolved = true;
|
|
335
|
-
});
|
|
336
|
-
|
|
337
|
-
await testUtils.flushPromises();
|
|
345
|
+
await simulateAudioMuteChange(true);
|
|
346
|
+
await simulateAudioMuteChange(false);
|
|
338
347
|
|
|
339
348
|
assert.calledOnce(MeetingUtil.remoteUpdateAudioVideo);
|
|
340
|
-
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, true, undefined
|
|
349
|
+
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, meeting, true, undefined);
|
|
341
350
|
|
|
342
351
|
// now allow the first request to complete
|
|
343
352
|
serverResponseResolve();
|
|
344
353
|
await testUtils.flushPromises();
|
|
345
|
-
assert.isTrue(firstClientPromiseResolved);
|
|
346
354
|
|
|
347
355
|
// that should trigger the second server request to be sent
|
|
348
356
|
assert.calledTwice(MeetingUtil.remoteUpdateAudioVideo);
|
|
349
|
-
assert.
|
|
350
|
-
|
|
351
|
-
|
|
357
|
+
assert.deepEqual(
|
|
358
|
+
[meeting, false, undefined],
|
|
359
|
+
MeetingUtil.remoteUpdateAudioVideo.getCall(1).args
|
|
360
|
+
);
|
|
352
361
|
|
|
353
362
|
serverResponseResolve();
|
|
354
|
-
await testUtils.flushPromises();
|
|
355
|
-
|
|
356
|
-
assert.isTrue(secondClientPromiseResolved);
|
|
357
|
-
});
|
|
358
|
-
|
|
359
|
-
it('rejects client request to unmute if hard mute is used', (done) => {
|
|
360
|
-
audio.handleServerRemoteMuteUpdate(true, false);
|
|
361
|
-
|
|
362
|
-
audio
|
|
363
|
-
.handleClientRequest(meeting, false)
|
|
364
|
-
.then(() => {
|
|
365
|
-
done(new Error('expected handleClientRequest to fail, but it did not!'));
|
|
366
|
-
})
|
|
367
|
-
.catch((e) => {
|
|
368
|
-
assert.isTrue(e instanceof PermissionError);
|
|
369
|
-
done();
|
|
370
|
-
});
|
|
371
363
|
});
|
|
372
364
|
|
|
373
365
|
it('does not send remote mute for video', async () => {
|
|
374
366
|
// mute
|
|
375
|
-
await
|
|
367
|
+
await simulateVideoMuteChange(true);
|
|
376
368
|
|
|
377
369
|
assert.isTrue(video.isMuted());
|
|
378
|
-
assert.isTrue(video.isSelf());
|
|
379
370
|
|
|
380
371
|
// check local mute is done, but not remote one
|
|
381
|
-
assert.calledWith(
|
|
382
|
-
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, undefined, true, meeting);
|
|
372
|
+
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, meeting, undefined, true);
|
|
383
373
|
assert.notCalled(meeting.members.muteMember);
|
|
384
374
|
|
|
385
|
-
Media.setLocalTrack.resetHistory();
|
|
386
375
|
MeetingUtil.remoteUpdateAudioVideo.resetHistory();
|
|
387
376
|
meeting.members.muteMember.resetHistory();
|
|
388
377
|
|
|
389
378
|
// unmute
|
|
390
|
-
await
|
|
379
|
+
await simulateVideoMuteChange(false);
|
|
391
380
|
|
|
392
381
|
assert.isFalse(video.isMuted());
|
|
393
|
-
assert.isFalse(video.isSelf());
|
|
394
382
|
|
|
395
|
-
assert.calledWith(
|
|
396
|
-
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, undefined, false, meeting);
|
|
383
|
+
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, meeting, undefined, false);
|
|
397
384
|
assert.notCalled(meeting.members.muteMember);
|
|
398
385
|
});
|
|
399
386
|
|
|
400
|
-
it('sends
|
|
387
|
+
it('sends undefined value for the other media type when sending local mute', async () => {
|
|
401
388
|
// make sure the meeting object has mute state machines for both audio and video
|
|
402
389
|
meeting.audio = audio;
|
|
403
390
|
meeting.video = video;
|
|
404
391
|
|
|
405
|
-
// mute audio ->
|
|
406
|
-
await
|
|
407
|
-
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo,
|
|
392
|
+
// mute audio -> the call to remoteUpdateAudioVideo should have video undefined
|
|
393
|
+
await simulateAudioMuteChange(true);
|
|
394
|
+
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, meeting, true, undefined);
|
|
408
395
|
MeetingUtil.remoteUpdateAudioVideo.resetHistory();
|
|
409
396
|
|
|
410
|
-
// now mute video ->
|
|
411
|
-
await
|
|
412
|
-
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo,
|
|
397
|
+
// now mute video -> the call to remoteUpdateAudioVideo should have unmute for video and undefined for audio
|
|
398
|
+
await simulateVideoMuteChange(true);
|
|
399
|
+
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, meeting, undefined, true);
|
|
413
400
|
MeetingUtil.remoteUpdateAudioVideo.resetHistory();
|
|
414
401
|
|
|
415
|
-
// now unmute the audio ->
|
|
416
|
-
await
|
|
417
|
-
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo,
|
|
402
|
+
// now unmute the audio -> the call to remoteUpdateAudioVideo should have video undefined
|
|
403
|
+
await simulateAudioMuteChange(false);
|
|
404
|
+
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, meeting, false, undefined);
|
|
418
405
|
MeetingUtil.remoteUpdateAudioVideo.resetHistory();
|
|
419
406
|
|
|
420
|
-
// unmute video ->
|
|
421
|
-
await
|
|
422
|
-
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo,
|
|
407
|
+
// unmute video -> the call to remoteUpdateAudioVideo should have audio undefined
|
|
408
|
+
await simulateVideoMuteChange(false);
|
|
409
|
+
assert.calledWith(MeetingUtil.remoteUpdateAudioVideo, meeting, undefined, false);
|
|
423
410
|
});
|
|
424
411
|
});
|
|
412
|
+
|
|
413
|
+
describe('#init, #handleLocalStreamChange', () => {
|
|
414
|
+
let meeting;
|
|
415
|
+
let muteState;
|
|
416
|
+
let setServerMutedSpy;
|
|
417
|
+
let setMutedSpy, setUnmuteAllowedSpy;
|
|
418
|
+
|
|
419
|
+
const setupMeeting = (
|
|
420
|
+
mediaType,
|
|
421
|
+
remoteMuted = false,
|
|
422
|
+
muted = false,
|
|
423
|
+
defineStreams = true
|
|
424
|
+
) => {
|
|
425
|
+
const remoteMuteField = mediaType === AUDIO ? 'remoteMuted' : 'remoteVideoMuted';
|
|
426
|
+
|
|
427
|
+
meeting = {
|
|
428
|
+
mediaProperties: {
|
|
429
|
+
audioStream: defineStreams
|
|
430
|
+
? createFakeLocalStream('fake audio stream', muted)
|
|
431
|
+
: undefined,
|
|
432
|
+
videoStream: defineStreams
|
|
433
|
+
? createFakeLocalStream('fake video stream', muted)
|
|
434
|
+
: undefined,
|
|
435
|
+
},
|
|
436
|
+
[remoteMuteField]: remoteMuted,
|
|
437
|
+
unmuteAllowed: true,
|
|
438
|
+
unmuteVideoAllowed: true,
|
|
439
|
+
|
|
440
|
+
locusInfo: {
|
|
441
|
+
onFullLocus: sinon.stub(),
|
|
442
|
+
},
|
|
443
|
+
members: {
|
|
444
|
+
selfId: 'fake self id',
|
|
445
|
+
muteMember: sinon.stub().resolves(),
|
|
446
|
+
},
|
|
447
|
+
};
|
|
448
|
+
};
|
|
449
|
+
|
|
450
|
+
const setup = async (mediaType, remoteMuted = false, muted = false, defineStreams = true) => {
|
|
451
|
+
setupMeeting(mediaType, remoteMuted, muted, defineStreams);
|
|
452
|
+
|
|
453
|
+
muteState = createMuteState(mediaType, meeting, true);
|
|
454
|
+
muteState.handleLocalStreamChange(meeting);
|
|
455
|
+
|
|
456
|
+
await testUtils.flushPromises();
|
|
457
|
+
|
|
458
|
+
MeetingUtil.remoteUpdateAudioVideo.resetHistory();
|
|
459
|
+
};
|
|
460
|
+
|
|
461
|
+
const setupSpies = (mediaType) => {
|
|
462
|
+
setUnmuteAllowedSpy =
|
|
463
|
+
mediaType === AUDIO
|
|
464
|
+
? meeting.mediaProperties.audioStream?.setUnmuteAllowed
|
|
465
|
+
: meeting.mediaProperties.videoStream?.setUnmuteAllowed;
|
|
466
|
+
setServerMutedSpy =
|
|
467
|
+
mediaType === AUDIO
|
|
468
|
+
? meeting.mediaProperties.audioStream?.setServerMuted
|
|
469
|
+
: meeting.mediaProperties.videoStream?.setServerMuted;
|
|
470
|
+
setMutedSpy =
|
|
471
|
+
mediaType === AUDIO
|
|
472
|
+
? meeting.mediaProperties.audioStream?.setMuted
|
|
473
|
+
: meeting.mediaProperties.videoStream?.setMuted;
|
|
474
|
+
|
|
475
|
+
clearSpies();
|
|
476
|
+
};
|
|
477
|
+
|
|
478
|
+
const clearSpies = () => {
|
|
479
|
+
setUnmuteAllowedSpy?.resetHistory();
|
|
480
|
+
setServerMutedSpy?.resetHistory();
|
|
481
|
+
setMutedSpy?.resetHistory();
|
|
482
|
+
};
|
|
483
|
+
const tests = [
|
|
484
|
+
{mediaType: AUDIO, title: 'audio'},
|
|
485
|
+
{mediaType: VIDEO, title: 'video'},
|
|
486
|
+
];
|
|
487
|
+
|
|
488
|
+
tests.forEach(({mediaType, title}) =>
|
|
489
|
+
describe(title, () => {
|
|
490
|
+
let originalRemoteUpdateAudioVideo;
|
|
491
|
+
|
|
492
|
+
beforeEach(() => {
|
|
493
|
+
originalRemoteUpdateAudioVideo = MeetingUtil.remoteUpdateAudioVideo;
|
|
494
|
+
MeetingUtil.remoteUpdateAudioVideo = sinon.stub().resolves({info: 'fake locus'});
|
|
495
|
+
});
|
|
496
|
+
|
|
497
|
+
afterEach(() => {
|
|
498
|
+
MeetingUtil.remoteUpdateAudioVideo = originalRemoteUpdateAudioVideo;
|
|
499
|
+
sinon.restore();
|
|
500
|
+
});
|
|
501
|
+
|
|
502
|
+
describe('#handleLocalStreamChange', () => {
|
|
503
|
+
it('calls init()', async () => {
|
|
504
|
+
await setup(mediaType);
|
|
505
|
+
const spy = sinon.spy(muteState, 'init');
|
|
506
|
+
muteState.handleLocalStreamChange(meeting);
|
|
507
|
+
assert.calledOnceWithExactly(spy, meeting);
|
|
508
|
+
});
|
|
509
|
+
});
|
|
510
|
+
|
|
511
|
+
describe('#init', () => {
|
|
512
|
+
// does the setup by calling new MuteState() so that MuteState.init() doesn't get called
|
|
513
|
+
const setupWithoutInit = async (
|
|
514
|
+
mediaType,
|
|
515
|
+
remoteMuted = false,
|
|
516
|
+
muted = false,
|
|
517
|
+
defineStreams = true
|
|
518
|
+
) => {
|
|
519
|
+
setupMeeting(mediaType, remoteMuted, muted, defineStreams);
|
|
520
|
+
|
|
521
|
+
muteState = new MuteState(mediaType, meeting, true);
|
|
522
|
+
};
|
|
523
|
+
|
|
524
|
+
it('nothing goes bad when stream is undefined', async () => {
|
|
525
|
+
await setupWithoutInit(mediaType, false, false, false);
|
|
526
|
+
setupSpies(mediaType);
|
|
527
|
+
|
|
528
|
+
muteState.init(meeting);
|
|
529
|
+
|
|
530
|
+
assert.isTrue(muteState.state.client.localMute);
|
|
531
|
+
});
|
|
532
|
+
|
|
533
|
+
it('tests when stream muted is true and remoteMuted is false', async () => {
|
|
534
|
+
await setupWithoutInit(mediaType, false, true);
|
|
535
|
+
setupSpies(mediaType);
|
|
536
|
+
|
|
537
|
+
muteState.init(meeting);
|
|
538
|
+
|
|
539
|
+
assert.calledWith(setUnmuteAllowedSpy, muteState.state.server.unmuteAllowed);
|
|
540
|
+
assert.notCalled(setServerMutedSpy);
|
|
541
|
+
assert.notCalled(MeetingUtil.remoteUpdateAudioVideo);
|
|
542
|
+
assert.isTrue(muteState.state.client.localMute);
|
|
543
|
+
});
|
|
544
|
+
|
|
545
|
+
it('tests when stream muted is false and remoteMuted is false', async () => {
|
|
546
|
+
await setupWithoutInit(mediaType, false, false);
|
|
547
|
+
setupSpies(mediaType);
|
|
548
|
+
|
|
549
|
+
muteState.init(meeting);
|
|
550
|
+
|
|
551
|
+
assert.calledWith(setUnmuteAllowedSpy, muteState.state.server.unmuteAllowed);
|
|
552
|
+
assert.notCalled(setServerMutedSpy);
|
|
553
|
+
assert.calledOnce(MeetingUtil.remoteUpdateAudioVideo);
|
|
554
|
+
assert.isFalse(muteState.state.client.localMute);
|
|
555
|
+
});
|
|
556
|
+
|
|
557
|
+
it('tests when remoteMuted is true', async () => {
|
|
558
|
+
// testing that muteLocalStream is called
|
|
559
|
+
await setupWithoutInit(mediaType, true);
|
|
560
|
+
setupSpies(mediaType);
|
|
561
|
+
|
|
562
|
+
muteState.init(meeting);
|
|
563
|
+
|
|
564
|
+
assert.calledWith(setUnmuteAllowedSpy, muteState.state.server.unmuteAllowed);
|
|
565
|
+
assert.calledOnceWithExactly(setServerMutedSpy, true, 'remotelyMuted');
|
|
566
|
+
});
|
|
567
|
+
});
|
|
568
|
+
|
|
569
|
+
describe('#handleLocalStreamMuteStateChange', () => {
|
|
570
|
+
it('checks when ignoreMuteStateChange is true nothing changes', async () => {
|
|
571
|
+
await setup(mediaType, false, false);
|
|
572
|
+
muteState.ignoreMuteStateChange = true;
|
|
573
|
+
|
|
574
|
+
muteState.handleLocalStreamMuteStateChange(meeting, true);
|
|
575
|
+
assert.notCalled(MeetingUtil.remoteUpdateAudioVideo);
|
|
576
|
+
|
|
577
|
+
assert.isFalse(muteState.state.client.localMute);
|
|
578
|
+
});
|
|
579
|
+
|
|
580
|
+
it('tests localMute - true to false', async () => {
|
|
581
|
+
await setup(mediaType, false, true);
|
|
582
|
+
|
|
583
|
+
muteState.handleLocalStreamMuteStateChange(meeting, false);
|
|
584
|
+
assert.equal(muteState.state.client.localMute, false);
|
|
585
|
+
assert.called(MeetingUtil.remoteUpdateAudioVideo);
|
|
586
|
+
});
|
|
587
|
+
|
|
588
|
+
it('tests localMute - false to true', async () => {
|
|
589
|
+
await setup(mediaType, false, false);
|
|
590
|
+
|
|
591
|
+
muteState.handleLocalStreamMuteStateChange(meeting, true);
|
|
592
|
+
assert.equal(muteState.state.client.localMute, true);
|
|
593
|
+
assert.called(MeetingUtil.remoteUpdateAudioVideo);
|
|
594
|
+
});
|
|
595
|
+
});
|
|
596
|
+
|
|
597
|
+
describe('#applyClientStateLocally', () => {
|
|
598
|
+
afterEach(() => {
|
|
599
|
+
sinon.restore();
|
|
600
|
+
});
|
|
601
|
+
|
|
602
|
+
it('calls setServerMuted on the stream', async () => {
|
|
603
|
+
await setup(mediaType);
|
|
604
|
+
setupSpies(mediaType);
|
|
605
|
+
|
|
606
|
+
muteState.applyClientStateLocally(meeting, 'somereason');
|
|
607
|
+
assert.calledOnceWithExactly(
|
|
608
|
+
setServerMutedSpy,
|
|
609
|
+
muteState.state.client.localMute,
|
|
610
|
+
'somereason'
|
|
611
|
+
);
|
|
612
|
+
assert.notCalled(setMutedSpy);
|
|
613
|
+
});
|
|
614
|
+
|
|
615
|
+
it('nothing explodes when streams are undefined', async () => {
|
|
616
|
+
await setup(mediaType, false, false, false);
|
|
617
|
+
setupSpies(mediaType);
|
|
618
|
+
|
|
619
|
+
muteState.applyClientStateLocally(meeting, 'somereason');
|
|
620
|
+
});
|
|
621
|
+
});
|
|
622
|
+
})
|
|
623
|
+
);
|
|
624
|
+
});
|
|
425
625
|
});
|
|
426
626
|
});
|