From 1a0771d91eccd9a0bc96f3b8fee60b70c68cd1c3 Mon Sep 17 00:00:00 2001
From: Martin Stamm <martin.stamm@camunda.com>
Date: Thu, 30 May 2024 14:41:41 +0200
Subject: [PATCH] fix(App): stop showing dialogs after first cancel

closes #4186
---
 client/src/app/App.js               | 13 ++---
 client/src/app/__tests__/AppSpec.js | 82 ++++++++++++++++++++++++++++-
 2 files changed, 88 insertions(+), 7 deletions(-)

diff --git a/client/src/app/App.js b/client/src/app/App.js
index 90690f700d..5e1ac3ed40 100644
--- a/client/src/app/App.js
+++ b/client/src/app/App.js
@@ -1992,13 +1992,14 @@ export class App extends PureComponent {
   async quit() {
     const { tabs } = this.state;
 
-    const quitTasks = tabs.map((tab) => {
-      return () => this.saveBeforeClose(tab);
-    });
-
-    const quitResults = await pSeries(quitTasks);
+    let canQuit = true;
+    for (const tab of tabs) {
+      canQuit = await this.saveBeforeClose(tab);
 
-    const canQuit = quitResults.every(result => result);
+      if (!canQuit) {
+        break;
+      }
+    }
 
     try {
       await this.workspaceChanged(false);
diff --git a/client/src/app/__tests__/AppSpec.js b/client/src/app/__tests__/AppSpec.js
index 74cdc4dc57..7910d9694e 100644
--- a/client/src/app/__tests__/AppSpec.js
+++ b/client/src/app/__tests__/AppSpec.js
@@ -720,6 +720,86 @@ describe('<App>', function() {
     });
 
 
+    it('should ask user to save or discard changes before quiting', async function() {
+
+      // given
+      const dialog = new Dialog();
+
+      const {
+        app
+      } = createApp({
+        globals: {
+          dialog
+        }
+      });
+
+      const showCloseFileDialogSpy = spy(dialog, 'showCloseFileDialog'),
+            saveTabSpy = spy(app, 'saveTab');
+
+      const tab = await app.createDiagram();
+
+      app.setState({
+        ...app.setDirty(tab)
+      });
+
+      dialog.setShowCloseFileDialogResponse({ button: 'discard' });
+
+      // when
+      await app.quit();
+
+      // then
+      expect(showCloseFileDialogSpy).to.have.been.calledWith({
+        name: tab.file.name
+      });
+
+      expect(saveTabSpy).not.to.have.been.called;
+    });
+
+
+    it('should stop asking the user on cancel', async function() {
+
+      // given
+      const dialog = new Dialog();
+
+      const {
+        app
+      } = createApp({
+        globals: {
+          dialog
+        }
+      });
+
+      const showCloseFileDialogSpy = spy(dialog, 'showCloseFileDialog'),
+            saveTabSpy = spy(app, 'saveTab');
+
+      const tab = await app.createDiagram();
+      const tab2 = await app.createDiagram();
+
+
+      app.setState({
+        ...app.setDirty(tab)
+      });
+
+      app.setState({
+        ...app.setDirty(tab2)
+      });
+
+      dialog.setShowCloseFileDialogResponse({ button: 'cancel' });
+
+      // when
+      await app.quit();
+
+      // then
+      expect(showCloseFileDialogSpy).to.have.been.calledOnce;
+
+      expect(showCloseFileDialogSpy).to.have.been.calledWith({
+        name: tab.file.name
+      });
+
+      expect(saveTabSpy).not.to.have.been.called;
+    });
+
+
     it('should not close tab when saving dialog is canceled', async function() {
 
       // given
@@ -984,7 +1064,7 @@ describe('<App>', function() {
     });
 
 
-    it('should save  multiple times', async function() {
+    it('should save multiple times', async function() {
 
       // given
       const file = createFile('diagram_1.bpmn');