@webex/plugin-meetings 3.10.0-next.13 → 3.10.0-next.15

package/test/unit/spec/hashTree/utils.ts

@@ -0,0 +1,103 @@
+import {deleteNestedObjectsWithHtMeta} from '../../../../src/hashTree/utils';
+
+import {assert} from '@webex/test-helper-chai';
+
+describe('Hash Tree Utils', () => {
+  describe('#deleteNestedObjectsWithHtMeta', () => {
+    it('should delete nested objects with htMeta', () => {
+      const locusPart = {
+        a: {
+          htMeta: {
+            id: '1',
+          },
+          value: 'to be deleted',
+        },
+        b: {
+          c: {
+            htMeta: {
+              id: '2',
+            },
+            value: 'to be deleted',
+          },
+          d: 'to be kept',
+        },
+        e: [
+          {
+            htMeta: {
+              id: '3',
+            },
+            value: 'to be deleted',
+          },
+          {
+            f: 'to be kept',
+          },
+          {
+            htMeta: {
+              id: '4',
+            },
+            value: 'to be deleted',
+          },
+          {
+            g: 'to be kept',
+          },
+        ],
+      };
+
+      deleteNestedObjectsWithHtMeta(locusPart);
+
+      assert.deepEqual(locusPart, {
+        b: {
+          d: 'to be kept',
+        },
+        e: [
+          {
+            f: 'to be kept',
+          },
+          {
+            g: 'to be kept',
+          },
+        ],
+      });
+    });
+
+    it('should handle arrays correctly', () => {
+      const locusPart = {
+        htMeta: {
+          id: '0', // this should not be deleted
+        },
+        participants: [
+          {
+            htMeta: {
+              id: '1',
+            },
+            id: 'participant1',
+            value: 'to be deleted',
+          },
+          {
+            htMeta: {
+              id: '2',
+            },
+            id: 'participant2',
+            value: 'to be deleted',
+          },
+        ],
+        self: {
+          htMeta: {
+            id: '3',
+          },
+          id: 'self1',
+          value: 'to be deleted',
+        },
+      };
+
+      deleteNestedObjectsWithHtMeta(locusPart);
+
+      assert.deepEqual(locusPart, {
+        htMeta: {
+          id: '0',
+        },
+        participants: [],
+      });
+    });
+  });
+});

package/test/unit/spec/locus-info/index.js

@@ -358,11 +358,47 @@ describe('plugin-meetings', () => {
           });
         });
 
+        it('should process locus update correctly when called with updated fullState', () => {
+          const newFullState = {
+            id: 'new-fullState',
+            visibleDataSets: ['dataset1', 'dataset2'],
+          };
+
+          // simulate an update from the HashTreeParser (normally this would be triggered by incoming locus messages)
+          locusInfoUpdateCallback(OBJECTS_UPDATED, {
+            updatedObjects: [{htMeta: {elementId: {type: 'fullState'}}, data: newFullState}],
+          });
+
+          // check onDeltaLocus() was called with correctly updated locus info
+          assert.calledOnceWithExactly(onDeltaLocusStub, {
+            ...expectedLocusInfo,
+            fullState: newFullState,
+          });
+        });
+
+        it('should process locus update correctly when called with updated info', () => {
+          const newInfo = {
+            id: 'new-info',
+            visibleDataSets: ['dataset1', 'dataset2'],
+          };
+
+          // simulate an update from the HashTreeParser (normally this would be triggered by incoming locus messages)
+          locusInfoUpdateCallback(OBJECTS_UPDATED, {
+            updatedObjects: [{htMeta: {elementId: {type: 'info'}}, data: newInfo}],
+          });
+
+          // check onDeltaLocus() was called with correctly updated locus info
+          assert.calledOnceWithExactly(onDeltaLocusStub, {
+            ...expectedLocusInfo,
+            info: newInfo,
+          });
+        });
+
         it('should process locus update correctly when called with updated LOCUS object', () => {
           // setup new updated locus that has many things missing
           const newLocusHtMeta = {elementId: {type: 'locus', version: 42}};
           const newLocus = {
-            conntrols: 'new-controls',
+            controls: 'new-controls',
             host: 'new-host',
             htMeta: newLocusHtMeta,
           };
@@ -374,6 +410,12 @@ describe('plugin-meetings', () => {
 
           // check onDeltaLocus() was called with correctly updated locus info
           assert.calledOnceWithExactly(onDeltaLocusStub, {
+            // these fields are not part of Locus object, so should keep their old values:
+            info: {id: 'fake-info'},
+            fullState: {id: 'fake-full-state'},
+            self: {id: 'fake-self'},
+            mediaShares: expectedLocusInfo.mediaShares,
+            // and now the new fields
             ...newLocus,
             htMeta: newLocusHtMeta,
             participants: [], // empty means there were no participant updates
@@ -381,6 +423,50 @@ describe('plugin-meetings', () => {
           });
         });
 
+        // this test is checking that we cope with an edge case if Locus
+        // sends us something that they shouldn't
+        it('should process locus update correctly when called with updated LOCUS object that contains info/fullState/self/participants etc', () => {
+          // setup new updated locus that has many things missing
+          const newLocusHtMeta = {elementId: {type: 'locus', version: 42}};
+          const newLocus = {
+            controls: 'new-controls',
+            host: 'new-host',
+            htMeta: newLocusHtMeta,
+          };
+
+          // simulate an update from the HashTreeParser (normally this would be triggered by incoming locus messages)
+          locusInfoUpdateCallback(OBJECTS_UPDATED, {
+            updatedObjects: [
+              {
+                htMeta: newLocusHtMeta,
+                data: {
+                  ...newLocus,
+                  // all these fields below should be ignored and not override the existing ones in our "old" Locus
+                  info: 'new-info',
+                  fullState: 'new-fullState',
+                  self: 'new-self',
+                  participants: 'new-participants',
+                  mediaShares: 'new-mediaShares',
+                },
+              },
+            ],
+          });
+
+          // check onDeltaLocus() was called with correctly updated locus info
+          // with old values for the fields that should be ignored (like "info" or "fullState")
+          assert.calledOnceWithExactly(onDeltaLocusStub, {
+            // these fields have the "old" values:
+            info: {id: 'fake-info'},
+            fullState: {id: 'fake-full-state'},
+            self: {id: 'fake-self'},
+            mediaShares: expectedLocusInfo.mediaShares,
+            participants: [], // empty means there were no participant updates
+            jsSdkMeta: {removedParticipantIds: []}, // no participants were removed
+            ...newLocus,
+            htMeta: newLocusHtMeta,
+          });
+        });
+
         it('should process locus update correctly when called with removed LOCUS object followed by updated LOCUS object', () => {
           // setup new updated locus that has many things missing
           const newLocusHtMeta = {elementId: {type: 'locus', version: 99}};
@@ -402,6 +488,12 @@ describe('plugin-meetings', () => {
 
           // check onDeltaLocus() was called with correctly updated locus info
           assert.calledOnceWithExactly(onDeltaLocusStub, {
+            // these fields are not part of Locus object, so should keep their old values:
+            info: {id: 'fake-info'},
+            fullState: {id: 'fake-full-state'},
+            self: {id: 'fake-self'},
+            mediaShares: expectedLocusInfo.mediaShares,
+            // and now the new fields
             ...newLocus,
             htMeta: newLocusHtMeta,
             participants: [], // empty means there were no participant updates
@@ -2578,7 +2670,7 @@ describe('plugin-meetings', () => {
 
         assert.calledWith(locusInfo.handleLocusDelta, fakeLocus, mockMeeting);
       });
-      it('does nothing when we are using hash trees', () => {
+      it('calls hash tree parser when we are using hash trees', () => {
         const fakeLocus = {eventType: LOCUSEVENT.DIFFERENCE};
         const fakeDataSets = [{name: 'dataset1', url: 'http://test.com'}];
         const responseBody = {locus: fakeLocus, dataSets: fakeDataSets};
@@ -2593,8 +2685,7 @@ describe('plugin-meetings', () => {
 
         locusInfo.handleLocusAPIResponse(mockMeeting, responseBody);
 
-        assert.notCalled(mockHashTreeParser.handleLocusUpdate);
-        assert.notCalled(locusInfo.onDeltaLocus);
+        assert.calledOnceWithExactly(mockHashTreeParser.handleLocusUpdate, responseBody);
       });
     });