Skip to content

Commit acb86c3

Browse files
committed
feat(api): migrate usecase and use into controller
1 parent 929eb27 commit acb86c3

File tree

5 files changed

+11
-7
lines changed

5 files changed

+11
-7
lines changed

api/src/prescription/learner-management/application/jobs/compute-certificability-job-controller.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { usecases } from '../../../../../lib/domain/usecases/index.js';
1+
import { usecases } from '../../../../../src/prescription/learner-management/domain/usecases/index.js';
22
import { JobController } from '../../../../shared/application/jobs/job-controller.js';
33
import { ComputeCertificabilityJob } from '../../domain/models/ComputeCertificabilityJob.js';
44

api/lib/domain/usecases/compute-organization-learner-certificability.js api/src/prescription/learner-management/domain/usecases/compute-organization-learner-certificability.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ const computeOrganizationLearnerCertificability = async function ({
33
organizationLearnerRepository,
44
placementProfileService,
55
}) {
6-
const organizationLearner = await organizationLearnerRepository.get(organizationLearnerId);
6+
const organizationLearner = await organizationLearnerRepository.getLearnerInfo(organizationLearnerId);
77

88
const placementProfile = await placementProfileService.getPlacementProfile({
99
userId: organizationLearner.userId,

api/src/prescription/learner-management/domain/usecases/index.js

+4
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import * as campaignRepository from '../../../../../lib/infrastructure/repositor
77
import * as libOrganizationLearnerRepository from '../../../../../lib/infrastructure/repositories/organization-learner-repository.js';
88
import * as userRepository from '../../../../identity-access-management/infrastructure/repositories/user.repository.js';
99
import * as organizationFeatureApi from '../../../../organizational-entities/application/api/organization-features-api.js';
10+
import * as placementProfileService from '../../../../shared/domain/services/placement-profile-service.js';
1011
import { logErrorWithCorrelationIds } from '../../../../shared/infrastructure/monitoring-tools.js';
1112
import * as organizationRepository from '../../../../shared/infrastructure/repositories/organization-repository.js';
1213
import { injectDependencies } from '../../../../shared/infrastructure/utils/dependency-injection.js';
@@ -46,6 +47,7 @@ import { importStorage } from '../../infrastructure/storage/import-storage.js';
4647
* @typedef {import ('../../infrastructure/repositories/organization-learner-import-format-repository.js')} OrganizationLearnerImportFormatRepository
4748
* @typedef {import ('../../infrastructure/repositories/organization-learner-repository.js')} OrganizationLearnerRepository
4849
* @typedef {import ('../../../../shared/infrastructure/repositories/organization-repository.js')} OrganizationRepository
50+
* @typedef {import ('../../../../shared/domain/services/placement-profile-service.js')} placementProfileService
4951
* @typedef {import('../../../organization-learner/infrastructure/repositories/registration-organization-learner-repository.js')} registrationOrganizationLearnerRepository
5052
* @typedef {import ('../../infrastructure/repositories/student-repository.js')} studentRepository
5153
* @typedef {import ('../../infrastructure/repositories/sup-organization-learner-repository.js')} SupOrganizationLearnerRepository
@@ -72,6 +74,7 @@ const dependencies = {
7274
organizationLearnerImportFormatRepository,
7375
organizationLearnerRepository,
7476
organizationRepository,
77+
placementProfileService,
7578
registrationOrganizationLearnerRepository,
7679
studentRepository,
7780
supOrganizationLearnerRepository,
@@ -96,6 +99,7 @@ const usecasesWithoutInjectedDependencies = {
9699

97100
/**
98101
* @typedef PrescriptionLearnerManagementUsecases
102+
* @property {computeOrganizationLearnerCertificability} computeOrganizationLearnerCertificability
99103
* @property {saveOrganizationLearnersFile} saveOrganizationLearnersFile
100104
* @property {sendOrganizationLearnersFile} sendOrganizationLearnersFile
101105
* @property {validateOrganizationLearnersFile} validateOrganizationLearnersFile

api/tests/prescription/learner-management/unit/application/jobs/compute-certificability-job-controller_test.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
import { usecases } from '../../../../../../lib/domain/usecases/index.js';
21
import { ComputeCertificabilityJobController } from '../../../../../../src/prescription/learner-management/application/jobs/compute-certificability-job-controller.js';
2+
import { usecases } from '../../../../../../src/prescription/learner-management/domain/usecases/index.js';
33
import { expect, sinon } from '../../../../../test-helper.js';
44

55
describe('Unit | Prescription | Application | Jobs | computeCertificabilityJobController', function () {

api/tests/unit/domain/usecases/compute-organization-learner-certificability_test.js api/tests/prescription/organization-learner/unit/domain/usecases/compute-organization-learner-certificability_test.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
import { usecases } from '../../../../lib/domain/usecases/index.js';
2-
import { domainBuilder, expect, sinon } from '../../../test-helper.js';
1+
import { usecases } from '../../../../../../src/prescription/learner-management/domain/usecases/index.js';
2+
import { domainBuilder, expect, sinon } from '../../../../../test-helper.js';
33

44
describe('Unit | UseCase | compute-organization-learner-certificabilty', function () {
55
let clock;
@@ -18,13 +18,13 @@ describe('Unit | UseCase | compute-organization-learner-certificabilty', functio
1818
it('should update certificability for an organization learner', async function () {
1919
// given
2020
const organizationLearnerRepository = {
21-
get: sinon.stub(),
21+
getLearnerInfo: sinon.stub(),
2222
updateCertificability: sinon.stub(),
2323
};
2424
const organizationLearnerId = 1;
2525
const organizationLearner = domainBuilder.buildOrganizationLearner({ id: organizationLearnerId });
2626
sinon.stub(organizationLearner, 'updateCertificability');
27-
organizationLearnerRepository.get.withArgs(organizationLearner.id).returns(organizationLearner);
27+
organizationLearnerRepository.getLearnerInfo.withArgs(organizationLearner.id).returns(organizationLearner);
2828
const placementProfileService = {
2929
getPlacementProfile: sinon.stub(),
3030
};

0 commit comments

Comments
 (0)