Merge branch 'fix/gb/notification-settings-when-no-repository' into 'master'

Show notifications settings even when repository feature is disabled

See merge request !9180

app/views/projects/_home_panel.html.haml

@@ -28,9 +28,11 @@
           .project-clone-holder
             = render "shared/clone_panel"
 
-        - if current_user && can?(current_user, :download_code, @project)
-          = render 'projects/buttons/download', project: @project, ref: @ref
-          = render 'projects/buttons/dropdown'
+        - if current_user
+          - if can?(current_user, :download_code, @project)
+            = render 'projects/buttons/download', project: @project, ref: @ref
+            = render 'projects/buttons/dropdown'
+            = render 'projects/buttons/koding'
+
           = render 'shared/notifications/button', notification_setting: @notification_setting
-          = render 'projects/buttons/koding'
           = render 'shared/members/access_request_buttons', source: @project

changelogs/unreleased/fix-gb-notification-settings-when-no-repository.yml

+---
+title: Show notifications settings dropdown even if repository feature is disabled
+merge_request: 9180
+author:

spec/views/projects/_home_panel.html.haml_spec.rb

+require 'spec_helper'
+
+describe 'projects/_home_panel', :view do
+  let(:project) { create(:empty_project, :public) }
+
+  let(:notification_settings) do
+    user.notification_settings_for(project) if user
+  end
+
+  before do
+    assign(:project, project)
+    assign(:notification_setting, notification_settings)
+
+    allow(view).to receive(:current_user).and_return(user)
+    allow(view).to receive(:can?).and_return(false)
+  end
+
+  context 'when user is signed in' do
+    let(:user) { create(:user) }
+
+    it 'makes it possible to set notification level' do
+      render
+
+      expect(view).to render_template('shared/notifications/_button')
+      expect(rendered).to have_selector('.notification-dropdown')
+    end
+  end
+
+  context 'when user is signed out' do
+    let(:user) { nil }
+
+    it 'is not possible to set notification level' do
+      render
+
+      expect(rendered).not_to have_selector('.notification_dropdown')
+    end
+  end
+end