Fix test cases

apps/api/src/controllers/__tests__/Workflows.test.ts

@@ -43,9 +43,9 @@ describe('Workflows Controller', () => {
       expect(responseBody.fields).toContain('contact.subscribed');
 
       // Should include custom contact data fields
-      expect(responseBody.fields).toContain('data.firstName');
+      expect(responseBody.fields).toContain('contact.data.firstName');
-      expect(responseBody.fields).toContain('data.lastName');
+      expect(responseBody.fields).toContain('contact.data.lastName');
-      expect(responseBody.fields).toContain('data.plan');
+      expect(responseBody.fields).toContain('contact.data.plan');
 
       // Should include event fields
       expect(responseBody.fields).toContain('event.subject');

apps/api/src/services/__tests__/SegmentService.test.ts

@@ -233,7 +233,9 @@ describe('SegmentService', () => {
 
       expect(segment.name).toBe('VIP Customers');
       expect(segment.projectId).toBe(projectId);
-      expect(Array.isArray(segment.filters)).toBe(true);
+      // Segments now use condition object with groups containing filters
+      expect(segment.condition).toBeDefined();
+      expect(typeof segment.condition).toBe('object');
     });
 
     it('should list all segments for a project', async () => {
@@ -387,7 +389,10 @@ describe('SegmentService', () => {
       await expect(
         SegmentService.create(projectId, {
           name: 'Invalid Segment',
-          filters: [],
+          condition: {
+            logic: 'AND',
+            groups: [{filters: []}],
+          },
         }),
       ).rejects.toThrow(/at least one filter/i);
     });
@@ -396,6 +401,10 @@ describe('SegmentService', () => {
       await expect(
         SegmentService.create(projectId, {
           name: 'Invalid Segment',
+          condition: {
+            logic: 'AND',
+            groups: [
+              {
                 filters: [
                   {
                     // Intentionally missing field, cast to bypass compile-time validation
@@ -403,6 +412,9 @@ describe('SegmentService', () => {
                     value: 'test',
                   } as InvalidFilterInput as SegmentFilter,
                 ],
+              },
+            ],
+          },
         }),
       ).rejects.toThrow(/field is required/i);
     });
@@ -411,6 +423,10 @@ describe('SegmentService', () => {
       await expect(
         SegmentService.create(projectId, {
           name: 'Invalid Segment',
+          condition: {
+            logic: 'AND',
+            groups: [
+              {
                 filters: [
                   {
                     field: 'email',
@@ -419,6 +435,9 @@ describe('SegmentService', () => {
                     value: 'test',
                   } as InvalidFilterInput as SegmentFilter,
                 ],
+              },
+            ],
+          },
         }),
       ).rejects.toThrow(/invalid operator/i);
     });
@@ -427,6 +446,10 @@ describe('SegmentService', () => {
       await expect(
         SegmentService.create(projectId, {
           name: 'Invalid Segment',
+          condition: {
+            logic: 'AND',
+            groups: [
+              {
                 filters: [
                   {
                     field: 'email',
@@ -434,6 +457,9 @@ describe('SegmentService', () => {
                     // Value intentionally omitted
                   } as InvalidFilterInput as SegmentFilter,
                 ],
+              },
+            ],
+          },
         }),
       ).rejects.toThrow(/requires a value/i);
     });
@@ -441,6 +467,10 @@ describe('SegmentService', () => {
     it('should ACCEPT valid filters', async () => {
       const segment = await SegmentService.create(projectId, {
         name: 'Valid Segment',
+        condition: {
+          logic: 'AND',
+          groups: [
+            {
               filters: [
                 {
                   field: 'subscribed',
@@ -448,6 +478,9 @@ describe('SegmentService', () => {
                   value: true,
                 },
               ],
+            },
+          ],
+        },
       });
 
       expect(segment.id).toBeDefined();

test/helpers/factories.ts

@@ -1,15 +1,15 @@
 import {
-  PrismaClient,
   AuthMethod,
+  CampaignStatus,
+  EmailSourceType,
+  EmailStatus,
+  Prisma,
+  PrismaClient,
   Role,
   TemplateType,
-  CampaignStatus,
-  EmailStatus,
-  EmailSourceType,
-  WorkflowTriggerType,
-  WorkflowStepType,
   WorkflowExecutionStatus,
-  StepExecutionStatus,
+  WorkflowStepType,
+  WorkflowTriggerType
 } from '@plunk/db';
 import {getPrismaClient} from './database';
 import bcrypt from 'bcrypt';
@@ -513,20 +513,31 @@ export class TestFactories {
 
   /**
    * Create a segment
+   * Supports both old format (filters array) and new format (condition object)
    */
   async createSegment(
     projectId: string,
     overrides: {
       name?: string;
-      filters?: unknown;
+      filters?: Array<{field: string; operator: string; value?: unknown; unit?: string}>;
+      condition?: {
+        logic: 'AND' | 'OR';
+        groups: Array<{filters: Array<{field: string; operator: string; value?: unknown; unit?: string}>}>;
+      };
       trackMembership?: boolean;
     } = {},
   ) {
+    // Convert filters array to condition format if condition not provided
+    const condition = overrides.condition || {
+      logic: 'AND' as const,
+      groups: [{filters: overrides.filters || []}],
+    };
+
     return this.prisma.segment.create({
       data: {
         projectId,
         name: overrides.name || `Segment ${uniqueId()}`,
-        filters: overrides.filters || [],
+        condition: condition as unknown as Prisma.InputJsonValue,
         trackMembership: overrides.trackMembership ?? false,
       },
     });

vitest.config.ts

@@ -33,7 +33,7 @@ export default defineConfig({
     // Run tests sequentially to avoid database cleanup conflicts
     fileParallelism: false,
     // Limit concurrent test files to reduce memory pressure
-    maxConcurrency: 1,
+    maxConcurrency: 3,
     // Only include our test files, not dependency tests
     include: [
       'apps/**/__tests__/**/*.{test,spec}.{ts,tsx}',