Always show warning before destroying a bundle (#1614)

## Changes

WIP: need to update integ tests

## Tests

<!-- How is this tested? -->

Author: ilia-db

packages/databricks-vscode/src/test/e2e/destroy.e2e.ts

@@ -128,8 +128,13 @@ describe("Deploy and destroy", async function () {
         assert(found, `Job ${jobName} not found in workspace`);
 
         await browser.executeWorkbench(async (vscode) => {
-            await vscode.commands.executeCommand("databricks.bundle.destroy");
+            await vscode.commands.executeCommand(
+                "databricks.bundle.destroy",
+                false, // Don't force it
+                false // Skip the modal warning dialog, as they don't work in tests
+            );
         });
+
         console.log("Waiting for bundle to destroy");
         // Wait for status to reach success
         await browser.waitUntil(

packages/databricks-vscode/src/ui/bundle-resource-explorer/BundleCommands.ts

@@ -313,16 +313,15 @@ export class BundleCommands implements Disposable {
         this.bundleRunStatusManager.cancel(treeNode.resourceKey);
     }
 
-    async destroy(force = false) {
-        if ((await this.configModel.get("mode")) !== "development") {
-            const confirm = await window.showErrorMessage(
+    async destroy(force = false, warn = true) {
+        if (warn) {
+            const choice = await window.showWarningMessage(
                 "Are you sure you want to destroy this bundle and all resources associated with it?",
                 {modal: true},
                 "Yes, continue",
                 "No"
             );
-
-            if (confirm !== "Yes, continue") {
+            if (choice !== "Yes, continue") {
                 return;
             }
         }