From 5aad68ec6218c4671e2846a33f9bd8b889ee2dc0 Mon Sep 17 00:00:00 2001
From: Faruk AYDIN <omerfaruk26@gmail.com>
Date: Sun, 25 Feb 2024 23:34:41 +0100
Subject: [PATCH] test: Use nested serializers explicitly for serializer tests

---
 packages/backend/src/serializers/role.test.js | 6 +++++-
 packages/backend/src/serializers/user.test.js | 9 +++++++--
 2 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/packages/backend/src/serializers/role.test.js b/packages/backend/src/serializers/role.test.js
index 900ad9c2..85f239ec 100644
--- a/packages/backend/src/serializers/role.test.js
+++ b/packages/backend/src/serializers/role.test.js
@@ -1,6 +1,7 @@
 import { describe, it, expect, beforeEach } from 'vitest';
 import { createRole } from '../../test/factories/role';
 import roleSerializer from './role';
+import permissionSerializer from './permission';
 import { createPermission } from '../../test/factories/permission';
 
 describe('roleSerializer', () => {
@@ -40,7 +41,10 @@ describe('roleSerializer', () => {
     role.permissions = [permissionOne, permissionTwo];
 
     const expectedPayload = {
-      permissions: [permissionOne, permissionTwo],
+      permissions: [
+        permissionSerializer(permissionOne),
+        permissionSerializer(permissionTwo),
+      ],
     };
 
     expect(roleSerializer(role)).toMatchObject(expectedPayload);
diff --git a/packages/backend/src/serializers/user.test.js b/packages/backend/src/serializers/user.test.js
index 07f6c8cd..8369bb60 100644
--- a/packages/backend/src/serializers/user.test.js
+++ b/packages/backend/src/serializers/user.test.js
@@ -4,6 +4,8 @@ import appConfig from '../config/app';
 import { createUser } from '../../test/factories/user';
 import { createPermission } from '../../test/factories/permission';
 import userSerializer from './user';
+import roleSerializer from './role';
+import permissionSerializer from './permission';
 
 describe('userSerializer', () => {
   let user, role, permissionOne, permissionTwo;
@@ -43,7 +45,7 @@ describe('userSerializer', () => {
     user.role = role;
 
     const expectedPayload = {
-      role,
+      role: roleSerializer(role),
     };
 
     expect(userSerializer(user)).toMatchObject(expectedPayload);
@@ -53,7 +55,10 @@ describe('userSerializer', () => {
     user.permissions = [permissionOne, permissionTwo];
 
     const expectedPayload = {
-      permissions: [permissionOne, permissionTwo],
+      permissions: [
+        permissionSerializer(permissionOne),
+        permissionSerializer(permissionTwo),
+      ],
     };
 
     expect(userSerializer(user)).toMatchObject(expectedPayload);