@imposium-hub/components 2.15.0-3 → 2.16.0

package/src/components/assets/AssetsTableSelectFilter.test.tsx

@@ -0,0 +1,119 @@
+import { describe, it, expect, vi } from 'vitest';
+import React from 'react';
+import { render, fireEvent } from '@testing-library/react';
+import { Provider } from 'react-redux';
+import { legacy_createStore as createStore } from 'redux';
+
+vi.mock('../checkbox-field/CheckboxField', () => ({
+    default: ({ label, propagate, value, onChange }: any) => (
+        <div
+            data-testid='mock-checkbox-field'
+            data-label={label}
+            data-propagate={propagate}
+            data-value={value}>
+            <input
+                type='checkbox'
+                checked={value}
+                onChange={() => onChange()}
+            />
+        </div>
+    )
+}));
+
+vi.mock('../../redux/actions/selected-assets', () => ({
+    selectAll: vi.fn(() => ({ type: 'selectedAssets/SELECT_ALL' })),
+    resetSelection: vi.fn(() => ({ type: 'selectedAssets/RESET' }))
+}));
+
+import AssetsTableSelectFilterMemoized from './AssetsTableSelectFilter';
+
+const createMockStore = (selectedAssets: string[] = []) => {
+    const initialState = { selectedAssets };
+    return createStore((state: any = initialState, action: any) => {
+        switch (action.type) {
+            case 'selectedAssets/SELECT_ALL':
+                return {
+                    ...state,
+                    selectedAssets: ['asset-1', 'asset-2', 'asset-3']
+                };
+            case 'selectedAssets/RESET':
+                return {
+                    ...state,
+                    selectedAssets: []
+                };
+            default:
+                return state;
+        }
+    });
+};
+
+const renderComponent = (selectedAssets: string[] = []) => {
+    const store = createMockStore(selectedAssets);
+
+    const result = render(
+        <Provider store={store}>
+            <AssetsTableSelectFilterMemoized />
+        </Provider>
+    );
+
+    return { ...result, store };
+};
+
+describe('AssetsTableSelectFilter', () => {
+    it('renders CheckboxField with correct props when no assets are selected', () => {
+        const { getByTestId } = renderComponent();
+        const checkbox = getByTestId('mock-checkbox-field');
+
+        expect(checkbox.getAttribute('data-label')).toBe('');
+        expect(checkbox.getAttribute('data-propagate')).toBe('false');
+        expect(checkbox.getAttribute('data-value')).toBe('false');
+    });
+
+    it('renders CheckboxField as checked when assets are selected', () => {
+        const { getByTestId } = renderComponent(['asset-1', 'asset-2']);
+        const checkbox = getByTestId('mock-checkbox-field');
+
+        expect(checkbox.getAttribute('data-value')).toBe('true');
+    });
+
+    it('dispatches selectAll when checkbox is clicked and no assets are selected', () => {
+        const { getByRole, store } = renderComponent();
+        const checkboxInput = getByRole('checkbox');
+
+        fireEvent.click(checkboxInput);
+
+        const storeState = store.getState();
+        expect(storeState.selectedAssets).toEqual(['asset-1', 'asset-2', 'asset-3']);
+    });
+
+    it('dispatches resetSelection when checkbox is clicked and assets are selected', () => {
+        const { getByRole, store } = renderComponent(['asset-1', 'asset-2']);
+        const checkboxInput = getByRole('checkbox');
+
+        fireEvent.click(checkboxInput);
+
+        const storeState = store.getState();
+        expect(storeState.selectedAssets).toEqual([]);
+    });
+
+    it('shows unchecked state when selectedAssets array is empty', () => {
+        const { getByTestId } = renderComponent([]);
+        const checkbox = getByTestId('mock-checkbox-field');
+
+        expect(checkbox.getAttribute('data-value')).toBe('false');
+    });
+
+    it('shows checked state when selectedAssets array has one item', () => {
+        const { getByTestId } = renderComponent(['single-asset']);
+        const checkbox = getByTestId('mock-checkbox-field');
+
+        expect(checkbox.getAttribute('data-value')).toBe('true');
+    });
+
+    it('shows checked state when selectedAssets array has multiple items', () => {
+        const { getByTestId } = renderComponent(['asset-1', 'asset-2', 'asset-3']);
+        const checkbox = getByTestId('mock-checkbox-field');
+
+        expect(checkbox.getAttribute('data-value')).toBe('true');
+    });
+});

package/src/components/assets/AssetsTableStatusCell.test.tsx

@@ -0,0 +1,60 @@
+import { describe, it, expect } from 'vitest';
+import React from 'react';
+import { render } from '@testing-library/react';
+import AssetsTableStatusCellMemoized from './AssetsTableStatusCell';
+
+const buildCellProp = (overrides: Record<string, any> = {}) => ({
+    row: {
+        original: {
+            processing: false,
+            ...overrides
+        }
+    }
+});
+
+const renderComponent = (overrides: Record<string, any> = {}) => {
+    return render(<AssetsTableStatusCellMemoized cell={buildCellProp(overrides)} />);
+};
+
+describe('AssetsTableStatusCell', () => {
+    it('renders with complete status when processing is false', () => {
+        const { container } = renderComponent({ processing: false });
+        const statusCell = container.querySelector('.asset-status-cell');
+        const statusIndicator = container.querySelector('.status-indicator');
+
+        expect(statusCell).toBeTruthy();
+        expect(statusIndicator?.className).toBe('status-indicator complete');
+    });
+
+    it('renders with processing status when processing is true', () => {
+        const { container } = renderComponent({ processing: true });
+        const statusIndicator = container.querySelector('.status-indicator');
+
+        expect(statusIndicator?.className).toBe('status-indicator processing');
+    });
+
+    it('renders with complete status when processing is undefined', () => {
+        const { container } = renderComponent({ processing: undefined });
+        const statusIndicator = container.querySelector('.status-indicator');
+
+        expect(statusIndicator?.className).toBe('status-indicator complete');
+    });
+
+    it('renders with complete status when processing is null', () => {
+        const { container } = renderComponent({ processing: null });
+        const statusIndicator = container.querySelector('.status-indicator');
+
+        expect(statusIndicator?.className).toBe('status-indicator complete');
+    });
+
+    it('renders the correct DOM structure', () => {
+        const { container } = renderComponent();
+        const statusCell = container.querySelector('.asset-status-cell');
+        const statusIndicator = statusCell?.querySelector('.status-indicator');
+
+        expect(statusCell).toBeTruthy();
+        expect(statusIndicator).toBeTruthy();
+        expect(statusCell?.children).toHaveLength(1);
+        expect(statusCell?.firstChild).toEqual(statusIndicator);
+    });
+});

package/src/components/assets/AssetsTypeIcon.tsx

@@ -24,6 +24,8 @@ interface IAssetsTypeIconProps {
 }
 
 const AssetsTypeIcon: React.FC<IAssetsTypeIconProps> = (p: IAssetsTypeIconProps) => {
+    if (!p.type) return null;
+
     const style = {
         color: ASSET_COLORS[p.type]
     };

package/src/components/change-report/ChangeReportTree.tsx

@@ -1,6 +1,54 @@
 import * as React from 'react';
 import { IChangeReportNode, IChangeReportSection } from '../../constants/snippets';
 import * as copy from '../../constants/copy';
+import {
+    ICON_IMAGE,
+    ICON_VIDEO,
+    ICON_LAYER_GROUP,
+    ICON_CODE,
+    ICON_FILE_ARCHIVE,
+    ICON_IMAGE_SEQ_TYPE,
+    ICON_FONT,
+    ICON_AUDIO_TYPE,
+    ICON_HASHTAG,
+    ICON_TOGGLE_ON,
+    ICON_BARS,
+    ICON_PALETTE,
+    ICON_QRCODE
+} from '../../constants/icons';
+
+const TYPE_ICONS: Record<string, React.ReactNode> = {
+    // Layer types
+    image: ICON_IMAGE,
+    video: ICON_VIDEO,
+    video_composition: ICON_LAYER_GROUP,
+    html: ICON_CODE,
+    template: ICON_FILE_ARCHIVE,
+    image_sequence: ICON_IMAGE_SEQ_TYPE,
+    text: ICON_FONT,
+    audio: ICON_AUDIO_TYPE,
+    solid: ICON_IMAGE,
+    after_effects: ICON_VIDEO,
+    // Variable types (matches getVariableIcon in Story Editor)
+    number: ICON_HASHTAG,
+    boolean: ICON_TOGGLE_ON,
+    enum: ICON_BARS,
+    color: ICON_PALETTE,
+    qr_code: ICON_QRCODE
+};
+
+const TYPE_COLORS: Record<string, string> = {
+    video: 'rgba(236, 22, 150, 1)',
+    image: 'rgba(241, 191, 21, 1)',
+    audio: 'rgba(72, 177, 222, 1)',
+    template: 'rgba(208, 80, 35, 1)',
+    video_composition: 'rgba(215, 22, 22, 1)',
+    text: 'rgba(71, 212, 125, 1)',
+    image_sequence: 'rgba(241, 191, 21, 1)',
+    html: '#ae6cff',
+    solid: '#AAAAAA',
+    after_effects: 'rgba(236, 22, 150, 1)'
+};
 
 interface IChangeReportTreeProps {
     sections: IChangeReportSection[] | null;
@@ -15,6 +63,33 @@ interface IChangeReportNodeProps {
     depth: number;
 }
 
+const isColorValue = (val: string): boolean => {
+    if (!val) return false;
+    return /^#[0-9a-fA-F]{3,8}$/.test(val) || /^rgba?\(/.test(val);
+};
+
+const ValueDisplay: React.FC<{ value: string }> = ({ value }) => {
+    if (value === '*default*' || value === '*empty*') {
+        // Check if this could be a transparent/default color
+        return <>{value}</>;
+    }
+    if (isColorValue(value)) {
+        return (
+            <>
+                <span className='change-report-color-chip-wrapper'>
+                    <span className='change-report-color-chip-checkerboard' />
+                    <span
+                        className='change-report-color-chip-color'
+                        style={{ backgroundColor: value }}
+                    />
+                </span>
+                {value}
+            </>
+        );
+    }
+    return <>{value}</>;
+};
+
 const STATUS_LABELS: Record<IChangeReportNode['status'], string> = {
     added: copy.publish.statusAdded,
     removed: copy.publish.statusRemoved,
@@ -25,16 +100,12 @@ const ChangeReportNode: React.FC<IChangeReportNodeProps> = ({ node, defaultExpan
     const [expanded, setExpanded] = React.useState(defaultExpanded);
 
     const hasChildren = node.children && node.children.length > 0;
-    const hasValues = node.oldValue !== undefined || node.newValue !== undefined;
+    const hasOldValue = node.oldValue !== undefined && node.oldValue !== null;
+    const hasNewValue = node.newValue !== undefined && node.newValue !== null;
+    const hasValues = hasOldValue || hasNewValue;
 
-    const oldDisplay =
-        node.oldValue === null || node.oldValue === '' || node.oldValue === undefined
-            ? '*empty*'
-            : node.oldValue;
-    const newDisplay =
-        node.newValue === null || node.newValue === '' || node.newValue === undefined
-            ? '*empty*'
-            : node.newValue;
+    const oldDisplay = !hasOldValue || node.oldValue === '' ? '*empty*' : node.oldValue;
+    const newDisplay = !hasNewValue || node.newValue === '' ? '*empty*' : node.newValue;
 
     return (
         <div
@@ -54,16 +125,33 @@ const ChangeReportNode: React.FC<IChangeReportNodeProps> = ({ node, defaultExpan
                 <span className={`change-report-status change-report-status--${node.status}`}>
                     {STATUS_LABELS[node.status]}
                 </span>
+                {(node.layerType || node.varType) && TYPE_ICONS[node.layerType || node.varType] && (
+                    <span
+                        className='change-report-layer-icon'
+                        style={
+                            TYPE_COLORS[node.layerType || node.varType]
+                                ? { color: TYPE_COLORS[node.layerType || node.varType] }
+                                : undefined
+                        }>
+                        {TYPE_ICONS[node.layerType || node.varType]}
+                    </span>
+                )}
                 <span className='change-report-label'>{node.label}</span>
                 {hasValues && (
                     <span className='change-report-values'>
-                        <span className='change-report-value change-report-value--old'>
-                            {oldDisplay}
-                        </span>
-                        <span className='change-report-value-arrow'>&rarr;</span>
-                        <span className='change-report-value change-report-value--new'>
-                            {newDisplay}
-                        </span>
+                        {hasOldValue && (
+                            <span className='change-report-value change-report-value--old'>
+                                <ValueDisplay value={oldDisplay} />
+                            </span>
+                        )}
+                        {hasOldValue && hasNewValue && (
+                            <span className='change-report-value-arrow'>&rarr;</span>
+                        )}
+                        {hasNewValue && (
+                            <span className='change-report-value change-report-value--new'>
+                                <ValueDisplay value={newDisplay} />
+                            </span>
+                        )}
                     </span>
                 )}
             </div>