diff options
author | Michael Kozono <mkozono@gmail.com> | 2017-08-09 19:24:49 +0300 |
---|---|---|
committer | Robert Speicher <robert@gitlab.com> | 2017-08-09 19:24:49 +0300 |
commit | 255be6c5ca805446ad29d8f45b3ef7ca9b11e23f (patch) | |
tree | 6409c560b9f429c59e86813c90e24b6452a92c9d /spec/models/user_spec.rb | |
parent | a0fa59f61e28837265f30cd35f265d9311df3f7a (diff) |
Prevent user from changing username with container registry tags
Diffstat (limited to 'spec/models/user_spec.rb')
-rw-r--r-- | spec/models/user_spec.rb | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index 0103fb6040e..6c8248eeb40 100644 --- a/spec/models/user_spec.rb +++ b/spec/models/user_spec.rb @@ -118,6 +118,17 @@ describe User do expect(user).to validate_uniqueness_of(:username).case_insensitive end + + context 'when username is changed' do + let(:user) { build_stubbed(:user, username: 'old_path', namespace: build_stubbed(:namespace)) } + + it 'validates move_dir is allowed for the namespace' do + expect(user.namespace).to receive(:any_project_has_container_registry_tags?).and_return(true) + user.username = 'new_path' + expect(user).to be_invalid + expect(user.errors.messages[:username].first).to match('cannot be changed if a personal project has container registry tags') + end + end end it { is_expected.to validate_presence_of(:projects_limit) } |