@contrail/flexplm 1.7.1-alpha.a7e0d3c → 1.7.1

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

@@ -183,7 +183,13 @@ 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 === 0 || firstColonIndex === entityReference.length - 1) {
+            throw new Error('Invalid identity entityReference: ' + entityReference);
+        }
         const entityName = entityReference.slice(0, firstColonIndex);
         const id = entityReference.slice(firstColonIndex + 1);
         const entity = await this.entities.get({

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

@@ -617,5 +617,50 @@ describe('BaseEntityProcessor', () => {
                 id: 'e5g8ugusNWOuCvdw:nju12Jw2jGFvcQSf'
             });
         });
+        it('should throw error when entityReference has no colon', async () => {
+            mockEntitiesGet.mockResolvedValue([{ entityReference: 'item1' }]);
+            await expect(btep.getEntityUsingIdentityService({
+                poolKey: 'item',
+                propertyName: 'itemNumber',
+                propertyValue: '12345'
+            })).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 () => {
+            mockEntitiesGet.mockResolvedValue([{ entityReference: ':1' }]);
+            await expect(btep.getEntityUsingIdentityService({
+                poolKey: 'item',
+                propertyName: 'itemNumber',
+                propertyValue: '12345'
+            })).rejects.toThrow('Invalid identity entityReference: :1');
+            expect(mockEntitiesGet).toHaveBeenCalledTimes(1);
+        });
+        it('should throw error when entityReference ends with a colon (missing id)', async () => {
+            mockEntitiesGet.mockResolvedValue([{ entityReference: 'item:' }]);
+            await expect(btep.getEntityUsingIdentityService({
+                poolKey: 'item',
+                propertyName: 'itemNumber',
+                propertyValue: '12345'
+            })).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.a7e0d3c",
+  "version": "1.7.1",
   "description": "Library used for integration with flexplm.",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",