Add feature specs for default dashboard preference

cherry-picked

app/views/profiles/preferences/show.html.haml

@@ -60,7 +60,7 @@
       Behavior
     .panel-body
       .form-group
-        = f.label :dashboard, class: 'control-label'
+        = f.label :dashboard, 'Default Dashboard', class: 'control-label'
         .col-sm-10
           = f.select :dashboard, dashboard_choices, {}, class: 'form-control'
           %p.help-block.hint

app/views/profiles/preferences/update.js.erb

 // Remove body class for any previous theme, re-add current one
 $('body').removeClass('<%= Gitlab::Theme.body_classes %>')
 $('body').addClass('<%= app_theme %> <%= theme_type %>')
+
+// Re-enable the "Save" button
+$('input[type=submit]').enable()
+
+// Show the notice flash message
 new Flash('<%= flash.discard(:notice) %>', 'notice')

spec/features/profiles/preferences_spec.rb

@@ -5,22 +5,25 @@ describe 'Profile > Preferences' do
 
   before do
     login_as(user)
+    visit profile_preferences_path
   end
 
   describe 'User changes their application theme', js: true do
     let(:default_class) { Gitlab::Theme.css_class_by_id(nil) }
     let(:theme_5_class) { Gitlab::Theme.css_class_by_id(5) }
 
-    before do
-      visit profile_preferences_path
-    end
-
     it 'creates a flash message' do
       choose "user_theme_id_#{theme.id}"
 
-      within('.flash-container') do
-        expect(page).to have_content('Preferences saved.')
+      expect_preferences_saved_message
     end
+
+    it 'updates their preference' do
+      choose "user_theme_id_#{theme.id}"
+
+      visit page.current_path
+
+      expect(page).to have_checked_field("user_theme_id_#{theme.id}")
     end
 
     it 'reflects the changes immediately' do
@@ -33,9 +36,45 @@ describe 'Profile > Preferences' do
     end
   end
 
-  describe 'User changes their syntax highlighting theme' do
-    before do
-      visit profile_preferences_path
+  describe 'User changes their syntax highlighting theme', js: true do
+    it 'creates a flash message' do
+      choose 'user_color_scheme_id_5'
+
+      expect_preferences_saved_message
+    end
+
+    it 'updates their preference' do
+      choose 'user_color_scheme_id_5'
+
+      visit page.current_path
+
+      expect(page).to have_checked_field('user_color_scheme_id_5')
+    end
+  end
+
+  describe 'User changes their default dashboard' do
+    it 'creates a flash message' do
+      select 'Starred Projects', from: 'user_dashboard'
+      click_button 'Save'
+
+      expect_preferences_saved_message
+    end
+
+    it 'updates their preference' do
+      select 'Starred Projects', from: 'user_dashboard'
+      click_button 'Save'
+
+      click_link 'Dashboard'
+      expect(page.current_path).to eq starred_dashboard_projects_path
+
+      click_link 'Your Projects'
+      expect(page.current_path).to eq dashboard_path
+    end
+  end
+
+  def expect_preferences_saved_message
+    within('.flash-container') do
+      expect(page).to have_content('Preferences saved.')
     end
   end
 end