reviewflow 3.19.2 → 3.21.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 +14 -0
- package/dist/config/projectConfig.d.ts +7 -0
- package/dist/config/projectConfig.d.ts.map +1 -1
- package/dist/config/projectConfig.js +18 -0
- package/dist/config/projectConfig.js.map +1 -1
- package/dist/dashboard/index.html +452 -142
- package/dist/dashboard/modules/cardCounters.d.ts +32 -0
- package/dist/dashboard/modules/cardCounters.d.ts.map +1 -0
- package/dist/dashboard/modules/cardCounters.js +40 -0
- package/dist/dashboard/modules/cardCounters.js.map +1 -0
- package/dist/dashboard/modules/constants.d.ts +1 -0
- package/dist/dashboard/modules/constants.d.ts.map +1 -1
- package/dist/dashboard/modules/constants.js +1 -0
- package/dist/dashboard/modules/constants.js.map +1 -1
- package/dist/dashboard/modules/managePanel.d.ts +49 -0
- package/dist/dashboard/modules/managePanel.d.ts.map +1 -0
- package/dist/dashboard/modules/managePanel.js +123 -0
- package/dist/dashboard/modules/managePanel.js.map +1 -0
- package/dist/dashboard/modules/overview.d.ts +65 -0
- package/dist/dashboard/modules/overview.d.ts.map +1 -0
- package/dist/dashboard/modules/overview.js +260 -0
- package/dist/dashboard/modules/overview.js.map +1 -0
- package/dist/dashboard/modules/settingsModal.d.ts +77 -0
- package/dist/dashboard/modules/settingsModal.d.ts.map +1 -0
- package/dist/dashboard/modules/settingsModal.js +182 -0
- package/dist/dashboard/modules/settingsModal.js.map +1 -0
- package/dist/dashboard/modules/tabBar.d.ts +60 -0
- package/dist/dashboard/modules/tabBar.d.ts.map +1 -0
- package/dist/dashboard/modules/tabBar.js +103 -0
- package/dist/dashboard/modules/tabBar.js.map +1 -0
- package/dist/dashboard/styles.css +936 -0
- package/dist/frameworks/config/configLoader.d.ts +8 -0
- package/dist/frameworks/config/configLoader.d.ts.map +1 -1
- package/dist/frameworks/config/configLoader.js +18 -0
- package/dist/frameworks/config/configLoader.js.map +1 -1
- package/dist/main/routes.d.ts.map +1 -1
- package/dist/main/routes.js +67 -11
- package/dist/main/routes.js.map +1 -1
- package/dist/modules/cli-configuration/entities/projectConfig/projectConfig.gateway.d.ts +20 -0
- package/dist/modules/cli-configuration/entities/projectConfig/projectConfig.gateway.d.ts.map +1 -0
- package/dist/modules/cli-configuration/entities/projectConfig/projectConfig.gateway.js +2 -0
- package/dist/modules/cli-configuration/entities/projectConfig/projectConfig.gateway.js.map +1 -0
- package/dist/modules/cli-configuration/entities/repositoryEntry/repositoryEntry.d.ts +13 -0
- package/dist/modules/cli-configuration/entities/repositoryEntry/repositoryEntry.d.ts.map +1 -0
- package/dist/modules/cli-configuration/entities/repositoryEntry/repositoryEntry.js +2 -0
- package/dist/modules/cli-configuration/entities/repositoryEntry/repositoryEntry.js.map +1 -0
- package/dist/modules/cli-configuration/interface-adapters/controllers/http/projectConfig.routes.d.ts +6 -1
- package/dist/modules/cli-configuration/interface-adapters/controllers/http/projectConfig.routes.d.ts.map +1 -1
- package/dist/modules/cli-configuration/interface-adapters/controllers/http/projectConfig.routes.js +116 -13
- package/dist/modules/cli-configuration/interface-adapters/controllers/http/projectConfig.routes.js.map +1 -1
- package/dist/modules/cli-configuration/interface-adapters/controllers/http/repositories.routes.d.ts +43 -0
- package/dist/modules/cli-configuration/interface-adapters/controllers/http/repositories.routes.d.ts.map +1 -0
- package/dist/modules/cli-configuration/interface-adapters/controllers/http/repositories.routes.js +82 -0
- package/dist/modules/cli-configuration/interface-adapters/controllers/http/repositories.routes.js.map +1 -0
- package/dist/modules/cli-configuration/interface-adapters/gateways/projectConfig.fileSystem.gateway.d.ts +7 -0
- package/dist/modules/cli-configuration/interface-adapters/gateways/projectConfig.fileSystem.gateway.d.ts.map +1 -0
- package/dist/modules/cli-configuration/interface-adapters/gateways/projectConfig.fileSystem.gateway.js +48 -0
- package/dist/modules/cli-configuration/interface-adapters/gateways/projectConfig.fileSystem.gateway.js.map +1 -0
- package/dist/modules/cli-configuration/usecases/cli/addRepositoriesToConfig.usecase.d.ts +1 -6
- package/dist/modules/cli-configuration/usecases/cli/addRepositoriesToConfig.usecase.d.ts.map +1 -1
- package/dist/modules/cli-configuration/usecases/cli/addRepositoriesToConfig.usecase.js.map +1 -1
- package/dist/modules/cli-configuration/usecases/cli/removeRepositoryFromConfig.usecase.d.ts +21 -0
- package/dist/modules/cli-configuration/usecases/cli/removeRepositoryFromConfig.usecase.d.ts.map +1 -0
- package/dist/modules/cli-configuration/usecases/cli/removeRepositoryFromConfig.usecase.js +27 -0
- package/dist/modules/cli-configuration/usecases/cli/removeRepositoryFromConfig.usecase.js.map +1 -0
- package/dist/modules/cli-configuration/usecases/cli/toggleRepositoryEnabled.usecase.d.ts +22 -0
- package/dist/modules/cli-configuration/usecases/cli/toggleRepositoryEnabled.usecase.d.ts.map +1 -0
- package/dist/modules/cli-configuration/usecases/cli/toggleRepositoryEnabled.usecase.js +27 -0
- package/dist/modules/cli-configuration/usecases/cli/toggleRepositoryEnabled.usecase.js.map +1 -0
- package/dist/modules/cli-configuration/usecases/cli/writeInitConfig.usecase.d.ts +1 -6
- package/dist/modules/cli-configuration/usecases/cli/writeInitConfig.usecase.d.ts.map +1 -1
- package/dist/modules/cli-configuration/usecases/cli/writeInitConfig.usecase.js.map +1 -1
- package/dist/modules/cli-configuration/usecases/dashboardRepositories/addRepositoryFromDashboard.usecase.d.ts +19 -0
- package/dist/modules/cli-configuration/usecases/dashboardRepositories/addRepositoryFromDashboard.usecase.d.ts.map +1 -0
- package/dist/modules/cli-configuration/usecases/dashboardRepositories/addRepositoryFromDashboard.usecase.js +30 -0
- package/dist/modules/cli-configuration/usecases/dashboardRepositories/addRepositoryFromDashboard.usecase.js.map +1 -0
- package/dist/modules/cli-configuration/usecases/dashboardRepositories/removeRepositoryFromDashboard.usecase.d.ts +16 -0
- package/dist/modules/cli-configuration/usecases/dashboardRepositories/removeRepositoryFromDashboard.usecase.d.ts.map +1 -0
- package/dist/modules/cli-configuration/usecases/dashboardRepositories/removeRepositoryFromDashboard.usecase.js +27 -0
- package/dist/modules/cli-configuration/usecases/dashboardRepositories/removeRepositoryFromDashboard.usecase.js.map +1 -0
- package/dist/modules/cli-configuration/usecases/dashboardRepositories/updateRepositoryEnabledFromDashboard.usecase.d.ts +17 -0
- package/dist/modules/cli-configuration/usecases/dashboardRepositories/updateRepositoryEnabledFromDashboard.usecase.d.ts.map +1 -0
- package/dist/modules/cli-configuration/usecases/dashboardRepositories/updateRepositoryEnabledFromDashboard.usecase.js +28 -0
- package/dist/modules/cli-configuration/usecases/dashboardRepositories/updateRepositoryEnabledFromDashboard.usecase.js.map +1 -0
- package/dist/modules/cli-configuration/usecases/projectConfig/updateProjectConfig.usecase.d.ts +31 -0
- package/dist/modules/cli-configuration/usecases/projectConfig/updateProjectConfig.usecase.d.ts.map +1 -0
- package/dist/modules/cli-configuration/usecases/projectConfig/updateProjectConfig.usecase.js +102 -0
- package/dist/modules/cli-configuration/usecases/projectConfig/updateProjectConfig.usecase.js.map +1 -0
- package/dist/modules/statistics-insights/interface-adapters/controllers/http/overview.routes.d.ts +16 -0
- package/dist/modules/statistics-insights/interface-adapters/controllers/http/overview.routes.d.ts.map +1 -0
- package/dist/modules/statistics-insights/interface-adapters/controllers/http/overview.routes.js +53 -0
- package/dist/modules/statistics-insights/interface-adapters/controllers/http/overview.routes.js.map +1 -0
- package/dist/modules/statistics-insights/interface-adapters/presenters/overview.presenter.d.ts +93 -0
- package/dist/modules/statistics-insights/interface-adapters/presenters/overview.presenter.d.ts.map +1 -0
- package/dist/modules/statistics-insights/interface-adapters/presenters/overview.presenter.js +145 -0
- package/dist/modules/statistics-insights/interface-adapters/presenters/overview.presenter.js.map +1 -0
- package/dist/tests/acceptance/177-dashboard-add-project-ui.acceptance.test.d.ts +12 -0
- package/dist/tests/acceptance/177-dashboard-add-project-ui.acceptance.test.d.ts.map +1 -0
- package/dist/tests/acceptance/177-dashboard-add-project-ui.acceptance.test.js +304 -0
- package/dist/tests/acceptance/177-dashboard-add-project-ui.acceptance.test.js.map +1 -0
- package/dist/tests/acceptance/178-dashboard-tabs-reposition.acceptance.test.d.ts +12 -0
- package/dist/tests/acceptance/178-dashboard-tabs-reposition.acceptance.test.d.ts.map +1 -0
- package/dist/tests/acceptance/178-dashboard-tabs-reposition.acceptance.test.js +131 -0
- package/dist/tests/acceptance/178-dashboard-tabs-reposition.acceptance.test.js.map +1 -0
- package/dist/tests/acceptance/179-dashboard-project-settings-modal.acceptance.test.d.ts +12 -0
- package/dist/tests/acceptance/179-dashboard-project-settings-modal.acceptance.test.d.ts.map +1 -0
- package/dist/tests/acceptance/179-dashboard-project-settings-modal.acceptance.test.js +312 -0
- package/dist/tests/acceptance/179-dashboard-project-settings-modal.acceptance.test.js.map +1 -0
- package/dist/tests/acceptance/91-dashboard-multi-project-overview.acceptance.test.d.ts +10 -0
- package/dist/tests/acceptance/91-dashboard-multi-project-overview.acceptance.test.d.ts.map +1 -0
- package/dist/tests/acceptance/91-dashboard-multi-project-overview.acceptance.test.js +275 -0
- package/dist/tests/acceptance/91-dashboard-multi-project-overview.acceptance.test.js.map +1 -0
- package/dist/tests/factories/projectStatsApiResponse.factory.d.ts +16 -0
- package/dist/tests/factories/projectStatsApiResponse.factory.d.ts.map +1 -0
- package/dist/tests/factories/projectStatsApiResponse.factory.js +39 -0
- package/dist/tests/factories/projectStatsApiResponse.factory.js.map +1 -0
- package/dist/tests/factories/recentReviewFile.factory.d.ts +5 -0
- package/dist/tests/factories/recentReviewFile.factory.d.ts.map +1 -0
- package/dist/tests/factories/recentReviewFile.factory.js +16 -0
- package/dist/tests/factories/recentReviewFile.factory.js.map +1 -0
- package/dist/tests/factories/repositoryConfig.factory.d.ts +5 -0
- package/dist/tests/factories/repositoryConfig.factory.d.ts.map +1 -0
- package/dist/tests/factories/repositoryConfig.factory.js +14 -0
- package/dist/tests/factories/repositoryConfig.factory.js.map +1 -0
- package/dist/tests/stubs/projectConfigGateway.stub.d.ts +15 -0
- package/dist/tests/stubs/projectConfigGateway.stub.d.ts.map +1 -0
- package/dist/tests/stubs/projectConfigGateway.stub.js +40 -0
- package/dist/tests/stubs/projectConfigGateway.stub.js.map +1 -0
- package/dist/tests/units/config/projectConfig.test.js +43 -0
- package/dist/tests/units/config/projectConfig.test.js.map +1 -1
- package/dist/tests/units/dashboard/modules/cardCounters.test.d.ts +2 -0
- package/dist/tests/units/dashboard/modules/cardCounters.test.d.ts.map +1 -0
- package/dist/tests/units/dashboard/modules/cardCounters.test.js +106 -0
- package/dist/tests/units/dashboard/modules/cardCounters.test.js.map +1 -0
- package/dist/tests/units/dashboard/modules/constants.test.js +2 -1
- package/dist/tests/units/dashboard/modules/constants.test.js.map +1 -1
- package/dist/tests/units/dashboard/modules/managePanel.test.d.ts +2 -0
- package/dist/tests/units/dashboard/modules/managePanel.test.d.ts.map +1 -0
- package/dist/tests/units/dashboard/modules/managePanel.test.js +112 -0
- package/dist/tests/units/dashboard/modules/managePanel.test.js.map +1 -0
- package/dist/tests/units/dashboard/modules/overview.test.d.ts +2 -0
- package/dist/tests/units/dashboard/modules/overview.test.d.ts.map +1 -0
- package/dist/tests/units/dashboard/modules/overview.test.js +268 -0
- package/dist/tests/units/dashboard/modules/overview.test.js.map +1 -0
- package/dist/tests/units/dashboard/modules/settingsModal.test.d.ts +2 -0
- package/dist/tests/units/dashboard/modules/settingsModal.test.d.ts.map +1 -0
- package/dist/tests/units/dashboard/modules/settingsModal.test.js +166 -0
- package/dist/tests/units/dashboard/modules/settingsModal.test.js.map +1 -0
- package/dist/tests/units/dashboard/modules/tabBar.test.d.ts +2 -0
- package/dist/tests/units/dashboard/modules/tabBar.test.d.ts.map +1 -0
- package/dist/tests/units/dashboard/modules/tabBar.test.js +128 -0
- package/dist/tests/units/dashboard/modules/tabBar.test.js.map +1 -0
- package/dist/tests/units/frameworks/config/configLoader.test.js +35 -1
- package/dist/tests/units/frameworks/config/configLoader.test.js.map +1 -1
- package/dist/tests/units/modules/cli-configuration/interface-adapters/controllers/http/projectConfig.routes.test.js +111 -0
- package/dist/tests/units/modules/cli-configuration/interface-adapters/controllers/http/projectConfig.routes.test.js.map +1 -1
- package/dist/tests/units/modules/cli-configuration/interface-adapters/controllers/http/repositories.routes.test.d.ts +2 -0
- package/dist/tests/units/modules/cli-configuration/interface-adapters/controllers/http/repositories.routes.test.d.ts.map +1 -0
- package/dist/tests/units/modules/cli-configuration/interface-adapters/controllers/http/repositories.routes.test.js +298 -0
- package/dist/tests/units/modules/cli-configuration/interface-adapters/controllers/http/repositories.routes.test.js.map +1 -0
- package/dist/tests/units/modules/cli-configuration/interface-adapters/gateways/projectConfig.fileSystem.gateway.test.d.ts +2 -0
- package/dist/tests/units/modules/cli-configuration/interface-adapters/gateways/projectConfig.fileSystem.gateway.test.d.ts.map +1 -0
- package/dist/tests/units/modules/cli-configuration/interface-adapters/gateways/projectConfig.fileSystem.gateway.test.js +72 -0
- package/dist/tests/units/modules/cli-configuration/interface-adapters/gateways/projectConfig.fileSystem.gateway.test.js.map +1 -0
- package/dist/tests/units/modules/cli-configuration/usecases/cli/removeRepositoryFromConfig.usecase.test.d.ts +2 -0
- package/dist/tests/units/modules/cli-configuration/usecases/cli/removeRepositoryFromConfig.usecase.test.d.ts.map +1 -0
- package/dist/tests/units/modules/cli-configuration/usecases/cli/removeRepositoryFromConfig.usecase.test.js +76 -0
- package/dist/tests/units/modules/cli-configuration/usecases/cli/removeRepositoryFromConfig.usecase.test.js.map +1 -0
- package/dist/tests/units/modules/cli-configuration/usecases/cli/toggleRepositoryEnabled.usecase.test.d.ts +2 -0
- package/dist/tests/units/modules/cli-configuration/usecases/cli/toggleRepositoryEnabled.usecase.test.d.ts.map +1 -0
- package/dist/tests/units/modules/cli-configuration/usecases/cli/toggleRepositoryEnabled.usecase.test.js +84 -0
- package/dist/tests/units/modules/cli-configuration/usecases/cli/toggleRepositoryEnabled.usecase.test.js.map +1 -0
- package/dist/tests/units/modules/cli-configuration/usecases/projectConfig/updateProjectConfig.usecase.test.d.ts +2 -0
- package/dist/tests/units/modules/cli-configuration/usecases/projectConfig/updateProjectConfig.usecase.test.d.ts.map +1 -0
- package/dist/tests/units/modules/cli-configuration/usecases/projectConfig/updateProjectConfig.usecase.test.js +141 -0
- package/dist/tests/units/modules/cli-configuration/usecases/projectConfig/updateProjectConfig.usecase.test.js.map +1 -0
- package/dist/tests/units/modules/statistics-insights/interface-adapters/controllers/http/overview.routes.test.d.ts +2 -0
- package/dist/tests/units/modules/statistics-insights/interface-adapters/controllers/http/overview.routes.test.d.ts.map +1 -0
- package/dist/tests/units/modules/statistics-insights/interface-adapters/controllers/http/overview.routes.test.js +200 -0
- package/dist/tests/units/modules/statistics-insights/interface-adapters/controllers/http/overview.routes.test.js.map +1 -0
- package/dist/tests/units/modules/statistics-insights/interface-adapters/presenters/overview.presenter.test.d.ts +2 -0
- package/dist/tests/units/modules/statistics-insights/interface-adapters/presenters/overview.presenter.test.d.ts.map +1 -0
- package/dist/tests/units/modules/statistics-insights/interface-adapters/presenters/overview.presenter.test.js +331 -0
- package/dist/tests/units/modules/statistics-insights/interface-adapters/presenters/overview.presenter.test.js.map +1 -0
- package/package.json +1 -1
|
@@ -0,0 +1,312 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* SPEC-179 — Configure Project Settings via a Modal
|
|
3
|
+
*
|
|
4
|
+
* Outer-loop acceptance test (SDD): exercises GET / PATCH /api/project-config
|
|
5
|
+
* through the Fastify plugin wired with the in-memory stub gateway, plus
|
|
6
|
+
* filesystem-grep assertions on src/dashboard/index.html and styles.css for
|
|
7
|
+
* the sidebar button, the <dialog> markup and the reduced-motion contract.
|
|
8
|
+
*
|
|
9
|
+
* Source of truth: docs/specs/179-dashboard-project-settings-modal.md (15 scenarios).
|
|
10
|
+
*/
|
|
11
|
+
import Fastify from 'fastify';
|
|
12
|
+
import { readFileSync } from 'node:fs';
|
|
13
|
+
import { fileURLToPath } from 'node:url';
|
|
14
|
+
import { dirname, join } from 'node:path';
|
|
15
|
+
import { beforeEach, describe, expect, it } from 'vitest';
|
|
16
|
+
import { projectConfigRoutes } from '../../modules/cli-configuration/interface-adapters/controllers/http/projectConfig.routes.js';
|
|
17
|
+
import { UpdateProjectConfigUseCase } from '../../modules/cli-configuration/usecases/projectConfig/updateProjectConfig.usecase.js';
|
|
18
|
+
import { StubProjectConfigGateway } from '../../tests/stubs/projectConfigGateway.stub.js';
|
|
19
|
+
import { validateExternalLink, buildSettingsViewModel, renderSettingsModalHtml, } from '../../dashboard/modules/settingsModal.js';
|
|
20
|
+
const __filename = fileURLToPath(import.meta.url);
|
|
21
|
+
const __dirname = dirname(__filename);
|
|
22
|
+
const PROJECT_ROOT = join(__dirname, '..', '..', '..');
|
|
23
|
+
const INDEX_HTML_PATH = join(PROJECT_ROOT, 'src', 'dashboard', 'index.html');
|
|
24
|
+
const STYLES_CSS_PATH = join(PROJECT_ROOT, 'src', 'dashboard', 'styles.css');
|
|
25
|
+
function baseProjectConfig(overrides = {}) {
|
|
26
|
+
return {
|
|
27
|
+
github: false,
|
|
28
|
+
gitlab: true,
|
|
29
|
+
defaultModel: 'sonnet',
|
|
30
|
+
reviewSkill: 'review-front',
|
|
31
|
+
reviewFollowupSkill: 'review-followup',
|
|
32
|
+
language: 'fr',
|
|
33
|
+
retentionDays: 14,
|
|
34
|
+
...overrides,
|
|
35
|
+
};
|
|
36
|
+
}
|
|
37
|
+
async function buildApp(options) {
|
|
38
|
+
const app = Fastify();
|
|
39
|
+
const updateProjectConfig = new UpdateProjectConfigUseCase(options.gateway, options.onUpdated);
|
|
40
|
+
await app.register(projectConfigRoutes, { updateProjectConfig });
|
|
41
|
+
return app;
|
|
42
|
+
}
|
|
43
|
+
describe('Acceptance — SPEC-179: Configure Project Settings via a Modal', () => {
|
|
44
|
+
describe('PATCH /api/project-config — save changes', () => {
|
|
45
|
+
it('language change persists and preserves agents / routingPolicy (S3)', async () => {
|
|
46
|
+
const gateway = new StubProjectConfigGateway();
|
|
47
|
+
gateway.set('/repo/A', baseProjectConfig({
|
|
48
|
+
language: 'fr',
|
|
49
|
+
agents: [{ name: 'security', displayName: 'Security' }],
|
|
50
|
+
routingPolicy: { haikuMaxLines: 50, sonnetMaxLines: 500 },
|
|
51
|
+
}));
|
|
52
|
+
const app = await buildApp({ gateway });
|
|
53
|
+
const response = await app.inject({
|
|
54
|
+
method: 'PATCH',
|
|
55
|
+
url: '/api/project-config?path=' + encodeURIComponent('/repo/A'),
|
|
56
|
+
payload: { language: 'en' },
|
|
57
|
+
});
|
|
58
|
+
expect(response.statusCode).toBe(200);
|
|
59
|
+
const body = response.json();
|
|
60
|
+
expect(body.success).toBe(true);
|
|
61
|
+
expect(body.config.language).toBe('en');
|
|
62
|
+
expect(body.config.agents).toEqual([{ name: 'security', displayName: 'Security' }]);
|
|
63
|
+
expect(body.config.routingPolicy).toEqual({ haikuMaxLines: 50, sonnetMaxLines: 500 });
|
|
64
|
+
const persisted = gateway.get('/repo/A');
|
|
65
|
+
expect(persisted?.language).toBe('en');
|
|
66
|
+
expect(persisted?.agents).toEqual([{ name: 'security', displayName: 'Security' }]);
|
|
67
|
+
await app.close();
|
|
68
|
+
});
|
|
69
|
+
it('defaultModel "sonnet" persists and next read returns sonnet (S4, S13)', async () => {
|
|
70
|
+
const gateway = new StubProjectConfigGateway();
|
|
71
|
+
gateway.set('/repo/A', baseProjectConfig({ defaultModel: 'haiku' }));
|
|
72
|
+
const app = await buildApp({ gateway });
|
|
73
|
+
const response = await app.inject({
|
|
74
|
+
method: 'PATCH',
|
|
75
|
+
url: '/api/project-config?path=' + encodeURIComponent('/repo/A'),
|
|
76
|
+
payload: { defaultModel: 'sonnet' },
|
|
77
|
+
});
|
|
78
|
+
expect(response.statusCode).toBe(200);
|
|
79
|
+
const body = response.json();
|
|
80
|
+
expect(body.config.defaultModel).toBe('sonnet');
|
|
81
|
+
const reread = gateway.read('/repo/A');
|
|
82
|
+
expect(reread.status).toBe('ok');
|
|
83
|
+
if (reread.status === 'ok') {
|
|
84
|
+
expect(reread.config.defaultModel).toBe('sonnet');
|
|
85
|
+
}
|
|
86
|
+
await app.close();
|
|
87
|
+
});
|
|
88
|
+
it('externalLink "https://notion.so/x" → 200 + persisted (S5)', async () => {
|
|
89
|
+
const gateway = new StubProjectConfigGateway();
|
|
90
|
+
gateway.set('/repo/A', baseProjectConfig());
|
|
91
|
+
const app = await buildApp({ gateway });
|
|
92
|
+
const response = await app.inject({
|
|
93
|
+
method: 'PATCH',
|
|
94
|
+
url: '/api/project-config?path=' + encodeURIComponent('/repo/A'),
|
|
95
|
+
payload: { externalLink: 'https://notion.so/team/projet' },
|
|
96
|
+
});
|
|
97
|
+
expect(response.statusCode).toBe(200);
|
|
98
|
+
const body = response.json();
|
|
99
|
+
expect(body.config.externalLink).toBe('https://notion.so/team/projet');
|
|
100
|
+
await app.close();
|
|
101
|
+
});
|
|
102
|
+
it('empty externalLink "" → 200 + key absent from persisted config (S6)', async () => {
|
|
103
|
+
const gateway = new StubProjectConfigGateway();
|
|
104
|
+
gateway.set('/repo/A', baseProjectConfig({ externalLink: 'https://old.example' }));
|
|
105
|
+
const app = await buildApp({ gateway });
|
|
106
|
+
const response = await app.inject({
|
|
107
|
+
method: 'PATCH',
|
|
108
|
+
url: '/api/project-config?path=' + encodeURIComponent('/repo/A'),
|
|
109
|
+
payload: { externalLink: '' },
|
|
110
|
+
});
|
|
111
|
+
expect(response.statusCode).toBe(200);
|
|
112
|
+
const persisted = gateway.get('/repo/A');
|
|
113
|
+
expect(persisted?.externalLink).toBeUndefined();
|
|
114
|
+
await app.close();
|
|
115
|
+
});
|
|
116
|
+
it('rejects http://insecure with "Le lien doit être en HTTPS" (S7)', async () => {
|
|
117
|
+
const gateway = new StubProjectConfigGateway();
|
|
118
|
+
gateway.set('/repo/A', baseProjectConfig());
|
|
119
|
+
const app = await buildApp({ gateway });
|
|
120
|
+
const response = await app.inject({
|
|
121
|
+
method: 'PATCH',
|
|
122
|
+
url: '/api/project-config?path=' + encodeURIComponent('/repo/A'),
|
|
123
|
+
payload: { externalLink: 'http://insecure.example' },
|
|
124
|
+
});
|
|
125
|
+
expect(response.statusCode).toBe(400);
|
|
126
|
+
const body = response.json();
|
|
127
|
+
expect(body.success).toBe(false);
|
|
128
|
+
expect(body.error).toBe('Le lien doit être en HTTPS');
|
|
129
|
+
await app.close();
|
|
130
|
+
});
|
|
131
|
+
it('rejects javascript:alert(1) with "URL invalide" (S8)', async () => {
|
|
132
|
+
const gateway = new StubProjectConfigGateway();
|
|
133
|
+
gateway.set('/repo/A', baseProjectConfig());
|
|
134
|
+
const app = await buildApp({ gateway });
|
|
135
|
+
const response = await app.inject({
|
|
136
|
+
method: 'PATCH',
|
|
137
|
+
url: '/api/project-config?path=' + encodeURIComponent('/repo/A'),
|
|
138
|
+
payload: { externalLink: 'javascript:alert(1)' },
|
|
139
|
+
});
|
|
140
|
+
expect(response.statusCode).toBe(400);
|
|
141
|
+
const body = response.json();
|
|
142
|
+
expect(body.error).toBe('URL invalide');
|
|
143
|
+
await app.close();
|
|
144
|
+
});
|
|
145
|
+
it('rejects free text "not a url" with "URL invalide" (S9)', async () => {
|
|
146
|
+
const gateway = new StubProjectConfigGateway();
|
|
147
|
+
gateway.set('/repo/A', baseProjectConfig());
|
|
148
|
+
const app = await buildApp({ gateway });
|
|
149
|
+
const response = await app.inject({
|
|
150
|
+
method: 'PATCH',
|
|
151
|
+
url: '/api/project-config?path=' + encodeURIComponent('/repo/A'),
|
|
152
|
+
payload: { externalLink: 'not a url' },
|
|
153
|
+
});
|
|
154
|
+
expect(response.statusCode).toBe(400);
|
|
155
|
+
expect(response.json().error).toBe('URL invalide');
|
|
156
|
+
await app.close();
|
|
157
|
+
});
|
|
158
|
+
it('missing project → 404', async () => {
|
|
159
|
+
const gateway = new StubProjectConfigGateway();
|
|
160
|
+
const app = await buildApp({ gateway });
|
|
161
|
+
const response = await app.inject({
|
|
162
|
+
method: 'PATCH',
|
|
163
|
+
url: '/api/project-config?path=' + encodeURIComponent('/unknown'),
|
|
164
|
+
payload: { language: 'en' },
|
|
165
|
+
});
|
|
166
|
+
expect(response.statusCode).toBe(404);
|
|
167
|
+
await app.close();
|
|
168
|
+
});
|
|
169
|
+
it('corrupt config.json → 422 "Configuration projet illisible" (S14)', async () => {
|
|
170
|
+
const gateway = new StubProjectConfigGateway();
|
|
171
|
+
gateway.set('/repo/A', baseProjectConfig());
|
|
172
|
+
gateway.forceMalformed('/repo/A');
|
|
173
|
+
const app = await buildApp({ gateway });
|
|
174
|
+
const response = await app.inject({
|
|
175
|
+
method: 'PATCH',
|
|
176
|
+
url: '/api/project-config?path=' + encodeURIComponent('/repo/A'),
|
|
177
|
+
payload: { language: 'en' },
|
|
178
|
+
});
|
|
179
|
+
expect(response.statusCode).toBe(422);
|
|
180
|
+
expect(response.json().error).toBe('Configuration projet illisible');
|
|
181
|
+
await app.close();
|
|
182
|
+
});
|
|
183
|
+
it('write failure → 500 "Échec de la sauvegarde" (S15)', async () => {
|
|
184
|
+
const gateway = new StubProjectConfigGateway();
|
|
185
|
+
gateway.set('/repo/A', baseProjectConfig());
|
|
186
|
+
gateway.forceIoError();
|
|
187
|
+
const app = await buildApp({ gateway });
|
|
188
|
+
const response = await app.inject({
|
|
189
|
+
method: 'PATCH',
|
|
190
|
+
url: '/api/project-config?path=' + encodeURIComponent('/repo/A'),
|
|
191
|
+
payload: { language: 'en' },
|
|
192
|
+
});
|
|
193
|
+
expect(response.statusCode).toBe(500);
|
|
194
|
+
expect(response.json().error).toBe('Échec de la sauvegarde');
|
|
195
|
+
await app.close();
|
|
196
|
+
});
|
|
197
|
+
it('ignores out-of-scope fields like "agents" in the payload silently', async () => {
|
|
198
|
+
const gateway = new StubProjectConfigGateway();
|
|
199
|
+
gateway.set('/repo/A', baseProjectConfig({
|
|
200
|
+
agents: [{ name: 'security', displayName: 'Security' }],
|
|
201
|
+
}));
|
|
202
|
+
const app = await buildApp({ gateway });
|
|
203
|
+
const response = await app.inject({
|
|
204
|
+
method: 'PATCH',
|
|
205
|
+
url: '/api/project-config?path=' + encodeURIComponent('/repo/A'),
|
|
206
|
+
payload: {
|
|
207
|
+
language: 'en',
|
|
208
|
+
agents: [{ name: 'evil', displayName: 'Evil' }],
|
|
209
|
+
},
|
|
210
|
+
});
|
|
211
|
+
expect(response.statusCode).toBe(200);
|
|
212
|
+
const persisted = gateway.get('/repo/A');
|
|
213
|
+
expect(persisted?.agents).toEqual([{ name: 'security', displayName: 'Security' }]);
|
|
214
|
+
expect(persisted?.language).toBe('en');
|
|
215
|
+
await app.close();
|
|
216
|
+
});
|
|
217
|
+
});
|
|
218
|
+
describe('In-memory propagation (S13)', () => {
|
|
219
|
+
it('next read after PATCH returns the new value (inflight review keeps old)', async () => {
|
|
220
|
+
const gateway = new StubProjectConfigGateway();
|
|
221
|
+
gateway.set('/repo/A', baseProjectConfig({ language: 'fr' }));
|
|
222
|
+
const inflight = gateway.read('/repo/A');
|
|
223
|
+
const app = await buildApp({ gateway });
|
|
224
|
+
await app.inject({
|
|
225
|
+
method: 'PATCH',
|
|
226
|
+
url: '/api/project-config?path=' + encodeURIComponent('/repo/A'),
|
|
227
|
+
payload: { language: 'en' },
|
|
228
|
+
});
|
|
229
|
+
const next = gateway.read('/repo/A');
|
|
230
|
+
if (inflight.status === 'ok') {
|
|
231
|
+
expect(inflight.config.language).toBe('fr');
|
|
232
|
+
}
|
|
233
|
+
else {
|
|
234
|
+
throw new Error('inflight read should succeed');
|
|
235
|
+
}
|
|
236
|
+
if (next.status === 'ok') {
|
|
237
|
+
expect(next.config.language).toBe('en');
|
|
238
|
+
}
|
|
239
|
+
else {
|
|
240
|
+
throw new Error('next read should succeed');
|
|
241
|
+
}
|
|
242
|
+
await app.close();
|
|
243
|
+
});
|
|
244
|
+
});
|
|
245
|
+
describe('Frontend humble module — validateExternalLink', () => {
|
|
246
|
+
it('accepts an empty string', () => {
|
|
247
|
+
expect(validateExternalLink('')).toEqual({ ok: true });
|
|
248
|
+
});
|
|
249
|
+
it('accepts an https url', () => {
|
|
250
|
+
expect(validateExternalLink('https://notion.so/x')).toEqual({ ok: true });
|
|
251
|
+
});
|
|
252
|
+
it('rejects http with the French HTTPS message (S7)', () => {
|
|
253
|
+
expect(validateExternalLink('http://insecure.example')).toEqual({
|
|
254
|
+
ok: false,
|
|
255
|
+
message: 'Le lien doit être en HTTPS',
|
|
256
|
+
});
|
|
257
|
+
});
|
|
258
|
+
it('rejects javascript: with "URL invalide" (S8)', () => {
|
|
259
|
+
expect(validateExternalLink('javascript:alert(1)')).toEqual({
|
|
260
|
+
ok: false,
|
|
261
|
+
message: 'URL invalide',
|
|
262
|
+
});
|
|
263
|
+
});
|
|
264
|
+
it('rejects free text with "URL invalide" (S9)', () => {
|
|
265
|
+
expect(validateExternalLink('not a url')).toEqual({ ok: false, message: 'URL invalide' });
|
|
266
|
+
});
|
|
267
|
+
});
|
|
268
|
+
describe('Frontend humble module — render', () => {
|
|
269
|
+
it('renders the modal with the 5 editable fields pre-filled from the config (S1)', () => {
|
|
270
|
+
const html = renderSettingsModalHtml(buildSettingsViewModel({
|
|
271
|
+
config: baseProjectConfig({
|
|
272
|
+
language: 'en',
|
|
273
|
+
defaultModel: 'opus',
|
|
274
|
+
reviewSkill: 'review-back',
|
|
275
|
+
reviewFollowupSkill: 'review-followup',
|
|
276
|
+
externalLink: 'https://notion.so/x',
|
|
277
|
+
}),
|
|
278
|
+
projectName: 'A',
|
|
279
|
+
}));
|
|
280
|
+
expect(html).toContain('A');
|
|
281
|
+
expect(html).toContain('name="language"');
|
|
282
|
+
expect(html).toContain('name="defaultModel"');
|
|
283
|
+
expect(html).toContain('name="reviewSkill"');
|
|
284
|
+
expect(html).toContain('name="reviewFollowupSkill"');
|
|
285
|
+
expect(html).toContain('name="externalLink"');
|
|
286
|
+
expect(html).toContain('https://notion.so/x');
|
|
287
|
+
});
|
|
288
|
+
});
|
|
289
|
+
describe('Dashboard markup contracts (cross-checks for S1, S2, S16)', () => {
|
|
290
|
+
let indexHtml;
|
|
291
|
+
let stylesCss;
|
|
292
|
+
beforeEach(() => {
|
|
293
|
+
indexHtml = readFileSync(INDEX_HTML_PATH, 'utf-8');
|
|
294
|
+
stylesCss = readFileSync(STYLES_CSS_PATH, 'utf-8');
|
|
295
|
+
});
|
|
296
|
+
it('sidebar Settings button is present with id="open-settings-modal-btn"', () => {
|
|
297
|
+
expect(indexHtml).toMatch(/id="open-settings-modal-btn"/);
|
|
298
|
+
});
|
|
299
|
+
it('<dialog id="settings-modal"> is present in index.html', () => {
|
|
300
|
+
expect(indexHtml).toMatch(/<dialog[^>]*id="settings-modal"/);
|
|
301
|
+
});
|
|
302
|
+
it('styles.css declares a .settings-modal selector', () => {
|
|
303
|
+
expect(stylesCss).toMatch(/\.settings-modal\b/);
|
|
304
|
+
});
|
|
305
|
+
it('styles.css honors prefers-reduced-motion with a rule touching .settings-modal', () => {
|
|
306
|
+
const reducedMotionBlocks = stylesCss.match(/@media[^{]*prefers-reduced-motion:\s*reduce[^{]*\{[\s\S]*?\n\}/g) ?? [];
|
|
307
|
+
const concatenated = reducedMotionBlocks.join('\n');
|
|
308
|
+
expect(concatenated).toMatch(/\.settings-modal/);
|
|
309
|
+
});
|
|
310
|
+
});
|
|
311
|
+
});
|
|
312
|
+
//# sourceMappingURL=179-dashboard-project-settings-modal.acceptance.test.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"179-dashboard-project-settings-modal.acceptance.test.js","sourceRoot":"","sources":["../../../src/tests/acceptance/179-dashboard-project-settings-modal.acceptance.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,OAAiC,MAAM,SAAS,CAAC;AACxD,OAAO,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AACvC,OAAO,EAAE,aAAa,EAAE,MAAM,UAAU,CAAC;AACzC,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AAC1C,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC1D,OAAO,EAAE,mBAAmB,EAAE,MAAM,yFAAyF,CAAC;AAC9H,OAAO,EAAE,0BAA0B,EAAE,MAAM,mFAAmF,CAAC;AAC/H,OAAO,EAAE,wBAAwB,EAAE,MAAM,4CAA4C,CAAC;AACtF,OAAO,EACL,oBAAoB,EACpB,sBAAsB,EACtB,uBAAuB,GACxB,MAAM,sCAAsC,CAAC;AAG9C,MAAM,UAAU,GAAG,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;AAClD,MAAM,SAAS,GAAG,OAAO,CAAC,UAAU,CAAC,CAAC;AACtC,MAAM,YAAY,GAAG,IAAI,CAAC,SAAS,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,CAAC,CAAC;AACvD,MAAM,eAAe,GAAG,IAAI,CAAC,YAAY,EAAE,KAAK,EAAE,WAAW,EAAE,YAAY,CAAC,CAAC;AAC7E,MAAM,eAAe,GAAG,IAAI,CAAC,YAAY,EAAE,KAAK,EAAE,WAAW,EAAE,YAAY,CAAC,CAAC;AAE7E,SAAS,iBAAiB,CAAC,YAAoC,EAAE;IAC/D,OAAO;QACL,MAAM,EAAE,KAAK;QACb,MAAM,EAAE,IAAI;QACZ,YAAY,EAAE,QAAQ;QACtB,WAAW,EAAE,cAAc;QAC3B,mBAAmB,EAAE,iBAAiB;QACtC,QAAQ,EAAE,IAAI;QACd,aAAa,EAAE,EAAE;QACjB,GAAG,SAAS;KACb,CAAC;AACJ,CAAC;AAOD,KAAK,UAAU,QAAQ,CAAC,OAAwB;IAC9C,MAAM,GAAG,GAAG,OAAO,EAAE,CAAC;IACtB,MAAM,mBAAmB,GAAG,IAAI,0BAA0B,CACxD,OAAO,CAAC,OAAO,EACf,OAAO,CAAC,SAAS,CAClB,CAAC;IACF,MAAM,GAAG,CAAC,QAAQ,CAAC,mBAAmB,EAAE,EAAE,mBAAmB,EAAE,CAAC,CAAC;IACjE,OAAO,GAAG,CAAC;AACb,CAAC;AAED,QAAQ,CAAC,+DAA+D,EAAE,GAAG,EAAE;IAC7E,QAAQ,CAAC,0CAA0C,EAAE,GAAG,EAAE;QACxD,EAAE,CAAC,oEAAoE,EAAE,KAAK,IAAI,EAAE;YAClF,MAAM,OAAO,GAAG,IAAI,wBAAwB,EAAE,CAAC;YAC/C,OAAO,CAAC,GAAG,CAAC,SAAS,EAAE,iBAAiB,CAAC;gBACvC,QAAQ,EAAE,IAAI;gBACd,MAAM,EAAE,CAAC,EAAE,IAAI,EAAE,UAAU,EAAE,WAAW,EAAE,UAAU,EAAE,CAAC;gBACvD,aAAa,EAAE,EAAE,aAAa,EAAE,EAAE,EAAE,cAAc,EAAE,GAAG,EAAE;aAC1D,CAAC,CAAC,CAAC;YACJ,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC;YAExC,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;gBAChC,MAAM,EAAE,OAAO;gBACf,GAAG,EAAE,2BAA2B,GAAG,kBAAkB,CAAC,SAAS,CAAC;gBAChE,OAAO,EAAE,EAAE,QAAQ,EAAE,IAAI,EAAE;aAC5B,CAAC,CAAC;YAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACtC,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAAiD,CAAC;YAC5E,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAChC,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACxC,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,CAAC,EAAE,IAAI,EAAE,UAAU,EAAE,WAAW,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC;YACpF,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,aAAa,CAAC,CAAC,OAAO,CAAC,EAAE,aAAa,EAAE,EAAE,EAAE,cAAc,EAAE,GAAG,EAAE,CAAC,CAAC;YACtF,MAAM,SAAS,GAAG,OAAO,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC;YACzC,MAAM,CAAC,SAAS,EAAE,QAAQ,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACvC,MAAM,CAAC,SAAS,EAAE,MAAM,CAAC,CAAC,OAAO,CAAC,CAAC,EAAE,IAAI,EAAE,UAAU,EAAE,WAAW,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC;YAEnF,MAAM,GAAG,CAAC,KAAK,EAAE,CAAC;QACpB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,uEAAuE,EAAE,KAAK,IAAI,EAAE;YACrF,MAAM,OAAO,GAAG,IAAI,wBAAwB,EAAE,CAAC;YAC/C,OAAO,CAAC,GAAG,CAAC,SAAS,EAAE,iBAAiB,CAAC,EAAE,YAAY,EAAE,OAAO,EAAE,CAAC,CAAC,CAAC;YACrE,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC;YAExC,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;gBAChC,MAAM,EAAE,OAAO;gBACf,GAAG,EAAE,2BAA2B,GAAG,kBAAkB,CAAC,SAAS,CAAC;gBAChE,OAAO,EAAE,EAAE,YAAY,EAAE,QAAQ,EAAE;aACpC,CAAC,CAAC;YAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACtC,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAA+B,CAAC;YAC1D,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YAChD,MAAM,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;YACvC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACjC,IAAI,MAAM,CAAC,MAAM,KAAK,IAAI,EAAE,CAAC;gBAC3B,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;YACpD,CAAC;YAED,MAAM,GAAG,CAAC,KAAK,EAAE,CAAC;QACpB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,2DAA2D,EAAE,KAAK,IAAI,EAAE;YACzE,MAAM,OAAO,GAAG,IAAI,wBAAwB,EAAE,CAAC;YAC/C,OAAO,CAAC,GAAG,CAAC,SAAS,EAAE,iBAAiB,EAAE,CAAC,CAAC;YAC5C,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC;YAExC,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;gBAChC,MAAM,EAAE,OAAO;gBACf,GAAG,EAAE,2BAA2B,GAAG,kBAAkB,CAAC,SAAS,CAAC;gBAChE,OAAO,EAAE,EAAE,YAAY,EAAE,+BAA+B,EAAE;aAC3D,CAAC,CAAC;YAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACtC,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAA+B,CAAC;YAC1D,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,+BAA+B,CAAC,CAAC;YAEvE,MAAM,GAAG,CAAC,KAAK,EAAE,CAAC;QACpB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qEAAqE,EAAE,KAAK,IAAI,EAAE;YACnF,MAAM,OAAO,GAAG,IAAI,wBAAwB,EAAE,CAAC;YAC/C,OAAO,CAAC,GAAG,CAAC,SAAS,EAAE,iBAAiB,CAAC,EAAE,YAAY,EAAE,qBAAqB,EAAE,CAAC,CAAC,CAAC;YACnF,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC;YAExC,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;gBAChC,MAAM,EAAE,OAAO;gBACf,GAAG,EAAE,2BAA2B,GAAG,kBAAkB,CAAC,SAAS,CAAC;gBAChE,OAAO,EAAE,EAAE,YAAY,EAAE,EAAE,EAAE;aAC9B,CAAC,CAAC;YAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACtC,MAAM,SAAS,GAAG,OAAO,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC;YACzC,MAAM,CAAC,SAAS,EAAE,YAAY,CAAC,CAAC,aAAa,EAAE,CAAC;YAEhD,MAAM,GAAG,CAAC,KAAK,EAAE,CAAC;QACpB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gEAAgE,EAAE,KAAK,IAAI,EAAE;YAC9E,MAAM,OAAO,GAAG,IAAI,wBAAwB,EAAE,CAAC;YAC/C,OAAO,CAAC,GAAG,CAAC,SAAS,EAAE,iBAAiB,EAAE,CAAC,CAAC;YAC5C,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC;YAExC,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;gBAChC,MAAM,EAAE,OAAO;gBACf,GAAG,EAAE,2BAA2B,GAAG,kBAAkB,CAAC,SAAS,CAAC;gBAChE,OAAO,EAAE,EAAE,YAAY,EAAE,yBAAyB,EAAE;aACrD,CAAC,CAAC;YAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACtC,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAAyC,CAAC;YACpE,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACjC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,4BAA4B,CAAC,CAAC;YAEtD,MAAM,GAAG,CAAC,KAAK,EAAE,CAAC;QACpB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;YACpE,MAAM,OAAO,GAAG,IAAI,wBAAwB,EAAE,CAAC;YAC/C,OAAO,CAAC,GAAG,CAAC,SAAS,EAAE,iBAAiB,EAAE,CAAC,CAAC;YAC5C,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC;YAExC,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;gBAChC,MAAM,EAAE,OAAO;gBACf,GAAG,EAAE,2BAA2B,GAAG,kBAAkB,CAAC,SAAS,CAAC;gBAChE,OAAO,EAAE,EAAE,YAAY,EAAE,qBAAqB,EAAE;aACjD,CAAC,CAAC;YAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACtC,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,EAAuB,CAAC;YAClD,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;YAExC,MAAM,GAAG,CAAC,KAAK,EAAE,CAAC;QACpB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,wDAAwD,EAAE,KAAK,IAAI,EAAE;YACtE,MAAM,OAAO,GAAG,IAAI,wBAAwB,EAAE,CAAC;YAC/C,OAAO,CAAC,GAAG,CAAC,SAAS,EAAE,iBAAiB,EAAE,CAAC,CAAC;YAC5C,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC;YAExC,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;gBAChC,MAAM,EAAE,OAAO;gBACf,GAAG,EAAE,2BAA2B,GAAG,kBAAkB,CAAC,SAAS,CAAC;gBAChE,OAAO,EAAE,EAAE,YAAY,EAAE,WAAW,EAAE;aACvC,CAAC,CAAC;YAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACtC,MAAM,CAAE,QAAQ,CAAC,IAAI,EAAwB,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;YAE1E,MAAM,GAAG,CAAC,KAAK,EAAE,CAAC;QACpB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,uBAAuB,EAAE,KAAK,IAAI,EAAE;YACrC,MAAM,OAAO,GAAG,IAAI,wBAAwB,EAAE,CAAC;YAC/C,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC;YAExC,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;gBAChC,MAAM,EAAE,OAAO;gBACf,GAAG,EAAE,2BAA2B,GAAG,kBAAkB,CAAC,UAAU,CAAC;gBACjE,OAAO,EAAE,EAAE,QAAQ,EAAE,IAAI,EAAE;aAC5B,CAAC,CAAC;YAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YAEtC,MAAM,GAAG,CAAC,KAAK,EAAE,CAAC;QACpB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,kEAAkE,EAAE,KAAK,IAAI,EAAE;YAChF,MAAM,OAAO,GAAG,IAAI,wBAAwB,EAAE,CAAC;YAC/C,OAAO,CAAC,GAAG,CAAC,SAAS,EAAE,iBAAiB,EAAE,CAAC,CAAC;YAC5C,OAAO,CAAC,cAAc,CAAC,SAAS,CAAC,CAAC;YAClC,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC;YAExC,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;gBAChC,MAAM,EAAE,OAAO;gBACf,GAAG,EAAE,2BAA2B,GAAG,kBAAkB,CAAC,SAAS,CAAC;gBAChE,OAAO,EAAE,EAAE,QAAQ,EAAE,IAAI,EAAE;aAC5B,CAAC,CAAC;YAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACtC,MAAM,CAAE,QAAQ,CAAC,IAAI,EAAwB,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,gCAAgC,CAAC,CAAC;YAE5F,MAAM,GAAG,CAAC,KAAK,EAAE,CAAC;QACpB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;YAClE,MAAM,OAAO,GAAG,IAAI,wBAAwB,EAAE,CAAC;YAC/C,OAAO,CAAC,GAAG,CAAC,SAAS,EAAE,iBAAiB,EAAE,CAAC,CAAC;YAC5C,OAAO,CAAC,YAAY,EAAE,CAAC;YACvB,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC;YAExC,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;gBAChC,MAAM,EAAE,OAAO;gBACf,GAAG,EAAE,2BAA2B,GAAG,kBAAkB,CAAC,SAAS,CAAC;gBAChE,OAAO,EAAE,EAAE,QAAQ,EAAE,IAAI,EAAE;aAC5B,CAAC,CAAC;YAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACtC,MAAM,CAAE,QAAQ,CAAC,IAAI,EAAwB,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAC;YAEpF,MAAM,GAAG,CAAC,KAAK,EAAE,CAAC;QACpB,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,mEAAmE,EAAE,KAAK,IAAI,EAAE;YACjF,MAAM,OAAO,GAAG,IAAI,wBAAwB,EAAE,CAAC;YAC/C,OAAO,CAAC,GAAG,CAAC,SAAS,EAAE,iBAAiB,CAAC;gBACvC,MAAM,EAAE,CAAC,EAAE,IAAI,EAAE,UAAU,EAAE,WAAW,EAAE,UAAU,EAAE,CAAC;aACxD,CAAC,CAAC,CAAC;YACJ,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC;YAExC,MAAM,QAAQ,GAAG,MAAM,GAAG,CAAC,MAAM,CAAC;gBAChC,MAAM,EAAE,OAAO;gBACf,GAAG,EAAE,2BAA2B,GAAG,kBAAkB,CAAC,SAAS,CAAC;gBAChE,OAAO,EAAE;oBACP,QAAQ,EAAE,IAAI;oBACd,MAAM,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,EAAE,CAAC;iBAChD;aACF,CAAC,CAAC;YAEH,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACtC,MAAM,SAAS,GAAG,OAAO,CAAC,GAAG,CAAC,SAAS,CAAC,CAAC;YACzC,MAAM,CAAC,SAAS,EAAE,MAAM,CAAC,CAAC,OAAO,CAAC,CAAC,EAAE,IAAI,EAAE,UAAU,EAAE,WAAW,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC;YACnF,MAAM,CAAC,SAAS,EAAE,QAAQ,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAEvC,MAAM,GAAG,CAAC,KAAK,EAAE,CAAC;QACpB,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,6BAA6B,EAAE,GAAG,EAAE;QAC3C,EAAE,CAAC,yEAAyE,EAAE,KAAK,IAAI,EAAE;YACvF,MAAM,OAAO,GAAG,IAAI,wBAAwB,EAAE,CAAC;YAC/C,OAAO,CAAC,GAAG,CAAC,SAAS,EAAE,iBAAiB,CAAC,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC;YAC9D,MAAM,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;YACzC,MAAM,GAAG,GAAG,MAAM,QAAQ,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC;YAExC,MAAM,GAAG,CAAC,MAAM,CAAC;gBACf,MAAM,EAAE,OAAO;gBACf,GAAG,EAAE,2BAA2B,GAAG,kBAAkB,CAAC,SAAS,CAAC;gBAChE,OAAO,EAAE,EAAE,QAAQ,EAAE,IAAI,EAAE;aAC5B,CAAC,CAAC;YAEH,MAAM,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;YACrC,IAAI,QAAQ,CAAC,MAAM,KAAK,IAAI,EAAE,CAAC;gBAC7B,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAC9C,CAAC;iBAAM,CAAC;gBACN,MAAM,IAAI,KAAK,CAAC,8BAA8B,CAAC,CAAC;YAClD,CAAC;YACD,IAAI,IAAI,CAAC,MAAM,KAAK,IAAI,EAAE,CAAC;gBACzB,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAC1C,CAAC;iBAAM,CAAC;gBACN,MAAM,IAAI,KAAK,CAAC,0BAA0B,CAAC,CAAC;YAC9C,CAAC;YAED,MAAM,GAAG,CAAC,KAAK,EAAE,CAAC;QACpB,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,+CAA+C,EAAE,GAAG,EAAE;QAC7D,EAAE,CAAC,yBAAyB,EAAE,GAAG,EAAE;YACjC,MAAM,CAAC,oBAAoB,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC;QACzD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,sBAAsB,EAAE,GAAG,EAAE;YAC9B,MAAM,CAAC,oBAAoB,CAAC,qBAAqB,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC;QAC5E,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,iDAAiD,EAAE,GAAG,EAAE;YACzD,MAAM,CAAC,oBAAoB,CAAC,yBAAyB,CAAC,CAAC,CAAC,OAAO,CAAC;gBAC9D,EAAE,EAAE,KAAK;gBACT,OAAO,EAAE,4BAA4B;aACtC,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,8CAA8C,EAAE,GAAG,EAAE;YACtD,MAAM,CAAC,oBAAoB,CAAC,qBAAqB,CAAC,CAAC,CAAC,OAAO,CAAC;gBAC1D,EAAE,EAAE,KAAK;gBACT,OAAO,EAAE,cAAc;aACxB,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;YACpD,MAAM,CAAC,oBAAoB,CAAC,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,cAAc,EAAE,CAAC,CAAC;QAC5F,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,iCAAiC,EAAE,GAAG,EAAE;QAC/C,EAAE,CAAC,8EAA8E,EAAE,GAAG,EAAE;YACtF,MAAM,IAAI,GAAG,uBAAuB,CAClC,sBAAsB,CAAC;gBACrB,MAAM,EAAE,iBAAiB,CAAC;oBACxB,QAAQ,EAAE,IAAI;oBACd,YAAY,EAAE,MAAM;oBACpB,WAAW,EAAE,aAAa;oBAC1B,mBAAmB,EAAE,iBAAiB;oBACtC,YAAY,EAAE,qBAAqB;iBACpC,CAAC;gBACF,WAAW,EAAE,GAAG;aACjB,CAAC,CACH,CAAC;YAEF,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC;YAC5B,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,iBAAiB,CAAC,CAAC;YAC1C,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,qBAAqB,CAAC,CAAC;YAC9C,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;YAC7C,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,4BAA4B,CAAC,CAAC;YACrD,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,qBAAqB,CAAC,CAAC;YAC9C,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,qBAAqB,CAAC,CAAC;QAChD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,2DAA2D,EAAE,GAAG,EAAE;QACzE,IAAI,SAAiB,CAAC;QACtB,IAAI,SAAiB,CAAC;QAEtB,UAAU,CAAC,GAAG,EAAE;YACd,SAAS,GAAG,YAAY,CAAC,eAAe,EAAE,OAAO,CAAC,CAAC;YACnD,SAAS,GAAG,YAAY,CAAC,eAAe,EAAE,OAAO,CAAC,CAAC;QACrD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,sEAAsE,EAAE,GAAG,EAAE;YAC9E,MAAM,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC,8BAA8B,CAAC,CAAC;QAC5D,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,uDAAuD,EAAE,GAAG,EAAE;YAC/D,MAAM,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC,iCAAiC,CAAC,CAAC;QAC/D,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;YACxD,MAAM,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC,oBAAoB,CAAC,CAAC;QAClD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,+EAA+E,EAAE,GAAG,EAAE;YACvF,MAAM,mBAAmB,GAAG,SAAS,CAAC,KAAK,CACzC,iEAAiE,CAClE,IAAI,EAAE,CAAC;YACR,MAAM,YAAY,GAAG,mBAAmB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACpD,MAAM,CAAC,YAAY,CAAC,CAAC,OAAO,CAAC,kBAAkB,CAAC,CAAC;QACnD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
|
|
@@ -0,0 +1,10 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* SPEC-91 — Dashboard Multi-Project Overview
|
|
3
|
+
*
|
|
4
|
+
* Outer-loop acceptance test (SDD): exercises the new GET /api/repositories
|
|
5
|
+
* endpoint and the OverviewPresenter aggregation end-to-end without infra
|
|
6
|
+
* (no DB, no real CLI). Covers Scenarios 2, 4, 6, 9, 10, 12 plus the
|
|
7
|
+
* /api/repositories shape per docs/specs/91-dashboard-multi-project-overview.md.
|
|
8
|
+
*/
|
|
9
|
+
export {};
|
|
10
|
+
//# sourceMappingURL=91-dashboard-multi-project-overview.acceptance.test.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"91-dashboard-multi-project-overview.acceptance.test.d.ts","sourceRoot":"","sources":["../../../src/tests/acceptance/91-dashboard-multi-project-overview.acceptance.test.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG"}
|
|
@@ -0,0 +1,275 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* SPEC-91 — Dashboard Multi-Project Overview
|
|
3
|
+
*
|
|
4
|
+
* Outer-loop acceptance test (SDD): exercises the new GET /api/repositories
|
|
5
|
+
* endpoint and the OverviewPresenter aggregation end-to-end without infra
|
|
6
|
+
* (no DB, no real CLI). Covers Scenarios 2, 4, 6, 9, 10, 12 plus the
|
|
7
|
+
* /api/repositories shape per docs/specs/91-dashboard-multi-project-overview.md.
|
|
8
|
+
*/
|
|
9
|
+
import Fastify from 'fastify';
|
|
10
|
+
import { describe, expect, it } from 'vitest';
|
|
11
|
+
import { repositoriesRoutes } from '../../modules/cli-configuration/interface-adapters/controllers/http/repositories.routes.js';
|
|
12
|
+
import { OverviewPresenter } from '../../modules/statistics-insights/interface-adapters/presenters/overview.presenter.js';
|
|
13
|
+
import { RepositoryConfigFactory } from '../../tests/factories/repositoryConfig.factory.js';
|
|
14
|
+
import { ProjectStatsApiResponseFactory } from '../../tests/factories/projectStatsApiResponse.factory.js';
|
|
15
|
+
import { RecentReviewFileFactory } from '../../tests/factories/recentReviewFile.factory.js';
|
|
16
|
+
import { ReviewStatsFactory } from '../../tests/factories/projectStats.factory.js';
|
|
17
|
+
const NOW = new Date('2026-05-25T12:00:00.000Z');
|
|
18
|
+
async function buildAcceptanceApp(repositories) {
|
|
19
|
+
const app = Fastify();
|
|
20
|
+
await app.register(repositoriesRoutes, {
|
|
21
|
+
getRepositories: () => repositories,
|
|
22
|
+
addRepository: () => ({ status: 'ok', repositories }),
|
|
23
|
+
removeRepository: () => ({ status: 'ok', repositories }),
|
|
24
|
+
patchRepository: () => ({ status: 'ok', repositories }),
|
|
25
|
+
});
|
|
26
|
+
return app;
|
|
27
|
+
}
|
|
28
|
+
describe('Acceptance — SPEC-91: Dashboard Multi-Project Overview', () => {
|
|
29
|
+
describe('GET /api/repositories — tab bar data source', () => {
|
|
30
|
+
it('returns enabled and disabled repos with name, localPath, platform, enabled', async () => {
|
|
31
|
+
const repositories = [
|
|
32
|
+
RepositoryConfigFactory.create({
|
|
33
|
+
name: 'frontend',
|
|
34
|
+
localPath: '/repos/frontend',
|
|
35
|
+
platform: 'gitlab',
|
|
36
|
+
enabled: true,
|
|
37
|
+
}),
|
|
38
|
+
RepositoryConfigFactory.create({
|
|
39
|
+
name: 'api',
|
|
40
|
+
localPath: '/repos/api',
|
|
41
|
+
platform: 'github',
|
|
42
|
+
enabled: false,
|
|
43
|
+
}),
|
|
44
|
+
];
|
|
45
|
+
const app = await buildAcceptanceApp(repositories);
|
|
46
|
+
const response = await app.inject({ method: 'GET', url: '/api/repositories' });
|
|
47
|
+
expect(response.statusCode).toBe(200);
|
|
48
|
+
const body = response.json();
|
|
49
|
+
expect(body.repositories).toHaveLength(2);
|
|
50
|
+
expect(body.repositories[0]).toEqual({
|
|
51
|
+
name: 'frontend',
|
|
52
|
+
localPath: '/repos/frontend',
|
|
53
|
+
platform: 'gitlab',
|
|
54
|
+
enabled: true,
|
|
55
|
+
});
|
|
56
|
+
expect(body.repositories[1]).toEqual({
|
|
57
|
+
name: 'api',
|
|
58
|
+
localPath: '/repos/api',
|
|
59
|
+
platform: 'github',
|
|
60
|
+
enabled: false,
|
|
61
|
+
});
|
|
62
|
+
await app.close();
|
|
63
|
+
});
|
|
64
|
+
it('returns empty array when no repositories configured', async () => {
|
|
65
|
+
const app = await buildAcceptanceApp([]);
|
|
66
|
+
const response = await app.inject({ method: 'GET', url: '/api/repositories' });
|
|
67
|
+
expect(response.statusCode).toBe(200);
|
|
68
|
+
const body = response.json();
|
|
69
|
+
expect(body.repositories).toEqual([]);
|
|
70
|
+
await app.close();
|
|
71
|
+
});
|
|
72
|
+
});
|
|
73
|
+
describe('OverviewPresenter — aggregation across projects', () => {
|
|
74
|
+
it('Scenario 2: active reviews across all projects, ordered by startedAt DESC', () => {
|
|
75
|
+
const presenter = new OverviewPresenter({ now: () => NOW });
|
|
76
|
+
const startedSevenMinutesAgo = new Date(NOW.getTime() - 7 * 60_000).toISOString();
|
|
77
|
+
const startedThreeMinutesAgo = new Date(NOW.getTime() - 3 * 60_000).toISOString();
|
|
78
|
+
const viewModel = presenter.present({
|
|
79
|
+
repositories: [
|
|
80
|
+
RepositoryConfigFactory.create({ name: 'frontend', localPath: '/repos/frontend', platform: 'gitlab' }),
|
|
81
|
+
RepositoryConfigFactory.create({ name: 'api', localPath: '/repos/api', platform: 'github' }),
|
|
82
|
+
],
|
|
83
|
+
activeJobs: [
|
|
84
|
+
{
|
|
85
|
+
id: 'github:api:28',
|
|
86
|
+
mrNumber: 28,
|
|
87
|
+
project: '/repos/api',
|
|
88
|
+
mrUrl: 'https://github.com/org/api/pull/28',
|
|
89
|
+
status: 'running',
|
|
90
|
+
startedAt: startedSevenMinutesAgo,
|
|
91
|
+
title: 'feat: new endpoint',
|
|
92
|
+
jobType: 'review',
|
|
93
|
+
},
|
|
94
|
+
{
|
|
95
|
+
id: 'gitlab:frontend:142',
|
|
96
|
+
mrNumber: 142,
|
|
97
|
+
project: '/repos/frontend',
|
|
98
|
+
mrUrl: 'https://gitlab.com/org/frontend/-/merge_requests/142',
|
|
99
|
+
status: 'running',
|
|
100
|
+
startedAt: startedThreeMinutesAgo,
|
|
101
|
+
title: 'feat: dashboard',
|
|
102
|
+
jobType: 'review',
|
|
103
|
+
},
|
|
104
|
+
],
|
|
105
|
+
projectStats: [],
|
|
106
|
+
recentReviews: [],
|
|
107
|
+
});
|
|
108
|
+
expect(viewModel.activeReviews.items).toHaveLength(2);
|
|
109
|
+
expect(viewModel.activeReviews.items[0]?.projectName).toBe('frontend');
|
|
110
|
+
expect(viewModel.activeReviews.items[0]?.mrNumber).toBe(142);
|
|
111
|
+
expect(viewModel.activeReviews.items[0]?.mrPrefix).toBe('MR');
|
|
112
|
+
expect(viewModel.activeReviews.items[0]?.elapsedLabel).toBe('3m');
|
|
113
|
+
expect(viewModel.activeReviews.items[1]?.projectName).toBe('api');
|
|
114
|
+
expect(viewModel.activeReviews.items[1]?.mrNumber).toBe(28);
|
|
115
|
+
expect(viewModel.activeReviews.items[1]?.mrPrefix).toBe('PR');
|
|
116
|
+
expect(viewModel.activeReviews.items[1]?.elapsedLabel).toBe('7m');
|
|
117
|
+
expect(viewModel.activeReviews.isEmpty).toBe(false);
|
|
118
|
+
});
|
|
119
|
+
it('Scenario 4: project cards with total reviews, average score, and sparkline points', () => {
|
|
120
|
+
const presenter = new OverviewPresenter({ now: () => NOW });
|
|
121
|
+
const frontendReviews = Array.from({ length: 12 }, (_, index) => ReviewStatsFactory.create({
|
|
122
|
+
id: `frontend-${index}`,
|
|
123
|
+
timestamp: new Date(NOW.getTime() - (12 - index) * 60_000).toISOString(),
|
|
124
|
+
mrNumber: 100 + index,
|
|
125
|
+
score: 7,
|
|
126
|
+
}));
|
|
127
|
+
const apiReviews = Array.from({ length: 8 }, (_, index) => ReviewStatsFactory.create({
|
|
128
|
+
id: `api-${index}`,
|
|
129
|
+
timestamp: new Date(NOW.getTime() - (8 - index) * 60_000).toISOString(),
|
|
130
|
+
mrNumber: 20 + index,
|
|
131
|
+
score: 8,
|
|
132
|
+
}));
|
|
133
|
+
const viewModel = presenter.present({
|
|
134
|
+
repositories: [
|
|
135
|
+
RepositoryConfigFactory.create({ name: 'frontend', localPath: '/repos/frontend' }),
|
|
136
|
+
RepositoryConfigFactory.create({ name: 'api', localPath: '/repos/api' }),
|
|
137
|
+
],
|
|
138
|
+
activeJobs: [],
|
|
139
|
+
projectStats: [
|
|
140
|
+
ProjectStatsApiResponseFactory.create({
|
|
141
|
+
project: 'frontend',
|
|
142
|
+
path: '/repos/frontend',
|
|
143
|
+
totalReviews: 12,
|
|
144
|
+
averageScore: 7.2,
|
|
145
|
+
reviews: frontendReviews,
|
|
146
|
+
}),
|
|
147
|
+
ProjectStatsApiResponseFactory.create({
|
|
148
|
+
project: 'api',
|
|
149
|
+
path: '/repos/api',
|
|
150
|
+
totalReviews: 8,
|
|
151
|
+
averageScore: 8.1,
|
|
152
|
+
reviews: apiReviews,
|
|
153
|
+
}),
|
|
154
|
+
],
|
|
155
|
+
recentReviews: [],
|
|
156
|
+
});
|
|
157
|
+
expect(viewModel.projectCards.items).toHaveLength(2);
|
|
158
|
+
const frontendCard = viewModel.projectCards.items.find((card) => card.projectName === 'frontend');
|
|
159
|
+
const apiCard = viewModel.projectCards.items.find((card) => card.projectName === 'api');
|
|
160
|
+
expect(frontendCard?.totalReviews).toBe(12);
|
|
161
|
+
expect(frontendCard?.averageScoreLabel).toBe('7.2');
|
|
162
|
+
expect(frontendCard?.sparklinePoints).toHaveLength(10);
|
|
163
|
+
expect(apiCard?.totalReviews).toBe(8);
|
|
164
|
+
expect(apiCard?.averageScoreLabel).toBe('8.1');
|
|
165
|
+
expect(apiCard?.sparklinePoints).toHaveLength(8);
|
|
166
|
+
expect(apiCard?.isEmptyHistory).toBe(false);
|
|
167
|
+
});
|
|
168
|
+
it('Scenario 6: recent reviews feed across projects, ordered DESC, capped at 10', () => {
|
|
169
|
+
const presenter = new OverviewPresenter({ now: () => NOW });
|
|
170
|
+
const recentReviews = Array.from({ length: 12 }, (_, index) => RecentReviewFileFactory.create({
|
|
171
|
+
filename: `2026-05-25-MR-${100 + index}.md`,
|
|
172
|
+
path: `/repos/${index % 2 === 0 ? 'frontend' : 'api'}/.claude/reviews/2026-05-25-MR-${100 + index}.md`,
|
|
173
|
+
mrNumber: String(100 + index),
|
|
174
|
+
type: 'MR',
|
|
175
|
+
mtime: new Date(NOW.getTime() - (12 - index) * 60_000).toISOString(),
|
|
176
|
+
title: `Review ${index}`,
|
|
177
|
+
}));
|
|
178
|
+
const viewModel = presenter.present({
|
|
179
|
+
repositories: [
|
|
180
|
+
RepositoryConfigFactory.create({ name: 'frontend', localPath: '/repos/frontend' }),
|
|
181
|
+
RepositoryConfigFactory.create({ name: 'api', localPath: '/repos/api' }),
|
|
182
|
+
],
|
|
183
|
+
activeJobs: [],
|
|
184
|
+
projectStats: [],
|
|
185
|
+
recentReviews,
|
|
186
|
+
});
|
|
187
|
+
expect(viewModel.recentReviewsFeed.items).toHaveLength(10);
|
|
188
|
+
expect(viewModel.recentReviewsFeed.items[0]?.filename).toBe('2026-05-25-MR-111.md');
|
|
189
|
+
expect(viewModel.recentReviewsFeed.items[0]?.projectName).toBe('api');
|
|
190
|
+
expect(viewModel.recentReviewsFeed.items[0]?.mrPrefix).toBe('MR');
|
|
191
|
+
expect(viewModel.recentReviewsFeed.isEmpty).toBe(false);
|
|
192
|
+
});
|
|
193
|
+
it('Scenario 9: no configured projects renders empty states with French messages', () => {
|
|
194
|
+
const presenter = new OverviewPresenter({ now: () => NOW });
|
|
195
|
+
const viewModel = presenter.present({
|
|
196
|
+
repositories: [],
|
|
197
|
+
activeJobs: [],
|
|
198
|
+
projectStats: [],
|
|
199
|
+
recentReviews: [],
|
|
200
|
+
});
|
|
201
|
+
expect(viewModel.activeReviews.isEmpty).toBe(true);
|
|
202
|
+
expect(viewModel.activeReviews.emptyMessage).toBe('Aucune review en cours');
|
|
203
|
+
expect(viewModel.projectCards.isEmpty).toBe(true);
|
|
204
|
+
expect(viewModel.projectCards.emptyMessage).toBe('Aucun projet configuré');
|
|
205
|
+
expect(viewModel.recentReviewsFeed.isEmpty).toBe(true);
|
|
206
|
+
expect(viewModel.recentReviewsFeed.emptyMessage).toBe('Aucune review récente');
|
|
207
|
+
});
|
|
208
|
+
it('Scenario 10: project with 0 reviews shows score "-" and empty sparkline', () => {
|
|
209
|
+
const presenter = new OverviewPresenter({ now: () => NOW });
|
|
210
|
+
const viewModel = presenter.present({
|
|
211
|
+
repositories: [RepositoryConfigFactory.create({ name: 'new-project', localPath: '/repos/new' })],
|
|
212
|
+
activeJobs: [],
|
|
213
|
+
projectStats: [
|
|
214
|
+
ProjectStatsApiResponseFactory.create({
|
|
215
|
+
project: 'new-project',
|
|
216
|
+
path: '/repos/new',
|
|
217
|
+
totalReviews: 0,
|
|
218
|
+
averageScore: null,
|
|
219
|
+
reviews: [],
|
|
220
|
+
}),
|
|
221
|
+
],
|
|
222
|
+
recentReviews: [],
|
|
223
|
+
});
|
|
224
|
+
expect(viewModel.projectCards.items).toHaveLength(1);
|
|
225
|
+
const card = viewModel.projectCards.items[0];
|
|
226
|
+
expect(card?.totalReviews).toBe(0);
|
|
227
|
+
expect(card?.averageScoreLabel).toBe('-');
|
|
228
|
+
expect(card?.sparklinePoints).toEqual([]);
|
|
229
|
+
expect(card?.isEmptyHistory).toBe(true);
|
|
230
|
+
});
|
|
231
|
+
it('Scenario 12: review completes — moves from active to recent on next present()', () => {
|
|
232
|
+
const presenter = new OverviewPresenter({ now: () => NOW });
|
|
233
|
+
const startedAt = new Date(NOW.getTime() - 5 * 60_000).toISOString();
|
|
234
|
+
const beforeCompletion = presenter.present({
|
|
235
|
+
repositories: [RepositoryConfigFactory.create({ name: 'frontend', localPath: '/repos/frontend' })],
|
|
236
|
+
activeJobs: [
|
|
237
|
+
{
|
|
238
|
+
id: 'gitlab:frontend:142',
|
|
239
|
+
mrNumber: 142,
|
|
240
|
+
project: '/repos/frontend',
|
|
241
|
+
mrUrl: 'https://gitlab.com/org/frontend/-/merge_requests/142',
|
|
242
|
+
status: 'running',
|
|
243
|
+
startedAt,
|
|
244
|
+
title: 'feat: dashboard',
|
|
245
|
+
jobType: 'review',
|
|
246
|
+
},
|
|
247
|
+
],
|
|
248
|
+
projectStats: [],
|
|
249
|
+
recentReviews: [],
|
|
250
|
+
});
|
|
251
|
+
const afterCompletion = presenter.present({
|
|
252
|
+
repositories: [RepositoryConfigFactory.create({ name: 'frontend', localPath: '/repos/frontend' })],
|
|
253
|
+
activeJobs: [],
|
|
254
|
+
projectStats: [],
|
|
255
|
+
recentReviews: [
|
|
256
|
+
RecentReviewFileFactory.create({
|
|
257
|
+
filename: '2026-05-25-MR-142.md',
|
|
258
|
+
path: '/repos/frontend/.claude/reviews/2026-05-25-MR-142.md',
|
|
259
|
+
mrNumber: '142',
|
|
260
|
+
type: 'MR',
|
|
261
|
+
mtime: NOW.toISOString(),
|
|
262
|
+
title: 'feat: dashboard',
|
|
263
|
+
}),
|
|
264
|
+
],
|
|
265
|
+
});
|
|
266
|
+
expect(beforeCompletion.activeReviews.items).toHaveLength(1);
|
|
267
|
+
expect(beforeCompletion.recentReviewsFeed.items).toHaveLength(0);
|
|
268
|
+
expect(afterCompletion.activeReviews.items).toHaveLength(0);
|
|
269
|
+
expect(afterCompletion.recentReviewsFeed.items).toHaveLength(1);
|
|
270
|
+
expect(afterCompletion.recentReviewsFeed.items[0]?.mrNumber).toBe('142');
|
|
271
|
+
expect(afterCompletion.recentReviewsFeed.items[0]?.projectName).toBe('frontend');
|
|
272
|
+
});
|
|
273
|
+
});
|
|
274
|
+
});
|
|
275
|
+
//# sourceMappingURL=91-dashboard-multi-project-overview.acceptance.test.js.map
|