groot/automatisch
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test: Use nested serializers explicitly for serializer tests

Browse Source
This commit is contained in:
Faruk AYDIN
2024-02-25 23:34:41 +01:00
parent 74fbc937a1
commit 5aad68ec62
2 changed files with 12 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
packages/backend/src/serializers/role.test.js
View File

@@ -1,6 +1,7 @@
import { describe, it, expect, beforeEach } from 'vitest'; import { describe, it, expect, beforeEach } from 'vitest';
import { createRole } from '../../test/factories/role'; import { createRole } from '../../test/factories/role';
import roleSerializer from './role'; import roleSerializer from './role';
import permissionSerializer from './permission';
import { createPermission } from '../../test/factories/permission'; import { createPermission } from '../../test/factories/permission';
describe('roleSerializer', () => { describe('roleSerializer', () => {
@@ -40,7 +41,10 @@ describe('roleSerializer', () => {
role.permissions = [permissionOne, permissionTwo]; role.permissions = [permissionOne, permissionTwo];
const expectedPayload = { const expectedPayload = {
permissions: [permissionOne, permissionTwo], permissions: [
permissionSerializer(permissionOne),
permissionSerializer(permissionTwo),
],
}; };
expect(roleSerializer(role)).toMatchObject(expectedPayload); expect(roleSerializer(role)).toMatchObject(expectedPayload);

9
packages/backend/src/serializers/user.test.js
View File

@@ -4,6 +4,8 @@ import appConfig from '../config/app';
import { createUser } from '../../test/factories/user'; import { createUser } from '../../test/factories/user';
import { createPermission } from '../../test/factories/permission'; import { createPermission } from '../../test/factories/permission';
import userSerializer from './user'; import userSerializer from './user';
import roleSerializer from './role';
import permissionSerializer from './permission';
describe('userSerializer', () => { describe('userSerializer', () => {
let user, role, permissionOne, permissionTwo; let user, role, permissionOne, permissionTwo;
@@ -43,7 +45,7 @@ describe('userSerializer', () => {
user.role = role; user.role = role;
const expectedPayload = { const expectedPayload = {
role, role: roleSerializer(role),
}; };
expect(userSerializer(user)).toMatchObject(expectedPayload); expect(userSerializer(user)).toMatchObject(expectedPayload);
@@ -53,7 +55,10 @@ describe('userSerializer', () => {
user.permissions = [permissionOne, permissionTwo]; user.permissions = [permissionOne, permissionTwo];
const expectedPayload = { const expectedPayload = {
permissions: [permissionOne, permissionTwo], permissions: [
permissionSerializer(permissionOne),
permissionSerializer(permissionTwo),
],
}; };
expect(userSerializer(user)).toMatchObject(expectedPayload); expect(userSerializer(user)).toMatchObject(expectedPayload);