Fix extractReferencedCustomPropertyNames when there's a default value

Munter · Jul 9, 2022 · 5063755 · 5063755
1 parent 5028f72
commit 5063755
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/lib/extractReferencedCustomPropertyNames.js b/lib/extractReferencedCustomPropertyNames.js
@@ -7,7 +7,7 @@ function extractReferencedCustomPropertyNames(cssValue) {
     if (
       node.type === 'function' &&
       node.value === 'var' &&
-      node.nodes.length === 1 &&
+      node.nodes.length >= 1 &&
       node.nodes[0].type === 'word' &&
       /^--/.test(node.nodes[0].value)
     ) {

diff --git a/test/extractReferencedCustomPropertyNames.js b/test/extractReferencedCustomPropertyNames.js
@@ -19,6 +19,16 @@ describe('extractReferencedCustomPropertyNames', function () {
     );
   });
 
+  it('should return the name of a referenced custom property with a default value', function () {
+    expect(
+      extractReferencedCustomPropertyNames(
+        "foo(bar), var(--abc, 'the default'), bla-bla"
+      ),
+      'to equal',
+      new Set(['--abc'])
+    );
+  });
+
   it('should return the names of multiple referenced custom properties', function () {
     expect(
       extractReferencedCustomPropertyNames(