Move parentBy to traversing namespace

spec/parentBy.spec.js → spec/traversing/parent-by.spec.js

@@ -1,19 +1,17 @@
-import { matches, parentBy } from "../src";
+import { matches, parentBy } from "../../src";
 
 describe("parentBy", () => {
   afterEach(clearFixtures);
 
   describe("when selector given", () => {
     it("returns parent element which matches by selector", () => {
-      useFixture(
-        `
+      useFixture(`
         <div class="parent">
           <div class="wrapper">
             <div class="root"></div>
           </div>
         </div>
-      `
-      );
+      `);
 
       const subject = document.querySelector(".root");
 
@@ -23,15 +21,13 @@ describe("parentBy", () => {
     });
 
     it("returns null if parent element which matches doesn't exists", () => {
-      useFixture(
-        `
+      useFixture(`
         <div class="outer-wrapper">
           <div class="inner-wrapper">
             <div class="root"></div>
           </div>
         </div>
-      `
-      );
+      `);
 
       const subject = document.querySelector(".root");
 
@@ -41,15 +37,13 @@ describe("parentBy", () => {
 
   describe("when predicate function is given", () => {
     it("returns parent element which matches by predicate", () => {
-      useFixture(
-        `
+      useFixture(`
         <div class="parent">
           <div class="wrapper">
             <div class="root"></div>
           </div>
         </div>
-      `
-      );
+      `);
 
       const subject = document.querySelector(".root");
       const predicate = e => matches(e, ".parent");
@@ -60,15 +54,13 @@ describe("parentBy", () => {
     });
 
     it("returns null if parent element which matches doesn't exists", () => {
-      useFixture(
-        `
+      useFixture(`
         <div class="outer-wrapper">
           <div class="inner-wrapper">
             <div class="root"></div>
           </div>
         </div>
-      `
-      );
+      `);
 
       const subject = document.querySelector(".root");
       const predicate = e => matches(e, ".parent");

src/index.js

@@ -7,6 +7,7 @@ import query from "./queries/query";
 import queryAll from "./queries/query-all";
 import matches from "./traversing/matches";
 import parent from "./traversing/parent";
+import parentBy from "./traversing/parent-by";
 
 /* Types */
 
@@ -97,27 +98,6 @@ function dataset(element: HTMLElement): Dataset {
  * Traverse
  */
 
-function parentBy(
-  element: Element,
-  condition: PredicateFn | Selector
-): ?Element {
-  const predicate = typeof condition === "string"
-    ? e => matches(e, condition)
-    : condition;
-
-  let current = parent(element);
-
-  while (current) {
-    if (predicate(current)) {
-      return current;
-    }
-
-    current = parent(current);
-  }
-
-  return null;
-}
-
 function parents(element: Element): Elements {
   const result = [];
 

src/traversing/parent-by.js

@@ -0,0 +1,26 @@
+/* @flow */
+
+import type { Predicate, Selector } from "../types";
+
+import matches from "./matches";
+import parent from "./parent";
+
+function parentBy(element: Element, condition: Predicate | Selector): ?Element {
+  const predicate = typeof condition === "string"
+    ? e => matches(e, condition)
+    : condition;
+
+  let current = parent(element);
+
+  while (current) {
+    if (predicate(current)) {
+      return current;
+    }
+
+    current = parent(current);
+  }
+
+  return null;
+}
+
+export default parentBy;

src/types.js

@@ -2,4 +2,6 @@
 
 export type Elements = Array<Element>;
 
+export type Predicate = (element: Element) => boolean;
+
 export type Selector = string;