reviewflow 3.41.0 → 3.42.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +26 -0
- package/dist/dashboard/index.html +32 -226
- package/dist/dashboard/modules/i18n.d.ts.map +1 -1
- package/dist/dashboard/modules/i18n.js +16 -2
- package/dist/dashboard/modules/i18n.js.map +1 -1
- package/dist/dashboard/modules/statsCharts.d.ts.map +1 -1
- package/dist/dashboard/modules/statsCharts.js +43 -28
- package/dist/dashboard/modules/statsCharts.js.map +1 -1
- package/dist/dashboard/modules/statsPage.d.ts +15 -0
- package/dist/dashboard/modules/statsPage.d.ts.map +1 -0
- package/dist/dashboard/modules/statsPage.js +250 -0
- package/dist/dashboard/modules/statsPage.js.map +1 -0
- package/dist/dashboard/stats.html +371 -0
- package/dist/dashboard/styles.css +23 -8
- package/dist/main/routes.d.ts.map +1 -1
- package/dist/main/routes.js +14 -4
- package/dist/main/routes.js.map +1 -1
- package/dist/modules/statistics-insights/entities/stats/keyInsights.d.ts +4 -3
- package/dist/modules/statistics-insights/entities/stats/keyInsights.d.ts.map +1 -1
- package/dist/modules/statistics-insights/entities/stats/keyInsights.js +27 -7
- package/dist/modules/statistics-insights/entities/stats/keyInsights.js.map +1 -1
- package/dist/modules/statistics-insights/interface-adapters/presenters/statsSummary.presenter.d.ts +7 -0
- package/dist/modules/statistics-insights/interface-adapters/presenters/statsSummary.presenter.d.ts.map +1 -1
- package/dist/modules/statistics-insights/interface-adapters/presenters/statsSummary.presenter.js +24 -1
- package/dist/modules/statistics-insights/interface-adapters/presenters/statsSummary.presenter.js.map +1 -1
- package/dist/modules/tracking/interface-adapters/controllers/http/mrTracking.routes.d.ts +9 -1
- package/dist/modules/tracking/interface-adapters/controllers/http/mrTracking.routes.d.ts.map +1 -1
- package/dist/modules/tracking/interface-adapters/controllers/http/mrTracking.routes.js +14 -25
- package/dist/modules/tracking/interface-adapters/controllers/http/mrTracking.routes.js.map +1 -1
- package/dist/modules/tracking/usecases/tracking/markReviewAsMerged.usecase.d.ts +33 -0
- package/dist/modules/tracking/usecases/tracking/markReviewAsMerged.usecase.d.ts.map +1 -0
- package/dist/modules/tracking/usecases/tracking/markReviewAsMerged.usecase.js +42 -0
- package/dist/modules/tracking/usecases/tracking/markReviewAsMerged.usecase.js.map +1 -0
- package/dist/modules/worktree-management/interface-adapters/gateways/worktree.fileSystem.gateway.d.ts.map +1 -1
- package/dist/modules/worktree-management/interface-adapters/gateways/worktree.fileSystem.gateway.js +2 -1
- package/dist/modules/worktree-management/interface-adapters/gateways/worktree.fileSystem.gateway.js.map +1 -1
- package/dist/modules/worktree-management/usecases/ensureWorktree.usecase.d.ts +1 -0
- package/dist/modules/worktree-management/usecases/ensureWorktree.usecase.d.ts.map +1 -1
- package/dist/modules/worktree-management/usecases/ensureWorktree.usecase.js +17 -1
- package/dist/modules/worktree-management/usecases/ensureWorktree.usecase.js.map +1 -1
- package/dist/tests/acceptance/170-prebuilt-worktree-lifecycle.acceptance.test.js +3 -0
- package/dist/tests/acceptance/170-prebuilt-worktree-lifecycle.acceptance.test.js.map +1 -1
- package/dist/tests/acceptance/180-quality-threshold-block-approval.acceptance.test.js +7 -0
- package/dist/tests/acceptance/180-quality-threshold-block-approval.acceptance.test.js.map +1 -1
- package/dist/tests/acceptance/215-mark-any-review-as-merged.acceptance.test.d.ts +12 -0
- package/dist/tests/acceptance/215-mark-any-review-as-merged.acceptance.test.d.ts.map +1 -0
- package/dist/tests/acceptance/215-mark-any-review-as-merged.acceptance.test.js +158 -0
- package/dist/tests/acceptance/215-mark-any-review-as-merged.acceptance.test.js.map +1 -0
- package/dist/tests/acceptance/80-split-stats-service.acceptance.test.js +8 -1
- package/dist/tests/acceptance/80-split-stats-service.acceptance.test.js.map +1 -1
- package/dist/tests/units/dashboard/dashboardLayout.test.js +11 -18
- package/dist/tests/units/dashboard/dashboardLayout.test.js.map +1 -1
- package/dist/tests/units/dashboard/modules/i18n.test.js +13 -0
- package/dist/tests/units/dashboard/modules/i18n.test.js.map +1 -1
- package/dist/tests/units/dashboard/modules/statsCharts.test.js +37 -0
- package/dist/tests/units/dashboard/modules/statsCharts.test.js.map +1 -1
- package/dist/tests/units/dashboard/modules/statsPage.test.d.ts +2 -0
- package/dist/tests/units/dashboard/modules/statsPage.test.d.ts.map +1 -0
- package/dist/tests/units/dashboard/modules/statsPage.test.js +169 -0
- package/dist/tests/units/dashboard/modules/statsPage.test.js.map +1 -0
- package/dist/tests/units/interface-adapters/presenters/statsSummary.presenter.test.js +61 -1
- package/dist/tests/units/interface-adapters/presenters/statsSummary.presenter.test.js.map +1 -1
- package/dist/tests/units/modules/statistics-insights/entities/stats/keyInsights.test.js +49 -0
- package/dist/tests/units/modules/statistics-insights/entities/stats/keyInsights.test.js.map +1 -1
- package/dist/tests/units/modules/statistics-insights/entities/stats/reviewOutput.parser.test.d.ts.map +1 -0
- package/dist/tests/units/{entities → modules/statistics-insights/entities}/stats/reviewOutput.parser.test.js +1 -1
- package/dist/tests/units/modules/statistics-insights/entities/stats/reviewOutput.parser.test.js.map +1 -0
- package/dist/tests/units/modules/statistics-insights/interface-adapters/presenters/keyInsights.presenter.test.js +24 -0
- package/dist/tests/units/modules/statistics-insights/interface-adapters/presenters/keyInsights.presenter.test.js.map +1 -1
- package/dist/tests/units/modules/statistics-insights/usecases/stats/addReviewStats.counter.test.d.ts +2 -0
- package/dist/tests/units/modules/statistics-insights/usecases/stats/addReviewStats.counter.test.d.ts.map +1 -0
- package/dist/tests/units/modules/statistics-insights/usecases/stats/addReviewStats.counter.test.js +95 -0
- package/dist/tests/units/modules/statistics-insights/usecases/stats/addReviewStats.counter.test.js.map +1 -0
- package/dist/tests/units/modules/tracking/interface-adapters/controllers/http/mrTracking.markAsMerged.routes.test.d.ts +2 -0
- package/dist/tests/units/modules/tracking/interface-adapters/controllers/http/mrTracking.markAsMerged.routes.test.d.ts.map +1 -0
- package/dist/tests/units/modules/tracking/interface-adapters/controllers/http/mrTracking.markAsMerged.routes.test.js +108 -0
- package/dist/tests/units/modules/tracking/interface-adapters/controllers/http/mrTracking.markAsMerged.routes.test.js.map +1 -0
- package/dist/tests/units/modules/tracking/interface-adapters/controllers/http/mrTracking.routes.test.js +11 -5
- package/dist/tests/units/modules/tracking/interface-adapters/controllers/http/mrTracking.routes.test.js.map +1 -1
- package/dist/tests/units/modules/tracking/usecases/tracking/markReviewAsMerged.usecase.test.d.ts +2 -0
- package/dist/tests/units/modules/tracking/usecases/tracking/markReviewAsMerged.usecase.test.d.ts.map +1 -0
- package/dist/tests/units/modules/tracking/usecases/tracking/markReviewAsMerged.usecase.test.js +138 -0
- package/dist/tests/units/modules/tracking/usecases/tracking/markReviewAsMerged.usecase.test.js.map +1 -0
- package/dist/tests/units/modules/worktree-management/usecases/ensureWorktree.usecase.test.js +71 -34
- package/dist/tests/units/modules/worktree-management/usecases/ensureWorktree.usecase.test.js.map +1 -1
- package/package.json +1 -1
- package/dist/tests/acceptance/182-mark-pending-fix-as-merged.acceptance.test.d.ts +0 -19
- package/dist/tests/acceptance/182-mark-pending-fix-as-merged.acceptance.test.d.ts.map +0 -1
- package/dist/tests/acceptance/182-mark-pending-fix-as-merged.acceptance.test.js +0 -157
- package/dist/tests/acceptance/182-mark-pending-fix-as-merged.acceptance.test.js.map +0 -1
- package/dist/tests/units/entities/stats/reviewOutput.parser.test.d.ts.map +0 -1
- package/dist/tests/units/entities/stats/reviewOutput.parser.test.js.map +0 -1
- /package/dist/tests/units/{entities → modules/statistics-insights/entities}/stats/reviewOutput.parser.test.d.ts +0 -0
package/dist/tests/units/modules/statistics-insights/usecases/stats/addReviewStats.counter.test.js
ADDED
|
@@ -0,0 +1,95 @@
|
|
|
1
|
+
import { describe, it, expect, beforeEach } from 'vitest';
|
|
2
|
+
import { parseReviewOutput } from '../../../../../../modules/statistics-insights/entities/stats/reviewOutput.parser.js';
|
|
3
|
+
import { AddReviewStatsUseCase } from '../../../../../../modules/statistics-insights/usecases/stats/addReviewStats.usecase.js';
|
|
4
|
+
import { ProjectStatsFactory, ReviewStatsFactory } from '../../../../../../tests/factories/projectStats.factory.js';
|
|
5
|
+
import { InMemoryStatsGateway } from '../../../../../../tests/stubs/stats.stub.js';
|
|
6
|
+
const PROJECT = '/project';
|
|
7
|
+
const SCORED_REVIEW = '[REVIEW_STATS:blocking=1:warnings=2:suggestions=3:score=7.5]';
|
|
8
|
+
describe('AddReviewStatsUseCase cumulative counters', () => {
|
|
9
|
+
let statsGateway;
|
|
10
|
+
let useCase;
|
|
11
|
+
beforeEach(() => {
|
|
12
|
+
statsGateway = new InMemoryStatsGateway();
|
|
13
|
+
useCase = new AddReviewStatsUseCase(statsGateway);
|
|
14
|
+
});
|
|
15
|
+
it('moves scoredReviewCount by exactly one per recorded review, never double-counting', () => {
|
|
16
|
+
useCase.execute({
|
|
17
|
+
projectPath: PROJECT,
|
|
18
|
+
mrNumber: 1,
|
|
19
|
+
duration: 60000,
|
|
20
|
+
parsed: parseReviewOutput(SCORED_REVIEW),
|
|
21
|
+
});
|
|
22
|
+
useCase.execute({
|
|
23
|
+
projectPath: PROJECT,
|
|
24
|
+
mrNumber: 2,
|
|
25
|
+
duration: 60000,
|
|
26
|
+
parsed: parseReviewOutput(SCORED_REVIEW),
|
|
27
|
+
});
|
|
28
|
+
const stats = statsGateway.loadProjectStats(PROJECT);
|
|
29
|
+
expect(stats?.scoredReviewCount).toBe(2);
|
|
30
|
+
expect(stats?.totalScoreSum).toBeCloseTo(15, 1);
|
|
31
|
+
expect(stats?.totalBlocking).toBe(2);
|
|
32
|
+
expect(stats?.totalWarnings).toBe(4);
|
|
33
|
+
});
|
|
34
|
+
it('counts a re-review of an already-recorded mrNumber once, not twice', () => {
|
|
35
|
+
statsGateway.saveProjectStats(PROJECT, ProjectStatsFactory.create({
|
|
36
|
+
totalReviews: 1,
|
|
37
|
+
totalScoreSum: 6,
|
|
38
|
+
scoredReviewCount: 1,
|
|
39
|
+
totalBlocking: 1,
|
|
40
|
+
totalWarnings: 2,
|
|
41
|
+
averageScore: 6,
|
|
42
|
+
reviews: [ReviewStatsFactory.create({ id: 'r1', mrNumber: 7, score: 6 })],
|
|
43
|
+
}));
|
|
44
|
+
useCase.execute({
|
|
45
|
+
projectPath: PROJECT,
|
|
46
|
+
mrNumber: 7,
|
|
47
|
+
duration: 60000,
|
|
48
|
+
parsed: parseReviewOutput(SCORED_REVIEW),
|
|
49
|
+
});
|
|
50
|
+
const stats = statsGateway.loadProjectStats(PROJECT);
|
|
51
|
+
expect(stats?.scoredReviewCount).toBe(2);
|
|
52
|
+
expect(stats?.totalScoreSum).toBeCloseTo(13.5, 1);
|
|
53
|
+
expect(stats?.totalBlocking).toBe(2);
|
|
54
|
+
expect(stats?.totalWarnings).toBe(4);
|
|
55
|
+
});
|
|
56
|
+
it('skips the counter when the new review carries a null score', () => {
|
|
57
|
+
statsGateway.saveProjectStats(PROJECT, ProjectStatsFactory.create({
|
|
58
|
+
totalReviews: 1,
|
|
59
|
+
totalScoreSum: 8,
|
|
60
|
+
scoredReviewCount: 1,
|
|
61
|
+
averageScore: 8,
|
|
62
|
+
reviews: [ReviewStatsFactory.create({ id: 'r1', mrNumber: 1, score: 8 })],
|
|
63
|
+
}));
|
|
64
|
+
useCase.execute({
|
|
65
|
+
projectPath: PROJECT,
|
|
66
|
+
mrNumber: 2,
|
|
67
|
+
duration: 60000,
|
|
68
|
+
parsed: { score: null, blocking: 0, warnings: 0, suggestions: 0, categoryBreakdown: null },
|
|
69
|
+
});
|
|
70
|
+
const stats = statsGateway.loadProjectStats(PROJECT);
|
|
71
|
+
expect(stats?.scoredReviewCount).toBe(1);
|
|
72
|
+
expect(stats?.totalScoreSum).toBe(8);
|
|
73
|
+
});
|
|
74
|
+
it('accumulates scoredReviewCount monotonically past the 100-review trim window', () => {
|
|
75
|
+
const reviews = Array.from({ length: 100 }, (_, index) => ReviewStatsFactory.create({ id: `review-${index}`, mrNumber: index + 1, score: 7 }));
|
|
76
|
+
statsGateway.saveProjectStats(PROJECT, ProjectStatsFactory.create({
|
|
77
|
+
totalReviews: 300,
|
|
78
|
+
totalScoreSum: 2100,
|
|
79
|
+
scoredReviewCount: 300,
|
|
80
|
+
averageScore: 7,
|
|
81
|
+
reviews,
|
|
82
|
+
}));
|
|
83
|
+
useCase.execute({
|
|
84
|
+
projectPath: PROJECT,
|
|
85
|
+
mrNumber: 301,
|
|
86
|
+
duration: 60000,
|
|
87
|
+
parsed: parseReviewOutput(SCORED_REVIEW),
|
|
88
|
+
});
|
|
89
|
+
const stats = statsGateway.loadProjectStats(PROJECT);
|
|
90
|
+
expect(stats?.reviews).toHaveLength(100);
|
|
91
|
+
expect(stats?.scoredReviewCount).toBe(301);
|
|
92
|
+
expect(stats?.totalReviews).toBe(301);
|
|
93
|
+
});
|
|
94
|
+
});
|
|
95
|
+
//# sourceMappingURL=addReviewStats.counter.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"addReviewStats.counter.test.js","sourceRoot":"","sources":["../../../../../../../src/tests/units/modules/statistics-insights/usecases/stats/addReviewStats.counter.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAE1D,OAAO,EAAE,iBAAiB,EAAE,MAAM,qEAAqE,CAAC;AACxG,OAAO,EAAE,qBAAqB,EAAE,MAAM,wEAAwE,CAAC;AAC/G,OAAO,EAAE,mBAAmB,EAAE,kBAAkB,EAAE,MAAM,2CAA2C,CAAC;AACpG,OAAO,EAAE,oBAAoB,EAAE,MAAM,6BAA6B,CAAC;AAEnE,MAAM,OAAO,GAAG,UAAU,CAAC;AAC3B,MAAM,aAAa,GAAG,8DAA8D,CAAC;AAErF,QAAQ,CAAC,2CAA2C,EAAE,GAAG,EAAE;IACzD,IAAI,YAAkC,CAAC;IACvC,IAAI,OAA8B,CAAC;IAEnC,UAAU,CAAC,GAAG,EAAE;QACd,YAAY,GAAG,IAAI,oBAAoB,EAAE,CAAC;QAC1C,OAAO,GAAG,IAAI,qBAAqB,CAAC,YAAY,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mFAAmF,EAAE,GAAG,EAAE;QAC3F,OAAO,CAAC,OAAO,CAAC;YACd,WAAW,EAAE,OAAO;YACpB,QAAQ,EAAE,CAAC;YACX,QAAQ,EAAE,KAAK;YACf,MAAM,EAAE,iBAAiB,CAAC,aAAa,CAAC;SACzC,CAAC,CAAC;QACH,OAAO,CAAC,OAAO,CAAC;YACd,WAAW,EAAE,OAAO;YACpB,QAAQ,EAAE,CAAC;YACX,QAAQ,EAAE,KAAK;YACf,MAAM,EAAE,iBAAiB,CAAC,aAAa,CAAC;SACzC,CAAC,CAAC;QAEH,MAAM,KAAK,GAAG,YAAY,CAAC,gBAAgB,CAAC,OAAO,CAAC,CAAC;QACrD,MAAM,CAAC,KAAK,EAAE,iBAAiB,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACzC,MAAM,CAAC,KAAK,EAAE,aAAa,CAAC,CAAC,WAAW,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC;QAChD,MAAM,CAAC,KAAK,EAAE,aAAa,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACrC,MAAM,CAAC,KAAK,EAAE,aAAa,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oEAAoE,EAAE,GAAG,EAAE;QAC5E,YAAY,CAAC,gBAAgB,CAC3B,OAAO,EACP,mBAAmB,CAAC,MAAM,CAAC;YACzB,YAAY,EAAE,CAAC;YACf,aAAa,EAAE,CAAC;YAChB,iBAAiB,EAAE,CAAC;YACpB,aAAa,EAAE,CAAC;YAChB,aAAa,EAAE,CAAC;YAChB,YAAY,EAAE,CAAC;YACf,OAAO,EAAE,CAAC,kBAAkB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC;SAC1E,CAAC,CACH,CAAC;QAEF,OAAO,CAAC,OAAO,CAAC;YACd,WAAW,EAAE,OAAO;YACpB,QAAQ,EAAE,CAAC;YACX,QAAQ,EAAE,KAAK;YACf,MAAM,EAAE,iBAAiB,CAAC,aAAa,CAAC;SACzC,CAAC,CAAC;QAEH,MAAM,KAAK,GAAG,YAAY,CAAC,gBAAgB,CAAC,OAAO,CAAC,CAAC;QACrD,MAAM,CAAC,KAAK,EAAE,iBAAiB,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACzC,MAAM,CAAC,KAAK,EAAE,aAAa,CAAC,CAAC,WAAW,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC;QAClD,MAAM,CAAC,KAAK,EAAE,aAAa,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACrC,MAAM,CAAC,KAAK,EAAE,aAAa,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;QACpE,YAAY,CAAC,gBAAgB,CAC3B,OAAO,EACP,mBAAmB,CAAC,MAAM,CAAC;YACzB,YAAY,EAAE,CAAC;YACf,aAAa,EAAE,CAAC;YAChB,iBAAiB,EAAE,CAAC;YACpB,YAAY,EAAE,CAAC;YACf,OAAO,EAAE,CAAC,kBAAkB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CAAC;SAC1E,CAAC,CACH,CAAC;QAEF,OAAO,CAAC,OAAO,CAAC;YACd,WAAW,EAAE,OAAO;YACpB,QAAQ,EAAE,CAAC;YACX,QAAQ,EAAE,KAAK;YACf,MAAM,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,iBAAiB,EAAE,IAAI,EAAE;SAC3F,CAAC,CAAC;QAEH,MAAM,KAAK,GAAG,YAAY,CAAC,gBAAgB,CAAC,OAAO,CAAC,CAAC;QACrD,MAAM,CAAC,KAAK,EAAE,iBAAiB,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACzC,MAAM,CAAC,KAAK,EAAE,aAAa,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IACvC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6EAA6E,EAAE,GAAG,EAAE;QACrF,MAAM,OAAO,GAAG,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,GAAG,EAAE,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,EAAE,CACvD,kBAAkB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,UAAU,KAAK,EAAE,EAAE,QAAQ,EAAE,KAAK,GAAG,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,CAAC,CACpF,CAAC;QACF,YAAY,CAAC,gBAAgB,CAC3B,OAAO,EACP,mBAAmB,CAAC,MAAM,CAAC;YACzB,YAAY,EAAE,GAAG;YACjB,aAAa,EAAE,IAAI;YACnB,iBAAiB,EAAE,GAAG;YACtB,YAAY,EAAE,CAAC;YACf,OAAO;SACR,CAAC,CACH,CAAC;QAEF,OAAO,CAAC,OAAO,CAAC;YACd,WAAW,EAAE,OAAO;YACpB,QAAQ,EAAE,GAAG;YACb,QAAQ,EAAE,KAAK;YACf,MAAM,EAAE,iBAAiB,CAAC,aAAa,CAAC;SACzC,CAAC,CAAC;QAEH,MAAM,KAAK,GAAG,YAAY,CAAC,gBAAgB,CAAC,OAAO,CAAC,CAAC;QACrD,MAAM,CAAC,KAAK,EAAE,OAAO,CAAC,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC;QACzC,MAAM,CAAC,KAAK,EAAE,iBAAiB,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAC3C,MAAM,CAAC,KAAK,EAAE,YAAY,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"mrTracking.markAsMerged.routes.test.d.ts","sourceRoot":"","sources":["../../../../../../../../src/tests/units/modules/tracking/interface-adapters/controllers/http/mrTracking.markAsMerged.routes.test.ts"],"names":[],"mappings":""}
|
|
@@ -0,0 +1,108 @@
|
|
|
1
|
+
import Fastify from 'fastify';
|
|
2
|
+
import { beforeEach, describe, expect, it, vi } from 'vitest';
|
|
3
|
+
import { mrTrackingRoutes } from '../../../../../../../modules/tracking/interface-adapters/controllers/http/mrTracking.routes.js';
|
|
4
|
+
import { TrackedMrFactory } from '../../../../../../../tests/factories/trackedMr.factory.js';
|
|
5
|
+
import { createCapturingLogger } from '../../../../../../../tests/stubs/capturingLogger.stub.js';
|
|
6
|
+
import { StubReviewContextGateway } from '../../../../../../../tests/stubs/reviewContextGateway.stub.js';
|
|
7
|
+
import { InMemoryReviewRequestTrackingGateway } from '../../../../../../../tests/stubs/reviewRequestTracking.stub.js';
|
|
8
|
+
const projectPath = '/home/user/proj';
|
|
9
|
+
async function buildHarness() {
|
|
10
|
+
const trackingGateway = new InMemoryReviewRequestTrackingGateway();
|
|
11
|
+
const contextGateway = new StubReviewContextGateway();
|
|
12
|
+
const capturing = createCapturingLogger();
|
|
13
|
+
const cancelJob = vi.fn((_jobId) => false);
|
|
14
|
+
const app = Fastify();
|
|
15
|
+
await app.register(mrTrackingRoutes, {
|
|
16
|
+
reviewRequestTrackingGateway: trackingGateway,
|
|
17
|
+
getQualityThreshold: () => null,
|
|
18
|
+
reviewContextGateway: contextGateway,
|
|
19
|
+
cancelJob,
|
|
20
|
+
buildJobId: (platform, path, mrNumber) => `${platform}:${path}:${mrNumber}`,
|
|
21
|
+
removeWorktree: async () => ({ status: 'removed' }),
|
|
22
|
+
logger: capturing.logger,
|
|
23
|
+
});
|
|
24
|
+
return { app, trackingGateway, contextGateway, cancelJob };
|
|
25
|
+
}
|
|
26
|
+
function seedMr(harness, state) {
|
|
27
|
+
harness.trackingGateway.create(projectPath, TrackedMrFactory.create({ id: 'mr-42', state }));
|
|
28
|
+
}
|
|
29
|
+
describe('mrTrackingRoutes — POST /api/mr-tracking/mark-as-merged (SPEC-215)', () => {
|
|
30
|
+
let harness;
|
|
31
|
+
beforeEach(async () => {
|
|
32
|
+
harness = await buildHarness();
|
|
33
|
+
});
|
|
34
|
+
it('marks a pending-approval review as merged → 200', async () => {
|
|
35
|
+
seedMr(harness, 'pending-approval');
|
|
36
|
+
const response = await harness.app.inject({
|
|
37
|
+
method: 'POST',
|
|
38
|
+
url: '/api/mr-tracking/mark-as-merged',
|
|
39
|
+
payload: { mrId: 'mr-42', projectPath },
|
|
40
|
+
});
|
|
41
|
+
expect(response.statusCode).toBe(200);
|
|
42
|
+
const body = response.json();
|
|
43
|
+
expect(body.success).toBe(true);
|
|
44
|
+
expect(body.mrId).toBe('mr-42');
|
|
45
|
+
expect(harness.trackingGateway.getById(projectPath, 'mr-42')?.state).toBe('merged');
|
|
46
|
+
});
|
|
47
|
+
it('marks an approved review as merged → 200 (no current-state restriction)', async () => {
|
|
48
|
+
seedMr(harness, 'approved');
|
|
49
|
+
const response = await harness.app.inject({
|
|
50
|
+
method: 'POST',
|
|
51
|
+
url: '/api/mr-tracking/mark-as-merged',
|
|
52
|
+
payload: { mrId: 'mr-42', projectPath },
|
|
53
|
+
});
|
|
54
|
+
expect(response.statusCode).toBe(200);
|
|
55
|
+
expect(harness.trackingGateway.getById(projectPath, 'mr-42')?.state).toBe('merged');
|
|
56
|
+
});
|
|
57
|
+
it('returns 404 with a French message for an unknown review', async () => {
|
|
58
|
+
const response = await harness.app.inject({
|
|
59
|
+
method: 'POST',
|
|
60
|
+
url: '/api/mr-tracking/mark-as-merged',
|
|
61
|
+
payload: { mrId: 'ghost', projectPath },
|
|
62
|
+
});
|
|
63
|
+
expect(response.statusCode).toBe(404);
|
|
64
|
+
const body = response.json();
|
|
65
|
+
expect(body.success).toBe(false);
|
|
66
|
+
expect(body.error).toBe('MR non trouvée');
|
|
67
|
+
});
|
|
68
|
+
it('rejects an empty review id with 400 "mrId requis"', async () => {
|
|
69
|
+
const response = await harness.app.inject({
|
|
70
|
+
method: 'POST',
|
|
71
|
+
url: '/api/mr-tracking/mark-as-merged',
|
|
72
|
+
payload: { mrId: '', projectPath },
|
|
73
|
+
});
|
|
74
|
+
expect(response.statusCode).toBe(400);
|
|
75
|
+
expect(response.json().error).toBe('mrId requis');
|
|
76
|
+
});
|
|
77
|
+
it('rejects an invalid project path with 400 "Chemin invalide"', async () => {
|
|
78
|
+
const response = await harness.app.inject({
|
|
79
|
+
method: 'POST',
|
|
80
|
+
url: '/api/mr-tracking/mark-as-merged',
|
|
81
|
+
payload: { mrId: 'mr-42', projectPath: '../etc' },
|
|
82
|
+
});
|
|
83
|
+
expect(response.statusCode).toBe(400);
|
|
84
|
+
expect(response.json().error).toBe('Chemin invalide');
|
|
85
|
+
});
|
|
86
|
+
});
|
|
87
|
+
describe('mrTrackingRoutes — GET /api/mr-tracking exposes the merged list (SPEC-215)', () => {
|
|
88
|
+
let harness;
|
|
89
|
+
beforeEach(async () => {
|
|
90
|
+
harness = await buildHarness();
|
|
91
|
+
});
|
|
92
|
+
it('returns merged reviews under a "merged" field', async () => {
|
|
93
|
+
seedMr(harness, 'pending-fix');
|
|
94
|
+
await harness.app.inject({
|
|
95
|
+
method: 'POST',
|
|
96
|
+
url: '/api/mr-tracking/mark-as-merged',
|
|
97
|
+
payload: { mrId: 'mr-42', projectPath },
|
|
98
|
+
});
|
|
99
|
+
const response = await harness.app.inject({
|
|
100
|
+
method: 'GET',
|
|
101
|
+
url: `/api/mr-tracking?path=${encodeURIComponent(projectPath)}`,
|
|
102
|
+
});
|
|
103
|
+
const body = response.json();
|
|
104
|
+
expect(body.success).toBe(true);
|
|
105
|
+
expect(body.merged.map((mr) => mr.id)).toContain('mr-42');
|
|
106
|
+
});
|
|
107
|
+
});
|
|
108
|
+
//# sourceMappingURL=mrTracking.markAsMerged.routes.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"mrTracking.markAsMerged.routes.test.js","sourceRoot":"","sources":["../../../../../../../../src/tests/units/modules/tracking/interface-adapters/controllers/http/mrTracking.markAsMerged.routes.test.ts"],"names":[],"mappings":"AAAA,OAAO,OAAiC,MAAM,SAAS,CAAC;AACxD,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAG9D,OAAO,EAAE,gBAAgB,EAAE,MAAM,6EAA6E,CAAC;AAE/G,OAAO,EAAE,gBAAgB,EAAE,MAAM,wCAAwC,CAAC;AAC1E,OAAO,EAAE,qBAAqB,EAAE,MAAM,uCAAuC,CAAC;AAC9E,OAAO,EAAE,wBAAwB,EAAE,MAAM,4CAA4C,CAAC;AACtF,OAAO,EAAE,oCAAoC,EAAE,MAAM,6CAA6C,CAAC;AASnG,MAAM,WAAW,GAAG,iBAAiB,CAAC;AAEtC,KAAK,UAAU,YAAY;IACzB,MAAM,eAAe,GAAG,IAAI,oCAAoC,EAAE,CAAC;IACnE,MAAM,cAAc,GAAG,IAAI,wBAAwB,EAAE,CAAC;IACtD,MAAM,SAAS,GAAG,qBAAqB,EAAE,CAAC;IAC1C,MAAM,SAAS,GAAG,EAAE,CAAC,EAAE,CAAC,CAAC,MAAc,EAAW,EAAE,CAAC,KAAK,CAAC,CAAC;IAE5D,MAAM,GAAG,GAAG,OAAO,EAAE,CAAC;IACtB,MAAM,GAAG,CAAC,QAAQ,CAAC,gBAAgB,EAAE;QACnC,4BAA4B,EAAE,eAAe;QAC7C,mBAAmB,EAAE,GAAG,EAAE,CAAC,IAAI;QAC/B,oBAAoB,EAAE,cAAc;QACpC,SAAS;QACT,UAAU,EAAE,CAAC,QAAgB,EAAE,IAAY,EAAE,QAAgB,EAAU,EAAE,CACvE,GAAG,QAAQ,IAAI,IAAI,IAAI,QAAQ,EAAE;QACnC,cAAc,EAAE,KAAK,IAA2B,EAAE,CAAC,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC;QAC1E,MAAM,EAAE,SAAS,CAAC,MAAM;KACzB,CAAC,CAAC;IAEH,OAAO,EAAE,GAAG,EAAE,eAAe,EAAE,cAAc,EAAE,SAAS,EAAE,CAAC;AAC7D,CAAC;AAED,SAAS,MAAM,CAAC,OAAgB,EAAE,KAA8B;IAC9D,OAAO,CAAC,eAAe,CAAC,MAAM,CAAC,WAAW,EAAE,gBAAgB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,OAAO,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC;AAC/F,CAAC;AAED,QAAQ,CAAC,oEAAoE,EAAE,GAAG,EAAE;IAClF,IAAI,OAAgB,CAAC;IAErB,UAAU,CAAC,KAAK,IAAI,EAAE;QACpB,OAAO,GAAG,MAAM,YAAY,EAAE,CAAC;IACjC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;QAC/D,MAAM,CAAC,OAAO,EAAE,kBAAkB,CAAC,CAAC;QAEpC,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC;YACxC,MAAM,EAAE,MAAM;YACd,GAAG,EAAE,iCAAiC;YACtC,OAAO,EAAE,EAAE,IAAI,EAAE,OAAO,EAAE,WAAW,EAAE;SACxC,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAAyD,CAAC;QACpF,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAChC,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;QAChC,MAAM,CAAC,OAAO,CAAC,eAAe,CAAC,OAAO,CAAC,WAAW,EAAE,OAAO,CAAC,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IACtF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yEAAyE,EAAE,KAAK,IAAI,EAAE;QACvF,MAAM,CAAC,OAAO,EAAE,UAAU,CAAC,CAAC;QAE5B,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC;YACxC,MAAM,EAAE,MAAM;YACd,GAAG,EAAE,iCAAiC;YACtC,OAAO,EAAE,EAAE,IAAI,EAAE,OAAO,EAAE,WAAW,EAAE;SACxC,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,CAAC,OAAO,CAAC,eAAe,CAAC,OAAO,CAAC,WAAW,EAAE,OAAO,CAAC,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IACtF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yDAAyD,EAAE,KAAK,IAAI,EAAE;QACvE,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC;YACxC,MAAM,EAAE,MAAM;YACd,GAAG,EAAE,iCAAiC;YACtC,OAAO,EAAE,EAAE,IAAI,EAAE,OAAO,EAAE,WAAW,EAAE;SACxC,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAAyC,CAAC;QACpE,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACjC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC;IAC5C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;QACjE,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC;YACxC,MAAM,EAAE,MAAM;YACd,GAAG,EAAE,iCAAiC;YACtC,OAAO,EAAE,EAAE,IAAI,EAAE,EAAE,EAAE,WAAW,EAAE;SACnC,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,CAAE,QAAQ,CAAC,IAAI,EAAwB,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;IAC3E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;QAC1E,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC;YACxC,MAAM,EAAE,MAAM;YACd,GAAG,EAAE,iCAAiC;YACtC,OAAO,EAAE,EAAE,IAAI,EAAE,OAAO,EAAE,WAAW,EAAE,QAAQ,EAAE;SAClD,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,CAAE,QAAQ,CAAC,IAAI,EAAwB,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC;IAC/E,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,4EAA4E,EAAE,GAAG,EAAE;IAC1F,IAAI,OAAgB,CAAC;IAErB,UAAU,CAAC,KAAK,IAAI,EAAE;QACpB,OAAO,GAAG,MAAM,YAAY,EAAE,CAAC;IACjC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;QAC7D,MAAM,CAAC,OAAO,EAAE,aAAa,CAAC,CAAC;QAC/B,MAAM,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC;YACvB,MAAM,EAAE,MAAM;YACd,GAAG,EAAE,iCAAiC;YACtC,OAAO,EAAE,EAAE,IAAI,EAAE,OAAO,EAAE,WAAW,EAAE;SACxC,CAAC,CAAC;QAEH,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC;YACxC,MAAM,EAAE,KAAK;YACb,GAAG,EAAE,yBAAyB,kBAAkB,CAAC,WAAW,CAAC,EAAE;SAChE,CAAC,CAAC;QAEH,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAAyD,CAAC;QACpF,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAChC,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC;IAC5D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -3,6 +3,8 @@ import { describe, it, expect, beforeEach } from 'vitest';
|
|
|
3
3
|
import { mrTrackingRoutes } from '../../../../../../../modules/tracking/interface-adapters/controllers/http/mrTracking.routes.js';
|
|
4
4
|
import { ProjectStatsFactory, ReviewStatsFactory } from '../../../../../../../tests/factories/projectStats.factory.js';
|
|
5
5
|
import { TrackedMrFactory } from '../../../../../../../tests/factories/trackedMr.factory.js';
|
|
6
|
+
import { createCapturingLogger } from '../../../../../../../tests/stubs/capturingLogger.stub.js';
|
|
7
|
+
import { StubReviewContextGateway } from '../../../../../../../tests/stubs/reviewContextGateway.stub.js';
|
|
6
8
|
import { InMemoryReviewRequestTrackingGateway } from '../../../../../../../tests/stubs/reviewRequestTracking.stub.js';
|
|
7
9
|
import { InMemoryStatsGateway } from '../../../../../../../tests/stubs/stats.stub.js';
|
|
8
10
|
async function buildApp(options) {
|
|
@@ -11,6 +13,11 @@ async function buildApp(options) {
|
|
|
11
13
|
reviewRequestTrackingGateway: options.gateway,
|
|
12
14
|
getQualityThreshold: () => options.qualityThreshold,
|
|
13
15
|
statsGateway: options.statsGateway,
|
|
16
|
+
reviewContextGateway: new StubReviewContextGateway(),
|
|
17
|
+
cancelJob: () => false,
|
|
18
|
+
buildJobId: (platform, path, mrNumber) => `${platform}:${path}:${mrNumber}`,
|
|
19
|
+
removeWorktree: async () => ({ status: 'removed' }),
|
|
20
|
+
logger: createCapturingLogger().logger,
|
|
14
21
|
});
|
|
15
22
|
return app;
|
|
16
23
|
}
|
|
@@ -140,7 +147,7 @@ describe('mrTrackingRoutes — POST /api/mr-tracking/mark-as-merged', () => {
|
|
|
140
147
|
expect(updated?.state).toBe('merged');
|
|
141
148
|
expect(updated?.mergedAt).not.toBeNull();
|
|
142
149
|
});
|
|
143
|
-
it('returns
|
|
150
|
+
it('returns 200 and transitions an approved MR to merged (SPEC-215: any state)', async () => {
|
|
144
151
|
gateway.create(projectPath, TrackedMrFactory.create({ id: 'mr-1', state: 'approved' }));
|
|
145
152
|
const app = await buildApp({ gateway, qualityThreshold: null });
|
|
146
153
|
const response = await app.inject({
|
|
@@ -148,11 +155,10 @@ describe('mrTrackingRoutes — POST /api/mr-tracking/mark-as-merged', () => {
|
|
|
148
155
|
url: '/api/mr-tracking/mark-as-merged',
|
|
149
156
|
payload: { mrId: 'mr-1', projectPath },
|
|
150
157
|
});
|
|
151
|
-
expect(response.statusCode).toBe(
|
|
158
|
+
expect(response.statusCode).toBe(200);
|
|
152
159
|
const body = response.json();
|
|
153
|
-
expect(body.success).toBe(
|
|
154
|
-
expect(
|
|
155
|
-
expect(gateway.getById(projectPath, 'mr-1')?.state).toBe('approved');
|
|
160
|
+
expect(body.success).toBe(true);
|
|
161
|
+
expect(gateway.getById(projectPath, 'mr-1')?.state).toBe('merged');
|
|
156
162
|
});
|
|
157
163
|
});
|
|
158
164
|
describe('mrTrackingRoutes — GET /api/mr-tracking diff stats enrichment', () => {
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"mrTracking.routes.test.js","sourceRoot":"","sources":["../../../../../../../../src/tests/units/modules/tracking/interface-adapters/controllers/http/mrTracking.routes.test.ts"],"names":[],"mappings":"AAAA,OAAO,OAAiC,MAAM,SAAS,CAAC;AACxD,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAG1D,OAAO,EAAE,gBAAgB,EAAE,MAAM,6EAA6E,CAAC;
|
|
1
|
+
{"version":3,"file":"mrTracking.routes.test.js","sourceRoot":"","sources":["../../../../../../../../src/tests/units/modules/tracking/interface-adapters/controllers/http/mrTracking.routes.test.ts"],"names":[],"mappings":"AAAA,OAAO,OAAiC,MAAM,SAAS,CAAC;AACxD,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAG1D,OAAO,EAAE,gBAAgB,EAAE,MAAM,6EAA6E,CAAC;AAE/G,OAAO,EAAE,mBAAmB,EAAE,kBAAkB,EAAE,MAAM,2CAA2C,CAAC;AACpG,OAAO,EAAE,gBAAgB,EAAE,MAAM,wCAAwC,CAAC;AAC1E,OAAO,EAAE,qBAAqB,EAAE,MAAM,uCAAuC,CAAC;AAC9E,OAAO,EAAE,wBAAwB,EAAE,MAAM,4CAA4C,CAAC;AACtF,OAAO,EAAE,oCAAoC,EAAE,MAAM,6CAA6C,CAAC;AACnG,OAAO,EAAE,oBAAoB,EAAE,MAAM,6BAA6B,CAAC;AAQnE,KAAK,UAAU,QAAQ,CAAC,OAAwB;IAC9C,MAAM,GAAG,GAAG,OAAO,EAAE,CAAC;IACtB,MAAM,GAAG,CAAC,QAAQ,CAAC,gBAAgB,EAAE;QACnC,4BAA4B,EAAE,OAAO,CAAC,OAAO;QAC7C,mBAAmB,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,gBAAgB;QACnD,YAAY,EAAE,OAAO,CAAC,YAAY;QAClC,oBAAoB,EAAE,IAAI,wBAAwB,EAAE;QACpD,SAAS,EAAE,GAAG,EAAE,CAAC,KAAK;QACtB,UAAU,EAAE,CAAC,QAAgB,EAAE,IAAY,EAAE,QAAgB,EAAU,EAAE,CACvE,GAAG,QAAQ,IAAI,IAAI,IAAI,QAAQ,EAAE;QACnC,cAAc,EAAE,KAAK,IAA2B,EAAE,CAAC,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC;QAC1E,MAAM,EAAE,qBAAqB,EAAE,CAAC,MAAM;KACvC,CAAC,CAAC;IACH,OAAO,GAAG,CAAC;AACb,CAAC;AAED,SAAS,WAAW,CAAC,YAAkC,EAAE;IACvD,OAAO;QACL,IAAI,EAAE,QAAQ;QACd,SAAS,EAAE,sBAAsB;QACjC,UAAU,EAAE,KAAK;QACjB,KAAK,EAAE,CAAC;QACR,QAAQ,EAAE,CAAC;QACX,QAAQ,EAAE,CAAC;QACX,WAAW,EAAE,CAAC;QACd,aAAa,EAAE,CAAC;QAChB,aAAa,EAAE,CAAC;QAChB,SAAS,EAAE,IAAI;QACf,GAAG,SAAS;KACb,CAAC;AACJ,CAAC;AAED,QAAQ,CAAC,+DAA+D,EAAE,GAAG,EAAE;IAC7E,MAAM,WAAW,GAAG,eAAe,CAAC;IACpC,IAAI,OAA6C,CAAC;IAElD,UAAU,CAAC,GAAG,EAAE;QACd,OAAO,GAAG,IAAI,oCAAoC,EAAE,CAAC;IACvD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,uEAAuE,EAAE,KAAK,IAAI,EAAE;QACrF,OAAO,CAAC,MAAM,CACZ,WAAW,EACX,gBAAgB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,MAAM,EAAE,WAAW,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CACxE,CAAC;QACF,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,gBAAgB,EAAE,CAAC,EAAE,CAAC,CAAC;QAE7D,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;YAChC,MAAM,EAAE,MAAM;YACd,GAAG,EAAE,0BAA0B;YAC/B,OAAO,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,WAAW,EAAE;SACvC,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAA0B,CAAC;QACrD,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAChC,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,WAAW,EAAE,MAAM,CAAC,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;IACvE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC7E,OAAO,CAAC,MAAM,CACZ,WAAW,EACX,gBAAgB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,MAAM,EAAE,WAAW,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CACxE,CAAC;QACF,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,gBAAgB,EAAE,CAAC,EAAE,CAAC,CAAC;QAE7D,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;YAChC,MAAM,EAAE,MAAM;YACd,GAAG,EAAE,0BAA0B;YAC/B,OAAO,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,WAAW,EAAE;SACvC,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAAyC,CAAC;QACpE,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACjC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,yCAAyC,CAAC,CAAC;QACnE,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,WAAW,EAAE,MAAM,CAAC,EAAE,KAAK,CAAC,CAAC,GAAG,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;IAC3E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2DAA2D,EAAE,KAAK,IAAI,EAAE;QACzE,OAAO,CAAC,MAAM,CACZ,WAAW,EACX,gBAAgB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,MAAM,EAAE,WAAW,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CACxE,CAAC;QACF,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,gBAAgB,EAAE,CAAC,EAAE,CAAC,CAAC;QAE7D,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;YAChC,MAAM,EAAE,MAAM;YACd,GAAG,EAAE,0BAA0B;YAC/B,OAAO,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,WAAW,EAAE;SACvC,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAAyC,CAAC;QACpE,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACjC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,gCAAgC,CAAC,CAAC;IAC5D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sEAAsE,EAAE,KAAK,IAAI,EAAE;QACpF,OAAO,CAAC,MAAM,CACZ,WAAW,EACX,gBAAgB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,MAAM,EAAE,WAAW,EAAE,IAAI,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CAC3E,CAAC;QACF,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,gBAAgB,EAAE,CAAC,EAAE,CAAC,CAAC;QAE7D,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;YAChC,MAAM,EAAE,MAAM;YACd,GAAG,EAAE,0BAA0B;YAC/B,OAAO,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,WAAW,EAAE;SACvC,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,WAAW,EAAE,MAAM,CAAC,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;IACvE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;QACnE,OAAO,CAAC,MAAM,CACZ,WAAW,EACX,gBAAgB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,MAAM,EAAE,WAAW,EAAE,CAAC,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC,CACxE,CAAC;QACF,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,gBAAgB,EAAE,IAAI,EAAE,CAAC,CAAC;QAEhE,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;YAChC,MAAM,EAAE,MAAM;YACd,GAAG,EAAE,0BAA0B;YAC/B,OAAO,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,WAAW,EAAE;SACvC,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,WAAW,EAAE,MAAM,CAAC,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;IACvE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wCAAwC,EAAE,KAAK,IAAI,EAAE;QACtD,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,gBAAgB,EAAE,CAAC,EAAE,CAAC,CAAC;QAE7D,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;YAChC,MAAM,EAAE,MAAM;YACd,GAAG,EAAE,0BAA0B;YAC/B,OAAO,EAAE,EAAE,IAAI,EAAE,SAAS,EAAE,WAAW,EAAE;SAC1C,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAA0B,CAAC;QACrD,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,KAAK,IAAI,EAAE;QACvD,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,gBAAgB,EAAE,IAAI,EAAE,CAAC,CAAC;QAEhE,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;YAChC,MAAM,EAAE,MAAM;YACd,GAAG,EAAE,0BAA0B;YAC/B,OAAO,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE;SAC1B,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,yDAAyD,EAAE,GAAG,EAAE;IACvE,MAAM,WAAW,GAAG,eAAe,CAAC;IACpC,IAAI,OAA6C,CAAC;IAElD,UAAU,CAAC,GAAG,EAAE;QACd,OAAO,GAAG,IAAI,oCAAoC,EAAE,CAAC;IACvD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wDAAwD,EAAE,KAAK,IAAI,EAAE;QACtE,OAAO,CAAC,MAAM,CAAC,WAAW,EAAE,gBAAgB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,aAAa,EAAE,CAAC,CAAC,CAAC;QAC3F,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,gBAAgB,EAAE,IAAI,EAAE,CAAC,CAAC;QAEhE,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;YAChC,MAAM,EAAE,MAAM;YACd,GAAG,EAAE,iCAAiC;YACtC,OAAO,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,WAAW,EAAE;SACvC,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAAwC,CAAC;QACnE,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAChC,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QAC/B,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC,WAAW,EAAE,MAAM,CAAC,CAAC;QACrD,MAAM,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACtC,MAAM,CAAC,OAAO,EAAE,QAAQ,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;IAC3C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;QAC1F,OAAO,CAAC,MAAM,CAAC,WAAW,EAAE,gBAAgB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC;QACxF,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,gBAAgB,EAAE,IAAI,EAAE,CAAC,CAAC;QAEhE,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;YAChC,MAAM,EAAE,MAAM;YACd,GAAG,EAAE,iCAAiC;YACtC,OAAO,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,WAAW,EAAE;SACvC,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAA0B,CAAC;QACrD,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAChC,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,WAAW,EAAE,MAAM,CAAC,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IACrE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,+DAA+D,EAAE,GAAG,EAAE;IAC7E,MAAM,WAAW,GAAG,eAAe,CAAC;IAEpC,EAAE,CAAC,sEAAsE,EAAE,KAAK,IAAI,EAAE;QACpF,MAAM,OAAO,GAAG,IAAI,oCAAoC,EAAE,CAAC;QAC3D,OAAO,CAAC,MAAM,CACZ,WAAW,EACX,gBAAgB,CAAC,MAAM,CAAC;YACtB,EAAE,EAAE,MAAM;YACV,QAAQ,EAAE,IAAI;YACd,KAAK,EAAE,aAAa;YACpB,OAAO,EAAE,CAAC,WAAW,EAAE,CAAC;SACzB,CAAC,CACH,CAAC;QACF,MAAM,YAAY,GAAG,IAAI,oBAAoB,EAAE,CAAC;QAChD,YAAY,CAAC,gBAAgB,CAC3B,WAAW,EACX,mBAAmB,CAAC,WAAW,CAAC;YAC9B,kBAAkB,CAAC,MAAM,CAAC;gBACxB,QAAQ,EAAE,IAAI;gBACd,SAAS,EAAE,EAAE,YAAY,EAAE,CAAC,EAAE,SAAS,EAAE,GAAG,EAAE,SAAS,EAAE,EAAE,EAAE;aAC9D,CAAC;SACH,CAAC,CACH,CAAC;QACF,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,gBAAgB,EAAE,IAAI,EAAE,YAAY,EAAE,CAAC,CAAC;QAE9E,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;YAChC,MAAM,EAAE,KAAK;YACb,GAAG,EAAE,yBAAyB,kBAAkB,CAAC,WAAW,CAAC,EAAE;SAChE,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAEzB,CAAC;QACF,MAAM,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,SAAS,CAAC,CAAC,OAAO,CAAC;YAC3D,YAAY,EAAE,CAAC;YACf,SAAS,EAAE,GAAG;YACd,SAAS,EAAE,EAAE;SACd,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC7E,MAAM,OAAO,GAAG,IAAI,oCAAoC,EAAE,CAAC;QAC3D,OAAO,CAAC,MAAM,CACZ,WAAW,EACX,gBAAgB,CAAC,MAAM,CAAC;YACtB,EAAE,EAAE,MAAM;YACV,QAAQ,EAAE,IAAI;YACd,KAAK,EAAE,aAAa;YACpB,OAAO,EAAE,CAAC,WAAW,EAAE,CAAC;SACzB,CAAC,CACH,CAAC;QACF,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,gBAAgB,EAAE,IAAI,EAAE,CAAC,CAAC;QAEhE,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;YAChC,MAAM,EAAE,KAAK;YACb,GAAG,EAAE,yBAAyB,kBAAkB,CAAC,WAAW,CAAC,EAAE;SAChE,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACtC,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAEzB,CAAC;QACF,MAAM,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,SAAS,CAAC,CAAC,QAAQ,EAAE,CAAC;IAClE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
package/dist/tests/units/modules/tracking/usecases/tracking/markReviewAsMerged.usecase.test.d.ts.map
ADDED
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"markReviewAsMerged.usecase.test.d.ts","sourceRoot":"","sources":["../../../../../../../src/tests/units/modules/tracking/usecases/tracking/markReviewAsMerged.usecase.test.ts"],"names":[],"mappings":""}
|
package/dist/tests/units/modules/tracking/usecases/tracking/markReviewAsMerged.usecase.test.js
ADDED
|
@@ -0,0 +1,138 @@
|
|
|
1
|
+
import { describe, it, expect, vi } from 'vitest';
|
|
2
|
+
import { MarkReviewAsMergedUseCase, } from '../../../../../../modules/tracking/usecases/tracking/markReviewAsMerged.usecase.js';
|
|
3
|
+
import { ReviewContextFactory } from '../../../../../../tests/factories/reviewContext.factory.js';
|
|
4
|
+
import { TrackedMrFactory } from '../../../../../../tests/factories/trackedMr.factory.js';
|
|
5
|
+
import { createCapturingLogger } from '../../../../../../tests/stubs/capturingLogger.stub.js';
|
|
6
|
+
import { StubReviewContextGateway } from '../../../../../../tests/stubs/reviewContextGateway.stub.js';
|
|
7
|
+
import { InMemoryReviewRequestTrackingGateway } from '../../../../../../tests/stubs/reviewRequestTracking.stub.js';
|
|
8
|
+
const projectPath = '/home/user/proj';
|
|
9
|
+
function buildHarness(overrides) {
|
|
10
|
+
const trackingGateway = new InMemoryReviewRequestTrackingGateway();
|
|
11
|
+
const contextGateway = new StubReviewContextGateway();
|
|
12
|
+
const capturing = createCapturingLogger();
|
|
13
|
+
const cancelJob = vi.fn((_jobId) => false);
|
|
14
|
+
const buildJobId = vi.fn((platform, path, mrNumber) => `${platform}:${path}:${mrNumber}`);
|
|
15
|
+
const removeWorktree = vi.fn(async () => ({ status: 'removed' }));
|
|
16
|
+
const deps = {
|
|
17
|
+
trackingGateway,
|
|
18
|
+
reviewContextGateway: contextGateway,
|
|
19
|
+
cancelJob,
|
|
20
|
+
buildJobId,
|
|
21
|
+
removeWorktree,
|
|
22
|
+
logger: capturing.logger,
|
|
23
|
+
...overrides,
|
|
24
|
+
};
|
|
25
|
+
return {
|
|
26
|
+
useCase: new MarkReviewAsMergedUseCase(deps),
|
|
27
|
+
trackingGateway,
|
|
28
|
+
contextGateway,
|
|
29
|
+
cancelJob,
|
|
30
|
+
buildJobId,
|
|
31
|
+
removeWorktree,
|
|
32
|
+
warnMessages: capturing.warnMessages,
|
|
33
|
+
};
|
|
34
|
+
}
|
|
35
|
+
function seedMr(harness, state, overrides = {}) {
|
|
36
|
+
harness.trackingGateway.create(projectPath, TrackedMrFactory.create({ id: 'mr-42', state, ...overrides }));
|
|
37
|
+
}
|
|
38
|
+
describe('MarkReviewAsMergedUseCase', () => {
|
|
39
|
+
const states = [
|
|
40
|
+
'pending-review',
|
|
41
|
+
'pending-fix',
|
|
42
|
+
'pending-approval',
|
|
43
|
+
'approved',
|
|
44
|
+
'closed',
|
|
45
|
+
];
|
|
46
|
+
for (const state of states) {
|
|
47
|
+
it(`transitions a ${state} review to merged with a mergedAt timestamp`, async () => {
|
|
48
|
+
const harness = buildHarness();
|
|
49
|
+
seedMr(harness, state);
|
|
50
|
+
const result = await harness.useCase.execute({ projectPath, mrId: 'mr-42' });
|
|
51
|
+
expect(result.ok).toBe(true);
|
|
52
|
+
const updated = harness.trackingGateway.getById(projectPath, 'mr-42');
|
|
53
|
+
expect(updated?.state).toBe('merged');
|
|
54
|
+
expect(updated?.mergedAt).not.toBeNull();
|
|
55
|
+
});
|
|
56
|
+
}
|
|
57
|
+
it('retains the record instead of archiving it', async () => {
|
|
58
|
+
const harness = buildHarness();
|
|
59
|
+
seedMr(harness, 'pending-fix');
|
|
60
|
+
const archiveSpy = vi.spyOn(harness.trackingGateway, 'archive');
|
|
61
|
+
const removeSpy = vi.spyOn(harness.trackingGateway, 'remove');
|
|
62
|
+
const result = await harness.useCase.execute({ projectPath, mrId: 'mr-42' });
|
|
63
|
+
expect(result).toEqual({
|
|
64
|
+
ok: true,
|
|
65
|
+
jobCancelled: false,
|
|
66
|
+
contextReleased: false,
|
|
67
|
+
recordRetained: true,
|
|
68
|
+
});
|
|
69
|
+
expect(archiveSpy).not.toHaveBeenCalled();
|
|
70
|
+
expect(removeSpy).not.toHaveBeenCalled();
|
|
71
|
+
expect(harness.trackingGateway.getById(projectPath, 'mr-42')).not.toBeNull();
|
|
72
|
+
});
|
|
73
|
+
it('cancels the running job using the built job id', async () => {
|
|
74
|
+
const harness = buildHarness();
|
|
75
|
+
harness.cancelJob.mockReturnValue(true);
|
|
76
|
+
seedMr(harness, 'pending-review', { platform: 'gitlab', mrNumber: 7 });
|
|
77
|
+
const result = await harness.useCase.execute({ projectPath, mrId: 'mr-42' });
|
|
78
|
+
expect(result.ok && result.jobCancelled).toBe(true);
|
|
79
|
+
expect(harness.buildJobId).toHaveBeenCalledWith('gitlab', projectPath, 7);
|
|
80
|
+
expect(harness.cancelJob).toHaveBeenCalledWith(`gitlab:${projectPath}:7`);
|
|
81
|
+
});
|
|
82
|
+
it('releases the review context keyed by the project path and the stored id', async () => {
|
|
83
|
+
const harness = buildHarness();
|
|
84
|
+
seedMr(harness, 'pending-fix');
|
|
85
|
+
harness.contextGateway.setContext('mr-42', ReviewContextFactory.create({ mergeRequestId: 'mr-42' }));
|
|
86
|
+
const deleteSpy = vi.spyOn(harness.contextGateway, 'delete');
|
|
87
|
+
const result = await harness.useCase.execute({ projectPath, mrId: 'mr-42' });
|
|
88
|
+
expect(result.ok && result.contextReleased).toBe(true);
|
|
89
|
+
expect(deleteSpy).toHaveBeenCalledWith(projectPath, 'mr-42');
|
|
90
|
+
});
|
|
91
|
+
it('removes the worktree with the project path as identity and source checkout', async () => {
|
|
92
|
+
const harness = buildHarness();
|
|
93
|
+
seedMr(harness, 'pending-fix', { platform: 'github', mrNumber: 9 });
|
|
94
|
+
await harness.useCase.execute({ projectPath, mrId: 'mr-42' });
|
|
95
|
+
expect(harness.removeWorktree).toHaveBeenCalledWith({
|
|
96
|
+
identity: { platform: 'github', projectPath, mrNumber: 9 },
|
|
97
|
+
sourceCheckoutPath: projectPath,
|
|
98
|
+
});
|
|
99
|
+
});
|
|
100
|
+
it('stays merged and succeeds when the review is already merged (idempotent)', async () => {
|
|
101
|
+
const harness = buildHarness();
|
|
102
|
+
seedMr(harness, 'merged', { mergedAt: '2026-01-01T00:00:00.000Z' });
|
|
103
|
+
const result = await harness.useCase.execute({ projectPath, mrId: 'mr-42' });
|
|
104
|
+
expect(result.ok).toBe(true);
|
|
105
|
+
const updated = harness.trackingGateway.getById(projectPath, 'mr-42');
|
|
106
|
+
expect(updated?.state).toBe('merged');
|
|
107
|
+
expect(harness.cancelJob).not.toHaveBeenCalled();
|
|
108
|
+
});
|
|
109
|
+
it('marks merged even when worktree removal reports a failure (best-effort)', async () => {
|
|
110
|
+
const harness = buildHarness({
|
|
111
|
+
removeWorktree: vi.fn(async () => ({ status: 'failed', warning: 'boom' })),
|
|
112
|
+
});
|
|
113
|
+
seedMr(harness, 'pending-fix');
|
|
114
|
+
const result = await harness.useCase.execute({ projectPath, mrId: 'mr-42' });
|
|
115
|
+
expect(result.ok).toBe(true);
|
|
116
|
+
expect(harness.trackingGateway.getById(projectPath, 'mr-42')?.state).toBe('merged');
|
|
117
|
+
expect(harness.warnMessages.some((message) => message.includes('boom'))).toBe(true);
|
|
118
|
+
});
|
|
119
|
+
it('marks merged even when worktree removal throws (best-effort)', async () => {
|
|
120
|
+
const harness = buildHarness({
|
|
121
|
+
removeWorktree: vi.fn(async () => {
|
|
122
|
+
throw new Error('exploded');
|
|
123
|
+
}),
|
|
124
|
+
});
|
|
125
|
+
seedMr(harness, 'pending-fix');
|
|
126
|
+
const result = await harness.useCase.execute({ projectPath, mrId: 'mr-42' });
|
|
127
|
+
expect(result.ok).toBe(true);
|
|
128
|
+
expect(harness.trackingGateway.getById(projectPath, 'mr-42')?.state).toBe('merged');
|
|
129
|
+
expect(harness.warnMessages.some((message) => message.includes('exploded'))).toBe(true);
|
|
130
|
+
});
|
|
131
|
+
it('returns not-found when the review does not exist', async () => {
|
|
132
|
+
const harness = buildHarness();
|
|
133
|
+
const result = await harness.useCase.execute({ projectPath, mrId: 'ghost' });
|
|
134
|
+
expect(result).toEqual({ ok: false, reason: 'not-found' });
|
|
135
|
+
expect(harness.cancelJob).not.toHaveBeenCalled();
|
|
136
|
+
});
|
|
137
|
+
});
|
|
138
|
+
//# sourceMappingURL=markReviewAsMerged.usecase.test.js.map
|
package/dist/tests/units/modules/tracking/usecases/tracking/markReviewAsMerged.usecase.test.js.map
ADDED
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"markReviewAsMerged.usecase.test.js","sourceRoot":"","sources":["../../../../../../../src/tests/units/modules/tracking/usecases/tracking/markReviewAsMerged.usecase.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAGlD,OAAO,EACL,yBAAyB,GAE1B,MAAM,oEAAoE,CAAC;AAE5E,OAAO,EAAE,oBAAoB,EAAE,MAAM,4CAA4C,CAAC;AAClF,OAAO,EAAE,gBAAgB,EAAE,MAAM,wCAAwC,CAAC;AAC1E,OAAO,EAAE,qBAAqB,EAAE,MAAM,uCAAuC,CAAC;AAC9E,OAAO,EAAE,wBAAwB,EAAE,MAAM,4CAA4C,CAAC;AACtF,OAAO,EAAE,oCAAoC,EAAE,MAAM,6CAA6C,CAAC;AAYnG,MAAM,WAAW,GAAG,iBAAiB,CAAC;AAEtC,SAAS,YAAY,CAAC,SAAmD;IACvE,MAAM,eAAe,GAAG,IAAI,oCAAoC,EAAE,CAAC;IACnE,MAAM,cAAc,GAAG,IAAI,wBAAwB,EAAE,CAAC;IACtD,MAAM,SAAS,GAAG,qBAAqB,EAAE,CAAC;IAE1C,MAAM,SAAS,GAAG,EAAE,CAAC,EAAE,CAAC,CAAC,MAAc,EAAW,EAAE,CAAC,KAAK,CAAC,CAAC;IAC5D,MAAM,UAAU,GAAG,EAAE,CAAC,EAAE,CACtB,CAAC,QAAgB,EAAE,IAAY,EAAE,QAAgB,EAAU,EAAE,CAAC,GAAG,QAAQ,IAAI,IAAI,IAAI,QAAQ,EAAE,CAChG,CAAC;IACF,MAAM,cAAc,GAAG,EAAE,CAAC,EAAE,CAAC,KAAK,IAA2B,EAAE,CAAC,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC;IAEzF,MAAM,IAAI,GAAmC;QAC3C,eAAe;QACf,oBAAoB,EAAE,cAAc;QACpC,SAAS;QACT,UAAU;QACV,cAAc;QACd,MAAM,EAAE,SAAS,CAAC,MAAM;QACxB,GAAG,SAAS;KACb,CAAC;IAEF,OAAO;QACL,OAAO,EAAE,IAAI,yBAAyB,CAAC,IAAI,CAAC;QAC5C,eAAe;QACf,cAAc;QACd,SAAS;QACT,UAAU;QACV,cAAc;QACd,YAAY,EAAE,SAAS,CAAC,YAAY;KACrC,CAAC;AACJ,CAAC;AAED,SAAS,MAAM,CACb,OAAgB,EAChB,KAA8B,EAC9B,YAA2D,EAAE;IAE7D,OAAO,CAAC,eAAe,CAAC,MAAM,CAC5B,WAAW,EACX,gBAAgB,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,OAAO,EAAE,KAAK,EAAE,GAAG,SAAS,EAAE,CAAC,CAC9D,CAAC;AACJ,CAAC;AAED,QAAQ,CAAC,2BAA2B,EAAE,GAAG,EAAE;IACzC,MAAM,MAAM,GAA8B;QACxC,gBAAgB;QAChB,aAAa;QACb,kBAAkB;QAClB,UAAU;QACV,QAAQ;KACT,CAAC;IAEF,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC3B,EAAE,CAAC,iBAAiB,KAAK,6CAA6C,EAAE,KAAK,IAAI,EAAE;YACjF,MAAM,OAAO,GAAG,YAAY,EAAE,CAAC;YAC/B,MAAM,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC;YAEvB,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,WAAW,EAAE,IAAI,EAAE,OAAO,EAAE,CAAC,CAAC;YAE7E,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAC7B,MAAM,OAAO,GAAG,OAAO,CAAC,eAAe,CAAC,OAAO,CAAC,WAAW,EAAE,OAAO,CAAC,CAAC;YACtE,MAAM,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YACtC,MAAM,CAAC,OAAO,EAAE,QAAQ,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;QAC3C,CAAC,CAAC,CAAC;IACL,CAAC;IAED,EAAE,CAAC,4CAA4C,EAAE,KAAK,IAAI,EAAE;QAC1D,MAAM,OAAO,GAAG,YAAY,EAAE,CAAC;QAC/B,MAAM,CAAC,OAAO,EAAE,aAAa,CAAC,CAAC;QAC/B,MAAM,UAAU,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,eAAe,EAAE,SAAS,CAAC,CAAC;QAChE,MAAM,SAAS,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,eAAe,EAAE,QAAQ,CAAC,CAAC;QAE9D,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,WAAW,EAAE,IAAI,EAAE,OAAO,EAAE,CAAC,CAAC;QAE7E,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC;YACrB,EAAE,EAAE,IAAI;YACR,YAAY,EAAE,KAAK;YACnB,eAAe,EAAE,KAAK;YACtB,cAAc,EAAE,IAAI;SACrB,CAAC,CAAC;QACH,MAAM,CAAC,UAAU,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QAC1C,MAAM,CAAC,SAAS,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QACzC,MAAM,CAAC,OAAO,CAAC,eAAe,CAAC,OAAO,CAAC,WAAW,EAAE,OAAO,CAAC,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAC;IAC/E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;QAC9D,MAAM,OAAO,GAAG,YAAY,EAAE,CAAC;QAC/B,OAAO,CAAC,SAAS,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC;QACxC,MAAM,CAAC,OAAO,EAAE,gBAAgB,EAAE,EAAE,QAAQ,EAAE,QAAQ,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC;QAEvE,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,WAAW,EAAE,IAAI,EAAE,OAAO,EAAE,CAAC,CAAC;QAE7E,MAAM,CAAC,MAAM,CAAC,EAAE,IAAI,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACpD,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC,oBAAoB,CAAC,QAAQ,EAAE,WAAW,EAAE,CAAC,CAAC,CAAC;QAC1E,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,oBAAoB,CAAC,UAAU,WAAW,IAAI,CAAC,CAAC;IAC5E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yEAAyE,EAAE,KAAK,IAAI,EAAE;QACvF,MAAM,OAAO,GAAG,YAAY,EAAE,CAAC;QAC/B,MAAM,CAAC,OAAO,EAAE,aAAa,CAAC,CAAC;QAC/B,OAAO,CAAC,cAAc,CAAC,UAAU,CAC/B,OAAO,EACP,oBAAoB,CAAC,MAAM,CAAC,EAAE,cAAc,EAAE,OAAO,EAAE,CAAC,CACzD,CAAC;QACF,MAAM,SAAS,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,cAAc,EAAE,QAAQ,CAAC,CAAC;QAE7D,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,WAAW,EAAE,IAAI,EAAE,OAAO,EAAE,CAAC,CAAC;QAE7E,MAAM,CAAC,MAAM,CAAC,EAAE,IAAI,MAAM,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACvD,MAAM,CAAC,SAAS,CAAC,CAAC,oBAAoB,CAAC,WAAW,EAAE,OAAO,CAAC,CAAC;IAC/D,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;QAC1F,MAAM,OAAO,GAAG,YAAY,EAAE,CAAC;QAC/B,MAAM,CAAC,OAAO,EAAE,aAAa,EAAE,EAAE,QAAQ,EAAE,QAAQ,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC,CAAC;QAEpE,MAAM,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,WAAW,EAAE,IAAI,EAAE,OAAO,EAAE,CAAC,CAAC;QAE9D,MAAM,CAAC,OAAO,CAAC,cAAc,CAAC,CAAC,oBAAoB,CAAC;YAClD,QAAQ,EAAE,EAAE,QAAQ,EAAE,QAAQ,EAAE,WAAW,EAAE,QAAQ,EAAE,CAAC,EAAE;YAC1D,kBAAkB,EAAE,WAAW;SAChC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0EAA0E,EAAE,KAAK,IAAI,EAAE;QACxF,MAAM,OAAO,GAAG,YAAY,EAAE,CAAC;QAC/B,MAAM,CAAC,OAAO,EAAE,QAAQ,EAAE,EAAE,QAAQ,EAAE,0BAA0B,EAAE,CAAC,CAAC;QAEpE,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,WAAW,EAAE,IAAI,EAAE,OAAO,EAAE,CAAC,CAAC;QAE7E,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC7B,MAAM,OAAO,GAAG,OAAO,CAAC,eAAe,CAAC,OAAO,CAAC,WAAW,EAAE,OAAO,CAAC,CAAC;QACtE,MAAM,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACtC,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACnD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yEAAyE,EAAE,KAAK,IAAI,EAAE;QACvF,MAAM,OAAO,GAAG,YAAY,CAAC;YAC3B,cAAc,EAAE,EAAE,CAAC,EAAE,CACnB,KAAK,IAA2B,EAAE,CAAC,CAAC,EAAE,MAAM,EAAE,QAAQ,EAAE,OAAO,EAAE,MAAM,EAAE,CAAC,CAC3E;SACF,CAAC,CAAC;QACH,MAAM,CAAC,OAAO,EAAE,aAAa,CAAC,CAAC;QAE/B,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,WAAW,EAAE,IAAI,EAAE,OAAO,EAAE,CAAC,CAAC;QAE7E,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC7B,MAAM,CAAC,OAAO,CAAC,eAAe,CAAC,OAAO,CAAC,WAAW,EAAE,OAAO,CAAC,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACpF,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACtF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;QAC5E,MAAM,OAAO,GAAG,YAAY,CAAC;YAC3B,cAAc,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAA2B,EAAE;gBACtD,MAAM,IAAI,KAAK,CAAC,UAAU,CAAC,CAAC;YAC9B,CAAC,CAAC;SACH,CAAC,CAAC;QACH,MAAM,CAAC,OAAO,EAAE,aAAa,CAAC,CAAC;QAE/B,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,WAAW,EAAE,IAAI,EAAE,OAAO,EAAE,CAAC,CAAC;QAE7E,MAAM,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC7B,MAAM,CAAC,OAAO,CAAC,eAAe,CAAC,OAAO,CAAC,WAAW,EAAE,OAAO,CAAC,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QACpF,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC1F,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,kDAAkD,EAAE,KAAK,IAAI,EAAE;QAChE,MAAM,OAAO,GAAG,YAAY,EAAE,CAAC;QAE/B,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,OAAO,CAAC,OAAO,CAAC,EAAE,WAAW,EAAE,IAAI,EAAE,OAAO,EAAE,CAAC,CAAC;QAE7E,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE,WAAW,EAAE,CAAC,CAAC;QAC3D,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACnD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|