✅ Complete test for all inclusion and exclusion logic

package.json

@@ -25,6 +25,7 @@
     "@rollup/plugin-commonjs": "^14.0.0",
     "@rollup/plugin-node-resolve": "^8.4.0",
     "chai": "^4.2.0",
+    "chai-match": "^1.1.1",
     "chalk": "^4.1.0",
     "eslint": "^7.5.0",
     "eslint-config-standard": "^14.1.1",

test/inclusionLogic.spec.js

@@ -3,9 +3,10 @@ const postcssRfsAutopilot = require('../index.js')
 
 const chai = require('chai')
 const expect = chai.expect
+chai.use(require('chai-match'))
 
 describe('Test shouldBeTransformed()', function () {
-  let css
+  let css, beforeTransformation, afterTransformation
 
   beforeEach(function () {
     css = `p #hello{
@@ -16,6 +17,19 @@ describe('Test shouldBeTransformed()', function () {
     body{
       padding: 5px;
     }`
+
+    beforeTransformation = []
+    afterTransformation = []
+
+    postcss
+      .parse(css, { from: undefined })
+      .walkDecls((decl) => {
+        beforeTransformation.push({
+          selector: decl.parent.selector,
+          prop: decl.prop,
+          value: decl.value
+        })
+      })
   })
 
   describe('if a value is already wrapped in rfs()', function () {
@@ -26,31 +40,104 @@ describe('Test shouldBeTransformed()', function () {
 
       await postcss([
         postcssRfsAutopilot(options)
-      ]).process(css, { from: undefined }).then(result => {
-        result.root.walkDecls((decl) => {
-          if (decl.prop === 'margin') {
-            expect(decl.value).to.equal('rfs(10rem)')
-          }
+      ])
+        .process(css, { from: undefined }).then(result => {
+          result.root.walkDecls((decl) => {
+            afterTransformation.push({
+              selector: decl.parent.selector,
+              prop: decl.prop,
+              value: decl.value
+            })
+          })
         })
+
+      beforeTransformation.forEach((decl, index) => {
+        if (/^rfs/g.test(decl.value)) {
+          expect(decl.value).to.equal(afterTransformation[index].value)
+        }
       })
     })
   })
 
   describe('if the unit of a value is not included in includedUnits', function () {
-    it('should not be transformed', function (done) {
-      done()
+    it('should not be transformed', async function () {
+      const options = {
+        includedUnits: ['rem']
+      }
+
+      await postcss([
+        postcssRfsAutopilot(options)
+      ])
+        .process(css, { from: undefined }).then(result => {
+          result.root.walkDecls((decl) => {
+            afterTransformation.push({
+              selector: decl.parent.selector,
+              prop: decl.prop,
+              value: decl.value
+            })
+          })
+        })
+
+      beforeTransformation.forEach((decl, index) => {
+        if (!/rem/g.test(decl.value)) {
+          expect(afterTransformation[index].value).to.not.match(/^rfs/g)
+        }
+      })
     })
   })
 
   describe('if the unit of a value is included in excludedUnits', function () {
-    it('should not be transformed', function (done) {
-      done()
+    it('should not be transformed', async function () {
+      const options = {
+        excludedUnits: ['px']
+      }
+
+      await postcss([
+        postcssRfsAutopilot(options)
+      ])
+        .process(css, { from: undefined }).then(result => {
+          result.root.walkDecls((decl) => {
+            afterTransformation.push({
+              selector: decl.parent.selector,
+              prop: decl.prop,
+              value: decl.value
+            })
+          })
+        })
+
+      beforeTransformation.forEach((decl, index) => {
+        if (/px/g.test(decl.value)) {
+          expect(afterTransformation[index].value).to.not.match(/^rfs/g)
+        }
+      })
     })
   })
 
   describe('if the unit of a value is included in includedUnits and excludedUnits', function () {
-    it('should not be transformed', function (done) {
-      done()
+    it('should not be transformed', async function () {
+      const options = {
+        includedUnits: ['px'],
+        excludedUnits: ['px']
+      }
+
+      await postcss([
+        postcssRfsAutopilot(options)
+      ])
+        .process(css, { from: undefined }).then(result => {
+          result.root.walkDecls((decl) => {
+            afterTransformation.push({
+              selector: decl.parent.selector,
+              prop: decl.prop,
+              value: decl.value
+            })
+          })
+        })
+
+      beforeTransformation.forEach((decl, index) => {
+        if (/px/g.test(decl.value)) {
+          expect(afterTransformation[index].value).to.not.match(/^rfs/g)
+        }
+      })
     })
   })
 })