Merge 21557e0dccb7c89ea2a1dc4325ab0623f4e1f12f into 4290ea753912378913b1f3f54e0fc5720afeda5d

src/web/App.mjs

@@ -828,7 +828,7 @@ class App {
 
 
     /**
-     * Handler for the history popstate event.
+     * Handler for history popstate and hashchange events.
      * Reloads parameters from the URL.
      *
      * @param {event} e

src/web/Manager.mjs

@@ -125,6 +125,7 @@ class Manager {
         window.addEventListener("focus", this.window.windowFocus.bind(this.window));
         window.addEventListener("statechange", this.app.stateChange.bind(this.app));
         window.addEventListener("popstate", this.app.popState.bind(this.app));
+        window.addEventListener("hashchange", this.app.popState.bind(this.app));
         window.addEventListener("message", this.input.handlePostMessage.bind(this.input));
 
         // Controls

src/web/waiters/WorkerWaiter.mjs

@@ -477,17 +477,24 @@ class WorkerWaiter {
             recipeConfig = this.recipeConfig;
 
         if (this.step) {
+            const stepProgress = Number.isInteger(this.app.progress) ? this.app.progress : 0,
+                breakpointSearchStart = Math.min(stepProgress, recipeConfig.length);
+
             // Remove all breakpoints from the recipe up to progress
-            if (nextInput.progress !== false) {
-                for (let i = 0; i < nextInput.progress; i++) {
-                    if ("breakpoint" in recipeConfig[i]) {
-                        delete recipeConfig[i].breakpoint;
-                    }
+            for (let i = 0; i < breakpointSearchStart; i++) {
+                if ("breakpoint" in recipeConfig[i]) {
+                    delete recipeConfig[i].breakpoint;
                 }
             }
 
-            // Set a breakpoint at the next operation so we stop baking there
-            if (recipeConfig[this.app.progress]) recipeConfig[this.app.progress].breakpoint = true;
+            // Set a breakpoint at the next enabled operation so disabled operations and comments do
+            // not cause the rest of the recipe to bake in a single step.
+            for (let i = breakpointSearchStart; i < recipeConfig.length; i++) {
+                if (!recipeConfig[i].disabled && recipeConfig[i].op !== "Comment") {
+                    recipeConfig[i].breakpoint = true;
+                    break;
+                }
+            }
         }
 
         let transferable;

tests/browser/04_step.js

@@ -0,0 +1,87 @@
+/**
+ * Regression tests for stepping through recipes.
+ *
+ * @copyright Crown Copyright
+ * @license Apache-2.0
+ */
+
+const utils = require("./browserUtils.js");
+
+module.exports = {
+    before: browser => {
+        browser
+            .resizeWindow(1280, 800)
+            .url(browser.launchUrl)
+            .useCss()
+            .waitForElementNotPresent("#preloader", 10000)
+            .click("#auto-bake-label");
+    },
+
+    "Step skips comments and disabled operations when choosing the next breakpoint": browser => {
+        const recipeConfig = [
+            {
+                op: "To Upper case",
+                args: ["All"]
+            },
+            {
+                op: "Comment",
+                args: ["Skip while stepping"]
+            },
+            {
+                op: "ROT13",
+                args: [true, true, false, 13],
+                disabled: true
+            },
+            {
+                op: "To Hex",
+                args: ["Space", 0]
+            }
+        ];
+
+        utils.setInput(browser, "a", false);
+
+        browser
+            .urlHash("recipe=" + JSON.stringify(recipeConfig))
+            .waitUntil(async function() {
+                const result = await this.execute(function() {
+                    return document.querySelectorAll("#rec-list li.operation").length;
+                });
+                return result.value === recipeConfig.length;
+            }, 5000)
+            .waitForElementNotVisible("#output-loader", 10000)
+            .click("#step")
+            .waitForElementNotVisible("#output-loader", 10000)
+            .waitUntil(async function() {
+                const result = await this.execute(function() {
+                    return window.app.manager.output.outputEditorView.state.doc.toString();
+                });
+                return result.value === "A";
+            }, 5000);
+
+        browser.execute(function() {
+            return document.querySelector("#rec-list li.operation.break .op-title")?.textContent;
+        }, [], function({value}) {
+            browser.expect(value).to.equal("To Hex");
+        });
+
+        browser
+            .click("#step")
+            .waitForElementNotVisible("#output-loader", 10000)
+            .waitUntil(async function() {
+                const result = await this.execute(function() {
+                    return window.app.manager.output.outputEditorView.state.doc.toString();
+                });
+                return result.value === "41";
+            }, 5000);
+
+        browser.execute(function() {
+            return document.querySelector("#rec-list li.operation.break .op-title")?.textContent || null;
+        }, [], function({value}) {
+            browser.expect(value).to.equal(null);
+        });
+    },
+
+    after: browser => {
+        browser.end();
+    }
+};

tests/operations/tests/Comment.mjs

@@ -58,6 +58,25 @@ TestRegister.addTests([
             }
         ]
     },
+    {
+        name: "Comment: preserves byte array dish type",
+        input: "00 ff 80 41",
+        expectedOutput: "00 ff 80 41",
+        recipeConfig: [
+            {
+                op: "From Hex",
+                args: ["Space"]
+            },
+            {
+                op: "Comment",
+                args: ["Must not translate the byte array to a string"]
+            },
+            {
+                op: "To Hex",
+                args: ["Space", 0]
+            }
+        ]
+    },
     {
         name: "Label, Comment: Complex content",
         input: ALL_BYTES,