Merge pull request #92 from codeclimate/abh-fingerprints

Fix fingerprint bug - don't use num occurrences

Author: Ashley Baldwin-Hunter

lib/cc/engine/analyzers/violation.rb

@@ -86,7 +86,7 @@ def fingerprint
           digest << "-"
           digest << current_sexp.mass.to_s
           digest << "-"
-          digest << occurrences.to_s
+          digest << check_name
           digest.to_s
         end
 

spec/cc/engine/analyzers/javascript/main_spec.rb

@@ -33,7 +33,7 @@
         {"path" => "foo.js", "lines" => { "begin" => 3, "end" => 3} }
       ])
       expect(json["content"]["body"]).to match /This issue has a mass of `11`/
-      expect(json["fingerprint"]).to eq("55ae5d0990647ef496e9e0d315f9727d")
+      expect(json["fingerprint"]).to eq("c4d29200c20d02297c6f550ad2c87c15")
     end
 
     it "prints an issue for similar code" do
@@ -61,7 +61,7 @@
         {"path" => "foo.js", "lines" => { "begin" => 3, "end" => 3} }
       ])
       expect(json["content"]["body"]).to match /This issue has a mass of `11`/
-      expect(json["fingerprint"]).to eq("55ae5d0990647ef496e9e0d315f9727d")
+      expect(json["fingerprint"]).to eq("d9dab8e4607e2a74da3b9eefb49eacec")
     end
 
     it "skips unparsable files" do

spec/cc/engine/analyzers/php/main_spec.rb

@@ -45,7 +45,7 @@
         {"path" => "foo.php", "lines" => { "begin" => 10, "end" => 14} },
       ])
       expect(json["content"]["body"]).to match /This issue has a mass of `11`/
-      expect(json["fingerprint"]).to eq("667da0e2bab866aa2fe9d014a65d57d9")
+      expect(json["fingerprint"]).to eq("8234e10d96fd6ef608085c22c91c9ab1")
     end
 
     it "runs against complex files" do

spec/cc/engine/analyzers/python/main_spec.rb

@@ -32,7 +32,7 @@
         {"path" => "foo.py", "lines" => { "begin" => 3, "end" => 3} }
       ])
       expect(json["content"]["body"]).to match /This issue has a mass of `6`/
-      expect(json["fingerprint"]).to eq("42b832387c997f54a2012efb2159aefc")
+      expect(json["fingerprint"]).to eq("3f3d34361bcaef98839d9da6ca9fcee4")
     end
 
     it "prints an issue for similar code" do
@@ -60,7 +60,7 @@
         {"path" => "foo.py", "lines" => { "begin" => 3, "end" => 3} }
       ])
       expect(json["content"]["body"]).to match /This issue has a mass of `6`/
-      expect(json["fingerprint"]).to eq("42b832387c997f54a2012efb2159aefc")
+      expect(json["fingerprint"]).to eq("019118ceed60bf40b35aad581aae1b02")
     end
 
 

spec/cc/engine/analyzers/ruby/main_spec.rb

@@ -44,7 +44,7 @@ module CC::Engine::Analyzers
           {"path" => "foo.rb", "lines" => { "begin" => 9, "end" => 13} },
         ])
         expect(json["content"]["body"]).to match /This issue has a mass of `18`/
-        expect(json["fingerprint"]).to eq("f21b75bbd135ec3ae6638364d5c73762")
+        expect(json["fingerprint"]).to eq("b7e46d8f5164922678e48942e26100f2")
       end
 
       it "creates an issue for each occurrence of the duplicated code" do

spec/cc/engine/analyzers/violations_spec.rb

@@ -43,7 +43,7 @@ module CC::Engine::Analyzers
           { :path => "file.rb", :lines => { :begin => 9, :end => 13} },
           { :path => "file.rb", :lines => { :begin => 17, :end => 21} },
         ])
-        expect(first_formatted[:fingerprint]).to eq("c2712b56bff2becf4ae2a8469e1171c7")
+        expect(first_formatted[:fingerprint]).to eq("f52d2f61a77c569513f8b6314a00d013")
 
         expect(second_formatted[:location]).to eq({:path=>"file.rb", :lines=>{:begin=>9, :end=>13}})
         expect(second_formatted[:other_locations]).to eq([