Update code, tests for 100% coverage

Author: wooorm

handlers/heading.js

@@ -5,6 +5,10 @@ module.exports = heading;
 var all = require('../all');
 
 function heading(h, node) {
-  var depth = Number(node.tagName.charAt(1)) || 1;
+  var depth = Number(node.tagName.charAt(1));
+
+  /* istanbul ignore next - `else` shouldn’t happen, of course… */
+  depth = depth || 1;
+
   return h(node, 'heading', {depth: depth}, all(h, node));
 }

handlers/list-item.js

@@ -5,9 +5,8 @@ module.exports = listItem;
 var is = require('hast-util-is-element');
 var all = require('../all');
 
-function listItem(h, node, parent) {
+function listItem(h, node) {
   var children = node.children;
-  var tail = !parent || parent.children[parent.children.length - 1] === node;
   var head = children[0];
   var checked = null;
   var loose = false;
@@ -64,10 +63,5 @@ function listItem(h, node, parent) {
     }
   }
 
-  /* Last list-item is never loose. */
-  if (tail) {
-    loose = false;
-  }
-
   return h(node, 'listItem', {loose: loose, checked: checked}, content);
 }

tests/fixtures/ol/index.html

@@ -15,9 +15,18 @@
 <p>Bar.</p>
 
 <ol start="3">
-  <li><p>Alpha</p></li>
-  <li><p>Bravo</p></li>
-  <li><p>Charlie</p></li>
+  <li>
+    <p>Alpha</p>
+    <p>Bravo</p>
+  </li>
+  <li>
+    <p>Charlie</p>
+    <p>Delta</p>
+  </li>
+  <li>
+    <p>Echo</p>
+    <p>Foxtrot</p>
+  </li>
 </ol>
 
 <p>Baz.</p>

tests/fixtures/ol/index.md

@@ -13,8 +13,16 @@ Foo.
 Bar.
 
 3.  Alpha
-4.  Bravo
-5.  Charlie
+
+    Bravo
+
+4.  Charlie
+
+    Delta
+
+5.  Echo
+
+    Foxtrot
 
 Baz.
 

tests/fixtures/ul/index.html

@@ -15,9 +15,18 @@
 <p>Bar.</p>
 
 <ul>
-  <li><p>Alpha</p></li>
-  <li><p>Bravo</p></li>
-  <li><p>Charlie</p></li>
+  <li>
+    <p>Alpha</p>
+    <p>Bravo</p>
+  </li>
+  <li>
+    <p>Charlie</p>
+    <p>Delta</p>
+  </li>
+  <li>
+    <p>Echo</p>
+    <p>Foxtrot</p>
+  </li>
 </ul>
 
 <p>Baz.</p>

tests/fixtures/ul/index.md

@@ -13,9 +13,17 @@ Foo.
 Bar.
 
 -   Alpha
--   Bravo
+
+    Bravo
+
 -   Charlie
 
+    Delta
+
+-   Echo
+
+    Foxtrot
+
 Baz.
 
 -···

tests/index.js

@@ -2,7 +2,8 @@
 
 var fs = require('fs');
 var path = require('path');
-
+var u = require('unist-builder');
+var h = require('hastscript');
 var test = require('tape');
 var hidden = require('is-hidden');
 var negate = require('negate');
@@ -12,10 +13,38 @@ var html = require('rehype-parse');
 var stringify = require('remark-stringify');
 var assert = require('mdast-util-assert');
 var remove = require('unist-util-remove-position');
-var toMDAST = require('../');
+var toMDAST = require('..');
 
 var fixtures = path.join(__dirname, 'fixtures');
 
+test('core', function (t) {
+  t.deepEqual(
+    toMDAST(u('root', [h('strong', 'Alpha')])),
+    u('root', [u('strong', [u('text', 'Alpha')])]),
+    'should transform HAST to MDAST'
+  );
+
+  t.deepEqual(
+    toMDAST(u('root', [u('unknown', 'text')])),
+    u('root', [u('text', 'text')]),
+    'should transform unknown texts to `text`'
+  );
+
+  t.deepEqual(
+    toMDAST(u('root', [u('unknown', [h('em')])])),
+    u('root', [u('emphasis', [])]),
+    'should unwrap unknown parents'
+  );
+
+  t.deepEqual(
+    toMDAST(u('root', [u('unknown')])),
+    u('root', []),
+    'should ignore unknown voids'
+  );
+
+  t.end();
+});
+
 test('fixtures', function (t) {
   var fromHTML = unified()
     .use(html)