Skip to content

Commit

Permalink
refactor(api): move route
Browse files Browse the repository at this point in the history
to src/team/certification-center-membership
  • Loading branch information
EmmanuelleBonnemay authored Nov 12, 2024
1 parent a076f53 commit 4263f75
Show file tree
Hide file tree
Showing 5 changed files with 45 additions and 29 deletions.
23 changes: 0 additions & 23 deletions api/lib/application/certification-centers/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -249,29 +249,6 @@ const register = async function (server) {
],
},
},
{
method: 'GET',
path: '/api/certification-centers/{certificationCenterId}/members',
config: {
pre: [
{
method: securityPreHandlers.checkUserIsMemberOfCertificationCenter,
assign: 'isMemberOfCertificationCenter',
},
],
validate: {
params: Joi.object({
certificationCenterId: identifiersType.certificationCenterId,
}),
},
handler: certificationCenterController.findCertificationCenterMemberships,
notes: [
'- **Cette route est restreinte aux utilisateurs appartenant à un centre de certification**\n' +
"- Récupération de tous les membres d'un centre de certification.\n",
],
tags: ['api', 'certification-center', 'members'],
},
},
]);
};

Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
import Joi from 'joi';

import { securityPreHandlers } from '../../../shared/application/security-pre-handlers.js';
import { identifiersType } from '../../../shared/domain/types/identifiers-type.js';
import { certificationCenterMembershipController } from './certification-center-membership.controller.js';

export const certificationCenterMembershipRoute = [
{
method: 'GET',
path: '/api/certification-centers/{certificationCenterId}/members',
config: {
pre: [
{
method: securityPreHandlers.checkUserIsMemberOfCertificationCenter,
assign: 'isMemberOfCertificationCenter',
},
],
validate: {
params: Joi.object({
certificationCenterId: identifiersType.certificationCenterId,
}),
},
handler: (request, h) => certificationCenterMembershipController.findCertificationCenterMemberships(request, h),
notes: [
'- **Cette route est restreinte aux utilisateurs appartenant à un centre de certification**\n' +
"- Récupération de tous les membres d'un centre de certification.\n",
],
tags: ['api', 'team', 'certification-center', 'members'],
},
},
];
2 changes: 2 additions & 0 deletions api/src/team/application/routes.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import { adminMemberRoutes } from './admin-member/admin-member.route.js';
import { certificationCenterInvitationAdminRoutes } from './certification-center-invitation/certification-center-invitation.admin.route.js';
import { certificationCenterInvitationRoutes } from './certification-center-invitation/certification-center-invitation.route.js';
import { certificationCenterMembershipRoute } from './certification-center-membership/certification-center-membership.route.js';
import { membershipAdminRoutes } from './membership/membership.admin.route.js';
import { membershipRoutes } from './membership/membership.route.js';
import { organizationInvitationAdminRoutes } from './organization-invitations/organization-invitation.admin.route.js';
Expand All @@ -12,6 +13,7 @@ const register = async function (server) {
...adminMemberRoutes,
...certificationCenterInvitationRoutes,
...certificationCenterInvitationAdminRoutes,
...certificationCenterMembershipRoute,
...membershipAdminRoutes,
...membershipRoutes,
...prescriberInformationsRoute,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,13 @@ import {
generateValidRequestAuthorizationHeader,
} from '../../../test-helper.js';

describe('Acceptance | Application | Certification-centers | Routes', function () {
describe('Acceptance | Application | Certification center membership | Routes', function () {
let server, options;

describe('GET /api/certification-centers/{certificationCenterId}/members', function () {
it('should return 200 http status code', async function () {
// given
server = await createServer();
const certificationCenter = databaseBuilder.factory.buildCertificationCenter();
const certificationCenterMember = databaseBuilder.factory.buildUser();
const user2 = databaseBuilder.factory.buildUser();
Expand All @@ -21,9 +24,8 @@ describe('Acceptance | Application | Certification-centers | Routes', function (
userId: user2.id,
});
await databaseBuilder.commit();
const server = await createServer();

const options = {
options = {
headers: {
authorization: generateValidRequestAuthorizationHeader(certificationCenterMember.id),
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,13 @@ describe('Unit | Team | Application | Controller | CertificationCenterMembership
.returns(serializedCertificationCenterMembership);

// when
const response = await certificationCenterMembershipController.findCertificationCenterMemberships(request, hFake, {
certificationCenterMembershipSerializer: certificationCenterMembershipSerializerStub,
});
const response = await certificationCenterMembershipController.findCertificationCenterMemberships(
request,
hFake,
{
certificationCenterMembershipSerializer: certificationCenterMembershipSerializerStub,
},
);

// then
expect(usecases.findCertificationCenterMembershipsByCertificationCenter).to.have.been.calledOnce;
Expand Down

0 comments on commit 4263f75

Please sign in to comment.