Refactor star btn logic for non-logged in user and fix tests

2 files changed, 19 insertions, 14 deletions

diff --git a/app/helpers/projects_helper.rb b/app/helpers/projects_helper.rb
index 4629de2ecad..e3734023be3 100644
--- a/app/helpers/projects_helper.rb
+++ b/app/helpers/projects_helper.rb
@@ -80,17 +80,17 @@ module ProjectsHelper
     @project.milestones.active.order("due_date, title ASC")
   end
 
-  def link_to_toggle_star(title, starred, signed_in)
+  def link_to_toggle_star(title, starred)
     cls = 'star-btn btn btn-sm btn-default'
-    cls << ' disabled' unless signed_in
 
-    toggle_html = content_tag('span', class: 'toggle') do
-      toggle_text = if starred
-                      ' Unstar'
-                    else
-                      ' Star'
-                    end
+    toggle_text =
+      if starred
+        ' Unstar'
+      else
+        ' Star'
+      end
 
+    toggle_html = content_tag('span', class: 'toggle') do
       icon('star') + toggle_text
     end
 
@@ -106,12 +106,10 @@ module ProjectsHelper
       data: { type: 'json' }
     }
 
+    path = toggle_star_namespace_project_path(@project.namespace, @project)
 
     content_tag 'span', class: starred ? 'turn-on' : 'turn-off' do
-      link_to(
-        toggle_star_namespace_project_path(@project.namespace, @project),
-        link_opts
-      ) do
+      link_to(path, link_opts) do
         toggle_html + ' ' + count_html
       end
     end
diff --git a/app/views/projects/_home_panel.html.haml b/app/views/projects/_home_panel.html.haml
index 774be6cd138..5689bdee1c6 100644
--- a/app/views/projects/_home_panel.html.haml
+++ b/app/views/projects/_home_panel.html.haml
@@ -17,8 +17,15 @@
     .project-repo-buttons
       .inline.star.js-toggler-container{class: @show_star ? 'on' : ''}
         - if current_user
-          = link_to_toggle_star('Star this project.', false, true)
-          = link_to_toggle_star('Unstar this project.', true, true)
+          = link_to_toggle_star('Star this project.', false)
+          = link_to_toggle_star('Unstar this project.', true)
+        - else
+          = link_to new_user_session_path, class: 'btn star-btn has_tooltip', title: 'You must sign in to star a project' do
+            %span
+              = icon('star')
+              Star
+            %span.count
+              = @project.star_count
       - unless @project.empty_repo?
         - if current_user && can?(current_user, :fork_project, @project) && @project.namespace != current_user.namespace
           .inline.fork-buttons.prepend-left-10