Skip to content

Commit

Permalink
fix: update unit-test to handle ownership system fields (#540)
Browse files Browse the repository at this point in the history
  • Loading branch information
maxmousse authored Dec 19, 2022
1 parent 4089576 commit 90609b7
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 7 deletions.
2 changes: 1 addition & 1 deletion hapify-models.json
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
"onUpdate": "Cascade",
"ownerStringPath": "user.id"
},
"name": "user",
"name": "owner",
"notes": "User that owns the answer",
"properties": ["internal", "ownership"],
"subtype": "oneMany",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,9 @@ describe('<<Model pascal>>Controller', () => {
<<< if (root.names.pascal !== 'User') { >>>const mockUser = mockUserFactory();<<< }>>>
mockReqUser.mockReturnValue(mockUser);
(mock<<Model pascal>>Service.getDefaultOwnerships as jest.Mock).mockReturnValue({
user: { connect: mockUser },
<<for Fields ownership and not primary and internal field>>
<<field camel>>: { connect: mockUser },
<<endfor>>
});
<<endif>>

Expand Down Expand Up @@ -182,7 +184,9 @@ describe('<<Model pascal>>Controller', () => {
<<< if (root.names.pascal !== 'User') { >>>const mockUser = mockUserFactory();<<< }>>>
mockReqUser.mockReturnValue(mockUser);
(mock<<Model pascal>>Service.getDefaultOwnerships as jest.Mock).mockReturnValue({
user: { connect: mockUser },
<<for Fields ownership and not primary and internal field>>
<<field camel>>: { connect: mockUser },
<<endfor>>
});
<<endif>>

Expand Down Expand Up @@ -706,7 +710,9 @@ describe('<<Model pascal>>Controller', () => {
<<< if (root.names.pascal !== 'User') { >>>const mockUser = mockUserFactory();<<< }>>>
mockReqUser.mockReturnValue(mockUser);
(mock<<Model pascal>>Service.getDefaultOwnerships as jest.Mock).mockReturnValue({
user: { connect: mockUser },
<<for Fields ownership and not primary and internal field>>
<<field camel>>: { connect: mockUser },
<<endfor>>
});

const { can, build } = new AbilityBuilder(AppAbility);
Expand Down Expand Up @@ -772,7 +778,9 @@ describe('<<Model pascal>>Controller', () => {
<<< if (root.names.pascal !== 'User') { >>>const mockUser = mockUserFactory();<<< }>>>
mockReqUser.mockReturnValue(mockUser);
(mock<<Model pascal>>Service.getDefaultOwnerships as jest.Mock).mockReturnValue({
user: { connect: mockUser },
<<for Fields ownership and not primary and internal field>>
<<field camel>>: { connect: mockUser },
<<endfor>>
});

const { can, build } = new AbilityBuilder(AppAbility);
Expand Down Expand Up @@ -838,7 +846,9 @@ describe('<<Model pascal>>Controller', () => {
<<< if (root.names.pascal !== 'User') { >>>const mockUser = mockUserFactory();<<< }>>>
mockReqUser.mockReturnValue(mockUser);
(mock<<Model pascal>>Service.getDefaultOwnerships as jest.Mock).mockReturnValue({
user: { connect: mockUser },
<<for Fields ownership and not primary and internal field>>
<<field camel>>: { connect: mockUser },
<<endfor>>
});

const { can, build } = new AbilityBuilder(AppAbility);
Expand Down Expand Up @@ -904,7 +914,9 @@ describe('<<Model pascal>>Controller', () => {
<<< if (root.names.pascal !== 'User') { >>>const mockUser = mockUserFactory();<<< }>>>
mockReqUser.mockReturnValue(mockUser);
(mock<<Model pascal>>Service.getDefaultOwnerships as jest.Mock).mockReturnValue({
user: { connect: mockUser },
<<for Fields ownership and not primary and internal field>>
<<field camel>>: { connect: mockUser },
<<endfor>>
});

const { can, build } = new AbilityBuilder(AppAbility);
Expand Down

0 comments on commit 90609b7

Please sign in to comment.