Change to remove support for passing a space directly

Author: wooorm

lib/index.js

@@ -42,6 +42,9 @@ import {stringify as spaces} from 'space-separated-tokens'
 import {webNamespaces} from 'web-namespaces'
 import {zwitch} from 'zwitch'
 
+/** @type {Options} */
+const emptyOptions = {}
+
 const own = {}.hasOwnProperty
 
 const one = zwitch('type', {handlers: {root, element, text, comment, doctype}})
@@ -51,13 +54,14 @@ const one = zwitch('type', {handlers: {root, element, text, comment, doctype}})
  *
  * @param {Nodes} tree
  *   Tree to transform.
- * @param {Options | Space | null | undefined} [options]
+ * @param {Options | null | undefined} [options]
  *   Configuration (optional).
  * @returns {Parse5Nodes}
  *   `parse5` node.
  */
 export function toParse5(tree, options) {
-  const space = options && typeof options === 'object' ? options.space : options
+  const settings = options || emptyOptions
+  const space = settings.space
   return one(tree, space === 'svg' ? svg : html)
 }
 

readme.md

@@ -17,7 +17,7 @@
 *   [Install](#install)
 *   [Use](#use)
 *   [API](#api)
-    *   [`toParse5(tree[, space|options])`](#toparse5tree-spaceoptions)
+    *   [`toParse5(tree[, options])`](#toparse5tree-options)
     *   [`Options`](#options)
     *   [`Space`](#space)
 *   [Types](#types)
@@ -93,16 +93,14 @@ Yields:
 This package exports the identifier [`toParse5`][toparse5].
 There is no default export.
 
-### `toParse5(tree[, space|options])`
+### `toParse5(tree[, options])`
 
 Transform a hast tree to Parse5’s AST.
 
 ###### Parameters
 
 *   `tree` ([`HastNode`][hast-node])
     — tree to transform
-*   `space` ([`Space`][space], optional)
-    — same as `{space: space}`
 *   `options` ([`Options`][options], optional)
     — configuration
 
@@ -235,7 +233,7 @@ abide by its terms.
 
 [hast-util-from-parse5]: https://github.com/syntax-tree/hast-util-from-parse5
 
-[toparse5]: #toparse5tree-spaceoptions
+[toparse5]: #toparse5tree-options
 
 [options]: #options
 

test.js

@@ -396,30 +396,6 @@ test('svg', async function (t) {
     )
   })
 
-  await t.test('should transform SVG (given a space)', async function () {
-    const expected = parseFragment(
-      '<svg><circle cx="60"  cy="60" r="50" fill="red"/></svg>'
-    ).childNodes[0]
-    assert(expected && 'childNodes' in expected)
-    const expectedChild = expected.childNodes[0]
-    expectedChild.parentNode = null
-
-    assert.deepEqual(
-      json(
-        toParse5(
-          {
-            type: 'element',
-            tagName: 'circle',
-            properties: {cx: '60', cy: '60', r: '50', fill: 'red'},
-            children: []
-          },
-          'svg'
-        )
-      ),
-      json(expectedChild)
-    )
-  })
-
   await t.test('should transform SVG (given options)', async function () {
     const expected = parseFragment(
       '<svg><circle cx="60"  cy="60" r="50" fill="red"/></svg>'