fix(locus): parse header and convert to ISO

Author: chrisadubois

packages/@webex/plugin-meetings/src/meeting/index.ts

@@ -1689,13 +1689,27 @@ export default class Meeting extends StatelessWebexPlugin {
   /**
    * Setter - sets isoLocalClientMeetingJoinTime
    * This will be set once on meeting join, and not updated again
-   * @param {string | undefined} time in ISO format
+   * this will always produce an ISO string
+   * If the iso string is invalid, it will fallback to the current system time
+   * @param {string | undefined} time
    */
   set isoLocalClientMeetingJoinTime(time: string | undefined) {
+    const fallback = new Date().toISOString();
     if (!time) {
-      this.#isoLocalClientMeetingJoinTime = new Date().toISOString();
+      this.#isoLocalClientMeetingJoinTime = fallback;
     } else {
-      this.#isoLocalClientMeetingJoinTime = time;
+      const date = new Date(time);
+
+      // Check if the date is valid
+      if (Number.isNaN(date.getTime())) {
+        LoggerProxy.logger.info(
+          // @ts-ignore
+          `Meeting:index#isoLocalClientMeetingJoinTime --> Invalid date provided: ${time}. Falling back to system clock.`
+        );
+        this.#isoLocalClientMeetingJoinTime = fallback;
+      } else {
+        this.#isoLocalClientMeetingJoinTime = date.toISOString();
+      }
     }
   }
 

packages/@webex/plugin-meetings/test/unit/spec/meeting/index.js

@@ -7527,22 +7527,24 @@ describe('plugin-meetings', () => {
         });
       });
 
-      describe('#setIsoLocalClientMeetingJoinTime', () => {
-        it('should set the isoLocalClientMeetingJoinTime when passed in', () => {
-          assert.equal(meeting.isoLocalClientMeetingJoinTime, isoLocalClientMeetingJoinTime);
-          meeting.isoLocalClientMeetingJoinTime = 'test';
-          assert.equal(meeting.isoLocalClientMeetingJoinTime, 'test');
-          meeting.isoLocalClientMeetingJoinTime = 'test2';
-          assert.equal(meeting.isoLocalClientMeetingJoinTime, 'test2');
-        });
-
-        it('should set the isoLocalClientMeetingJoin time once and only once when not passed in', () => {
-          assert.equal(meeting.isoLocalClientMeetingJoinTime, isoLocalClientMeetingJoinTime);
+      describe('#setIsoLocalClientMeetingJoinTime', () => {      
+        it('should fallback to system clock ISO string when given an undefined value', () => {
+          const currentSystemTime = new Date().toISOString();
           meeting.isoLocalClientMeetingJoinTime = undefined;
-          const time = meeting.isoLocalClientMeetingJoinTime;
-          assert.equal(meeting.isoLocalClientMeetingJoinTime, time);
-          meeting.isoLocalClientMeetingJoinTime = 'test2';
-          assert.equal(meeting.isoLocalClientMeetingJoinTime, 'test2');
+          assert.equal(meeting.isoLocalClientMeetingJoinTime, currentSystemTime);
+        });
+      
+        it('should fallback to system clock ISO string when given an invalid value', () => {
+          const currentSystemTime = new Date().toISOString();
+          meeting.isoLocalClientMeetingJoinTime = 'invalid-date';
+          assert.equal(meeting.isoLocalClientMeetingJoinTime, currentSystemTime);
+        });
+      
+        it('should set the isoLocalClientMeetingJoinTime correctly for a valid date string', () => {
+          const validDateString = 'Tue, 01 Apr 2025 13:00:36 GMT';
+          const expectedISOString = new Date(validDateString).toISOString();
+          meeting.isoLocalClientMeetingJoinTime = validDateString;
+          assert.equal(meeting.isoLocalClientMeetingJoinTime, expectedISOString);
         });
       });