color-name-list 13.2.0 → 13.4.0

package/tests/duplicate-allowlist.json

@@ -36,5 +36,48 @@
   "Dark as Night",
   "Green With Envy",
   "Lost in Space",
-  "Fruit of Passion"
+  "Fruit of Passion",
+  "Aroma Garden",
+  "Back in Black",
+  "Tea Rose",
+  "Red Violet",
+  "Blue Green",
+  "Blue Grey",
+  "Pink Red",
+  "Black Green",
+  "Red Purple",
+  "Red Revival",
+  "Purple Pink",
+  "Pearl White",
+  "Milk Chocolate",
+  "Violet Blue",
+  "Black Violet",
+  "Black Velvet",
+  "Rose Pearl",
+  "Yellow Orange",
+  "Red Orange",
+  "Pink Orange",
+  "Pine Mountain",
+  "Star of the Morning",
+  "Rose Madder",
+  "Lily the Pink",
+  "Purple Grey",
+  "Yellow Green",
+  "White Green",
+  "Turquoise Green",
+  "Green Teal",
+  "Yellow Brown",
+  "Grey Green",
+  "Grape Wine",
+  "Blue Light",
+  "Indigo Blue",
+  "Blue Purple",
+  "Blue Turquoise",
+  "Brown Green",
+  "Brown Grey",
+  "Brown Orange",
+  "Brown Red",
+  "Brown Rose",
+  "Honey Butter",
+  "Custard Cream"
 ]

package/tests/duplicates.test.js

@@ -74,6 +74,13 @@ describe('Duplicate-like color names', () => {
       groups.get(key).push({ name: item.name, lineNumber: item.lineNumber, order });
     }
 
+    // Build allowlist set once
+    const allowSet = new Set(
+      (Array.isArray(allowlist) ? allowlist : [])
+        .filter((v) => typeof v === 'string' && v.trim().length)
+        .map((v) => normalize(v))
+    );
+
     const conflicts = [];
     for (const [key, entries] of groups.entries()) {
       // We have a potential conflict if we see both orders "a b" and "b a"
@@ -99,13 +106,9 @@ describe('Duplicate-like color names', () => {
           })
           .sort((a, b) => a.lineNumber - b.lineNumber);
 
-        // Respect allowlist: if either direction string is allowlisted, skip
-        const allowSet = new Set(
-          (Array.isArray(allowlist) ? allowlist : [])
-            .filter((v) => typeof v === 'string' && v.trim().length)
-            .map((v) => normalize(v))
-        );
-        if (allowSet.has(forward) || allowSet.has(backward)) continue;
+        // Respect allowlist: check if any of the actual names are allowlisted
+        const hasAllowlisted = unique.some((e) => allowSet.has(normalize(e.name)));
+        if (hasAllowlisted) continue;
 
         conflicts.push({ key, tokens: [t1, t2], entries: unique });
       }
@@ -250,7 +253,7 @@ describe('Duplicate-like color names', () => {
     expect(conflicts[0].entries.length).toBe(2);
   });
 
-  it.skip('should not contain two-word names that are exact reversals of each other', () => {
+  it('should not contain two-word names that are exact reversals of each other', () => {
     expect(csvTestData.lineCount).toBeGreaterThan(1);
 
     const conflicts = findTwoWordReversedPairs(csvTestData.items);