Update resume on revisit dialog and add test

assets/javascripts/discourse/models/custom-wizard.js.es6

@@ -5,6 +5,7 @@ import discourseComputed from "discourse-common/utils/decorators";
 import getUrl from "discourse-common/lib/get-url";
 import CustomWizardField from "./custom-wizard-field";
 import CustomWizardStep from "./custom-wizard-step";
+import DiscourseURL from "discourse/lib/url";
 
 const CustomWizard = EmberObject.extend({
   @discourseComputed("steps.length")
@@ -34,7 +35,7 @@ CustomWizard.reopenClass({
   restart(wizardId) {
     ajax({ url: `/w/${wizardId}/skip`, type: "PUT" })
       .then(() => {
-        window.location.href = `/w/${wizardId}`;
+        DiscourseURL.redirectTo(getUrl(`/w/${wizardId}`));
       })
       .catch(popupAjaxError);
   },
@@ -44,7 +45,7 @@ CustomWizard.reopenClass({
     if (result.redirect_on_complete) {
       url = result.redirect_on_complete;
     }
-    window.location.href = getUrl(url);
+    DiscourseURL.redirectTo(getUrl(url));
   },
 
   build(wizardJson) {

assets/javascripts/discourse/routes/custom-wizard.js.es6

@@ -1,9 +1,11 @@
 import { findCustomWizard, updateCachedWizard } from "../models/custom-wizard";
 import I18n from "I18n";
 import DiscourseRoute from "discourse/routes/discourse";
-import bootbox from "bootbox";
+import { inject as service } from "@ember/service";
 
 export default DiscourseRoute.extend({
+  dialog: service(),
+
   titleToken() {
     const wizard = this.modelFor("custom-wizard");
     return wizard ? wizard.name || wizard.id : I18n.t("wizard.custom_title");
@@ -30,7 +32,7 @@ export default DiscourseRoute.extend({
       {
         label: I18n.t("wizard.incomplete_submission.restart"),
         class: "btn btn-default",
-        callback: () => {
+        action: () => {
           wizardModel.restart();
         },
       },
@@ -40,11 +42,7 @@ export default DiscourseRoute.extend({
       },
     ];
 
-    const options = {
-      onEscape: false,
-    };
-
-    bootbox.dialog(title, buttons, options);
+    this.dialog.dialog({ title, buttons, type: 'confirm' });
   },
 
   afterModel(model) {

test/javascripts/acceptance/wizard-test.js

@@ -1,4 +1,4 @@
-import { visit } from "@ember/test-helpers";
+import { click, visit } from "@ember/test-helpers";
 import { test } from "qunit";
 import {
   acceptance,
@@ -12,9 +12,12 @@ import {
   wizardGuest,
   wizardNoUser,
   wizardNotPermitted,
+  wizardResumeOnRevisit,
 } from "../helpers/wizard";
 import DiscourseURL from "discourse/lib/url";
 import sinon from "sinon";
+import pretender, { response } from "discourse/tests/helpers/create-pretender";
+import I18n from "I18n";
 
 acceptance("Wizard | Not logged in", function (needs) {
   needs.pretender((server, helper) => {
@@ -194,3 +197,53 @@ acceptance("Wizard | Guest access", function (needs) {
     assert.strictEqual($("body.custom-wizard").length, 0);
   });
 });
+
+acceptance("Wizard | Resume on revisit", function (needs) {
+  needs.user();
+
+  test("Shows dialog", async function (assert) {
+    pretender.get("/w/wizard.json", () => {
+      return response(wizardResumeOnRevisit);
+    });
+
+    await visit("/w/wizard");
+
+    assert.strictEqual(count(".dialog-content:visible"), 1);
+    assert.strictEqual(
+      query(".dialog-header h3").textContent.trim(),
+      I18n.t("wizard.incomplete_submission.title", {
+        date: moment(wizardResumeOnRevisit.submission_last_updated_at).format(
+          "MMMM Do YYYY"
+        ),
+      })
+    );
+  });
+
+  test("Resumes when resumed", async function (assert) {
+    pretender.get("/w/wizard.json", () => {
+      return response(wizardResumeOnRevisit);
+    });
+    await visit("/w/wizard");
+    await click(".dialog-footer .btn-primary");
+    assert.strictEqual(count(".dialog-content:visible"), 0);
+  });
+
+  test("Restarts when restarted", async function (assert) {
+    sinon.stub(DiscourseURL, "redirectTo");
+    let skips = 0;
+    pretender.get("/w/wizard.json", () => {
+      return response(wizardResumeOnRevisit);
+    });
+    pretender.put("/w/wizard/skip", () => {
+      skips++;
+      return response({});
+    });
+    await visit("/w/wizard");
+    await click(".dialog-footer .btn-default");
+    assert.strictEqual(skips, 1);
+    assert.ok(
+      DiscourseURL.redirectTo.calledWith("/w/wizard"),
+      "resuming wizard works"
+    );
+  });
+});

test/javascripts/helpers/wizard.js

@@ -23,6 +23,10 @@ wizard.resume_on_revisit = false;
 wizard.submission_last_updated_at = "2022-03-11T20:00:18+01:00";
 wizard.subscribed = false;
 
+const wizardResumeOnRevisit = cloneJSON(wizard);
+wizardResumeOnRevisit.start = "step_2";
+wizardResumeOnRevisit.resume_on_revisit = true;
+
 const stepNotPermitted = cloneJSON(wizard);
 stepNotPermitted.steps[0].permitted = false;
 
@@ -44,6 +48,7 @@ export {
   wizardNotPermitted,
   wizardCompleted,
   wizardGuest,
+  wizardResumeOnRevisit,
   stepNotPermitted,
   allFieldsWizard,
   wizard,