hyperapp

Author: mrginglymus

libraries/hyperapp/src/basic-tests.js

@@ -25,7 +25,8 @@ import {
   ComponentWithDifferentViews,
   ComponentWithProperties,
   ComponentWithImperativeEvent,
-  ComponentWithDeclarativeEvent
+  ComponentWithDeclarativeEvent,
+  ComponentWithoutProperties
 } from "./components";
 
 // Setup the test harness. This will get cleaned out with every test.
@@ -130,6 +131,17 @@ describe("basic support", function () {
       let data = wc.str || wc.getAttribute("str");
       expect(data).to.eql("Hyperapp");
     });
+
+    it('will not overwrite unwritable properties', async function() {
+      this.weight = 3;
+      ComponentWithoutProperties(root);
+      await new Promise(requestAnimationFrame);
+      let wc = testContainer.querySelector('#wc');
+      expect(wc.getAttribute('amethod')).to.eql('method');
+      expect(wc.getAttribute('agetter')).to.eql('getter');
+      expect(wc.getAttribute('areadonly')).to.eql('readonly');
+      expect(wc.innerHTML).to.eql('Success');
+    })
   });
 
   describe("events", function () {

libraries/hyperapp/src/components.js

@@ -19,6 +19,7 @@ import "ce-without-children"
 import "ce-with-children"
 import "ce-with-properties"
 import "ce-with-event"
+import "ce-without-properties"
 
 import { app, h, text } from "hyperapp"
 
@@ -81,6 +82,20 @@ export const ComponentWithProperties = node =>
       ]),
   })
 
+export const ComponentWithoutProperties = node =>
+  app({
+    node,
+    init: {},
+    view: () =>
+      h('div', {}, [
+        h('ce-without-properties', {
+          amethod: 'method',
+          agetter: 'getter',
+          areadonly: 'readonly'
+        }),
+      ]),
+  })
+
 export const ComponentWithImperativeEvent = node => {
   const _withElem = (_, opts) =>
     requestAnimationFrame(_ => {