add specificity.max (#156)

Closes #114

Author: bartveneman

src/analyzer/selectors/specificity.js

@@ -1,26 +1,42 @@
-const specificity = require('specificity')
+const {compare, calculate} = require('specificity')
+const uniquer = require('../../utils/uniquer')
 
 module.exports = selectors => {
-  const all = [...selectors]
-    .sort()
-    .reverse()
-    .sort(specificity.compare)
-    .reverse()
+  if (selectors.length === 0) {
+    return {
+      max: {
+        count: 0,
+        selectors: [],
+        value: null
+      },
+      top: []
+    }
+  }
 
-  const top = count => {
-    return [...all].slice(0, count).map(selector => {
-      const [a, b, c, d] = specificity
-        .calculate(selector)
-        .shift().specificityArray
+  const all = uniquer(selectors)
+    .unique.map(({value, count}) => {
+      const [a, b, c, d] = calculate(value).shift().specificityArray
 
       return {
-        value: selector,
+        count,
+        value,
         specificity: {a, b, c, d}
       }
     })
-  }
+    .sort((a, b) => compare(b.value, a.value))
+
+  const [maxSpecificitySelector] = all
+  const maxSpecificity = maxSpecificitySelector.specificity
+  const maxSpecificitySelectors = all.filter(
+    selector => compare(selector.value, maxSpecificitySelector.value) === 0
+  )
 
   return {
-    top: top(5)
+    top: [...all].slice(0, 5),
+    max: {
+      value: maxSpecificity,
+      count: maxSpecificitySelectors.length,
+      selectors: maxSpecificitySelectors
+    }
   }
 }

test/analyzer/index.js

@@ -91,8 +91,16 @@ test('it returns the correct analysis object structure', async t => {
     'selectors.js.total': 0,
     'selectors.js.totalUnique': 0,
     'selectors.js.unique': [],
+    'selectors.specificity.max.count': 1,
+    'selectors.specificity.max.selectors': [
+      {value: 'foo', specificity: {a: 0, b: 0, c: 0, d: 1}, count: 1}
+    ],
+    'selectors.specificity.max.value.a': 0,
+    'selectors.specificity.max.value.b': 0,
+    'selectors.specificity.max.value.c': 0,
+    'selectors.specificity.max.value.d': 1,
     'selectors.specificity.top': [
-      {value: 'foo', specificity: {a: 0, b: 0, c: 0, d: 1}}
+      {value: 'foo', specificity: {a: 0, b: 0, c: 0, d: 1}, count: 1}
     ],
     'selectors.total': 1,
     'selectors.totalUnique': 1,

test/analyzer/selectors/specificity.js

@@ -6,10 +6,37 @@ test('it responds with the correct structure', t => {
   const actual = analyze(fixture)
 
   t.deepEqual(actual, {
-    top: []
+    top: [],
+    max: {
+      value: null,
+      count: 0,
+      selectors: []
+    }
   })
 })
 
+test('it finds the max specificity selectors', t => {
+  const fixture = [
+    'a',
+    'a b',
+    'a b c',
+    '#a .b c', // <- Max.
+    'a .b #c', // <- Max.
+    'a .b #c' // <- Max. (dupe)
+  ]
+  const {max: actual} = analyze(fixture)
+  const expected = {
+    count: 2,
+    value: {a: 0, b: 1, c: 1, d: 1},
+    selectors: [
+      {count: 1, value: '#a .b c', specificity: {a: 0, b: 1, c: 1, d: 1}},
+      {count: 2, value: 'a .b #c', specificity: {a: 0, b: 1, c: 1, d: 1}}
+    ]
+  }
+
+  t.deepEqual(actual, expected)
+})
+
 test('it finds the top 5 specificity selectors and sorts them by specificity', t => {
   const fixture = [
     'a10',
@@ -24,24 +51,29 @@ test('it finds the top 5 specificity selectors and sorts them by specificity', t
     {
       value:
         '.Foo > .Bar ~ .Baz [type="text"] + span:before #bazz #fizz #buzz #brick #house',
-      specificity: {a: 0, b: 5, c: 4, d: 2}
+      specificity: {a: 0, b: 5, c: 4, d: 2},
+      count: 1
     },
     {
       value:
         '.a .b .c .d .e .f .g .h .i .j .k .l .m .n .o .p .q .r .s .t .u .v .w .x .y .z',
-      specificity: {a: 0, b: 0, c: 26, d: 0}
+      specificity: {a: 0, b: 0, c: 26, d: 0},
+      count: 1
     },
     {
       value: 'a',
-      specificity: {a: 0, b: 0, c: 0, d: 1}
+      specificity: {a: 0, b: 0, c: 0, d: 1},
+      count: 1
     },
     {
       value: 'a1',
-      specificity: {a: 0, b: 0, c: 0, d: 1}
+      specificity: {a: 0, b: 0, c: 0, d: 1},
+      count: 1
     },
     {
       value: 'a10',
-      specificity: {a: 0, b: 0, c: 0, d: 1}
+      specificity: {a: 0, b: 0, c: 0, d: 1},
+      count: 1
     }
   ]