fix: update properly html block internal state

ged-odoo · ged-odoo · commit 7b69aea7723f · 2022-02-09T13:45:57.000+01:00
diff --git a/src/html.ts b/src/html.ts
@@ -61,6 +61,7 @@ class VHtml {
       // remove current content
       this.remove();
       this.content = content;
+      this.html = other.html;
     }
   }
 
diff --git a/tests/html.test.ts b/tests/html.test.ts
@@ -1,4 +1,4 @@
-import { createBlock, html, mount, patch, text } from "../src";
+import { createBlock, html, mount, multi, patch, text } from "../src";
 import { makeTestFixture } from "./helpers";
 
 //------------------------------------------------------------------------------
@@ -45,4 +45,16 @@ describe("html block", () => {
     patch(tree, html(`<tr><td>potato</td></tr>`));
     expect(fixture.innerHTML).toBe("<tr><td>potato</td></tr>");
   });
+
+  test("html vnode can be used in a multi, and updated", () => {
+    const tree = multi([html("<p>a</p>"), html("<p>b</p>")]);
+    mount(tree, fixture);
+    expect(fixture.innerHTML).toBe("<p>a</p><p>b</p>");
+
+    patch(tree, multi([html("<p>b</p>"), html("<p>a</p>")]));
+    expect(fixture.innerHTML).toBe("<p>b</p><p>a</p>");
+
+    patch(tree, multi([html("<p>a</p>"), html("<p>b</p>")]));
+    expect(fixture.innerHTML).toBe("<p>a</p><p>b</p>");
+  });
 });
