admin/twenty
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Improve flaky storybook test (#8515)

Browse Source 
I don't think that will solve the flaky test but at least it will
cleanup the error message to avoid confusion
This commit is contained in:
Félix Malfait
2024-11-15 14:29:39 +01:00
committed by GitHub
parent bd18eb8b42
commit 34a3a66451
15 changed files with 341 additions and 131 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
packages/twenty-front/src/modules/companies/types/Company.ts
View File

@ -11,8 +11,19 @@ export type Company = {
__typename?: 'Links';
primaryLinkUrl: string;
primaryLinkLabel: string;
secondaryLinks: never[] | [];
};
address: string;
address: {
__typename?: 'Address';
addressLine1: string;
addressLine2: string;
addressCity: string;
addressState: string;
addressPostcode: string;
addressCountry: string;
addressLat: number | null;
addressLng: number | null;
};
accountOwnerId?: string | null;
position?: number;
linkedinLink: {

4
packages/twenty-front/src/modules/object-record/hooks/__mocks__/useCreateManyRecords.ts
View File

@ -26,7 +26,7 @@ export const responseData = {
xLink: {
primaryLinkUrl: '',
primaryLinkLabel: '',
secondaryLinks: null,
secondaryLinks: [],
},
createdAt: '',
city: '',
@ -43,7 +43,7 @@ export const responseData = {
linkedinLink: {
primaryLinkUrl: '',
primaryLinkLabel: '',
secondaryLinks: null,
secondaryLinks: [],
},
updatedAt: '',
avatarUrl: '',

4
packages/twenty-front/src/modules/object-record/hooks/__mocks__/useCreateOneRecord.ts
View File

@ -16,7 +16,7 @@ export const responseData = {
xLink: {
primaryLinkUrl: '',
primaryLinkLabel: '',
secondaryLinks: null,
secondaryLinks: [],
},
pointOfContactForOpportunities: {
edges: [],
@ -48,7 +48,7 @@ export const responseData = {
linkedinLink: {
primaryLinkUrl: '',
primaryLinkLabel: '',
secondaryLinks: null,
secondaryLinks: [],
},
updatedAt: '',
avatarUrl: '',

4
packages/twenty-front/src/modules/object-record/hooks/__mocks__/useUpdateOneRecord.ts
View File

@ -14,7 +14,7 @@ const basePerson = {
xLink: {
primaryLinkUrl: '',
primaryLinkLabel: '',
secondaryLinks: null,
secondaryLinks: [],
},
createdAt: '',
city: '',
@ -26,7 +26,7 @@ const basePerson = {
linkedinLink: {
primaryLinkUrl: '',
primaryLinkLabel: '',
secondaryLinks: null,
secondaryLinks: [],
},
updatedAt: '',
avatarUrl: '',

2
packages/twenty-front/src/modules/object-record/spreadsheet-import/utils/buildRecordFromImportedStructuredRow.ts
View File

@ -128,7 +128,7 @@ export const buildRecordFromImportedStructuredRow = (
primaryLinkUrl: castToString(
importedStructuredRow[`${primaryLinkUrlLabel} (${field.name})`],
),
secondaryLinks: null,
secondaryLinks: [],
} satisfies FieldLinksValue;
}
break;

2
packages/twenty-front/src/modules/object-record/utils/generateEmptyFieldValue.ts
View File

@ -15,7 +15,7 @@ export const generateEmptyFieldValue = (
return { primaryEmail: '', additionalEmails: null };
}
case FieldMetadataType.Links: {
return { primaryLinkUrl: '', primaryLinkLabel: '', secondaryLinks: null };
return { primaryLinkUrl: '', primaryLinkLabel: '', secondaryLinks: [] };
}
case FieldMetadataType.FullName: {
return {

6
packages/twenty-front/src/modules/sign-in-background-mock/constants/SignInBackgroundMockCompanies.ts
View File

@ -45,7 +45,7 @@ export const SIGN_IN_BACKGROUND_MOCK_COMPANIES = [
xLink: {
primaryLinkLabel: 'asd',
primaryLinkUrl: 'asd.com',
secondaryLinks: null,
secondaryLinks: [],
__typename: 'Links',
},
name: {
@ -58,7 +58,7 @@ export const SIGN_IN_BACKGROUND_MOCK_COMPANIES = [
linkedinLink: {
primaryLinkLabel: 'asd',
primaryLinkUrl: 'asd.com',
secondaryLinks: null,
secondaryLinks: [],
__typename: 'Links',
},
},
@ -85,7 +85,7 @@ export const SIGN_IN_BACKGROUND_MOCK_COMPANIES = [
xLink: {
primaryLinkLabel: null,
primaryLinkUrl: null,
secondaryLinks: null,
secondaryLinks: [],
__typename: 'Links',
},
annualRecurringRevenue: {