+ {{#if auditDetailView.hasChanges}}
+
+ {{#each auditDetailView.changeRows}}
+
+ {{path}}
+ From: {{from}}
+ {{#if (eq direction "changed")}}→{{else}}{{/if}}
+ To: {{to}}
+
+ {{/each}}
+ |
{{/each}}
{{else}}
diff --git a/test/admin-account-password-route.test.js b/test/admin-account-password-route.test.js
index c4179a1..16ed55c 100644
--- a/test/admin-account-password-route.test.js
+++ b/test/admin-account-password-route.test.js
@@ -1,6 +1,8 @@
const test = require('node:test');
const assert = require('node:assert/strict');
+require('../src/common');
+
const registerAccountRoutes = require('../src/web/routes/admin/account');
const { validatePasswordStrength } = require('../src/auth');
diff --git a/test/admin-users-create-no-roles-route.test.js b/test/admin-users-create-no-roles-route.test.js
index c910a3c..5bb7dbb 100644
--- a/test/admin-users-create-no-roles-route.test.js
+++ b/test/admin-users-create-no-roles-route.test.js
@@ -1,6 +1,8 @@
const test = require('node:test');
const assert = require('node:assert/strict');
+require('../src/common');
+
const registerUsersRoutes = require('../src/web/routes/admin/users');
test('user create route allows users without roles', async () => {
diff --git a/test/admin-users-edit-save-new-route.test.js b/test/admin-users-edit-save-new-route.test.js
index 69457e9..f0d1f00 100644
--- a/test/admin-users-edit-save-new-route.test.js
+++ b/test/admin-users-edit-save-new-route.test.js
@@ -1,6 +1,8 @@
const test = require('node:test');
const assert = require('node:assert/strict');
+require('../src/common');
+
const registerUsersRoutes = require('../src/web/routes/admin/users');
test('user edit save and new goes to the blank create page', async () => {
diff --git a/test/admin-users-password-route.test.js b/test/admin-users-password-route.test.js
index 43f9e2d..7904ac6 100644
--- a/test/admin-users-password-route.test.js
+++ b/test/admin-users-password-route.test.js
@@ -1,6 +1,8 @@
const test = require('node:test');
const assert = require('node:assert/strict');
+require('../src/common');
+
const registerUsersRoutes = require('../src/web/routes/admin/users');
const { validatePasswordStrength } = require('../src/auth');
diff --git a/test/audit-log-route.test.js b/test/audit-log-route.test.js
index d8e1ec1..5f74591 100644
--- a/test/audit-log-route.test.js
+++ b/test/audit-log-route.test.js
@@ -24,7 +24,26 @@ test('audit log uses its own read permission and shared pagination partial', asy
category: 'authentication',
event_type: 'login.success',
target_label: 'newer',
- details_json: null
+ details_json: JSON.stringify({
+ changes: {
+ content: {
+ from: { title: 'Same', body: { color: 'red', keep: 'same' } },
+ to: { title: 'Same', body: { color: 'blue', keep: 'same' } }
+ },
+ permissions: {
+ from: ['dashboard.read', 'screens.read'],
+ to: ['screens.read', 'playlists.read']
+ },
+ backgroundColor: {
+ from: null,
+ to: '#111111'
+ },
+ logoPath: {
+ from: '',
+ to: '/media/logo.png'
+ }
+ }
+ })
},
{
id: 2,
@@ -60,10 +79,23 @@ test('audit log uses its own read permission and shared pagination partial', asy
throw error;
});
assert.match(response.body, /Audit log/);
+ assert.ok(response.body.indexOf('>All categories') < response.body.indexOf('>announcements'));
+ assert.ok(response.body.indexOf('>announcements') < response.body.indexOf('>api-sources'));
+ assert.ok(response.body.indexOf('>api-sources') < response.body.indexOf('>canvas-sizes'));
assert.match(response.body, /table-pagination/);
assert.match(response.body, /data-local-datetime/);
assert.match(response.body, /audit-event-type-options/);
assert.match(response.body, /login\.success/);
+ assert.match(response.body, /content\.body\.color/);
+ assert.match(response.body, />redblueExport {
assert.equal(pool.inserts.length, 1);
assert.equal(pool.inserts[0][6], null);
assert.equal(pool.inserts[0][7], null);
+});
+
+test('audit changes include only fields with different from and to values', () => {
+ assert.deepEqual(buildAuditChanges({
+ name: 'Old name',
+ playlistId: 4,
+ roleIds: [1, 2]
+ }, {
+ name: 'New name',
+ playlistId: 4,
+ roleIds: [2, 3]
+ }), {
+ name: { from: 'Old name', to: 'New name' },
+ roleIds: { from: [1, 2], to: [2, 3] }
+ });
});
\ No newline at end of file
diff --git a/test/playlists-routes.test.js b/test/playlists-routes.test.js
index 653117d..4f11354 100644
--- a/test/playlists-routes.test.js
+++ b/test/playlists-routes.test.js
@@ -1,6 +1,8 @@
const test = require('node:test');
const assert = require('node:assert/strict');
+require('../src/common');
+
const registerPlaylistRoutes = require('../src/web/routes/admin/playlists');
function createAppAndHandlers() {
diff --git a/test/timetables-duplicate.test.js b/test/timetables-duplicate.test.js
index 3d9ffe5..809281b 100644
--- a/test/timetables-duplicate.test.js
+++ b/test/timetables-duplicate.test.js
@@ -3,6 +3,8 @@ const assert = require('node:assert/strict');
const fs = require('node:fs');
const path = require('node:path');
+require('../src/common');
+
const {
buildDuplicateTimetableGroupName,
buildDuplicateTimetableGroup,
diff --git a/test/users-duplicate-route.test.js b/test/users-duplicate-route.test.js
index 544781e..bd137a2 100644
--- a/test/users-duplicate-route.test.js
+++ b/test/users-duplicate-route.test.js
@@ -1,6 +1,8 @@
const test = require('node:test');
const assert = require('node:assert/strict');
+require('../src/common');
+
const registerUsersRoutes = require('../src/web/routes/admin/users');
test('user duplicate route pre-fills the add form', async () => {
diff --git a/test/web-manage-routes.test.js b/test/web-manage-routes.test.js
index 595fd13..87ff64c 100644
--- a/test/web-manage-routes.test.js
+++ b/test/web-manage-routes.test.js
@@ -1,6 +1,8 @@
const test = require('node:test');
const assert = require('node:assert/strict');
+require('../src/common');
+
const registerManageRoutes = require('../src/web/routes/admin/manage');
test('screen update keeps the existing slug on edit', async () => {