@contrail/flexplm 1.1.50 → 1.1.52

package/lib/publish/base-process-publish-assortment.d.ts

@@ -18,10 +18,16 @@ export declare class BaseProcessPublishAssortment {
     process(event: any): Promise<{
         results: {
             message: any;
+            event: any;
         };
         skip_await: boolean;
     } | {
         results: any;
+        event: {
+            assortmentId: string;
+            assortmentPublishChangeId: string;
+        };
+        resultsCount: any;
         skip_await?: undefined;
     }>;
     getPublishInfo(assortmentId: string, assortmentPublishChangeId: string, apcHistory: any, publisher: any): Promise<any>;
@@ -50,12 +56,24 @@ export declare class BaseProcessPublishAssortment {
     processPublish(pcd: PublishChangeData, changeDetail: any, fullChange: any, deleteChanges: any): Promise<{
         results: {
             message: string;
+            event: {
+                assortmentId: string;
+                assortmentPublishChangeId: string;
+            };
         };
         skip_await: boolean;
+        event?: undefined;
+        resultsCount?: undefined;
     } | {
         results: any;
+        event: {
+            assortmentId: string;
+            assortmentPublishChangeId: string;
+        };
+        resultsCount: any;
         skip_await?: undefined;
     }>;
+    getResultsCount(events: any): any;
     private sendEvents;
     private sendToFlexPLM;
     private saveToLocalFile;

package/lib/publish/base-process-publish-assortment.js

@@ -10,9 +10,17 @@ const type_conversion_utils_1 = require("../util/type-conversion-utils");
 const fsPromise = require("fs/promises");
 const path = require("path");
 const app_framework_1 = require("@contrail/app-framework");
+var EventStatuses;
+(function (EventStatuses) {
+    EventStatuses["NOT_PUBLISHABLE"] = "Not_Publishable";
+    EventStatuses["NO_FEDERATION_INFO"] = "No_Federation_Information";
+    EventStatuses["NO_EVENTS_TO_SEND"] = "No_Events_to_Send";
+    EventStatuses["FAILURE"] = "Failure";
+    EventStatuses["SUCCESS"] = "Success";
+})(EventStatuses || (EventStatuses = {}));
 class BaseProcessPublishAssortment {
     constructor(_config, _dc, _mapFileUtil) {
-        this.TTL = 7 * 24 * 60 * 60 * 1000;
+        this.TTL = 30 * 24 * 60 * 60 * 1000;
         this.cache = {
             carriedFromSeason: {}
         };
@@ -34,9 +42,15 @@ class BaseProcessPublishAssortment {
             }
             catch (e) {
                 const message = e.message;
+                const eventStatus = (message.includes(BaseProcessPublishAssortment.ASSORTMENT_NOT_PUBLISHABLE)) ? EventStatuses.NOT_PUBLISHABLE :
+                    (message.startsWith(BaseProcessPublishAssortment.ASSORTMENT_NO_FED_INFO)) ? EventStatuses.NO_FEDERATION_INFO :
+                        EventStatuses.FAILURE;
+                const processStatus = 'BaseProcessPublishAssortment: ' + eventStatus
+                    + ': assortmentId: ' + event.assortmentId + ': assortmentPublishChangeId: ' + event.assortmentPublishChangeId;
+                console.log(JSON.stringify(processStatus));
                 console.log(message);
                 const output = {
-                    results: { message },
+                    results: { message, event },
                     skip_await: true
                 };
                 return output;
@@ -502,20 +516,44 @@ class BaseProcessPublishAssortment {
     }
     async processPublish(pcd, changeDetail, fullChange, deleteChanges) {
         console.info('processPublish-start');
+        const event = {
+            assortmentId: pcd.assortmentId,
+            assortmentPublishChangeId: pcd.assortmentPublishChangeId
+        };
         const assortmentItemFullChangeMap = this.getFullChangeAssortmentMap(fullChange);
         const assortmentItemDeleteMap = this.getDeleteChangesAssortmentMap(deleteChanges);
         pcd.itemFamilyChanges = this.getItemFamilyChanges(pcd, changeDetail, assortmentItemFullChangeMap, assortmentItemDeleteMap);
         const events = await this.getEventsForPublishChangeData(pcd);
         if (events.length === 0) {
+            const processStatus = 'BaseProcessPublishAssortment: ' + EventStatuses.NO_EVENTS_TO_SEND
+                + ': assortmentId: ' + pcd.assortmentId + ': assortmentPublishChangeId: ' + pcd.assortmentPublishChangeId;
+            console.log(processStatus);
             const message = 'No Events to Send; returning';
             console.log(message);
             return {
-                results: { message },
+                results: { message, event },
                 skip_await: true
             };
         }
         const results = await this.sendEvents(events);
-        return { results };
+        const resultsCount = this.getResultsCount(events);
+        const processStatus = 'BaseProcessPublishAssortment: ' + EventStatuses.NO_EVENTS_TO_SEND
+            + ': assortmentId: ' + pcd.assortmentId + ': assortmentPublishChangeId: ' + pcd.assortmentPublishChangeId
+            + ': resultsCount: ' + JSON.stringify(resultsCount);
+        console.log(processStatus);
+        return { results, event, resultsCount };
+    }
+    getResultsCount(events) {
+        return events.reduce((acc, event) => {
+            const objectClass = event?.objectClass;
+            if (objectClass === 'LCSProductSeasonLink') {
+                acc.productSeasons++;
+            }
+            else if (objectClass === 'LCSSKUSeasonLink') {
+                acc.colorwaySeasons++;
+            }
+            return acc;
+        }, { productSeasons: 0, colorwaySeasons: 0 });
     }
     async sendEvents(events) {
         console.info('sendEvents()');

package/lib/util/data-converter.d.ts

@@ -9,6 +9,10 @@ export declare class DataConverter {
     private verboseDebug;
     private objRefCache;
     private userRefCache;
+    static staticUserCache: {};
+    static clearStaticUserCache(): void;
+    static getFromStaticCache(id: string): any;
+    static setToStaticCache(id: string, value: any): void;
     constructor(config: FCConfig, mapFileUtil: MapFileUtil);
     setVerboseDebug(val?: boolean): void;
     isVerboseDebugOn(): boolean;

package/lib/util/data-converter.js

@@ -8,6 +8,15 @@ const util_1 = require("@contrail/util");
 const type_conversion_utils_1 = require("./type-conversion-utils");
 const map_utils_1 = require("./map-utils");
 class DataConverter {
+    static clearStaticUserCache() {
+        DataConverter.staticUserCache = {};
+    }
+    static getFromStaticCache(id) {
+        return DataConverter.staticUserCache[id];
+    }
+    static setToStaticCache(id, value) {
+        DataConverter.staticUserCache[id] = value;
+    }
     constructor(config, mapFileUtil) {
         this.config = config;
         this.mapFileUtil = mapFileUtil;
@@ -460,17 +469,18 @@ class DataConverter {
         if (!nd?.email) {
             return "";
         }
-        if (this.userRefCache[nd.email]) {
+        let cacheUser = DataConverter.getFromStaticCache(nd.email);
+        if (cacheUser) {
             if (app_framework_1.Logger.isDebugOn()) {
                 console.debug('user cache hit: ' + nd.email);
             }
             await this.processGroupMemberCheck(prop, nd.email);
-            return this.userRefCache[nd.email];
+            return cacheUser;
         }
         const user = await this.getUserByEmail(nd);
         const userId = (user) ? user.id : undefined;
         if (userId) {
-            this.userRefCache[nd.email] = userId;
+            DataConverter.setToStaticCache(nd.email, userId);
             await this.processGroupMemberCheck(prop, nd.email);
         }
         return userId;
@@ -522,11 +532,12 @@ class DataConverter {
         if (!entityId) {
             return {};
         }
-        if (this.userRefCache[entityId]) {
+        const cacheUser = DataConverter.getFromStaticCache(entityId);
+        if (cacheUser) {
             if (app_framework_1.Logger.isDebugOn()) {
                 console.debug('user cache hit: ' + entityId);
             }
-            return this.userRefCache[entityId];
+            return Object.assign({}, cacheUser);
         }
         const user = await this.getUserById(entityId);
         const value = (user) ? {
@@ -536,7 +547,7 @@ class DataConverter {
             isSsoUser: user.isSsoUser,
         } : undefined;
         if (value) {
-            this.userRefCache[entityId] = value;
+            DataConverter.setToStaticCache(entityId, value);
         }
         return value;
     }
@@ -557,3 +568,4 @@ class DataConverter {
     }
 }
 exports.DataConverter = DataConverter;
+DataConverter.staticUserCache = {};

package/lib/util/data-converter.spec.js

@@ -652,3 +652,200 @@ describe('checkKeysAndValues', () => {
         expect(results.length).toBe(1);
     });
 });
+describe('setUserListValue', () => {
+    const config = {
+        apiHost: 'host',
+        userName: () => 'user',
+        password: () => 'pass',
+        urlContext: 'xxx',
+        vibeEventEndpoint: '/rfa/vibeiq/vibeEvents',
+        csrfEndpoint: '/servlet/rest/security/csrf',
+        itemPreDevelopmentLifecycleStages: ['concept']
+    };
+    const userListProp1 = {};
+    const userEmailMapping = [
+        {
+            id: "gxgr46kvdeEOn4cH",
+            email: "jessica.smith@vibeiq.com"
+        },
+        {
+            id: "Rl5D0sjuAwelScEK",
+            email: "adam.smith@vibeiq.com"
+        }
+    ];
+    const mapFileUtil = new transform_data_1.MapFileUtil(new sdk_1.Entities());
+    const dc = new data_converter_1.DataConverter(config, mapFileUtil);
+    let spyGetUserByEmail = jest.spyOn(dc, 'getUserByEmail')
+        .mockImplementation(async (nd) => {
+        let emaildInput = nd?.email.toLowerCase();
+        return userEmailMapping.find((user) => user.email === emaildInput);
+    });
+    let spyProcessGroupMemberCheck = jest.spyOn(dc, 'processGroupMemberCheck')
+        .mockImplementation(async (nd) => { return; });
+    afterEach(() => {
+        spyGetUserByEmail.mockClear();
+        spyProcessGroupMemberCheck.mockClear();
+        data_converter_1.DataConverter.clearStaticUserCache();
+    });
+    afterAll(() => {
+        spyGetUserByEmail.mockRestore();
+        spyProcessGroupMemberCheck.mockRestore();
+    });
+    it('no value', async () => {
+        const newData = {};
+        const returnValue = await dc.setUserListValue(userListProp1, newData);
+        expect(returnValue).toEqual('');
+    });
+    it('miss cache', async () => {
+        const newData = { email: 'adam.smith@vibeiq.com' };
+        const returnValue = await dc.setUserListValue(userListProp1, newData);
+        console.log('returnValue', JSON.stringify(returnValue));
+        expect(returnValue).toEqual('Rl5D0sjuAwelScEK');
+        expect(spyGetUserByEmail).toHaveBeenCalledTimes(1);
+        expect(spyProcessGroupMemberCheck).toHaveBeenCalledTimes(1);
+    });
+    it('miss cache - two different emails', async () => {
+        let newData = { email: 'adam.smith@vibeiq.com' };
+        let returnValue = await dc.setUserListValue(userListProp1, newData);
+        expect(returnValue).toEqual('Rl5D0sjuAwelScEK');
+        expect(spyGetUserByEmail).toHaveBeenCalledTimes(1);
+        expect(spyProcessGroupMemberCheck).toHaveBeenCalledTimes(1);
+        newData = { email: 'jessica.smith@vibeiq.com' };
+        returnValue = await dc.setUserListValue(userListProp1, newData);
+        expect(spyGetUserByEmail).toHaveBeenCalledTimes(2);
+        expect(spyProcessGroupMemberCheck).toHaveBeenCalledTimes(2);
+    });
+    it('hit cache same email', async () => {
+        const newData = { email: 'adam.smith@vibeiq.com' };
+        let returnValue = await dc.setUserListValue(userListProp1, newData);
+        expect(returnValue).toEqual('Rl5D0sjuAwelScEK');
+        expect(spyGetUserByEmail).toHaveBeenCalledTimes(1);
+        expect(spyProcessGroupMemberCheck).toHaveBeenCalledTimes(1);
+        returnValue = await dc.setUserListValue(userListProp1, newData);
+        expect(returnValue).toEqual('Rl5D0sjuAwelScEK');
+        expect(spyGetUserByEmail).toHaveBeenCalledTimes(1);
+        expect(spyProcessGroupMemberCheck).toHaveBeenCalledTimes(2);
+    });
+    it('hit cache same email - new DataConverter entity', async () => {
+        const newData = { email: 'adam.smith@vibeiq.com' };
+        let returnValue = await dc.setUserListValue(userListProp1, newData);
+        expect(returnValue).toEqual('Rl5D0sjuAwelScEK');
+        expect(spyGetUserByEmail).toHaveBeenCalledTimes(1);
+        expect(spyProcessGroupMemberCheck).toHaveBeenCalledTimes(1);
+        const dc2 = new data_converter_1.DataConverter(config, mapFileUtil);
+        let spyGetUserByEmail2 = jest.spyOn(dc2, 'getUserByEmail')
+            .mockImplementation(async (nd) => {
+            let emaildInput = nd?.email.toLowerCase();
+            return userEmailMapping.find((user) => user.email === emaildInput);
+        });
+        let spyProcessGroupMemberCheck2 = jest.spyOn(dc2, 'processGroupMemberCheck')
+            .mockImplementation(async (nd) => { return; });
+        returnValue = await dc2.setUserListValue(userListProp1, newData);
+        expect(returnValue).toEqual('Rl5D0sjuAwelScEK');
+        expect(spyGetUserByEmail2).toHaveBeenCalledTimes(0);
+        expect(spyProcessGroupMemberCheck2).toHaveBeenCalledTimes(1);
+        spyGetUserByEmail2.mockRestore();
+        spyProcessGroupMemberCheck2.mockRestore();
+    });
+});
+describe('getUserListValue', () => {
+    const config = {
+        apiHost: 'host',
+        userName: () => 'user',
+        password: () => 'pass',
+        urlContext: 'xxx',
+        vibeEventEndpoint: '/rfa/vibeiq/vibeEvents',
+        csrfEndpoint: '/servlet/rest/security/csrf',
+        itemPreDevelopmentLifecycleStages: ['concept']
+    };
+    const userListProp1 = {
+        slug: 'userList1'
+    };
+    const userEmailMapping = [
+        {
+            id: "gxgr46kvdeEOn4cH",
+            email: "jessica.smith@vibeiq.com",
+            first: 'Jessica',
+            last: 'Smith'
+        },
+        {
+            id: "Rl5D0sjuAwelScEK",
+            email: "adam.smith@vibeiq.com",
+            first: 'Adam',
+            last: 'Smith'
+        }
+    ];
+    const mapFileUtil = new transform_data_1.MapFileUtil(new sdk_1.Entities());
+    const dc = new data_converter_1.DataConverter(config, mapFileUtil);
+    let spyGetUserById = jest.spyOn(dc, 'getUserById')
+        .mockImplementation(async (nd) => {
+        return userEmailMapping.find((user) => user.id === nd);
+    });
+    afterEach(() => {
+        spyGetUserById.mockClear();
+        data_converter_1.DataConverter.clearStaticUserCache();
+    });
+    afterAll(() => {
+        spyGetUserById.mockRestore();
+    });
+    it('no value', async () => {
+        const newData = {};
+        const returnValue = await dc.getUserListValue(userListProp1, newData);
+        expect(returnValue).toEqual({});
+    });
+    it('miss cache', async () => {
+        const newData = {
+            userList1Id: 'gxgr46kvdeEOn4cH'
+        };
+        const returnValue = await dc.getUserListValue(userListProp1, newData);
+        expect(spyGetUserById).toHaveBeenCalledTimes(1);
+        expect(returnValue.email).toEqual('jessica.smith@vibeiq.com');
+        expect(returnValue.firstName).toEqual('Jessica');
+        expect(returnValue.lastName).toEqual('Smith');
+    });
+    it('miss cache - two different emails', async () => {
+        let newData = { userList1Id: 'gxgr46kvdeEOn4cH' };
+        let returnValue = await dc.getUserListValue(userListProp1, newData);
+        expect(spyGetUserById).toHaveBeenCalledTimes(1);
+        expect(returnValue.email).toEqual('jessica.smith@vibeiq.com');
+        expect(returnValue.firstName).toEqual('Jessica');
+        expect(returnValue.lastName).toEqual('Smith');
+        newData = { userList1Id: 'Rl5D0sjuAwelScEK' };
+        returnValue = await dc.getUserListValue(userListProp1, newData);
+        expect(spyGetUserById).toHaveBeenCalledTimes(2);
+        expect(returnValue.email).toEqual('adam.smith@vibeiq.com');
+        expect(returnValue.firstName).toEqual('Adam');
+        expect(returnValue.lastName).toEqual('Smith');
+    });
+    it('hit cache same email', async () => {
+        const newData = { userList1Id: 'gxgr46kvdeEOn4cH' };
+        let returnValue = await dc.getUserListValue(userListProp1, newData);
+        expect(spyGetUserById).toHaveBeenCalledTimes(1);
+        expect(returnValue.email).toEqual('jessica.smith@vibeiq.com');
+        expect(returnValue.firstName).toEqual('Jessica');
+        expect(returnValue.lastName).toEqual('Smith');
+        returnValue = await dc.getUserListValue(userListProp1, newData);
+        expect(spyGetUserById).toHaveBeenCalledTimes(1);
+        expect(returnValue.email).toEqual('jessica.smith@vibeiq.com');
+        expect(returnValue.firstName).toEqual('Jessica');
+        expect(returnValue.lastName).toEqual('Smith');
+    });
+    it('hit cache same email - new DataConverter entity', async () => {
+        const newData = { userList1Id: 'gxgr46kvdeEOn4cH' };
+        let returnValue = await dc.getUserListValue(userListProp1, newData);
+        expect(spyGetUserById).toHaveBeenCalledTimes(1);
+        expect(returnValue.email).toEqual('jessica.smith@vibeiq.com');
+        expect(returnValue.firstName).toEqual('Jessica');
+        expect(returnValue.lastName).toEqual('Smith');
+        const dc2 = new data_converter_1.DataConverter(config, mapFileUtil);
+        let spyGetUserById2 = jest.spyOn(dc2, 'getUserById')
+            .mockImplementation(async (nd) => {
+            return userEmailMapping.find((user) => user.id === nd);
+        });
+        returnValue = await dc2.getUserListValue(userListProp1, newData);
+        expect(spyGetUserById2).toHaveBeenCalledTimes(0);
+        expect(returnValue.email).toEqual('jessica.smith@vibeiq.com');
+        expect(returnValue.firstName).toEqual('Jessica');
+        expect(returnValue.lastName).toEqual('Smith');
+    });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contrail/flexplm",
-  "version": "1.1.50",
+  "version": "1.1.52",
   "description": "Library used for integration with flexplm.",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",

package/src/publish/base-process-publish-assortment.ts

@@ -12,9 +12,17 @@ import path = require('path');
 import { MapFileUtil } from '@contrail/transform-data';
 import { Logger } from '@contrail/app-framework';
 
+enum EventStatuses {
+  NOT_PUBLISHABLE = 'Not_Publishable',
+  NO_FEDERATION_INFO = 'No_Federation_Information',
+  NO_EVENTS_TO_SEND = 'No_Events_to_Send',
+  FAILURE = 'Failure',
+  SUCCESS = 'Success',
+}
+
 export class BaseProcessPublishAssortment {
 
-  private TTL = 7 * 24 * 60 * 60 * 1_000; // 1 week
+  private TTL = 30 * 24 * 60 * 60 * 1_000; // 30 days
   static ASSORTMENT_NOT_PUBLISHABLE = 'Assortment isn\'t marked for publishing to FlexPLM';
   static ASSORTMENT_NO_FED_INFO = 'Assortment doesn\'t have all "identifier" properties, so there is no processing. Identifier properties: ';
   static NOT_ABLE_TO_PROCESS_DELETE_CHANGES = 'Error: Not able to process delete changes';
@@ -47,9 +55,16 @@ export class BaseProcessPublishAssortment {
         seasonFed = await this.getSeasonFederation(assortmentId);
       } catch (e) {
         const message = e.message;
+        const eventStatus =
+        (message.includes(BaseProcessPublishAssortment.ASSORTMENT_NOT_PUBLISHABLE)) ? EventStatuses.NOT_PUBLISHABLE :
+        (message.startsWith(BaseProcessPublishAssortment.ASSORTMENT_NO_FED_INFO)) ? EventStatuses.NO_FEDERATION_INFO :
+        EventStatuses.FAILURE;
+        const processStatus = 'BaseProcessPublishAssortment: ' + eventStatus
+        + ': assortmentId: ' + event.assortmentId + ': assortmentPublishChangeId: ' + event.assortmentPublishChangeId;
+        console.log(JSON.stringify(processStatus));
         console.log(message);
         const output = {
-          results: { message },
+          results: { message, event },
           skip_await: true
         };
         return output;
@@ -607,6 +622,10 @@ export class BaseProcessPublishAssortment {
   }
   async processPublish(pcd: PublishChangeData, changeDetail, fullChange, deleteChanges) {
     console.info('processPublish-start');
+    const event = {
+      assortmentId: pcd.assortmentId,
+      assortmentPublishChangeId: pcd.assortmentPublishChangeId
+    };
 
     const assortmentItemFullChangeMap = this.getFullChangeAssortmentMap(fullChange);
     const assortmentItemDeleteMap = this.getDeleteChangesAssortmentMap(deleteChanges);
@@ -614,16 +633,39 @@ export class BaseProcessPublishAssortment {
 
     const events = await this.getEventsForPublishChangeData(pcd);
     if (events.length === 0) {
-      const message = 'No Events to Send; returning';
-      console.log(message);
-      return {
-        results: { message },
-        skip_await: true
-      };
+      const processStatus = 'BaseProcessPublishAssortment: ' + EventStatuses.NO_EVENTS_TO_SEND
+      + ': assortmentId: ' + pcd.assortmentId + ': assortmentPublishChangeId: ' + pcd.assortmentPublishChangeId;
+     console.log(processStatus);
+     const message = 'No Events to Send; returning';
+     console.log(message);
+     return {
+         results: { message, event },
+         skip_await: true
+     };
+
     }
 
     const results = await this.sendEvents(events);
-    return { results };
+    const resultsCount = this.getResultsCount(events);
+    const processStatus = 'BaseProcessPublishAssortment: ' + EventStatuses.NO_EVENTS_TO_SEND
+    + ': assortmentId: ' + pcd.assortmentId + ': assortmentPublishChangeId: ' + pcd.assortmentPublishChangeId
+     + ': resultsCount: ' + JSON.stringify(resultsCount);
+   console.log(processStatus);
+    return { results, event, resultsCount };
+
+  }
+
+  getResultsCount(events) {
+    return events.reduce((acc, event) => {
+        const objectClass = event?.objectClass;
+        if (objectClass === 'LCSProductSeasonLink') {
+            acc.productSeasons++;
+        }
+        else if (objectClass === 'LCSSKUSeasonLink') {
+            acc.colorwaySeasons++;
+        }
+        return acc;
+    }, { productSeasons: 0, colorwaySeasons: 0 });
   }
 
   private async sendEvents(events: SeasonalPayload[]): Promise<any> {

package/src/util/data-converter.spec.ts

@@ -392,7 +392,7 @@ describe('getEnumerationValue multi_select', () => {
     ],
   };
 
-  
+
   it('multi_select_null', async () => {
     const nd = null;
     const returnValue = await dc.getEnumerationValue(enumAtt, nd);
@@ -940,4 +940,241 @@ describe('checkKeysAndValues', () =>{
     expect(results.length).toBe(1);
   });
 
+});
+
+describe('setUserListValue', () =>{
+  const config: FCConfig = {
+    apiHost: 'host',
+    userName: () => 'user',
+    password: () => 'pass',
+    urlContext: 'xxx',
+    vibeEventEndpoint: '/rfa/vibeiq/vibeEvents',
+    csrfEndpoint: '/servlet/rest/security/csrf',
+    itemPreDevelopmentLifecycleStages: ['concept']
+  };
+  const userListProp1 = {};
+  const userEmailMapping = [
+    {
+      id: "gxgr46kvdeEOn4cH",
+      email: "jessica.smith@vibeiq.com"
+    },
+
+    {
+      id: "Rl5D0sjuAwelScEK",
+      email: "adam.smith@vibeiq.com"
+    }
+  ];
+
+  const mapFileUtil = new MapFileUtil(new Entities());
+  const dc = new DataConverter(config, mapFileUtil);
+  let spyGetUserByEmail = jest.spyOn(dc, 'getUserByEmail')
+  .mockImplementation(async (nd) => {
+    let emaildInput = nd?.email.toLowerCase();
+    return userEmailMapping.find((user) => user.email === emaildInput);
+  });
+  let spyProcessGroupMemberCheck = jest.spyOn(dc, 'processGroupMemberCheck')
+  .mockImplementation(async (nd) => { return; });
+
+  afterEach(() => {
+    spyGetUserByEmail.mockClear();
+    spyProcessGroupMemberCheck.mockClear();
+    DataConverter.clearStaticUserCache();
+  });
+  afterAll(() => {
+    spyGetUserByEmail.mockRestore();
+    spyProcessGroupMemberCheck.mockRestore();
+  });
+
+  it('no value', async () =>{
+    const newData = {};
+    const returnValue = await dc.setUserListValue(userListProp1, newData);
+
+    expect(returnValue).toEqual('');
+  });
+
+  it('miss cache', async () =>{
+    const newData = {email: 'adam.smith@vibeiq.com'};
+    const returnValue = await dc.setUserListValue(userListProp1, newData);
+    console.log('returnValue', JSON.stringify(returnValue));
+    expect(returnValue).toEqual('Rl5D0sjuAwelScEK');
+    expect(spyGetUserByEmail).toHaveBeenCalledTimes(1);
+    expect(spyProcessGroupMemberCheck).toHaveBeenCalledTimes(1);
+  });
+
+  it('miss cache - two different emails', async () =>{
+    let newData = {email: 'adam.smith@vibeiq.com'};
+    let returnValue = await dc.setUserListValue(userListProp1, newData);
+    expect(returnValue).toEqual('Rl5D0sjuAwelScEK');
+    expect(spyGetUserByEmail).toHaveBeenCalledTimes(1);
+    expect(spyProcessGroupMemberCheck).toHaveBeenCalledTimes(1);
+
+    newData = {email: 'jessica.smith@vibeiq.com'};
+    returnValue = await dc.setUserListValue(userListProp1, newData);
+    expect(spyGetUserByEmail).toHaveBeenCalledTimes(2);
+    expect(spyProcessGroupMemberCheck).toHaveBeenCalledTimes(2);
+  });
+
+  it('hit cache same email', async () =>{
+    const newData = {email: 'adam.smith@vibeiq.com'};
+    let returnValue = await dc.setUserListValue(userListProp1, newData);
+    expect(returnValue).toEqual('Rl5D0sjuAwelScEK');
+    expect(spyGetUserByEmail).toHaveBeenCalledTimes(1);
+    expect(spyProcessGroupMemberCheck).toHaveBeenCalledTimes(1);
+
+    returnValue = await dc.setUserListValue(userListProp1, newData);
+    expect(returnValue).toEqual('Rl5D0sjuAwelScEK');
+    expect(spyGetUserByEmail).toHaveBeenCalledTimes(1);
+    expect(spyProcessGroupMemberCheck).toHaveBeenCalledTimes(2);
+  });
+
+  it('hit cache same email - new DataConverter entity', async () =>{
+    const newData = {email: 'adam.smith@vibeiq.com'};
+    let returnValue = await dc.setUserListValue(userListProp1, newData);
+    expect(returnValue).toEqual('Rl5D0sjuAwelScEK');
+    expect(spyGetUserByEmail).toHaveBeenCalledTimes(1);
+    expect(spyProcessGroupMemberCheck).toHaveBeenCalledTimes(1);
+
+    const dc2 = new DataConverter(config, mapFileUtil);
+    let spyGetUserByEmail2 = jest.spyOn(dc2, 'getUserByEmail')
+    .mockImplementation(async (nd) => {
+      let emaildInput = nd?.email.toLowerCase();
+      return userEmailMapping.find((user) => user.email === emaildInput);
+    });
+    let spyProcessGroupMemberCheck2 = jest.spyOn(dc2, 'processGroupMemberCheck')
+    .mockImplementation(async (nd) => { return; });
+
+    returnValue = await dc2.setUserListValue(userListProp1, newData);
+    expect(returnValue).toEqual('Rl5D0sjuAwelScEK');
+    expect(spyGetUserByEmail2).toHaveBeenCalledTimes(0);
+    expect(spyProcessGroupMemberCheck2).toHaveBeenCalledTimes(1);
+
+    spyGetUserByEmail2.mockRestore();
+    spyProcessGroupMemberCheck2.mockRestore();
+  });
+
+});
+
+describe('getUserListValue', () =>{
+  const config: FCConfig = {
+    apiHost: 'host',
+    userName: () => 'user',
+    password: () => 'pass',
+    urlContext: 'xxx',
+    vibeEventEndpoint: '/rfa/vibeiq/vibeEvents',
+    csrfEndpoint: '/servlet/rest/security/csrf',
+    itemPreDevelopmentLifecycleStages: ['concept']
+  };
+  const userListProp1 = {
+    slug: 'userList1'
+  };
+  const userEmailMapping = [
+    {
+      id: "gxgr46kvdeEOn4cH",
+      email: "jessica.smith@vibeiq.com",
+      first: 'Jessica',
+      last: 'Smith'
+    },
+
+    {
+      id: "Rl5D0sjuAwelScEK",
+      email: "adam.smith@vibeiq.com",
+      first: 'Adam',
+      last: 'Smith'
+    }
+  ];
+
+  const mapFileUtil = new MapFileUtil(new Entities());
+  const dc = new DataConverter(config, mapFileUtil);
+
+  //getUserById
+  let spyGetUserById = jest.spyOn(dc, 'getUserById')
+  .mockImplementation(async (nd) => {
+    return userEmailMapping.find((user) => user.id === nd);
+  });
+
+  afterEach(() => {
+    spyGetUserById.mockClear();
+    DataConverter.clearStaticUserCache();
+  });
+  afterAll(() => {
+    spyGetUserById.mockRestore();
+  });
+
+  it('no value', async () =>{
+    const newData = {};
+    const returnValue = await dc.getUserListValue(userListProp1, newData);
+
+    expect(returnValue).toEqual({});
+  });
+
+  it('miss cache', async () =>{
+    const newData = {
+      userList1Id: 'gxgr46kvdeEOn4cH'
+    };
+
+    const returnValue = await dc.getUserListValue(userListProp1, newData);
+
+    expect(spyGetUserById).toHaveBeenCalledTimes(1);
+    expect(returnValue.email).toEqual('jessica.smith@vibeiq.com');
+    expect(returnValue.firstName).toEqual('Jessica');
+    expect(returnValue.lastName).toEqual('Smith');
+
+  });
+
+  it('miss cache - two different emails', async () =>{
+    let newData = { userList1Id: 'gxgr46kvdeEOn4cH'};
+    let returnValue = await dc.getUserListValue(userListProp1, newData);
+
+    expect(spyGetUserById).toHaveBeenCalledTimes(1);
+    expect(returnValue.email).toEqual('jessica.smith@vibeiq.com');
+    expect(returnValue.firstName).toEqual('Jessica');
+    expect(returnValue.lastName).toEqual('Smith');
+
+    newData = { userList1Id: 'Rl5D0sjuAwelScEK' };
+    returnValue = await dc.getUserListValue(userListProp1, newData);
+
+    expect(spyGetUserById).toHaveBeenCalledTimes(2);
+    expect(returnValue.email).toEqual('adam.smith@vibeiq.com');
+    expect(returnValue.firstName).toEqual('Adam');
+    expect(returnValue.lastName).toEqual('Smith');
+  });
+
+  it('hit cache same email', async () =>{
+    const newData = { userList1Id: 'gxgr46kvdeEOn4cH'};
+    let returnValue = await dc.getUserListValue(userListProp1, newData);
+
+    expect(spyGetUserById).toHaveBeenCalledTimes(1);
+    expect(returnValue.email).toEqual('jessica.smith@vibeiq.com');
+    expect(returnValue.firstName).toEqual('Jessica');
+    expect(returnValue.lastName).toEqual('Smith');
+
+    returnValue = await dc.getUserListValue(userListProp1, newData);
+    expect(spyGetUserById).toHaveBeenCalledTimes(1);
+    expect(returnValue.email).toEqual('jessica.smith@vibeiq.com');
+    expect(returnValue.firstName).toEqual('Jessica');
+    expect(returnValue.lastName).toEqual('Smith');
+  });
+
+  it('hit cache same email - new DataConverter entity', async () =>{
+    const newData = { userList1Id: 'gxgr46kvdeEOn4cH'};
+    let returnValue = await dc.getUserListValue(userListProp1, newData);
+
+    expect(spyGetUserById).toHaveBeenCalledTimes(1);
+    expect(returnValue.email).toEqual('jessica.smith@vibeiq.com');
+    expect(returnValue.firstName).toEqual('Jessica');
+    expect(returnValue.lastName).toEqual('Smith');
+
+    const dc2 = new DataConverter(config, mapFileUtil);
+    let spyGetUserById2 = jest.spyOn(dc2, 'getUserById')
+    .mockImplementation(async (nd) => {
+      return userEmailMapping.find((user) => user.id === nd);
+    });
+
+    returnValue = await dc2.getUserListValue(userListProp1, newData);
+    expect(spyGetUserById2).toHaveBeenCalledTimes(0);
+    expect(returnValue.email).toEqual('jessica.smith@vibeiq.com');
+    expect(returnValue.firstName).toEqual('Jessica');
+    expect(returnValue.lastName).toEqual('Smith');
+  });
+
 });

package/src/util/data-converter.ts

@@ -14,6 +14,17 @@ export class DataConverter {
   private verboseDebug = false;
   private objRefCache = {};
   private userRefCache = {};
+  static staticUserCache = {};
+  static clearStaticUserCache() {
+    DataConverter.staticUserCache = {};
+  }
+  static getFromStaticCache(id: string) {
+    return DataConverter.staticUserCache[id];
+  }
+  static setToStaticCache(id: string, value: any) {
+    DataConverter.staticUserCache[id] = value;
+  }
+
   constructor(private config: FCConfig, private mapFileUtil: MapFileUtil){
     this.typeUtils = new TypeUtils();
     this.transformMapFile = this.config['transformMapFile'];
@@ -575,21 +586,21 @@ export class DataConverter {
     if(!nd?.email) {
         return "";
     }    
-
-    if (this.userRefCache[nd.email]) {
+    let cacheUser = DataConverter.getFromStaticCache(nd.email);
+    if (cacheUser) {
         if (Logger.isDebugOn()) {
             console.debug('user cache hit: ' + nd.email);
         }
 
         await this.processGroupMemberCheck(prop, nd.email);
-        return this.userRefCache[nd.email];
+        return cacheUser;
     }
 
     const user = await this.getUserByEmail(nd);
     const userId = (user)?user.id:undefined;
 
     if(userId) {
-      this.userRefCache[nd.email] = userId;
+      DataConverter.setToStaticCache(nd.email, userId);
       await this.processGroupMemberCheck(prop, nd.email);
     }
 
@@ -671,11 +682,12 @@ export class DataConverter {
           return {};
       }
 
-      if (this.userRefCache[entityId]) {
+      const cacheUser = DataConverter.getFromStaticCache(entityId);
+      if (cacheUser) {
           if (Logger.isDebugOn()) {
               console.debug('user cache hit: ' + entityId);
           }
-          return this.userRefCache[entityId];
+          return Object.assign({}, cacheUser);
       }
 
       const user = await this.getUserById(entityId);
@@ -688,7 +700,7 @@ export class DataConverter {
       } : undefined;
 
       if(value) {
-          this.userRefCache[entityId] = value;
+          DataConverter.setToStaticCache(entityId, value);
       }
 
       return value;