riot

Author: mrginglymus

libraries/riot/src/basic-tests.js

@@ -23,6 +23,7 @@ import {
   ComponentWithChildrenRerender,
   ComponentWithDifferentViews,
   ComponentWithProperties,
+  ComponentWithoutProperties,
   ComponentWithUnregistered,
   ComponentWithImperativeEvent
 } from './components'
@@ -122,6 +123,16 @@ describe('basic support', function() {
       expect(data).to.eql('riot')
     })
 
+    it("will not overwrite unwriteable properties", function () {
+      this.weight = 3;
+      component(ComponentWithoutProperties)(scratch)
+      let wc = scratch.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');
+    });
+
     // Riot.js passes all the following tests but they must be commented out
     // being consistent with the other frameworks ¯\_(ツ)_/¯
     /*

libraries/riot/src/components.js

@@ -18,13 +18,15 @@
 import 'ce-without-children'
 import 'ce-with-children'
 import 'ce-with-properties'
+import 'ce-without-properties'
 import 'ce-with-event'
 
 export { default as ComponentWithoutChildren } from './components/ComponentWithoutChildren.riot'
 export { default as ComponentWithChildren } from './components/ComponentWithChildren.riot'
 export { default as ComponentWithChildrenRerender } from './components/ComponentWithChildrenRerender.riot'
 export { default as ComponentWithDifferentViews } from './components/ComponentWithDifferentViews.riot'
 export { default as ComponentWithProperties } from './components/ComponentWithProperties.riot'
+export { default as ComponentWithoutProperties } from './components/ComponentWithoutProperties.riot'
 export { default as ComponentWithUnregistered } from './components/ComponentWithUnregistered.riot'
 export { default as ComponentWithImperativeEvent } from './components/ComponentWithImperativeEvent.riot'
 export { default as ComponentWithDeclarativeEvent } from './components/ComponentWithDeclarativeEvent.riot'

libraries/riot/src/components/ComponentWithoutProperties.riot

@@ -0,0 +1,12 @@
+<component-with-properties>
+  <ce-without-properties id="wc" {...state}></ce-without-properties>
+  <script>
+    export default {
+      state: {
+        agetter: "getter",
+        amethod: "method",
+        areadonly: "readonly"
+      }
+    }
+  </script>
+</component-with-properties>