@axinom/mosaic-ui 0.65.4 → 0.65.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@axinom/mosaic-ui",
-  "version": "0.65.4",
+  "version": "0.65.5",
   "description": "UI components for building Axinom Mosaic applications",
   "author": "Axinom",
   "license": "PROPRIETARY",
@@ -112,5 +112,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "ac37c5be8b4c739425573c41be818692020f1f25"
+  "gitHead": "2dccfe1c1492cbb54be6b6e9af3602e717cdce57"
 }

package/src/components/Utils/Postgraphile/CreateConnectionRenderer.spec.ts

@@ -1,14 +1,5 @@
-import { TagsRenderer } from '../../List';
 import { createConnectionRenderer } from './CreateConnectionRenderer';
 
-jest.mock('../../List', () => ({
-  TagsRenderer: jest.fn(),
-}));
-
-const mockTagsRenderer = TagsRenderer as jest.MockedFunction<
-  typeof TagsRenderer
->;
-
 interface TestConnection {
   nodes: { id: string; name: string }[];
 }
@@ -18,70 +9,7 @@ interface StringConnection {
 }
 
 describe('createConnectionRenderer', () => {
-  beforeEach(() => {
-    jest.clearAllMocks();
-  });
-
-  describe('with renderAsTags = true (default)', () => {
-    it('should call TagsRenderer with mapped values when renderAsTags is true', () => {
-      const mockReturnValue = 'Mocked Tags JSX Element';
-      mockTagsRenderer.mockReturnValue(mockReturnValue as any);
-
-      const connection: TestConnection = {
-        nodes: [
-          { id: '1', name: 'Item 1' },
-          { id: '2', name: 'Item 2' },
-        ],
-      };
-
-      const selector = (item: { id: string; name: string }) => item.name;
-      const renderer = createConnectionRenderer<TestConnection>(selector);
-      const result = renderer(connection);
-
-      expect(mockTagsRenderer).toHaveBeenCalledWith(['Item 1', 'Item 2']);
-      expect(result).toBe(mockReturnValue);
-    });
-
-    it('should throw error when nodes is undefined', () => {
-      const connection = { nodes: undefined } as unknown as TestConnection;
-      const selector = (item: { id: string; name: string }) => item.name;
-      const renderer = createConnectionRenderer<TestConnection>(selector);
-
-      expect(() => renderer(connection)).toThrow();
-      expect(mockTagsRenderer).not.toHaveBeenCalled();
-    });
-
-    it('should throw error when value is null', () => {
-      const selector = (item: { id: string; name: string }) => item.name;
-      const renderer = createConnectionRenderer<TestConnection>(selector);
-
-      expect(() => renderer(null)).toThrow();
-      expect(mockTagsRenderer).not.toHaveBeenCalled();
-    });
-
-    it('should throw error when value is undefined', () => {
-      const selector = (item: { id: string; name: string }) => item.name;
-      const renderer = createConnectionRenderer<TestConnection>(selector);
-
-      expect(() => renderer(undefined)).toThrow();
-      expect(mockTagsRenderer).not.toHaveBeenCalled();
-    });
-
-    it('should call TagsRenderer with empty array when nodes is empty', () => {
-      const mockReturnValue = 'Empty Tags JSX Element';
-      mockTagsRenderer.mockReturnValue(mockReturnValue as any);
-
-      const connection: TestConnection = { nodes: [] };
-      const selector = (item: { id: string; name: string }) => item.name;
-      const renderer = createConnectionRenderer<TestConnection>(selector);
-      const result = renderer(connection);
-
-      expect(mockTagsRenderer).toHaveBeenCalledWith([]);
-      expect(result).toBe(mockReturnValue);
-    });
-  });
-
-  describe('with renderAsTags = false', () => {
+  describe('basic functionality', () => {
     it('should return comma-separated string using selector', () => {
       const connection: TestConnection = {
         nodes: [
@@ -92,13 +20,9 @@ describe('createConnectionRenderer', () => {
       };
 
       const selector = (item: { id: string; name: string }) => item.name;
-      const renderer = createConnectionRenderer<TestConnection>(
-        selector,
-        false,
-      );
+      const renderer = createConnectionRenderer<TestConnection>(selector);
       const result = renderer(connection);
 
-      expect(mockTagsRenderer).not.toHaveBeenCalled();
       expect(result).toBe('Item 1, Item 2, Item 3');
     });
 
@@ -108,10 +32,7 @@ describe('createConnectionRenderer', () => {
       };
 
       const selector = (item: string) => item;
-      const renderer = createConnectionRenderer<StringConnection>(
-        selector,
-        false,
-      );
+      const renderer = createConnectionRenderer<StringConnection>(selector);
       const result = renderer(connection);
 
       expect(result).toBe('apple, banana, cherry');
@@ -127,10 +48,7 @@ describe('createConnectionRenderer', () => {
 
       const selector = (item: { id: string; name: string }, index: number) =>
         `${index + 1}. ${item.name} (${item.id})`;
-      const renderer = createConnectionRenderer<TestConnection>(
-        selector,
-        false,
-      );
+      const renderer = createConnectionRenderer<TestConnection>(selector);
       const result = renderer(connection);
 
       expect(result).toBe('1. Item 1 (1), 2. Item 2 (2)');
@@ -139,10 +57,7 @@ describe('createConnectionRenderer', () => {
     it('should return empty string when nodes array is empty', () => {
       const connection: TestConnection = { nodes: [] };
       const selector = (item: { id: string; name: string }) => item.name;
-      const renderer = createConnectionRenderer<TestConnection>(
-        selector,
-        false,
-      );
+      const renderer = createConnectionRenderer<TestConnection>(selector);
       const result = renderer(connection);
 
       expect(result).toBe('');
@@ -160,25 +75,33 @@ describe('createConnectionRenderer', () => {
       };
 
       const selector = (item: { id: number; value: number }) => item.value;
-      const renderer = createConnectionRenderer<NumberConnection>(
-        selector,
-        false,
-      );
+      const renderer = createConnectionRenderer<NumberConnection>(selector);
       const result = renderer(connection);
 
       expect(result).toBe('100, 200');
     });
 
-    it('should throw error when connection.nodes is undefined and renderAsTags is false', () => {
+    it('should throw error when connection.nodes is undefined', () => {
       const connection = { nodes: undefined } as unknown as TestConnection;
       const selector = (item: { id: string; name: string }) => item.name;
-      const renderer = createConnectionRenderer<TestConnection>(
-        selector,
-        false,
-      );
+      const renderer = createConnectionRenderer<TestConnection>(selector);
 
       expect(() => renderer(connection)).toThrow();
     });
+
+    it('should throw error when value is null', () => {
+      const selector = (item: { id: string; name: string }) => item.name;
+      const renderer = createConnectionRenderer<TestConnection>(selector);
+
+      expect(() => renderer(null)).toThrow();
+    });
+
+    it('should throw error when value is undefined', () => {
+      const selector = (item: { id: string; name: string }) => item.name;
+      const renderer = createConnectionRenderer<TestConnection>(selector);
+
+      expect(() => renderer(undefined)).toThrow();
+    });
   });
 
   describe('edge cases', () => {
@@ -188,10 +111,7 @@ describe('createConnectionRenderer', () => {
       };
 
       const selector = (item: string) => item;
-      const renderer = createConnectionRenderer<StringConnection>(
-        selector,
-        false,
-      );
+      const renderer = createConnectionRenderer<StringConnection>(selector);
       const result = renderer(connection);
 
       expect(result).toBe('item, with comma, item with "quotes", normal item');
@@ -207,10 +127,7 @@ describe('createConnectionRenderer', () => {
       };
 
       const selector = (item: { id: string; name: string }) => item.name;
-      const renderer = createConnectionRenderer<TestConnection>(
-        selector,
-        false,
-      );
+      const renderer = createConnectionRenderer<TestConnection>(selector);
       const result = renderer(connection);
 
       expect(result).toBe(', Item 2, ');
@@ -222,10 +139,7 @@ describe('createConnectionRenderer', () => {
       };
 
       const selector = (item: { id: string; name: string }) => item.name;
-      const renderer = createConnectionRenderer<TestConnection>(
-        selector,
-        false,
-      );
+      const renderer = createConnectionRenderer<TestConnection>(selector);
       const result = renderer(connection);
 
       expect(result).toBe('Single Item');
@@ -247,10 +161,7 @@ describe('createConnectionRenderer', () => {
 
       const selector = (item: { customField: number; label: string }) =>
         `${item.label}: ${item.customField}`;
-      const renderer = createConnectionRenderer<CustomConnection>(
-        selector,
-        false,
-      );
+      const renderer = createConnectionRenderer<CustomConnection>(selector);
       const result = renderer(connection);
 
       expect(result).toBe('Custom 1: 42, Custom 2: 84');

package/src/components/Utils/Postgraphile/CreateConnectionRenderer.tsx

@@ -1,5 +1,4 @@
 import { ReactNode } from 'react';
-import { TagsRenderer } from '../../List';
 
 interface Connection {
   nodes: unknown[];
@@ -19,14 +18,14 @@ export type SelectorFunction<T> = (
  */
 export function createConnectionRenderer<T extends Connection>(
   selector: SelectorFunction<T['nodes'][number]>,
-  renderAsTags = true,
+  // renderAsTags = true,
 ): (val: unknown) => string | ReactNode {
   const ConnectionRenderer = (val: unknown): string | ReactNode => {
     const value = val as T;
 
-    if (renderAsTags) {
-      return TagsRenderer(value.nodes.map(selector));
-    }
+    // if (renderAsTags) {
+    //   return TagsRenderer(value.nodes.map(selector));
+    // }
 
     return value.nodes.map(selector).join(', ');
   };