From 5197a234a8ad72100fd710658bb6a8637b9e0cfd Mon Sep 17 00:00:00 2001 From: Jan Rose Date: Thu, 29 Oct 2026 11:51:31 +0100 Subject: [PATCH] Test sync deletion of BOM-prefixed notebooks --- acceptance/bundle/sync/replace-node-type/output.txt | 13 +++++++++++++ acceptance/bundle/sync/replace-node-type/script | 7 +++++++ acceptance/bundle/sync/replace-node-type/test.toml | 2 ++ 3 files changed, 22 insertions(+) diff --git a/acceptance/bundle/sync/replace-node-type/output.txt b/acceptance/bundle/sync/replace-node-type/output.txt index 9e482f6a28e..bb144a93bcd 100644 --- a/acceptance/bundle/sync/replace-node-type/output.txt +++ b/acceptance/bundle/sync/replace-node-type/output.txt @@ -20,3 +20,16 @@ Uploaded plain.sql >>> list NOTEBOOK bom FILE plain.sql + +=== Remove both local files and sync again + +>>> [CLI] bundle sync --output text +Action: DELETE: bom.sql, plain.sql +Deleted bom.sql +Deleted plain.sql +Initial Sync Complete + +=== BUG: bom remains after sync reports deleting bom.sql + +>>> list +NOTEBOOK bom diff --git a/acceptance/bundle/sync/replace-node-type/script b/acceptance/bundle/sync/replace-node-type/script index 32dd2c3cc53..11446a48db6 100644 --- a/acceptance/bundle/sync/replace-node-type/script +++ b/acceptance/bundle/sync/replace-node-type/script @@ -23,3 +23,10 @@ trace list title "Sync replaces both\n" trace $CLI bundle sync --output text 2>&1 | sort trace list + +title "Remove both local files and sync again\n" +rm bom.sql plain.sql +trace $CLI bundle sync --output text 2>&1 | sort + +title "BUG: bom remains after sync reports deleting bom.sql\n" +trace list diff --git a/acceptance/bundle/sync/replace-node-type/test.toml b/acceptance/bundle/sync/replace-node-type/test.toml index aab5e85edef..4ea183843a3 100644 --- a/acceptance/bundle/sync/replace-node-type/test.toml +++ b/acceptance/bundle/sync/replace-node-type/test.toml @@ -1,3 +1,5 @@ +Badness = "After a BOM-prefixed SQL notebook is removed locally, sync reports deleting bom.sql but leaves the remote notebook bom because the sync snapshot retains the .sql extension." + Cloud = true Env.MSYS_NO_PATHCONV = "1" Ignore = [".databricks"]