forked from twentyhq/twenty
-
Notifications
You must be signed in to change notification settings - Fork 3
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: fix Apollo client cache update error for Links field (twentyhq#5473
) Fixes twentyhq#5437
- Loading branch information
1 parent
d1cbd70
commit 474dfd7
Showing
8 changed files
with
159 additions
and
15 deletions.
There are no files selected for viewing
80 changes: 80 additions & 0 deletions
80
...nty-front/src/modules/object-record/cache/utils/__tests__/getRecordNodeFromRecord.test.ts
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,80 @@ | ||
| import { ObjectMetadataItem } from '@/object-metadata/types/ObjectMetadataItem'; | ||
| import { | ||
| mockedObjectMetadataItems, | ||
| mockedPersonObjectMetadataItem, | ||
| } from '~/testing/mock-data/metadata'; | ||
| import { mockedPeopleData } from '~/testing/mock-data/people'; | ||
|
|
||
| import { getRecordNodeFromRecord } from '../getRecordNodeFromRecord'; | ||
|
|
||
| describe('getRecordNodeFromRecord', () => { | ||
| it('computes relation records cache references by default', () => { | ||
| // Given | ||
| const objectMetadataItems: ObjectMetadataItem[] = mockedObjectMetadataItems; | ||
| const objectMetadataItem: Pick< | ||
| ObjectMetadataItem, | ||
| 'fields' | 'namePlural' | 'nameSingular' | ||
| > = mockedPersonObjectMetadataItem; | ||
| const recordGqlFields = { | ||
| name: true, | ||
| company: true, | ||
| }; | ||
| const record = mockedPeopleData[0]; | ||
|
|
||
| // When | ||
| const result = getRecordNodeFromRecord({ | ||
| objectMetadataItems, | ||
| objectMetadataItem, | ||
| recordGqlFields, | ||
| record, | ||
| }); | ||
|
|
||
| // Then | ||
| expect(result).toEqual({ | ||
| __typename: 'Person', | ||
| company: { | ||
| __ref: 'Company:5c21e19e-e049-4393-8c09-3e3f8fb09ecb', | ||
| }, | ||
| name: { | ||
| __typename: 'FullName', | ||
| firstName: 'Alexandre', | ||
| lastName: 'Prot', | ||
| }, | ||
| }); | ||
| }); | ||
|
|
||
| it('does not compute relation records cache references when `computeReferences` is false', () => { | ||
| // Given | ||
| const objectMetadataItems: ObjectMetadataItem[] = mockedObjectMetadataItems; | ||
| const objectMetadataItem: Pick< | ||
| ObjectMetadataItem, | ||
| 'fields' | 'namePlural' | 'nameSingular' | ||
| > = mockedPersonObjectMetadataItem; | ||
| const recordGqlFields = { | ||
| name: true, | ||
| company: true, | ||
| }; | ||
| const record = mockedPeopleData[0]; | ||
| const computeReferences = false; | ||
|
|
||
| // When | ||
| const result = getRecordNodeFromRecord({ | ||
| objectMetadataItems, | ||
| objectMetadataItem, | ||
| recordGqlFields, | ||
| record, | ||
| computeReferences, | ||
| }); | ||
|
|
||
| // Then | ||
| expect(result).toEqual({ | ||
| __typename: 'Person', | ||
| company: record.company, | ||
| name: { | ||
| __typename: 'FullName', | ||
| firstName: 'Alexandre', | ||
| lastName: 'Prot', | ||
| }, | ||
| }); | ||
| }); | ||
| }); |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
58 changes: 58 additions & 0 deletions
58
packages/twenty-front/src/utils/string/__tests__/pascalCase.test.ts
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,58 @@ | ||
| import { pascalCase } from '../pascalCase'; | ||
|
|
||
| describe('pascalCase', () => { | ||
| it('converts a string to pascal case', () => { | ||
| // Given | ||
| const input = 'HELLO_WORLD'; | ||
|
|
||
| // When | ||
| const result = pascalCase(input); | ||
|
|
||
| // Then | ||
| expect(result).toBe('HelloWorld'); | ||
| }); | ||
|
|
||
| it('handles empty strings', () => { | ||
| // Given | ||
| const input = ''; | ||
|
|
||
| // When | ||
| const result = pascalCase(input); | ||
|
|
||
| // Then | ||
| expect(result).toBe(''); | ||
| }); | ||
|
|
||
| it('handles strings with only one word', () => { | ||
| // Given | ||
| const input = 'hello'; | ||
|
|
||
| // When | ||
| const result = pascalCase(input); | ||
|
|
||
| // Then | ||
| expect(result).toBe('Hello'); | ||
| }); | ||
|
|
||
| it('handles strings with several words, spaces and special characters', () => { | ||
| // Given | ||
| const input = '& Hello world! How are you today? #'; | ||
|
|
||
| // When | ||
| const result = pascalCase(input); | ||
|
|
||
| // Then | ||
| expect(result).toBe('HelloWorldHowAreYouToday'); | ||
| }); | ||
|
|
||
| it('handles strings with leading and trailing spaces', () => { | ||
| // Given | ||
| const input = ' hello_world '; | ||
|
|
||
| // When | ||
| const result = pascalCase(input); | ||
|
|
||
| // Then | ||
| expect(result).toBe('HelloWorld'); | ||
| }); | ||
| }); |
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| import camelCase from 'lodash.camelcase'; | ||
|
|
||
| import { capitalize } from '~/utils/string/capitalize'; | ||
|
|
||
| export const pascalCase = (str: string) => capitalize(camelCase(str)); |