Updated tests

app/helpers/button_helper.rb

@@ -22,6 +22,14 @@ module ButtonHelper
       type: :button
   end
 
+  def clipboard_button_with_class(data = {}, css_class: 'btn-clipboard')
+    content_tag :button,
+      icon('clipboard'),
+      class: "btn #{css_class}",
+      data: data,
+      type: :button
+  end
+
   def http_clone_button(project)
     klass = 'http-selector'
     klass << ' has-tooltip' if current_user.try(:require_password?)

app/helpers/commits_helper.rb

@@ -200,7 +200,6 @@ module CommitsHelper
     source_email = clean(commit.send "#{options[:source]}_email".to_sym)
 
     person_name = user.try(:name) || source_name
-    person_email = user.try(:email) || source_email
 
     text =
       if options[:avatar]

app/views/projects/commits/_commit.html.haml

@@ -19,7 +19,7 @@
       .commit-actions
         - if commit.status
           = render_commit_status(commit, cssclass: 'btn btn-transparent')
-        = clipboard_button({ clipboard_text: commit.id }, css_class: 'btn-transparent')
+        = clipboard_button_with_class({ clipboard_text: commit.id }, css_class: 'btn-transparent')
         = link_to commit.short_id, namespace_project_commit_path(project.namespace, project, commit), class: "commit-short-id btn btn-transparent"
         = link_to_browse_code(project, commit)