Skip to content

Commit 5dc7527

Browse files
CCM-13135: Updated location to store DL clients
1 parent 5a43cff commit 5dc7527

File tree

2 files changed

+10
-12
lines changed

2 files changed

+10
-12
lines changed

utils/client-management/src/__test__/infra/client-repository/ssm.test.ts renamed to utils/client-management/src/__test__/infra/client-repository/repository.test.ts

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,6 @@ import { Client, IParameterStore } from 'utils';
33
import { mockDeep } from 'jest-mock-extended';
44
import { ClientRepository } from '../../../infra/client-repository/repository';
55

6-
// update name repository.test.ts
7-
86
let logger: any;
97

108
function setup() {
@@ -79,7 +77,7 @@ describe('putClient', () => {
7977

8078
expect(mocks.parameterStore.addParameter).toHaveBeenCalledTimes(1);
8179
expect(mocks.parameterStore.addParameter).toHaveBeenCalledWith(
82-
`/comms/${mocks.config.environment}/clients/${data.newClient.clientId}`,
80+
`/dl/${mocks.config.environment}/clients/${data.newClient.clientId}`,
8381
JSON.stringify(data.newClient),
8482
);
8583
});
@@ -100,7 +98,7 @@ describe('putClient', () => {
10098

10199
expect(mocks.parameterStore.addParameter).toHaveBeenCalledTimes(1);
102100
expect(mocks.parameterStore.addParameter).toHaveBeenCalledWith(
103-
`/comms/${mocks.config.environment}/clients/${existingClient.clientId}`,
101+
`/dl/${mocks.config.environment}/clients/${existingClient.clientId}`,
104102
JSON.stringify(updatedClient),
105103
);
106104
});
@@ -116,7 +114,7 @@ describe('listClients', () => {
116114

117115
expect(mocks.parameterStore.getAllParameters).toHaveBeenCalledTimes(1);
118116
expect(mocks.parameterStore.getAllParameters).toHaveBeenCalledWith(
119-
`/comms/${mocks.config.environment}/clients/`,
117+
`/dl/${mocks.config.environment}/clients/`,
120118
{ force: undefined },
121119
);
122120

@@ -128,16 +126,16 @@ describe('listClients', () => {
128126

129127
const invalidClientParams: Parameter[] = [
130128
{
131-
Name: `/comms/${mocks.config.environment}/clients/client-has-no-value`,
129+
Name: `/dl/${mocks.config.environment}/clients/client-has-no-value`,
132130
Version: 2,
133131
},
134132
{
135-
Name: `/comms/${mocks.config.environment}/clients/client-is-not-json`,
133+
Name: `/dl/${mocks.config.environment}/clients/client-is-not-json`,
136134
Value: 'not-a-client',
137135
Version: 2,
138136
},
139137
{
140-
Name: `/comms/${mocks.config.environment}/clients/client-is-missing-required-fields`,
138+
Name: `/dl/${mocks.config.environment}/clients/client-is-missing-required-fields`,
141139
Value: '{"clientId":"client-id"}',
142140
Version: 2,
143141
},
@@ -170,7 +168,7 @@ describe('getClient', () => {
170168

171169
expect(mocks.parameterStore.getParameter).toHaveBeenCalledTimes(1);
172170
expect(mocks.parameterStore.getParameter).toHaveBeenCalledWith(
173-
`/comms/${mocks.config.environment}/clients/${client.clientId}`,
171+
`/dl/${mocks.config.environment}/clients/${client.clientId}`,
174172
);
175173

176174
expect(result).toEqual(client);
@@ -223,7 +221,7 @@ describe('deleteClient', () => {
223221

224222
expect(mocks.parameterStore.deleteParameter).toHaveBeenCalledTimes(1);
225223
expect(mocks.parameterStore.deleteParameter).toHaveBeenCalledWith(
226-
`/comms/${mocks.config.environment}/clients/some-id`,
224+
`/dl/${mocks.config.environment}/clients/some-id`,
227225
);
228226

229227
expect(1).toEqual(1);
@@ -244,7 +242,7 @@ describe('deleteClient', () => {
244242

245243
expect(mocks.parameterStore.deleteParameter).toHaveBeenCalledTimes(1);
246244
expect(mocks.parameterStore.deleteParameter).toHaveBeenCalledWith(
247-
`/comms/${mocks.config.environment}/clients/some-id`,
245+
`/dl/${mocks.config.environment}/clients/some-id`,
248246
);
249247
});
250248

utils/client-management/src/infra/client-repository/repository.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ export class ClientRepository implements IClientRepository {
6161
}
6262

6363
private get parameterPathPrefix(): string {
64-
return `/comms/${this.config.environment}/clients/`;
64+
return `/dl/${this.config.environment}/clients/`;
6565
}
6666

6767
private getParameterName(id: string) {

0 commit comments

Comments
 (0)