@operato/shell 10.29.39 → 10.29.43

package/package.json

@@ -2,7 +2,7 @@
   "name": "@operato/shell",
   "description": "WebApplication architecturing shell following open-wc recommendations",
   "author": "heartyoh",
-  "version": "10.29.39",
+  "version": "10.29.43",
   "type": "module",
   "main": "dist/src/index.js",
   "module": "dist/src/index.js",
@@ -60,11 +60,11 @@
   },
   "dependencies": {
     "@material/web": "^2.0.0",
-    "@operato/graphql": "^10.29.39",
+    "@operato/graphql": "^10.29.43",
     "@operato/i18n": "^10.29.27",
-    "@operato/popup": "^10.29.39",
+    "@operato/popup": "^10.29.43",
     "@operato/styles": "^10.29.30",
-    "@operato/utils": "^10.29.39",
+    "@operato/utils": "^10.29.43",
     "@webcomponents/scoped-custom-element-registry": "^0.0.9",
     "lit": "^3.1.2",
     "lodash-es": "^4.17.21",
@@ -103,5 +103,5 @@
       "prettier --write"
     ]
   },
-  "gitHead": "f48742480ad2b9ce87f0b3b4c3ac5923f4866ad6"
+  "gitHead": "ccc06fed313fcc20a076a1cf7bc586fdacaba594"
 }

package/test/leaving-edits.test.ts

@@ -0,0 +1,94 @@
+import { expect, fixture } from '@open-wc/testing'
+import { html } from 'lit'
+
+import { PageView, staysOn } from '../src/app/pages/page-view.js'
+
+/**
+ * What leaving with unsaved edits asks (address-state-ownership §6.4, ruling 2026-10-19): only a change that would
+ * discard them. Grid edits stay in the cached page when another page is shown, as they did before 2026-10-17; a
+ * page's own unsaved state (a modeller's model) is asked about whenever it is left, as it was then.
+ */
+
+class ListPage extends PageView {
+  render() {
+    return html`<ox-grist id="bound"></ox-grist><ox-grist id="detail"></ox-grist>`
+  }
+}
+customElements.define('leaving-list-page', ListPage)
+
+class ModellerPage extends PageView {
+  unsaved = false
+  get addressDirty() {
+    return this.unsaved
+  }
+}
+customElements.define('leaving-modeller-page', ModellerPage)
+
+const query = { route: false, query: true }
+const route = { route: true, query: false }
+
+const listPage = async (bound: number, detail: number) => {
+  const page = await fixture<ListPage>(html`<leaving-list-page active></leaving-list-page>`)
+  Object.assign(page.renderRoot.querySelector('#bound')!, { urlParamsSensitive: true, dirtyRecords: new Array(bound) })
+  Object.assign(page.renderRoot.querySelector('#detail')!, { dirtyRecords: new Array(detail) })
+  return page
+}
+
+describe('leaving with unsaved edits', () => {
+  it('showing another page asks nothing: the grid edits stay in the cached page', async () => {
+    const page = await listPage(1, 1)
+    expect(page.discards(route, false)).to.equal(false)
+  })
+
+  it('a new query for the url-bound grid that holds edits asks', async () => {
+    const page = await listPage(1, 0)
+    expect(page.discards(query, true)).to.equal(true)
+  })
+
+  it('a new query asks nothing when only a grid not bound to the address holds edits', async () => {
+    const page = await listPage(0, 1)
+    expect(page.discards(query, true)).to.equal(false)
+  })
+
+  it('new route arguments for the page on screen ask about every grid', async () => {
+    const page = await listPage(0, 1)
+    expect(page.discards(route, true)).to.equal(true)
+  })
+
+  it("a page's own unsaved state is asked about when the page is left", async () => {
+    const page = await fixture<ModellerPage>(html`<leaving-modeller-page></leaving-modeller-page>`)
+    page.unsaved = true
+    expect(page.discards(route, false)).to.equal(true)
+  })
+
+  it('closing the instance asks first when it holds unsaved edits, and answers with the user', async () => {
+    class Asking extends ListPage {
+      answer = false
+      asked = 0
+      async canDeactivate() {
+        this.asked++
+        return this.answer
+      }
+    }
+    customElements.define('leaving-asking-page', Asking)
+    const page = await fixture<Asking>(html`<leaving-asking-page active></leaving-asking-page>`)
+    Object.assign(page.renderRoot.querySelector('#detail')!, { dirtyRecords: [{}], reset() {} })
+    expect(await page.confirmClose()).to.equal(false)
+    page.answer = true
+    expect(await page.confirmClose()).to.equal(true)
+    expect(page.asked).to.equal(2)
+  })
+
+  it('closing a clean instance asks nothing', async () => {
+    const page = await listPage(0, 0)
+    expect(await page.confirmClose()).to.equal(true)
+  })
+
+  it('a change stays on the page when the path is the same, or names the same page', async () => {
+    const page = await fixture<ListPage>(html`<leaving-list-page data-page="ledger"></leaving-list-page>`)
+    const at = (path: string) => new URL(path, location.origin)
+    expect(staysOn(page, at('/ledger?filters=x'), at('/ledger'))).to.equal(true)
+    expect(staysOn(page, at('/ledger/2'), at('/ledger/1'))).to.equal(true)
+    expect(staysOn(page, at('/employees'), at('/ledger'))).to.equal(false)
+  })
+})