Improve some tests

Author: Bart Veneman

src/aggregate-collection.js

@@ -76,6 +76,7 @@ class AggregateCollection {
       }
     }
 
+    // TODO: can we avoid this sort()? It's slow
     /** @type Number[] */
     const sorted = this._items.slice().sort((a, b) => a - b)
     const min = sorted[0]

src/values/colors.test.js

@@ -662,6 +662,7 @@ Colors('does not report false positives for color keywords', () => {
 
       /* Not the black color keyword */
       font-family: Arial Black, Arial Bold, Gadget, sans-serif;
+      font: 1em/1 Black;
     }
   `).values.colors
   const expected = {

src/values/vendor-prefix.test.js

@@ -11,21 +11,28 @@ VendorPrefix('finds simple prefixes', () => {
       width: -webkit-max-content;
       box-shadow: 0 0 0 3px -moz-mac-focusring;
       position: -webkit-sticky;
+      -webkit-transition: -webkit-transform 0.3s ease-out;
+      -moz-transition: -moz-transform 0.3s ease-out;
+      -o-transition: -o-transform 0.3s ease-out;
     }
 
-    false-positive {
+    not-a-prefix {
       background: var(--test);
+      margin: -0.3em;
     }
   `
   const actual = analyze(fixture).values.prefixes
   const expected = {
-    total: 4,
-    totalUnique: 4,
+    total: 7,
+    totalUnique: 7,
     unique: {
       '-moz-max-content': 1,
       '-webkit-max-content': 1,
       '0 0 0 3px -moz-mac-focusring': 1,
       '-webkit-sticky': 1,
+      '-webkit-transform 0.3s ease-out': 1,
+      '-moz-transform 0.3s ease-out': 1,
+      '-o-transform 0.3s ease-out': 1,
     },
     uniquenessRatio: 1
   }
@@ -61,4 +68,23 @@ VendorPrefix('finds nested prefixes', () => {
   assert.equal(actual, expected)
 })
 
+VendorPrefix.skip('finds DEEPLY nested prefixes', () => {
+  const fixture = `
+    value-vendor-prefix-deeply-nested {
+      width: var(--test, -webkit-max-content);
+    }
+  `
+  const actual = analyze(fixture).values.prefixes
+  const expected = {
+    total: 1,
+    totalUnique: 1,
+    unique: {
+      'var(--test, -webkit-max-content)': 1,
+    },
+    uniquenessRatio: 1
+  }
+
+  assert.equal(actual, expected)
+})
+
 VendorPrefix.run()