nodejs-quickstart-structure 1.18.1 → 1.19.1

package/templates/clean-architecture/js/src/interfaces/controllers/userController.spec.js.ejs

@@ -1,138 +1,234 @@
-const UserController = require('@/interfaces/controllers/userController');
-const CreateUser = require('@/usecases/CreateUser');
-const GetAllUsers = require('@/usecases/GetAllUsers');
-
-jest.mock('@/usecases/CreateUser');
-jest.mock('@/usecases/GetAllUsers');
-<%_ if (communication === 'Kafka') { -%>
-jest.mock('@/infrastructure/messaging/kafkaClient', () => ({
-    sendMessage: jest.fn().mockResolvedValue(undefined)
-}));
-<%_ } -%>
-
-
-describe('UserController (Clean Architecture)', () => {
-    let userController;
-    let mockCreateUserUseCase;
-    let mockGetAllUsersUseCase;
-<%_ if (communication !== 'GraphQL') { -%>
-    let mockRequest;
-    let mockResponse;
-    let mockNext;
-<%_ } -%>
-
-    beforeEach(() => {
-        jest.clearAllMocks();
-
-        userController = new UserController();
-
-        // Retrieve the mocked instances created inside UserController constructor
-        mockCreateUserUseCase = CreateUser.mock.instances[0];
-        mockGetAllUsersUseCase = GetAllUsers.mock.instances[0];
-
-<%_ if (communication !== 'GraphQL') { -%>
-        mockRequest = {
-            body: {}
-        };
-        mockResponse = {
-            status: jest.fn().mockReturnThis(),
-            json: jest.fn()
-        };
-        mockNext = jest.fn();
-<%_ } -%>
-    });
-
-    describe('getUsers', () => {
-        it('should return successfully (Happy Path)', async () => {
-            const usersMock = [{ id: '1', name: 'Test', email: 'test@example.com' }];
-            mockGetAllUsersUseCase.execute.mockResolvedValue(usersMock);
-
-<%_ if (communication === 'GraphQL') { -%>
-            const result = await userController.getUsers();
-            expect(result).toEqual(usersMock);
-<%_ } else { -%>
-            await userController.getUsers(mockRequest, mockResponse, mockNext);
-            expect(mockResponse.json).toHaveBeenCalledWith(usersMock);
-<%_ } -%>
-            expect(mockGetAllUsersUseCase.execute).toHaveBeenCalled();
-        });
-
-        it('should handle errors correctly (Error Handling)', async () => {
-            const error = new Error('UseCase Error');
-            mockGetAllUsersUseCase.execute.mockRejectedValue(error);
-
-<%_ if (communication === 'GraphQL') { -%>
-            await expect(userController.getUsers()).rejects.toThrow(error);
-<%_ } else { -%>
-            await userController.getUsers(mockRequest, mockResponse, mockNext);
-            expect(mockNext).toHaveBeenCalledWith(error);
-<%_ } -%>
-        });
-    });
-
-    describe('createUser', () => {
-        it('should successfully create a new user (Happy Path)', async () => {
-            const payload = { name: 'Alice', email: 'alice@example.com' };
-<%_ if (communication === 'GraphQL') { -%>
-            const dataArg = payload;
-<%_ } else { -%>
-            mockRequest.body = payload;
-<%_ } -%>
-            const expectedUser = { id: '1', ...payload };
-            
-            mockCreateUserUseCase.execute.mockResolvedValue(expectedUser);
-
-<%_ if (communication === 'GraphQL') { -%>
-            const result = await userController.createUser(dataArg);
-            expect(result).toEqual(expectedUser);
-<%_ } else { -%>
-            await userController.createUser(mockRequest, mockResponse, mockNext);
-<%_ if (communication === 'Kafka') { -%>
-            const { sendMessage } = require('@/infrastructure/messaging/kafkaClient');
-            expect(sendMessage).toHaveBeenCalled();
-<%_ } -%>
-            expect(mockResponse.status).toHaveBeenCalledWith(201);
-
-            expect(mockResponse.json).toHaveBeenCalledWith(expectedUser);
-<%_ } -%>
-            expect(mockCreateUserUseCase.execute).toHaveBeenCalledWith(payload.name, payload.email);
-        });
-
-        it('should handle errors when creation fails (Error Handling)', async () => {
-            const error = new Error('Creation Error');
-            mockCreateUserUseCase.execute.mockRejectedValue(error);
-
-<%_ if (communication === 'GraphQL') { -%>
-            await expect(userController.createUser({ name: 'Bob', email: 'bob@example.com' })).rejects.toThrow(error);
-<%_ } else { -%>
-            await userController.createUser(mockRequest, mockResponse, mockNext);
-            expect(mockNext).toHaveBeenCalledWith(error);
-<%_ } -%>
-        });
-
-<%_ if (communication === 'Kafka') { -%>
-        it('should successfully create a new user with _id for Kafka (Happy Path)', async () => {
-            const payload = { name: 'Bob', email: 'bob@example.com' };
-<% if (communication === 'GraphQL') { -%>
-            const dataArg = payload;
-<% } else { -%>
-            mockRequest.body = payload;
-<% } -%>
-            const expectedUser = { _id: '2', ...payload };
-            
-            mockCreateUserUseCase.execute.mockResolvedValue(expectedUser);
-
-<% if (communication === 'GraphQL') { -%>
-            await userController.createUser(dataArg);
-<% } else { -%>
-            await userController.createUser(mockRequest, mockResponse, mockNext);
-<% } -%>
-            const { sendMessage } = require('@/infrastructure/messaging/kafkaClient');
-            expect(sendMessage).toHaveBeenCalledWith(
-                'user-topic',
-                expect.stringContaining('"id":"2"')
-            );
-        });
-<%_ } -%>
-    });
-});
+const UserController = require('@/interfaces/controllers/userController');
+const ERROR_MESSAGES = require('@/utils/errorMessages');
+const CreateUser = require('@/usecases/CreateUser');
+const GetAllUsers = require('@/usecases/GetAllUsers');
+const UpdateUser = require('@/usecases/UpdateUser');
+const DeleteUser = require('@/usecases/DeleteUser');
+
+jest.mock('@/usecases/CreateUser');
+jest.mock('@/usecases/GetAllUsers');
+jest.mock('@/usecases/UpdateUser');
+jest.mock('@/usecases/DeleteUser');
+<%_ if (communication === 'Kafka') { -%>
+jest.mock('@/infrastructure/messaging/kafkaClient', () => ({
+    sendMessage: jest.fn().mockResolvedValue(undefined)
+}));
+<%_ } -%>
+
+
+describe('UserController (Clean Architecture)', () => {
+    let userController;
+    let mockCreateUserUseCase;
+    let mockGetAllUsersUseCase;
+    let mockUpdateUserUseCase;
+    let mockDeleteUserUseCase;
+<%_ if (communication !== 'GraphQL') { -%>
+    let mockRequest;
+    let mockResponse;
+    let mockNext;
+<%_ } -%>
+
+    beforeEach(() => {
+        jest.resetAllMocks();
+
+        userController = new UserController();
+
+        // Retrieve the mocked instances created inside UserController constructor
+        mockCreateUserUseCase = CreateUser.mock.instances[0];
+        mockGetAllUsersUseCase = GetAllUsers.mock.instances[0];
+        mockUpdateUserUseCase = UpdateUser.mock.instances[0];
+        mockDeleteUserUseCase = DeleteUser.mock.instances[0];
+
+<%_ if (communication !== 'GraphQL') { -%>
+        mockRequest = {
+            body: {},
+            params: {}
+        };
+        mockResponse = {
+            status: jest.fn().mockReturnThis(),
+            json: jest.fn()
+        };
+        mockNext = jest.fn();
+<%_ } -%>
+    });
+
+    describe('getUsers', () => {
+        it('should return successfully (Happy Path)', async () => {
+            const usersMock = [{ id: '1', name: 'Test', email: 'test@example.com' }];
+            mockGetAllUsersUseCase.execute.mockResolvedValue(usersMock);
+
+<%_ if (communication === 'GraphQL') { -%>
+            const result = await userController.getUsers();
+            expect(result).toEqual(usersMock);
+<%_ } else { -%>
+            await userController.getUsers(mockRequest, mockResponse, mockNext);
+            expect(mockResponse.json).toHaveBeenCalledWith(usersMock);
+<%_ } -%>
+            expect(mockGetAllUsersUseCase.execute).toHaveBeenCalled();
+        });
+
+        it('should handle errors correctly (Error Handling)', async () => {
+            const error = new Error('UseCase Error');
+            mockGetAllUsersUseCase.execute.mockRejectedValue(error);
+
+<%_ if (communication === 'GraphQL') { -%>
+            await expect(userController.getUsers()).rejects.toThrow(error);
+<%_ } else { -%>
+            await userController.getUsers(mockRequest, mockResponse, mockNext);
+            expect(mockNext).toHaveBeenCalledWith(error);
+<%_ } -%>
+        });
+    });
+
+    describe('createUser', () => {
+        it('should successfully create a new user (Happy Path)', async () => {
+            const payload = { name: 'Alice', email: 'alice@example.com' };
+<%_ if (communication === 'GraphQL') { -%>
+            const dataArg = payload;
+<%_ } else { -%>
+            mockRequest.body = payload;
+<%_ } -%>
+            const expectedUser = { id: '1', ...payload };
+            
+            mockCreateUserUseCase.execute.mockResolvedValue(expectedUser);
+
+<%_ if (communication === 'GraphQL') { -%>
+            const result = await userController.createUser(dataArg);
+            expect(result).toEqual(expectedUser);
+<%_ } else { -%>
+            await userController.createUser(mockRequest, mockResponse, mockNext);
+<%_ if (communication === 'Kafka') { -%>
+            const { sendMessage } = require('@/infrastructure/messaging/kafkaClient');
+            expect(sendMessage).toHaveBeenCalled();
+<%_ } -%>
+            expect(mockResponse.status).toHaveBeenCalledWith(201);
+
+            expect(mockResponse.json).toHaveBeenCalledWith(expectedUser);
+<%_ } -%>
+            expect(mockCreateUserUseCase.execute).toHaveBeenCalledWith(payload.name, payload.email);
+        });
+    });
+
+    describe('updateUser', () => {
+        it('should successfully update a user (Happy Path)', async () => {
+            const id = '1';
+            const payload = { name: 'Alice Updated' };
+<%_ if (communication === 'GraphQL') { -%>
+            const idArg = id;
+            const dataArg = payload;
+<%_ } else { -%>
+            mockRequest.params = { id };
+            mockRequest.body = payload;
+<%_ } -%>
+            const expectedUser = { id, ...payload };
+            
+            mockUpdateUserUseCase.execute.mockResolvedValue(expectedUser);
+
+<%_ if (communication === 'GraphQL') { -%>
+            const result = await userController.updateUser(idArg, dataArg);
+            expect(result.name).toBe(payload.name);
+<%_ } else { -%>
+            await userController.updateUser(mockRequest, mockResponse, mockNext);
+            expect(mockResponse.json).toHaveBeenCalled();
+<%_ } -%>
+<%_ if (communication === 'Kafka') { -%>
+            const { sendMessage } = require('@/infrastructure/messaging/kafkaClient');
+            expect(sendMessage).toHaveBeenCalled();
+<%_ } -%>
+            expect(mockUpdateUserUseCase.execute).toHaveBeenCalledWith(id, payload);
+        });
+
+        it('should handle database errors during update (Error Handling)', async () => {
+            const id = '1';
+            const error = new Error('Database Error');
+            mockUpdateUserUseCase.execute.mockRejectedValue(error);
+<%_ if (communication === 'GraphQL') { -%>
+            await expect(userController.updateUser(id, { name: 'Fail' })).rejects.toThrow(error);
+<%_ } else { -%>
+            await userController.updateUser(mockRequest, mockResponse, mockNext);
+            expect(mockNext).toHaveBeenCalledWith(error);
+<%_ } -%>
+        });
+    });
+
+    describe('deleteUser', () => {
+        it('should successfully delete a user (Happy Path)', async () => {
+            const id = '1';
+<%_ if (communication === 'GraphQL') { -%>
+            const idArg = id;
+<%_ } else { -%>
+            mockRequest.params = { id };
+<%_ } -%>
+            mockDeleteUserUseCase.execute.mockResolvedValue(true);
+
+<%_ if (communication === 'GraphQL') { -%>
+            const result = await userController.deleteUser(idArg);
+            expect(result).toBe(true);
+<%_ } else { -%>
+            await userController.deleteUser(mockRequest, mockResponse, mockNext);
+            expect(mockResponse.status).toHaveBeenCalledWith(200);
+<%_ } -%>
+<%_ if (communication === 'Kafka') { -%>
+            const { sendMessage } = require('@/infrastructure/messaging/kafkaClient');
+            expect(sendMessage).toHaveBeenCalled();
+<%_ } -%>
+            expect(mockDeleteUserUseCase.execute).toHaveBeenCalledWith(id);
+        });
+
+        it('should throw error if user not found during deletion (Error Handling)', async () => {
+            const id = '999';
+<%_ if (communication === 'GraphQL') { -%>
+            mockDeleteUserUseCase.execute.mockResolvedValue(false);
+            await expect(userController.deleteUser(id)).rejects.toThrow(ERROR_MESSAGES.USER_NOT_FOUND);
+<%_ } else { -%>
+            mockRequest.params = { id };
+            mockDeleteUserUseCase.execute.mockResolvedValue(false);
+            await userController.deleteUser(mockRequest, mockResponse, mockNext);
+            expect(mockResponse.status).toHaveBeenCalledWith(404);
+<%_ } -%>
+        });
+
+        it('should handle database errors during deletion (Error Handling)', async () => {
+            const id = '1';
+            const error = new Error('Database Error');
+            mockDeleteUserUseCase.execute.mockRejectedValue(error);
+<%_ if (communication === 'GraphQL') { -%>
+            await expect(userController.deleteUser(id)).rejects.toThrow(error);
+<%_ } else { -%>
+            mockRequest.params = { id };
+            await userController.deleteUser(mockRequest, mockResponse, mockNext);
+            expect(mockNext).toHaveBeenCalledWith(error);
+<%_ } -%>
+        });
+    });
+
+    describe('createUser Error Paths', () => {
+        it('should handle database errors during creation (Error Handling)', async () => {
+            const error = new Error('Database Error');
+            mockCreateUserUseCase.execute.mockRejectedValue(error);
+<%_ if (communication === 'GraphQL') { -%>
+            await expect(userController.createUser({ name: 'Alice', email: 'alice@example.com' })).rejects.toThrow(error);
+<%_ } else { -%>
+            mockRequest.body = { name: 'Alice', email: 'alice@example.com' };
+            await userController.createUser(mockRequest, mockResponse, mockNext);
+            expect(mockNext).toHaveBeenCalledWith(error);
+<%_ } -%>
+        });
+    });
+
+    describe('updateUser Error Paths', () => {
+        it('should handle database errors during update (Error Handling)', async () => {
+            const id = '1';
+            const error = new Error('Database Error');
+            mockUpdateUserUseCase.execute.mockRejectedValue(error);
+<%_ if (communication === 'GraphQL') { -%>
+            await expect(userController.updateUser(id, { name: 'Bob' })).rejects.toThrow(error);
+<%_ } else { -%>
+            mockRequest.params = { id };
+            mockRequest.body = { name: 'Bob' };
+            await userController.updateUser(mockRequest, mockResponse, mockNext);
+            expect(mockNext).toHaveBeenCalledWith(error);
+<%_ } -%>
+        });
+    });
+});

package/templates/clean-architecture/js/src/interfaces/graphql/context.js.ejs

@@ -1,13 +1,13 @@
-const UserRepository = require('../../infrastructure/repositories/UserRepository');
-
-const gqlContext = async ({ req }) => {
-  const token = req.headers.authorization || '';
-  const userRepository = new UserRepository();
-  
-  return { 
-    token, 
-    userRepository 
-  };
-};
-
-module.exports = { gqlContext };
+const UserRepository = require('../../infrastructure/repositories/UserRepository');
+
+const gqlContext = async ({ req }) => {
+  const token = req.headers.authorization || '';
+  const userRepository = new UserRepository();
+  
+  return { 
+    token, 
+    userRepository 
+  };
+};
+
+module.exports = { gqlContext };

package/templates/clean-architecture/js/src/interfaces/graphql/context.spec.js.ejs

@@ -1,31 +1,31 @@
-const { gqlContext } = require('@/interfaces/graphql/context');
-const { resolvers } = require('@/interfaces/graphql/resolvers');
-const { typeDefs } = require('@/interfaces/graphql/typeDefs');
-
-jest.mock('@/infrastructure/repositories/UserRepository');
-
-describe('GraphQL Context', () => {
-    it('should exercise GraphQL index entry points', () => {
-        expect(resolvers).toBeDefined();
-        expect(typeDefs).toBeDefined();
-    });
-  it('should return context with token when authorization header is present', async () => {
-    const mockRequest = {
-      headers: {
-        authorization: 'Bearer token123',
-      },
-    };
-
-    const context = await gqlContext({ req: mockRequest });
-    expect(context.token).toBe('Bearer token123');
-  });
-
-  it('should return context with empty token when authorization header is missing', async () => {
-    const mockRequest = {
-      headers: {},
-    };
-
-    const context = await gqlContext({ req: mockRequest });
-    expect(context.token).toBe('');
-  });
-});
+const { gqlContext } = require('@/interfaces/graphql/context');
+const { resolvers } = require('@/interfaces/graphql/resolvers');
+const { typeDefs } = require('@/interfaces/graphql/typeDefs');
+
+jest.mock('@/infrastructure/repositories/UserRepository');
+
+describe('GraphQL Context', () => {
+    it('should exercise GraphQL index entry points', () => {
+        expect(resolvers).toBeDefined();
+        expect(typeDefs).toBeDefined();
+    });
+  it('should return context with token when authorization header is present', async () => {
+    const mockRequest = {
+      headers: {
+        authorization: 'Bearer token123',
+      },
+    };
+
+    const context = await gqlContext({ req: mockRequest });
+    expect(context.token).toBe('Bearer token123');
+  });
+
+  it('should return context with empty token when authorization header is missing', async () => {
+    const mockRequest = {
+      headers: {},
+    };
+
+    const context = await gqlContext({ req: mockRequest });
+    expect(context.token).toBe('');
+  });
+});

package/templates/clean-architecture/js/src/interfaces/graphql/index.js.ejs

@@ -1,5 +1,5 @@
-const { typeDefs } = require('./typeDefs');
-const { resolvers } = require('./resolvers');
-const { gqlContext } = require('./context');
-
-module.exports = { typeDefs, resolvers, gqlContext };
+const { typeDefs } = require('./typeDefs');
+const { resolvers } = require('./resolvers');
+const { gqlContext } = require('./context');
+
+module.exports = { typeDefs, resolvers, gqlContext };

package/templates/clean-architecture/js/src/interfaces/graphql/resolvers/index.js.ejs

@@ -1,6 +1,6 @@
-const { mergeResolvers } = require('@graphql-tools/merge');
-const { userResolvers } = require('./user.resolvers');
-
-const resolvers = mergeResolvers([userResolvers]);
-
-module.exports = { resolvers };
+const { mergeResolvers } = require('@graphql-tools/merge');
+const { userResolvers } = require('./user.resolvers');
+
+const resolvers = mergeResolvers([userResolvers]);
+
+module.exports = { resolvers };

package/templates/clean-architecture/js/src/interfaces/graphql/resolvers/user.resolvers.js.ejs

@@ -1,21 +1,27 @@
-const UserController = require('../../controllers/userController');
-
-const userController = new UserController();
-
-const userResolvers = {
-  Query: {
-    getAllUsers: async () => {
-      return await userController.getUsers();
-    }
-  },
-  Mutation: {
-    createUser: async (_, { name, email }) => {
-      return await userController.createUser({ name, email });
-    }
-  }<%_ if (database === 'MongoDB') { -%>,
-  User: {
-    id: (parent) => parent.id || parent._id
-  }<%_ } %>
-};
-
-module.exports = { userResolvers };
+const UserController = require('../../controllers/userController');
+
+const userController = new UserController();
+
+const userResolvers = {
+  Query: {
+    getAllUsers: async () => {
+      return await userController.getUsers();
+    }
+  },
+  Mutation: {
+    createUser: async (_, { name, email }) => {
+      return await userController.createUser({ name, email });
+    },
+    updateUser: async (_, { id, name, email }) => {
+      return await userController.updateUser(id, { name, email });
+    },
+    deleteUser: async (_, { id }) => {
+      return await userController.deleteUser(id);
+    }
+  }<%_ if (database === 'MongoDB') { -%>,
+  User: {
+    id: (parent) => parent.id || parent._id
+  }<%_ } %>
+};
+
+module.exports = { userResolvers };