@contrail/flexplm 1.7.1-alpha.09c0de6 → 1.7.1-alpha.261418a

package/lib/util/thumbnail-util.js

@@ -136,8 +136,8 @@ class ThumbnailUtil {
         const eventData = event.data || {};
         const newThumbnailId = eventData[ThumbnailUtil.NEW_THUMBNAIL_ID];
         const existingThumbnailId = eventData[ThumbnailUtil.EXISTING_THUMBNAIL_ID];
-        const thumbnailUrl = eventData[ThumbnailUtil.THUMBNAIL];
-        // const thumbnailUrl = newThumbnailId || existingThumbnailId;
+        const iteratedThumbnailId = eventData[ThumbnailUtil.THUMBNAIL];
+        const thumbnailUrl = iteratedThumbnailId || newThumbnailId || existingThumbnailId;
         // Case 1: REMOVE_THUMBNAIL
         if (newThumbnailId === ThumbnailUtil.REMOVE_THUMBNAIL) {
             if (primaryViewableId) {

package/lib/util/thumbnail-util.spec.js

@@ -437,4 +437,123 @@ describe('ThumbnailUtil Tests', () => {
             expect(mockEntitiesDelete).not.toHaveBeenCalled();
         });
     });
+    describe('ThumbnailUtil - iteratedThumbnailId (THUMBNAIL key)', () => {
+        const config = {};
+        let tu;
+        beforeEach(() => {
+            jest.clearAllMocks();
+            tu = new thumbnail_util_1.ThumbnailUtil(config);
+            mockEntitiesGet.mockImplementation((opts) => {
+                if (opts.entityName === 'content-custom-size')
+                    return Promise.resolve([]);
+                return Promise.resolve({});
+            });
+            mockEntitiesUpdate.mockImplementation((opts) => Promise.resolve({ id: opts.id }));
+            mockEntitiesDelete.mockImplementation((opts) => Promise.resolve({ id: opts.id }));
+        });
+        it('does not break when THUMBNAIL value is not available in the event', async () => {
+            // No THUMBNAIL key (and no other thumbnail keys) -> iteratedThumbnailId is undefined
+            const event = { data: {} };
+            let result;
+            await expect((async () => {
+                result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
+            })()).resolves.not.toThrow();
+            // Falls through to the "no thumbnail URL" early return
+            expect(result).toBeUndefined();
+            expect(mockContentCreate).not.toHaveBeenCalled();
+            expect(mockEntitiesUpdate).not.toHaveBeenCalled();
+            expect(mockEntitiesDelete).not.toHaveBeenCalled();
+        });
+        it('does not break when THUMBNAIL key is present but explicitly empty/falsy', async () => {
+            // THUMBNAIL present but falsy -> iteratedThumbnailId is falsy, no crash
+            const event = { data: { [thumbnail_util_1.ThumbnailUtil.THUMBNAIL]: '' } };
+            let result;
+            await expect((async () => {
+                result = await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'item' });
+            })()).resolves.not.toThrow();
+            expect(result).toBeUndefined();
+            expect(mockContentCreate).not.toHaveBeenCalled();
+            expect(mockEntitiesUpdate).not.toHaveBeenCalled();
+            expect(mockEntitiesDelete).not.toHaveBeenCalled();
+        });
+        it('syncs using iteratedThumbnailId when a THUMBNAIL value is available (no existing primaryViewable)', async () => {
+            const mockResponse = {
+                arrayBuffer: jest.fn().mockResolvedValue(new ArrayBuffer(8)),
+                headers: { get: jest.fn().mockReturnValue('image/png') },
+            };
+            mockGetRequest.mockResolvedValue(mockResponse);
+            const createdContent = {
+                id: 'iteratedContent1',
+                contentType: 'image/png',
+                fileName: 'iterated.png',
+                primaryFileUrl: 'https://files/primary.png',
+                largeViewableUrl: 'https://files/large.png',
+                mediumLargeViewableUrl: null,
+                mediumViewableUrl: null,
+                smallViewableUrl: null,
+                tinyViewableUrl: null,
+            };
+            mockContentCreate.mockResolvedValue(createdContent);
+            const thumbnailUrl = '/rest/thumbnail/iterated.png';
+            const event = { data: { [thumbnail_util_1.ThumbnailUtil.THUMBNAIL]: thumbnailUrl } };
+            await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
+            // Uses the THUMBNAIL value to fetch + create the content
+            expect(mockGetRequest).toHaveBeenCalledWith({
+                urlPath: thumbnailUrl,
+                includeUrlContext: false,
+                returnFullResponse: true,
+            });
+            expect(mockContentCreate).toHaveBeenCalledWith(expect.objectContaining({
+                fileName: 'iterated.png',
+                contentType: 'image/png',
+                contentHolderReference: 'color:entity1',
+            }));
+            // Stamps the content with the THUMBNAIL url
+            expect(mockEntitiesUpdate).toHaveBeenCalledWith(expect.objectContaining({
+                entityName: 'content',
+                id: 'iteratedContent1',
+                object: { flexplmThumbnailUrl: thumbnailUrl },
+            }));
+            // Updates the main entity
+            expect(mockEntitiesUpdate).toHaveBeenCalledWith(expect.objectContaining({ entityName: 'color', id: 'entity1' }));
+        });
+        it('iteratedThumbnailId (THUMBNAIL) takes precedence over NEW/EXISTING thumbnail ids', async () => {
+            const mockResponse = {
+                arrayBuffer: jest.fn().mockResolvedValue(new ArrayBuffer(8)),
+                headers: { get: jest.fn().mockReturnValue('image/png') },
+            };
+            mockGetRequest.mockResolvedValue(mockResponse);
+            mockContentCreate.mockResolvedValue({
+                id: 'iteratedContent2',
+                contentType: 'image/png',
+                fileName: 'iterated.png',
+                primaryFileUrl: 'https://files/primary.png',
+                largeViewableUrl: null,
+                mediumLargeViewableUrl: null,
+                mediumViewableUrl: null,
+                smallViewableUrl: null,
+                tinyViewableUrl: null,
+            });
+            const iteratedUrl = '/rest/thumbnail/iterated.png';
+            const event = {
+                data: {
+                    [thumbnail_util_1.ThumbnailUtil.THUMBNAIL]: iteratedUrl,
+                    [thumbnail_util_1.ThumbnailUtil.NEW_THUMBNAIL_ID]: '/rest/thumbnail/new.png',
+                    [thumbnail_util_1.ThumbnailUtil.EXISTING_THUMBNAIL_ID]: '/rest/thumbnail/existing.png',
+                },
+            };
+            await tu.syncThumbnailToVibeIQ({ entityId: 'entity1', event, entityName: 'color' });
+            // The THUMBNAIL value wins over NEW/EXISTING
+            expect(mockGetRequest).toHaveBeenCalledWith({
+                urlPath: iteratedUrl,
+                includeUrlContext: false,
+                returnFullResponse: true,
+            });
+            expect(mockEntitiesUpdate).toHaveBeenCalledWith(expect.objectContaining({
+                entityName: 'content',
+                id: 'iteratedContent2',
+                object: { flexplmThumbnailUrl: iteratedUrl },
+            }));
+        });
+    });
 });

package/package.json

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