@integry/sdk 4.6.10 → 4.6.12

package/generateTests.js

@@ -6,45 +6,74 @@ import dotenv from 'dotenv';
 dotenv.config();
 
 const openai = new OpenAI({
-  apiKey: "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",
+  apiKey: "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",
 });
 
-async function generateTest(filePath) {
+async function generateTest(filePath, feedback = '') {
   const code = fs.readFileSync(filePath, 'utf-8');
 
-  const prompt = `Write Vitest unit tests for the following Preact component:
-${code}
+  let messages = [
+    { role: 'system', content: 'You are a senior Preact developer with expertise in testing. Use Vitest framework.' },
+    { role: 'user', content: `Write Vitest unit tests for the following Preact component:\n${code}\n
+    Make sure to cover:
+  - Props validation
+  - Event handlers
+  - Conditional rendering
+  - User interactions
+  - Edge cases
+    ` }
+  ];
 
-Make sure to cover:
-- Props validation
-- Event handlers
-- Conditional rendering
-- User interactions
-- Edge cases
-`;
+  if (feedback) {
+    messages.push({
+      role: 'user',
+      content: `Here is the previous generated test:\n${feedback}\nPlease improve the tests based on the following feedback: 
+- Add more edge cases.
+- You didn't test fields inside objects represented by field.properties or field.template_fields
+- If object has no field, it should render a textarea.`
+    });
+  }
 
   const completion = await openai.chat.completions.create({
     model: 'gpt-4-turbo',
-    messages: [
-      { role: 'system', content: 'You are a senior frontend developer with expertise in testing.' },
-      { role: 'user', content: prompt },
-    ],
-    temperature: 0.2,
+    messages,
+    temperature: 0.3,
   });
 
   const generatedTests = completion.choices[0]?.message?.content || '';
 
-  const testFilePath = path.join(path.dirname(filePath), `${path.basename(filePath, '.tsx')}.test.tsx`);
+  const testFilePath = filePath.replace('.ts', '.test.ts');
   fs.writeFileSync(testFilePath, generatedTests);
-  console.log(`Test file generated: ${testFilePath}`);
+
+  console.log(`✅ Test file generated: ${testFilePath}`);
+
+  return generatedTests;
 }
 
 async function main() {
-  const componentsFolder = './src/components/form/ObjectField/'; // Update with your components folder
+  // const componentsFolder = './src/components/form/ObjectField'; // Update this path
+  const componentsFolder = process.argv[2]; // Get folder path from command line argument
+
+  if (!componentsFolder) {
+    console.error('❌ Please provide the components folder path as an argument.');
+    process.exit(1);
+  }
+
+  if (!fs.existsSync(componentsFolder)) {
+    console.error(`❌ Path "${componentsFolder}" does not exist.`);
+    process.exit(1);
+  }
   const files = fs.readdirSync(componentsFolder).filter((file) => file.endsWith('.ts'));
-  console.log(files);
+
   for (const file of files) {
-    await generateTest(path.join(componentsFolder, file));
+    const testFilePath = path.join(componentsFolder, `${file.replace('.ts', '.test.ts')}`);
+    let previousTest = '';
+
+    if (fs.existsSync(testFilePath)) {
+      previousTest = fs.readFileSync(testFilePath, 'utf-8');
+    }
+
+    await generateTest(path.join(componentsFolder, file), previousTest);
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@integry/sdk",
-  "version": "4.6.10",
+  "version": "4.6.12",
   "description": "Integry SDK",
   "main": "dist/umd/index.umd.js",
   "module": "dist/esm/index.csm.js",
@@ -51,6 +51,7 @@
     "@storybook/preact": "^8.5.7",
     "@storybook/preact-webpack5": "^8.5.7",
     "@storybook/preset-scss": "^1.0.3",
+    "@testing-library/jest-dom": "^6.6.3",
     "@testing-library/preact": "^2.0.1",
     "@testing-library/user-event": "^14.6.1",
     "@types/jest": "^26.0.23",

package/src/components/form/ObjectField/__snapshots__/index.ts.test.tsx.snap

@@ -0,0 +1,3 @@
+// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
+
+exports[`ObjectField Component > renders with minimal props 1`] = `<div />`;

package/src/components/form/ObjectField/index.ts

@@ -367,7 +367,11 @@ const ObjectField = (props: ObjectFieldProps) => {
                 isMappable=${activityOutputDataRaw &&
                 Object.entries(activityOutputDataRaw).length > 0}
                 allowTagsInText=${activityOutputDataRaw}
-                value=${formatValueForTextarea(objectValue)}
+                value=${objectHasFields
+                  ? (objectArray[objectIndex] || {})[
+                      fieldDetails?.meta?.machine_name || ''
+                    ] || ''
+                  : formatValueForTextarea(objectValue)}
               ><//>
             `}
             ${fieldType === 'CHECKBOX' &&

package/src/components/form/ObjectField/index.ts.test.tsx

@@ -1,108 +0,0 @@
-To write Vitest unit tests for the `ObjectField` Preact component, we need to cover various aspects such as props validation, event handlers, conditional rendering, user interactions, and edge cases. Below are examples of how these tests can be structured using Vitest along with the Testing Library for Preact.
-
-### Setup
-
-First, ensure you have the necessary dependencies:
-
-```bash
-npm install vitest @testing-library/preact @testing-library/user-event --save-dev
-```
-
-### Test File: `ObjectField.test.tsx`
-
-```tsx
-import { describe, it, expect, vi } from 'vitest';
-import { render, fireEvent, screen } from '@testing-library/preact';
-import userEvent from '@testing-library/user-event';
-import { ObjectField } from './ObjectField';
-
-describe('ObjectField Component', () => {
-  const mockOnChange = vi.fn();
-  const mockOnChangeInFlow = vi.fn();
-  const mockApiHandler = vi.fn();
-
-  const baseProps = {
-    field: {
-      title: 'Test Field',
-      description: 'Test Description',
-      template_fields: [],
-      type: 'TEXTFIELD',
-      properties: {},
-    },
-    functionArguments: {},
-    connectedAccount: 'test-account',
-    appName: 'Test App',
-    onChange: mockOnChange,
-    onChangeInFlow: mockOnChangeInFlow,
-    apiHandler: mockApiHandler,
-    activityOutputData: {},
-    activityOutputDataRaw: {},
-  };
-
-  it('renders without crashing', () => {
-    const { container } = render(<ObjectField {...baseProps} />);
-    expect(container).toBeInTheDocument();
-  });
-
-  it('calls onChange when input changes', async () => {
-    const user = userEvent.setup();
-    render(<ObjectField {...baseProps} />);
-    const input = screen.getByRole('textbox');
-    await user.type(input, 'New Value');
-    expect(mockOnChange).toHaveBeenCalled();
-  });
-
-  it('conditionally renders add more button based on isArray prop', () => {
-    const { rerender } = render(<ObjectField {...baseProps} isArray={true} />);
-    expect(screen.queryByText('+ Add another Test Field')).toBeInTheDocument();
-
-    rerender(<ObjectField {...baseProps} isArray={false} />);
-    expect(screen.queryByText('+ Add another Test Field')).not.toBeInTheDocument();
-  });
-
-  it('handles array of objects and adds more on button click', async () => {
-    const user = userEvent.setup();
-    render(<ObjectField {...baseProps} isArray={true} />);
-    await user.click(screen.getByText('+ Add another Test Field'));
-    expect(screen.getAllByText('Test Field').length).toBe(2);
-  });
-
-  it('removes an item from the list when delete is clicked', async () => {
-    const user = userEvent.setup();
-    render(<ObjectField {...baseProps} isArray={true} objectValue={[{}, {}]} />);
-    expect(screen.getAllByText('Test Field').length).toBe(2);
-    await user.click(screen.getAllByRole('button')[0]); // Assuming the delete button is the first button
-    expect(screen.getAllByText('Test Field').length).toBe(1);
-  });
-
-  it('validates props and throws if required props are missing', () => {
-    const consoleError = vi.spyOn(console, 'error');
-    consoleError.mockImplementation(() => {});
-
-    expect(() => render(<ObjectField />)).toThrow();
-    expect(consoleError).toHaveBeenCalled();
-
-    consoleError.mockRestore();
-  });
-
-  // Additional tests can be added here to cover more complex interactions and edge cases
-});
-```
-
-### Explanation
-
-1. **Setup and Imports**: We import necessary functions from Vitest and Testing Library. We also mock functions that will be used as props.
-
-2. **Base Props**: A base props object is defined to reuse in multiple tests, ensuring consistency and reducing redundancy.
-
-3. **Simple Render Test**: Checks if the component renders without throwing.
-
-4. **Event Handler Test**: Simulates user input and checks if the appropriate handler (`onChange`) is called.
-
-5. **Conditional Rendering**: Tests the conditional rendering of the "Add another" button based on the `isArray` prop.
-
-6. **Dynamic List Manipulation**: Tests adding and removing items from a list, which is crucial for components managing arrays of data.
-
-7. **Props Validation**: Optionally, you can test for required props and ensure the component throws or logs errors when props are missing or invalid.
-
-This setup provides a robust starting point for testing the `ObjectField` component comprehensively, ensuring it behaves correctly under various conditions and interactions.

package/src/features/common/FunctionForm/index.ts

@@ -193,6 +193,15 @@ class FunctionForm extends Component<
             value = value === '' || value === '[]' ? [] : value;
             if (fieldDetails.items && fieldDetails.items.type === 'number') {
               value = value.map((item: string) => Number(item));
+            } else if (
+              fieldDetails.items &&
+              fieldDetails.items.type === 'string'
+            ) {
+              try {
+                value = JSON.parse(value);
+              } catch (e) {
+                value = [];
+              }
             }
 
             break;