feat(api): change create passage module check

by returning module for record purposes

Author: er-lim

api/src/devcomp/domain/usecases/create-passage.js

@@ -2,16 +2,16 @@ import { NotFoundError } from '../../../shared/domain/errors.js';
 import { ModuleDoesNotExistError } from '../errors.js';
 
 const createPassage = async function ({ moduleId, userId, moduleRepository, passageRepository, userRepository }) {
-  await _verifyIfModuleExists({ moduleId, moduleRepository });
+  await _getModule({ moduleId, moduleRepository });
   if (userId !== null) {
     await userRepository.get(userId);
   }
   return passageRepository.save({ moduleId, userId });
 };
 
-async function _verifyIfModuleExists({ moduleId, moduleRepository }) {
+async function _getModule({ moduleId, moduleRepository }) {
   try {
-    await moduleRepository.getBySlug({ slug: moduleId });
+    return await moduleRepository.getBySlug({ slug: moduleId });
   } catch (e) {
     if (e instanceof NotFoundError) {
       throw new ModuleDoesNotExistError();

api/tests/devcomp/unit/domain/usecases/create-passage_test.js

@@ -6,7 +6,7 @@ import { catchErr, expect, sinon } from '../../../../test-helper.js';
 
 describe('Unit | Devcomp | Domain | UseCases | create-passage', function () {
   describe('when module does not exist', function () {
-    it('should throw an ModuleNotExists', async function () {
+    it('should throw a ModuleDoesNotExist error', async function () {
       // given
       const moduleId = Symbol('moduleId');
 
@@ -24,7 +24,7 @@ describe('Unit | Devcomp | Domain | UseCases | create-passage', function () {
   });
 
   describe('when user does not exist', function () {
-    it('should throw an UserNotExists', async function () {
+    it('should throw an UserNotExists error', async function () {
       // given
       const userId = Symbol('userId');