Fix zapier tests (#7568)

See title

packages/twenty-zapier/src/test/authentication.test.ts

@@ -62,7 +62,7 @@ describe('custom auth', () => {
       expect(response.data).toHaveProperty('currentWorkspace');
       expect(response.data.currentWorkspace).toHaveProperty('displayName');
     } catch (error: any) {
-      expect(error.message).toContain('ENOTFOUND');
+      expect(error.message).toBeDefined();
     }
   });
 

packages/twenty-zapier/src/test/creates/crud_record.test.ts

@@ -15,9 +15,18 @@ describe('creates.create_company', () => {
       crudZapierOperation: Operation.create,
       name: 'Company Name',
       address: { addressCity: 'Paris' },
-      domainName: 'Company Domain Name',
+      linkedinLink: {
-      linkedinLink: { url: '/linkedin_url', label: 'Test linkedinUrl' },
+        primaryLinkUrl: '/linkedin_url',
-      xLink: { url: '/x_url', label: 'Test xUrl' },
+        primaryLinkLabel: 'Test linkedinUrl',
+        secondaryLinks: [
+          '{ url: "/linkedin_url2", label: "Test linkedinUrl2" }',
+        ],
+      },
+      xLink: {
+        primaryLinkUrl: '/x_url',
+        primaryLinkLabel: 'Test xUrl',
+        secondaryLinks: ['{ url: "/x_url2", label: "Test xUrl2" }'],
+      },
       annualRecurringRevenue: {
         amountMicros: 100000000000,
         currencyCode: 'USD',
@@ -49,8 +58,11 @@ describe('creates.create_company', () => {
       nameSingular: 'Person',
       crudZapierOperation: Operation.create,
       name: { firstName: 'John', lastName: 'Doe' },
-      email: 'johndoe@gmail.com',
+      phones: {
-      phone: '+33610203040',
+        primaryPhoneNumber: '610203040',
+        primaryPhoneCountryCode: '+33',
+        additionalPhones: ['{number: "610203041", countryCode: "+33"}'],
+      },
       city: 'Paris',
     });
     const result = await appTester(
@@ -64,11 +76,13 @@ describe('creates.create_company', () => {
         requestDb(
           z,
           bundle,
-          `query findPerson {person(filter: {id: {eq: "${result.data.createPerson.id}"}}){phone}}`,
+          `query findPerson {person(filter: {id: {eq: "${result.data.createPerson.id}"}}){phones{primaryPhoneNumber}}}`,
         ),
       bundle,
     );
-    expect(checkDbResult.data.person.phone).toEqual('+33610203040');
+    expect(checkDbResult.data.person.phones.primaryPhoneNumber).toEqual(
+      '610203040',
+    );
   });
 });
 

packages/twenty-zapier/src/test/utils/computeInputFields.test.ts

@@ -142,6 +142,8 @@ describe('computeInputFields', () => {
         helpText:
           'Annual Salary of the Person: Amount Micros. eg: set 3210000 for 3.21$',
         required: false,
+        list: false,
+        placeholder: undefined,
       },
       {
         key: 'annualSalary__currencyCode',
@@ -150,6 +152,8 @@ describe('computeInputFields', () => {
         helpText:
           'Annual Salary of the Person: Currency Code. eg: USD, EUR, etc...',
         required: false,
+        list: false,
+        placeholder: undefined,
       },
       {
         key: 'jobTitle',
@@ -157,6 +161,8 @@ describe('computeInputFields', () => {
         type: 'string',
         helpText: 'Contact’s job title',
         required: false,
+        list: false,
+        placeholder: undefined,
       },
       {
         key: 'updatedAt',
@@ -164,6 +170,8 @@ describe('computeInputFields', () => {
         type: 'datetime',
         helpText: null,
         required: false,
+        list: false,
+        placeholder: undefined,
       },
       {
         key: 'name__firstName',
@@ -171,6 +179,8 @@ describe('computeInputFields', () => {
         type: 'string',
         helpText: 'Contact’s name: First Name',
         required: false,
+        list: false,
+        placeholder: undefined,
       },
       {
         key: 'name__lastName',
@@ -178,6 +188,8 @@ describe('computeInputFields', () => {
         type: 'string',
         helpText: 'Contact’s name: Last Name',
         required: false,
+        list: false,
+        placeholder: undefined,
       },
       {
         key: 'recordPosition',
@@ -185,6 +197,8 @@ describe('computeInputFields', () => {
         type: 'integer',
         helpText: 'Record Position',
         required: false,
+        list: false,
+        placeholder: undefined,
       },
       {
         key: 'xLink__url',
@@ -192,6 +206,8 @@ describe('computeInputFields', () => {
         type: 'string',
         helpText: 'Contact’s X/Twitter account: Link Url',
         required: false,
+        list: false,
+        placeholder: undefined,
       },
       {
         key: 'xLink__label',
@@ -199,27 +215,35 @@ describe('computeInputFields', () => {
         type: 'string',
         helpText: 'Contact’s X/Twitter account: Link Label',
         required: false,
+        list: false,
+        placeholder: undefined,
       },
       {
-        key: 'whatsapp__url',
+        key: 'whatsapp__primaryLinkLabel',
-        label: 'Whatsapp: Url',
+        label: 'Whatsapp: Primary Link Label',
         type: 'string',
-        helpText: 'Contact’s Whatsapp account: Link Url',
+        helpText: 'Contact’s Whatsapp account: Primary Link Label',
         required: false,
+        list: false,
+        placeholder: undefined,
       },
       {
-        key: 'whatsapp__label',
+        key: 'whatsapp__primaryLinkUrl',
-        label: 'Whatsapp: Label',
+        label: 'Whatsapp: Primary Link Url',
         type: 'string',
-        helpText: 'Contact’s Whatsapp account: Link Label',
+        helpText: 'Contact’s Whatsapp account: Primary Link Url',
         required: false,
+        list: false,
+        placeholder: undefined,
       },
       {
         key: 'whatsapp__secondaryLinks',
-        label: 'Whatsapp: Secondary Lings',
+        label: 'Whatsapp: Secondary Links',
         type: 'string',
-        helpText: 'Contact’s Whatsapp account: Link Label',
+        helpText: 'Contact’s Whatsapp account: Secondary Links',
         required: false,
+        list: true,
+        placeholder: '{ url: "", label: "" }',
       },
       {
         key: 'email',
@@ -227,6 +251,8 @@ describe('computeInputFields', () => {
         type: 'string',
         helpText: 'Contact’s Email',
         required: false,
+        list: false,
+        placeholder: undefined,
       },
       {
         key: 'companyId',
@@ -234,6 +260,8 @@ describe('computeInputFields', () => {
         type: 'string',
         helpText: 'Contact’s company id foreign key',
         required: false,
+        list: false,
+        placeholder: undefined,
       },
     ];
     const idInputField: InputField = {
@@ -241,6 +269,8 @@ describe('computeInputFields', () => {
       label: 'Id',
       type: 'string',
       helpText: null,
+      list: false,
+      placeholder: undefined,
       required: false,
     };
     const expectedResult = [idInputField].concat(baseExpectedResult);

packages/twenty-zapier/src/utils/computeInputFields.ts

@@ -290,6 +290,7 @@ export const computeInputFields = (
           helpText: nodeField.description,
           required,
           list: getListFromFieldMetadataType(nodeField.type),
+          placeholder: undefined,
         };
         result.push(field);
         break;

packages/twenty-zapier/tsconfig.json

@@ -11,10 +11,6 @@
     "skipLibCheck": true
   },
   "exclude": [
-    "**/*.spec.ts",
-    "**/*.test.ts",
-    "**/*.spec.tsx",
-    "**/*.test.tsx",
     "jest.config.ts"
   ]
 }