Merge pull request #102 from achou11/69/set-map-weakmap-props

add support for passing Map, Set, and WeakMap instances as props
socketsupply · Jan 12, 2023 · e84d21e · e84d21e
2 parents e340536 + de7428a
commit e84d21e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/src/index.js b/src/index.js
@@ -181,7 +181,10 @@ export class Tonic extends window.HTMLElement {
           return this._prop(o)
         }
         case '[object Object]':
-        case '[object Function]': return this._prop(o)
+        case '[object Function]':
+        case '[object Set]':
+        case '[object Map]':
+        case '[object WeakMap]': return this._prop(o)
         case '[object NamedNodeMap]':
           return this._prop(Tonic._normalizeAttrs(o))
         case '[object Number]': return `${o}__float`

diff --git a/test/index.js b/test/index.js
@@ -236,7 +236,10 @@ test('pass props', async t => {
           id="y"
           data=${test}
           number=${42.42}
-          fn=${() => 'hello, world'}>
+          fn=${() => 'hello, world'}
+          set=${new Set(['foo'])}
+          map=${new Map([['bar', 'bar']])}
+          weakmap=${new WeakMap([[document, 'baz']])}>
         </component-b-b>
       `
     }
@@ -256,6 +259,9 @@ test('pass props', async t => {
     const props = bb.props
     t.equal(props.fn(), 'hello, world', 'passed a function')
     t.equal(props.number, 42.42, 'float parsed properly')
+    t.equal(props.set.has('foo'), true, 'set parsed properly')
+    t.equal(props.map.get('bar'), 'bar', 'map parsed properly')
+    t.equal(props.weakmap.get(document), 'baz', 'weak map parsed properly')
   }
 
   const div1 = document.getElementsByTagName('div')[0]