Repro for #12651 (#12919)

* Add repro for #12651

* Refactor test

- stick to CSS classes when targetting icons

* Skip the test

- skip the test until the issue is resolved
- added reference to the original issue in the test description

frontend/test/metabase/scenarios/question/new.cy.spec.js

-import { restore, signInAsAdmin, popover } from "__support__/cypress";
+import {
+  restore,
+  signInAsAdmin,
+  popover,
+  openOrdersTable,
+} from "__support__/cypress";
 
 // test various entry points into the query builder
 
@@ -25,6 +30,20 @@ describe("scenarios > question > new", () => {
       cy.contains("Orders").click();
       cy.contains("37.65");
     });
+
+    it.skip("should remove `/notebook` from URL when converting question to SQL/Native (Issue #12651)", () => {
+      cy.server();
+      cy.route("POST", "/api/dataset").as("dataset");
+      openOrdersTable();
+      cy.wait("@dataset");
+      cy.url().should("include", "question#");
+      // Isolate icons within "QueryBuilder" scope because there is also `.Icon-sql` in top navigation
+      cy.get(".QueryBuilder .Icon-notebook").click();
+      cy.url().should("include", "question/notebook#");
+      cy.get(".QueryBuilder .Icon-sql").click();
+      cy.findByText("Convert this question to SQL").click();
+      cy.url().should("include", "question#");
+    });
   });
 
   describe("ask a (custom) question", () => {