From 430c15ebd766f6cb34d31aadeb2e2fb0cc0f9b83 Mon Sep 17 00:00:00 2001 From: Vito Sansevero Date: Sun, 1 Nov 2026 03:46:12 -0800 Subject: [PATCH] fix(system): one full stop in the move-back review (#320) --- tests/js/workspace-system-thumbnail-location.test.js | 4 ++++ web/js/workspace/system-thumbnail-location.mjs | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/tests/js/workspace-system-thumbnail-location.test.js b/tests/js/workspace-system-thumbnail-location.test.js index 5859cd23..fe93e78b 100644 --- a/tests/js/workspace-system-thumbnail-location.test.js +++ b/tests/js/workspace-system-thumbnail-location.test.js @@ -80,6 +80,10 @@ test('the review of a move back names the folder and the removed sources', async assert.match(rows['Removed after the move'][0], /The folder itself is removed once empty/); assert.deepEqual(rows['Removed after the move'][1], ['1 source no longer listed: 240 thumbnails are removed; they are made again if it returns']); + // Nothing else removed: one full stop, never "created it.." (live test of #592). + const plain = Object.fromEntries(reviewRows({ ...PLAN, from: { mode: 'folder', path: '/data/thumbnails' }, + to: { mode: 'source', path: null }, unlisted: [], totals: { sources: 1, files: 1, bytes: 10 } })); + assert.match(plain['Removed after the move'][0], /Prompt Manager created it\.$/); const fresh = Object.fromEntries(reviewRows({ ...PLAN, to: { mode: 'folder', path: '/d', default: true, will_create: true } })); assert.equal(fresh.To, '/d (new, next to your library)'); }); diff --git a/web/js/workspace/system-thumbnail-location.mjs b/web/js/workspace/system-thumbnail-location.mjs index d9766f0d..0d496085 100644 --- a/web/js/workspace/system-thumbnail-location.mjs +++ b/web/js/workspace/system-thumbnail-location.mjs @@ -109,7 +109,7 @@ export function reviewRows(plan) { + 'they are made again if it returns'); } const oldCopies = plan.from?.mode === 'folder' - ? 'The thumbnail folders of these sources inside it. The folder itself is removed once empty if Prompt Manager created it.' + ? 'The thumbnail folders of these sources inside it. The folder itself is removed once empty if Prompt Manager created it' : 'The old copies in each source’s thumbnails folder'; rows.push(['Removed after the move', [removed.length ? `${oldCopies}, and:` : `${oldCopies}.`, removed]]); if (totals.left_in_place > 0) {