From f2dc09fc01b8c9b14306ac4b0dc59c815aae972e Mon Sep 17 00:00:00 2001
From: Jarrod Davis <developer@jarrodldavis.com>
Date: Sun, 9 Dec 2018 21:29:43 -0700
Subject: [PATCH 1/6] Fix Test

Set the Jest test environment to `node` to avoid a JSDom error.
---
 jest.config.js | 3 +++
 1 file changed, 3 insertions(+)
 create mode 100644 jest.config.js

diff --git a/jest.config.js b/jest.config.js
new file mode 100644
index 0000000..c1c5f83
--- /dev/null
+++ b/jest.config.js
@@ -0,0 +1,3 @@
+module.exports = {
+    testEnvironment: "node"
+}

From e90bce19008081b239b708c9923758e371fc8d3a Mon Sep 17 00:00:00 2001
From: Jarrod Davis <developer@jarrodldavis.com>
Date: Sun, 9 Dec 2018 21:42:11 -0700
Subject: [PATCH 2/6] Add tests for template tags

Add failing tests for extracting selectors from HTML tags under
`<template>` tags.
---
 __tests__/data.js  | 34 ++++++++++++++++++++++++++++++++++
 __tests__/index.js | 32 ++++++++++++++++++++++++++++++++
 2 files changed, 66 insertions(+)

diff --git a/__tests__/data.js b/__tests__/data.js
index b13fc62..843ea17 100644
--- a/__tests__/data.js
+++ b/__tests__/data.js
@@ -18,3 +18,37 @@ export const TEST_1_TAG = ["html", "head", "title", "body", "div", "a", "input"]
 export const TEST_1_CLASS = ["test-container", "test-footer", "a-link"]
 
 export const TEST_1_ID = ["a-link", "blo"]
+
+export const TEST_2_CONTENT = `
+<template>
+    <div id="app">
+        <div class="test-container">Well</div>
+        <div class="test-footer" id="an-id"></div>
+        <a href="#" id="a-link" class="a-link"></a>
+        <input id="blo" type="text" disabled/>
+    </div>
+</template>
+
+<script>
+export default {
+    name: 'its-just-a-test'
+}
+</script>
+
+<style>
+    .test-container {
+        color: darkgray;
+        text-align: center;
+    }
+
+    .test-footer {
+        font-weight: bold;
+    }
+</style>
+`
+
+export const TEST_2_TAG = ["div", "a", "input"]
+
+export const TEST_2_CLASS = ["test-container", "test-footer", "a-link"]
+
+export const TEST_2_ID = ["a-link", "blo"]
diff --git a/__tests__/index.js b/__tests__/index.js
index f86b9fc..d4f26a6 100644
--- a/__tests__/index.js
+++ b/__tests__/index.js
@@ -1,5 +1,6 @@
 import purgehtml from "./../index.js"
 import { TEST_1_CONTENT, TEST_1_TAG, TEST_1_CLASS, TEST_1_ID } from "./data"
+import { TEST_2_CONTENT, TEST_2_TAG, TEST_2_CLASS, TEST_2_ID } from "./data"
 
 describe("purgehtml", () => {
     it("finds tag selectors", () => {
@@ -30,4 +31,35 @@ describe("purgehtml", () => {
             expect(received.includes(item)).toBe(true)
         }
     })
+
+    describe("from a template tag", () => {
+        it("finds tag selectors", () => {
+            const received = purgehtml.extract(TEST_2_CONTENT)
+            for (let item of TEST_2_TAG) {
+                expect(received.includes(item)).toBe(true)
+            }
+        })
+
+        it("finds classes selectors", () => {
+            const received = purgehtml.extract(TEST_2_CONTENT)
+            for (let item of TEST_2_CLASS) {
+                expect(received.includes(item)).toBe(true)
+            }
+        })
+
+        it("finds id selectors", () => {
+            const received = purgehtml.extract(TEST_2_CONTENT)
+            for (let item of TEST_2_ID) {
+                expect(received.includes(item)).toBe(true)
+            }
+        })
+
+        it("finds all selectors", () => {
+            const received = purgehtml.extract(TEST_2_CONTENT)
+            const selectors = [...TEST_2_TAG, ...TEST_2_CLASS, ...TEST_2_ID]
+            for (let item of selectors) {
+                expect(received.includes(item)).toBe(true)
+            }
+        })
+    })
 })

From 36619988ce121a4a8d909894459e615aa40344fb Mon Sep 17 00:00:00 2001
From: Jarrod Davis <developer@jarrodldavis.com>
Date: Sun, 9 Dec 2018 21:44:16 -0700
Subject: [PATCH 3/6] Organize existing tests

Organize the existing tests for normal HTML documents into a sub-group.
---
 __tests__/index.js | 52 ++++++++++++++++++++++++----------------------
 1 file changed, 27 insertions(+), 25 deletions(-)

diff --git a/__tests__/index.js b/__tests__/index.js
index d4f26a6..9c4decf 100644
--- a/__tests__/index.js
+++ b/__tests__/index.js
@@ -3,34 +3,36 @@ import { TEST_1_CONTENT, TEST_1_TAG, TEST_1_CLASS, TEST_1_ID } from "./data"
 import { TEST_2_CONTENT, TEST_2_TAG, TEST_2_CLASS, TEST_2_ID } from "./data"
 
 describe("purgehtml", () => {
-    it("finds tag selectors", () => {
-        const received = purgehtml.extract(TEST_1_CONTENT)
-        for (let item of TEST_1_TAG) {
-            expect(received.includes(item)).toBe(true)
-        }
-    })
+    describe("from a normal html document", () => {
+        it("finds tag selectors", () => {
+            const received = purgehtml.extract(TEST_1_CONTENT)
+            for (let item of TEST_1_TAG) {
+                expect(received.includes(item)).toBe(true)
+            }
+        })
 
-    it("finds classes selectors", () => {
-        const received = purgehtml.extract(TEST_1_CONTENT)
-        for (let item of TEST_1_CLASS) {
-            expect(received.includes(item)).toBe(true)
-        }
-    })
+        it("finds classes selectors", () => {
+            const received = purgehtml.extract(TEST_1_CONTENT)
+            for (let item of TEST_1_CLASS) {
+                expect(received.includes(item)).toBe(true)
+            }
+        })
 
-    it("finds id selectors", () => {
-        const received = purgehtml.extract(TEST_1_CONTENT)
-        for (let item of TEST_1_ID) {
-            expect(received.includes(item)).toBe(true)
-        }
-    })
+        it("finds id selectors", () => {
+            const received = purgehtml.extract(TEST_1_CONTENT)
+            for (let item of TEST_1_ID) {
+                expect(received.includes(item)).toBe(true)
+            }
+        })
 
-    it("finds all selectors", () => {
-        const received = purgehtml.extract(TEST_1_CONTENT)
-        const selectors = [...TEST_1_TAG, ...TEST_1_CLASS, ...TEST_1_ID]
-        for (let item of selectors) {
-            expect(received.includes(item)).toBe(true)
-        }
-    })
+        it("finds all selectors", () => {
+            const received = purgehtml.extract(TEST_1_CONTENT)
+            const selectors = [...TEST_1_TAG, ...TEST_1_CLASS, ...TEST_1_ID]
+            for (let item of selectors) {
+                expect(received.includes(item)).toBe(true)
+            }
+        })
+    });
 
     describe("from a template tag", () => {
         it("finds tag selectors", () => {

From 42abc77eb2881202c05644c2844440a44d79d72b Mon Sep 17 00:00:00 2001
From: Jarrod Davis <developer@jarrodldavis.com>
Date: Sun, 9 Dec 2018 22:02:41 -0700
Subject: [PATCH 4/6] Implement fix for template tags

If a node type is `root`, gather selectors from children.
---
 index.js | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/index.js b/index.js
index e308c60..2e9e3d2 100644
--- a/index.js
+++ b/index.js
@@ -20,6 +20,8 @@ const getSelectorsInNodes = node => {
                 ...ids,
                 ...getSelectorsInNodes(childNode)
             ]
+        } else if (childNode.type === "root") {
+            return getSelectorsInNodes(childNode);
         }
     }
     return selectors

From b7b521398805cf2d7558bce5a86014d3d4f9adf8 Mon Sep 17 00:00:00 2001
From: Jarrod Davis <developer@jarrodldavis.com>
Date: Sun, 9 Dec 2018 22:15:33 -0700
Subject: [PATCH 5/6] Don't early-exit when encountering `root` nodes

Add to the existing `selectors` return array when gathering selectors
from a `root` type node instead of immediately returning.
---
 index.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/index.js b/index.js
index 2e9e3d2..85ef960 100644
--- a/index.js
+++ b/index.js
@@ -21,7 +21,10 @@ const getSelectorsInNodes = node => {
                 ...getSelectorsInNodes(childNode)
             ]
         } else if (childNode.type === "root") {
-            return getSelectorsInNodes(childNode);
+            selectors = [
+                ...selectors,
+                getSelectorsInNodes(childNode)
+            ]
         }
     }
     return selectors

From 0f47feea1a1124c76bab949091d1c2cfdd0ee8e7 Mon Sep 17 00:00:00 2001
From: Jarrod Davis <developer@jarrodldavis.com>
Date: Sun, 9 Dec 2018 22:25:18 -0700
Subject: [PATCH 6/6] Fix root node array spread.

Actually use the spread operator when gathering selectors from `root`
type nodes.
---
 index.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/index.js b/index.js
index 85ef960..3b912f6 100644
--- a/index.js
+++ b/index.js
@@ -23,7 +23,7 @@ const getSelectorsInNodes = node => {
         } else if (childNode.type === "root") {
             selectors = [
                 ...selectors,
-                getSelectorsInNodes(childNode)
+                ...getSelectorsInNodes(childNode)
             ]
         }
     }