test: add tests for detailed extractor

FullHuman · Nov 29, 2021 · ca69ec9 · ca69ec9
1 parent 18c5e80
commit ca69ec9
Show file tree

Hide file tree

Showing 5 changed files with 38 additions and 6 deletions.
diff --git a/jest.config.js b/jest.config.js
@@ -6,6 +6,7 @@ module.exports = {
   moduleFileExtensions: ["ts", "tsx", "js", "json"],
   moduleNameMapper: {
     "^purgecss$": "<rootDir>/packages/purgecss/src",
+    "^@fullhuman/purgecss-from-html$": "<rootDir>/packages/purgecss-from-html/src",
   },
   rootDir: __dirname,
   testMatch: ["<rootDir>/packages/**/__tests__/**/*test.ts"],

diff --git a/packages/purgecss/__tests__/index.test.ts b/packages/purgecss/__tests__/index.test.ts
@@ -1,7 +1,7 @@
-import PurgeCSS from "./../src/index";
+import purgecssFromHtml from "@fullhuman/purgecss-from-html";
 import { ExtractorResult } from "../src/types";
-
-import { ROOT_TEST_EXAMPLES } from "./utils";
+import PurgeCSS from "./../src/index";
+import { notFindInCSS, ROOT_TEST_EXAMPLES } from "./utils";
 
 describe("purgecss with config file", () => {
   it("initialize without error with a config file specified", () => {
@@ -71,3 +71,27 @@ describe("special characters, with custom Extractor", () => {
     expect(purgedCSS.includes("\\32 -panel")).toBe(false);
   });
 });
+
+describe("PurgeCSS with detailed extractor for html", () => {
+  let purgedCSS: string;
+  beforeAll(async () => {
+    const resultsPurge = await new PurgeCSS().purge({
+      content: [`${ROOT_TEST_EXAMPLES}chaining-rules/index.html`],
+      css: [`${ROOT_TEST_EXAMPLES}chaining-rules/index.css`],
+      extractors: [
+        {
+          extensions: ["html"],
+          extractor: purgecssFromHtml,
+        },
+      ],
+    });
+    purgedCSS = resultsPurge[0].css;
+  });
+
+  it("keeps parent1 selector", () => {
+    expect(purgedCSS.includes("parent1")).toBe(true);
+  });
+  it("removes parent3, d33ef1, .parent2", () => {
+    notFindInCSS(expect, ["parent3", "d33ef1", "parent2"], purgedCSS);
+  });
+});
diff --git a/packages/purgecss/__tests__/rejected.test.ts b/packages/purgecss/__tests__/rejected.test.ts
@@ -53,6 +53,7 @@ describe("rejected", () => {
       ".parent.d33ef1",
       ".parent2.def",
       ".parent3.def1",
+      "[href^='#']",
     ]);
   });
 });
diff --git a/packages/purgecss/__tests__/test_examples/chaining-rules/index.css b/packages/purgecss/__tests__/test_examples/chaining-rules/index.css
@@ -36,4 +36,8 @@
 }
 .parent3.def1{
     color:red;
+}
+
+[href^='#'] {
+    color: green;
 }
diff --git a/packages/purgecss/src/VariablesStructure.ts b/packages/purgecss/src/VariablesStructure.ts
@@ -40,8 +40,10 @@ class VariablesStructure {
       const variableName = variableMatch[1];
       if (this.nodes.has(variableName)) {
         const usedVariableNodes = this.nodes.get(variableName);
-        nodes?.forEach(node => {
-          usedVariableNodes?.forEach(usedVariableNode => node.nodes.push(usedVariableNode))
+        nodes?.forEach((node) => {
+          usedVariableNodes?.forEach((usedVariableNode) =>
+            node.nodes.push(usedVariableNode)
+          );
         });
       }
     }
@@ -74,7 +76,7 @@ class VariablesStructure {
     for (const used of this.usedVariables) {
       const usedNodes = this.nodes.get(used);
       if (usedNodes) {
-        for (const usedNode of usedNodes) {   
+        for (const usedNode of usedNodes) {
           const usedVariablesMatchesInDeclaration = matchAll(
             usedNode.value.value,
             /var\((.+?)[,)]/g