adds type test

Author: guillaumegarcia13

types/index.d.ts

@@ -17,14 +17,19 @@ declare namespace remove {
  * Mutate the given tree by removing all nodes that pass test. The tree is walked in preorder (NLR), visiting the node itself, then its head, etc.
  *
  * @param tree      Tree to filter
- * @param options   Whether to drop parent nodes if they had children, but all their children were filtered out
  * @param test      is-compatible test (such as a type)
  */
-declare function remove(tree: Node, test?: Test<Node>): Node
-declare function remove(
+export function remove(tree: Node, test?: Test<Node>): Node
+/**
+ *
+ * Mutate the given tree by removing all nodes that pass test. The tree is walked in preorder (NLR), visiting the node itself, then its head, etc.
+ *
+ * @param tree      Tree to filter
+ * @param options   Whether to drop parent nodes if they had children, but all their children were filtered out. Default is {cascade: true}
+ * @param test      is-compatible test (such as a type)
+ */
+export function remove(
   tree: Node,
-  options: remove.RemoveOptions = {cascade: true},
+  options?: remove.RemoveOptions,
   test?: Test<Node>
 ): Node
-
-export = remove

types/test.ts

@@ -0,0 +1,19 @@
+import {remove} from 'uninst-util-remove';
+import * as u from 'unist-builder';
+
+let tree = u('root', [
+  u('leaf', '1'),
+  u('node', [
+    u('leaf', '2'),
+    u('node', [u('leaf', '3'), u('other', '4')]),
+    u('node', [u('leaf', '5')])
+  ]),
+  u('leaf', '6')
+]);
+
+remove()          // $ExpectError
+remove('leaf')    // $ExpectError
+
+remove(tree)
+remove(tree, 'leaf');
+remove(tree, {cascade: false}, 'leaf');