Mirrors/unleash.unleash
1
0
Fork 0
mirror of https://github.com/Unleash/unleash.git synced 2025-09-01 13:47:27 +02:00
Code Issues Projects Releases Wiki Activity

feat: update query to include CR titles

Browse Source
This commit is contained in:
Thomas Heartman 2023-11-22 13:11:03 +01:00
parent 548700e924
commit 8169ea6996
No known key found for this signature in database
GPG Key ID: BD1F880DAED1EE78
3 changed files with 28 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
src/lib/features/change-request-segment-usage-service/change-request-segment-usage-read-model.test.ts
View File

@ -10,6 +10,9 @@ let user: IUser;
const CR_ID = 123456;
const CR_ID_2 = 234567;
const CR_TITLE = 'My change request';
const FLAG_NAME = 'crarm-test-flag';
let readModel: IChangeRequestSegmentUsageReadModel;
@ -46,7 +49,11 @@ afterEach(async () => {
.delete();
});
const createCR = async (state, changeRequestId = CR_ID) => {
const createCR = async (
state,
changeRequestId = CR_ID,
changeRequestTitle: string | null = CR_TITLE,
) => {
await db.rawDatabase.table('change_requests').insert({
id: changeRequestId,
environment: 'default',
@ -55,7 +62,7 @@ const createCR = async (state, changeRequestId = CR_ID) => {
created_by: user.id,
created_at: '2023-01-01 00:00:00',
min_approvals: 1,
title: 'My change request',
title: changeRequestTitle,
});
};
@ -154,7 +161,7 @@ test.each([
strategyName: 'flexibleRollout',
environment: 'default',
featureName: FLAG_NAME,
changeRequestIds: [CR_ID],
changeRequests: [{ id: CR_ID, title: CR_TITLE }],
},
]);
} else {
@ -193,7 +200,7 @@ test.each([
strategyName: 'flexibleRollout',
environment: 'default',
featureName: FLAG_NAME,
changeRequestIds: [CR_ID],
changeRequests: [{ id: CR_ID, title: CR_TITLE }],
},
]);
} else {
@ -203,8 +210,8 @@ test.each([
);
test(`If the same strategy appears in multiple CRs with the same segment, they should all be listed in its changeRequestIds`, async () => {
await createCR('In review', CR_ID);
await createCR('In review', CR_ID_2);
await createCR('In review', CR_ID, CR_TITLE);
await createCR('In review', CR_ID_2, null);
const segmentId = 3;
const strategyId = randomId();
@ -228,8 +235,8 @@ test(`If the same strategy appears in multiple CRs with the same segment, they s
},
]);
const crIds = result[0].changeRequestIds;
expect(crIds).toContain(CR_ID);
expect(crIds).toContain(CR_ID_2);
expect(crIds).toHaveLength(2);
const crData = result[0].changeRequests;
expect(crData).toContainEqual({ id: CR_ID, title: CR_TITLE });
expect(crData).toContainEqual({ id: CR_ID_2, title: null });
expect(crData).toHaveLength(2);
});

4
src/lib/features/change-request-segment-usage-service/change-request-segment-usage-read-model.ts
View File

@ -1,9 +1,11 @@
type ChangeRequestInfo = { id: string; title: string | null };
type NewStrategy = {
projectId: string;
featureName: string;
strategyName: string;
environment: string;
changeRequestIds: [string, ...string[]];
changeRequests: [ChangeRequestInfo, ...ChangeRequestInfo[]];
};
type ExistingStrategy = NewStrategy & { id: string };

13
src/lib/features/change-request-segment-usage-service/sql-change-request-segment-usage-read-model.ts
View File

@ -17,7 +17,7 @@ export class ChangeRequestSegmentUsageReadModel
segmentId: number,
): Promise<ChangeRequestStrategy[]> {
const query = this.db.raw(
`SELECT events.*, cr.project, cr.environment
`SELECT events.*, cr.project, cr.environment, cr.title
FROM change_request_events events
JOIN change_requests cr ON events.change_request_id = cr.id
WHERE cr.state NOT IN ('Applied', 'Cancelled', 'Rejected')
@ -35,21 +35,24 @@ export class ChangeRequestSegmentUsageReadModel
environment: environment,
strategyName: payload.name,
...(payload.id ? { id: payload.id } : {}),
changeRequestId: row.change_request_id,
changeRequest: {
id: row.change_request_id,
title: row.title || null,
},
};
});
const deduped = strategies.reduce((acc, strategy) => {
const { changeRequestId, ...rest } = strategy;
const { changeRequest, ...rest } = strategy;
const existingData = acc[strategy.id];
if (existingData) {
existingData.changeRequestIds.push(strategy.changeRequestId);
existingData.changeRequests.push(changeRequest);
} else {
acc[strategy.id] = {
...rest,
changeRequestIds: [strategy.changeRequestId],
changeRequests: [changeRequest],
};
}