Validate username uniq in scope of namespace

Author: dzaporozhets

app/models/user.rb

@@ -67,6 +67,8 @@ class User < ActiveRecord::Base
                       message: "only letters, digits & '_' '-' '.' allowed. Letter should be first" }
 
 
+  validate :namespace_uniq, if: ->(user) { user.username_changed? }
+
   before_validation :generate_password, on: :create
   before_save :ensure_authentication_token
   alias_attribute :private_token, :authentication_token
@@ -135,6 +137,12 @@ def generate_password
     end
   end
 
+  def namespace_uniq
+    namespace_name = self.username
+    if Namespace.find_by_path(namespace_name)
+      self.errors.add :username, "already exist"
+    end
+  end
 
   # Namespaces user has access to
   def namespaces

app/views/profiles/account.html.haml

@@ -69,7 +69,7 @@
           %i.icon-ok
           Saved
         %span.update-failed.cred.hide
-          %i.icon-ok
+          %i.icon-remove
           Failed
         %ul.cred
           %li It will change web url for personal projects.

features/steps/project/project_milestones.rb

@@ -50,12 +50,12 @@ class ProjectMilestones < Spinach::FeatureSteps
   end
 
   Then "I should see 3 issues" do
-    page.should have_selector('.milestone-issue-filter li', count: 4)
-    page.should have_selector('.milestone-issue-filter li.hide', count: 1)
+    page.should have_selector('.milestone-issue-filter .well-list li', count: 4)
+    page.should have_selector('.milestone-issue-filter .well-list li.hide', count: 1)
   end
 
   Then "I should see 4 issues" do
-    page.should have_selector('.milestone-issue-filter li', count: 4)
-    page.should_not have_selector('.milestone-issue-filter li.hide')
+    page.should have_selector('.milestone-issue-filter .well-list li', count: 4)
+    page.should_not have_selector('.milestone-issue-filter .well-list li.hide')
   end
 end

lib/tasks/travis.rake

@@ -1,7 +1,5 @@
-task :travis do
-  ["rake spinach", "rake spec"].each do |cmd|
-    puts "Starting to run #{cmd}..."
-    system("export DISPLAY=:99.0 && bundle exec #{cmd}")
-    raise "#{cmd} failed!" unless $?.exitstatus == 0
-  end
-end
+desc "Travis run tests"
+task :travis => [
+  :spinach,
+  :spec
+]