@pyreon/runtime-dom 0.5.5 → 0.5.6

package/src/tests/coverage.test.ts

@@ -13,6 +13,7 @@ import {
   onMount,
   onUnmount,
   onUpdate,
+  Portal,
 } from "@pyreon/core"
 import { signal } from "@pyreon/reactivity"
 import { installDevTools, registerComponent, unregisterComponent } from "../devtools"
@@ -360,6 +361,37 @@ describe("mount.ts — uncovered branches", () => {
     warnSpy.mockRestore()
   })
 
+  test("component returning Promise triggers dev warning", () => {
+    const el = container()
+    const warnSpy = vi.spyOn(console, "warn").mockImplementation(() => {})
+
+    const AsyncComp = (() => Promise.resolve(null)) as unknown as ComponentFn
+    mount(h(AsyncComp, null), el)
+
+    expect(warnSpy).toHaveBeenCalledWith(expect.stringContaining("returned a Promise"))
+    warnSpy.mockRestore()
+  })
+
+  test("void element with children triggers dev warning", () => {
+    const el = container()
+    const warnSpy = vi.spyOn(console, "warn").mockImplementation(() => {})
+
+    mount(h("img", null, "child text"), el)
+
+    expect(warnSpy).toHaveBeenCalledWith(expect.stringContaining("void element"))
+    warnSpy.mockRestore()
+  })
+
+  test("Portal with falsy target warns", () => {
+    const el = container()
+    const warnSpy = vi.spyOn(console, "warn").mockImplementation(() => {})
+
+    mount(h(Portal, { target: null }), el)
+
+    expect(warnSpy).toHaveBeenCalledWith(expect.stringContaining("Portal"))
+    warnSpy.mockRestore()
+  })
+
   test("component subtree mount error with propagateError (lines 298-309)", () => {
     const el = container()
     const errorSpy = vi.spyOn(console, "error").mockImplementation(() => {})
@@ -914,6 +946,16 @@ describe("props.ts — uncovered branches", () => {
     unmount()
   })
 
+  test("non-function event handler triggers dev warning", () => {
+    const el = container()
+    const warnSpy = vi.spyOn(console, "warn").mockImplementation(() => {})
+
+    mount(h("button", { onClick: "not a function" }), el)
+
+    expect(warnSpy).toHaveBeenCalledWith(expect.stringContaining("non-function value"))
+    warnSpy.mockRestore()
+  })
+
   test("n-show prop toggles display (lines 273-277)", () => {
     const el = container()
     const visible = signal(true)

package/src/tests/mount.test.ts

@@ -975,6 +975,23 @@ describe("mount — props (extended)", () => {
     expect(div.style.marginTop).toBe("10px")
   })
 
+  test("style object auto-appends px to numeric values", () => {
+    const el = container()
+    mount(h("div", { style: { height: 100, marginTop: 20, opacity: 0.5, zIndex: 10 } }), el)
+    const div = el.querySelector("div") as HTMLElement
+    expect(div.style.height).toBe("100px")
+    expect(div.style.marginTop).toBe("20px")
+    expect(div.style.opacity).toBe("0.5")
+    expect(div.style.zIndex).toBe("10")
+  })
+
+  test("style object handles CSS custom properties", () => {
+    const el = container()
+    mount(h("div", { style: { "--my-color": "red" } }), el)
+    const div = el.querySelector("div") as HTMLElement
+    expect(div.style.getPropertyValue("--my-color")).toBe("red")
+  })
+
   test("className sets class attribute", () => {
     const el = container()
     mount(h("div", { className: "my-class" }), el)
@@ -1841,7 +1858,6 @@ describe("hydrateRoot — extended", () => {
     const Comp = defineComponent(() => {
       onMount(() => {
         mountCalled = true
-        return undefined
       })
       return h("span", null, "mounted")
     })

package/src/transition-group.ts

@@ -251,7 +251,6 @@ export function TransitionGroup<T = unknown>(props: TransitionGroupProps<T>): VN
   // Fire the effect once the container is in the DOM
   onMount(() => {
     ready.set(true)
-    return undefined
   })
 
   onUnmount(() => {