Fixes bug where the shorthand messes with minimum trimming

Author: zmoazeni

lib/csscss/redundancy_analyzer.rb

@@ -56,12 +56,6 @@ def redundancies(minimum = nil)
         end
       end
 
-      if minimum
-        inverted_matches.delete_if do |key, declarations|
-          declarations.size < minimum
-        end
-      end
-
       # combines selector keys by common declarations
       final_inverted_matches = inverted_matches.dup
       inverted_matches.to_a[0..-2].each_with_index do |(selector_group1, declarations1), index|
@@ -97,6 +91,11 @@ def redundancies(minimum = nil)
         end
       end
 
+      if minimum
+        final_inverted_matches.delete_if do |key, declarations|
+          declarations.size < minimum
+        end
+      end
 
       # sort hash by number of matches
       sorted_array = final_inverted_matches.sort {|(_, v1), (_, v2)| v2.size <=> v1.size }

test/csscss/redundancy_analyzer_test.rb

@@ -171,6 +171,10 @@ module Csscss
         [sel(".bar"), sel(".baz")] => [dec("border", "4px solid #4f4f4f")],
         [sel(".bar"), sel(".baz"), sel(".foo")] => [dec("border-style", "solid"), dec("border-width", "4px")]
       })
+
+      RedundancyAnalyzer.new(css).redundancies(2).must_equal({
+        [sel(".bar"), sel(".baz"), sel(".foo")] => [dec("border-style", "solid"), dec("border-width", "4px")]
+      })
     end
 
     it "reduces padding and margin" do