Fixes sorting of strings (#7)

* Fixes sorting of strings

* Destructure string compare fn

Author: bartveneman

package-lock.json

@@ -26,7 +26,8 @@
   "homepage": "https://github.com/bartveneman/css-analyzer-diff#readme",
   "dependencies": {
     "color-sorter": "^2.2.1",
-    "css-unit-sort": "^1.1.1"
+    "css-unit-sort": "^1.1.1",
+    "string-natural-compare": "^2.0.3"
   },
   "devDependencies": {
     "ava": "^0.25.0",

package.json

@@ -1,5 +1,8 @@
 const cssUnitSort = require('css-unit-sort')
 const cssColorSort = require('color-sorter')
+const {
+	caseInsensitive: caseInsensitiveCompare
+} = require('string-natural-compare')
 
 const METRICS_SORTED_BY_CSS_UNIT = ['values.fontsizes.unique']
 const METRICS_SORTED_BY_COLOR = [
@@ -13,7 +16,7 @@ function getSortingFnByKey(key) {
 	}
 
 	return (a, b) => {
-		return b.toLowerCase().localeCompare(a.toLowerCase())
+		return caseInsensitiveCompare(b, a)
 	}
 }
 

src/diff-lists.js

@@ -123,6 +123,44 @@ test('It handles the second listable stat being absent in one of the stats', t =
 	t.deepEqual(actual, expectedDiff)
 })
 
+test('It sorts selectors/values/properties correctly', t => {
+	const actual = diff(
+		['a:after', 'a:before'],
+		['a:after', 'aa:before', 'b:before']
+	)
+	const expectedDiff = {
+		changed: true,
+		diff: [
+			{
+				value: 'a:after',
+				changed: false,
+				removed: false,
+				added: false
+			},
+			{
+				value: 'a:before',
+				changed: true,
+				removed: true,
+				added: false
+			},
+			{
+				value: 'aa:before',
+				added: true,
+				changed: true,
+				removed: false
+			},
+			{
+				value: 'b:before',
+				added: true,
+				changed: true,
+				removed: false
+			}
+		]
+	}
+
+	t.deepEqual(actual, expectedDiff)
+})
+
 test('It sorts fontsizes correctly after comparing them', t => {
 	const actual = diff(
 		['0', '32px', '4em'],