Move migration to post_migrate; fix shortcuts_spec

db/migrate/20170213172852_remove_theme_id_from_users.rb → db/post_migrate/20170213172852_remove_theme_id_from_users.rb

@@ -5,7 +5,7 @@ class RemoveThemeIdFromUsers < ActiveRecord::Migration
   include Gitlab::Database::MigrationHelpers
 
   # Set this constant to true if this migration requires downtime.
-  DOWNTIME = true
+  DOWNTIME = false
 
   # When a migration requires downtime you **must** uncomment the following
   # constant and define a short and easy to understand explanation as to why the

spec/features/dashboard/shortcuts_spec.rb

@@ -24,6 +24,7 @@ feature 'Dashboard shortcuts', feature: true, js: true do
   end
 
   def ensure_active_main_tab(content)
-    expect(find('.nav-sidebar li.active')).to have_content(content)
+    find('.global-dropdown-toggle').trigger('click')
+    expect(find('.global-dropdown-menu li.active')).to have_content(content)
   end
 end