@contrail/flexplm 1.7.1-alpha.e5225c7 → 1.7.1

package/lib/entity-processor/base-entity-processor.js

@@ -183,9 +183,12 @@ class BaseEntityProcessor {
             return undefined;
         }
         const entityReference = identityEntity.entityReference;
+        if (typeof entityReference !== 'string' || !entityReference.includes(':')) {
+            throw new Error(`Invalid identity entityReference: ${String(entityReference)}`);
+        }
         const firstColonIndex = entityReference.indexOf(':');
-        if (firstColonIndex === -1 || firstColonIndex === 0 || firstColonIndex === entityReference.length - 1) {
-            throw new Error('Invalid entityReference: ' + entityReference);
+        if (firstColonIndex === 0 || firstColonIndex === entityReference.length - 1) {
+            throw new Error('Invalid identity entityReference: ' + entityReference);
         }
         const entityName = entityReference.slice(0, firstColonIndex);
         const id = entityReference.slice(firstColonIndex + 1);

package/lib/entity-processor/base-entity-processor.spec.js

@@ -623,7 +623,25 @@ describe('BaseEntityProcessor', () => {
                 poolKey: 'item',
                 propertyName: 'itemNumber',
                 propertyValue: '12345'
-            })).rejects.toThrow('Invalid entityReference: item1');
+            })).rejects.toThrow('Invalid identity entityReference: item1');
+            expect(mockEntitiesGet).toHaveBeenCalledTimes(1);
+        });
+        it('should throw error when entityReference is not a string', async () => {
+            mockEntitiesGet.mockResolvedValue([{ entityReference: 12345 }]);
+            await expect(btep.getEntityUsingIdentityService({
+                poolKey: 'item',
+                propertyName: 'itemNumber',
+                propertyValue: '12345'
+            })).rejects.toThrow('Invalid identity entityReference: 12345');
+            expect(mockEntitiesGet).toHaveBeenCalledTimes(1);
+        });
+        it('should throw error when entityReference is undefined', async () => {
+            mockEntitiesGet.mockResolvedValue([{}]);
+            await expect(btep.getEntityUsingIdentityService({
+                poolKey: 'item',
+                propertyName: 'itemNumber',
+                propertyValue: '12345'
+            })).rejects.toThrow('Invalid identity entityReference: undefined');
             expect(mockEntitiesGet).toHaveBeenCalledTimes(1);
         });
         it('should throw error when entityReference starts with a colon (missing entityName)', async () => {
@@ -632,7 +650,7 @@ describe('BaseEntityProcessor', () => {
                 poolKey: 'item',
                 propertyName: 'itemNumber',
                 propertyValue: '12345'
-            })).rejects.toThrow('Invalid entityReference: :1');
+            })).rejects.toThrow('Invalid identity entityReference: :1');
             expect(mockEntitiesGet).toHaveBeenCalledTimes(1);
         });
         it('should throw error when entityReference ends with a colon (missing id)', async () => {
@@ -641,7 +659,7 @@ describe('BaseEntityProcessor', () => {
                 poolKey: 'item',
                 propertyName: 'itemNumber',
                 propertyValue: '12345'
-            })).rejects.toThrow('Invalid entityReference: item:');
+            })).rejects.toThrow('Invalid identity entityReference: item:');
             expect(mockEntitiesGet).toHaveBeenCalledTimes(1);
         });
     });

package/package.json

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