fix: show all assigned tasks on leader dashboard
This commit is contained in:
1 parent
9d1a6b4379
commit
03d01315a9
4 files changed
+30
-14
No files matched your search
@@ -150,14 +150,16 @@ test('administrators manage task-type grants and manual intake enforces them bef
|
||||
assert.match(parseResult, /taskAuthorization\.allowed && shouldAutomaticallyConfirm/);
|
||||
});
|
||||
|
||||
test('operations dashboard is leadership-gated, business-facing, searchable, and separate from normal task authority', async () => {
|
||||
test('operations dashboard is leadership-gated, cross-source, business-facing, and separate from normal task authority', async () => {
|
||||
const [tasks, server] = await Promise.all([
|
||||
source('../src/task-service.ts'),
|
||||
source('../src/server.ts')
|
||||
]);
|
||||
assert.match(tasks, /canViewOperationsDashboard[\s\S]+role === 'admin' \|\| role === 'team_lead'/);
|
||||
assert.match(tasks, /isTaskOwnerRestricted[\s\S]+role === 'team_lead' \|\| role === 'user'/);
|
||||
assert.match(tasks, /async listOperationsDashboard[\s\S]+t\.source = 'manual'/);
|
||||
assert.match(tasks, /async listOperationsDashboard[\s\S]+t\.assigned_user_id IS NOT NULL[\s\S]+t\.source IN \('manual', 'agentbus'\)/);
|
||||
assert.match(tasks, /actorUserId[\s\S]+t\.assigned_user_id = \$\$\{params\.length\}/);
|
||||
assert.match(tasks, /LEFT JOIN users assignee ON assignee\.id = t\.assigned_user_id/);
|
||||
assert.match(tasks, /operations_dashboard_range_too_large/);
|
||||
assert.match(tasks, /businessRouteId\?: string/);
|
||||
assert.match(tasks, /operations_dashboard_business_invalid/);
|
||||
@@ -194,7 +196,9 @@ test('operations dashboard is leadership-gated, business-facing, searchable, and
|
||||
tasks.indexOf('async getOperationsDashboardTask('),
|
||||
tasks.indexOf('async listAuditEvents(')
|
||||
);
|
||||
assert.match(detailMethod, /source = 'manual'/);
|
||||
assert.match(detailMethod, /t\.assigned_user_id IS NOT NULL/);
|
||||
assert.match(detailMethod, /t\.source IN \('manual', 'agentbus'\)/);
|
||||
assert.doesNotMatch(detailMethod, /t\.source = 'manual'/);
|
||||
assert.match(detailMethod, /getTaskInputHistory/);
|
||||
assert.match(detailMethod, /instructions,[\s\S]+attachments:/);
|
||||
assert.doesNotMatch(detailMethod, /this\.getTask\(/);
|
||||
|
||||
Reference in new issue
Block a user