diff options
| author | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2013-09-26 14:52:17 +0300 |
|---|---|---|
| committer | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2013-09-26 14:52:17 +0300 |
| commit | 0a2a34a4d18d0489f7db6aad5b5518b503692b5b (patch) | |
| tree | f169695667def43200565907636fae145469385d /spec/models | |
| parent | 65b9768ccfbdc3de682d66430601cf3af1b2a2f8 (diff) | |
| download | gitlab-ce-0a2a34a4d18d0489f7db6aad5b5518b503692b5b.tar.gz | |
Ignore owner_id for Group in tests
Diffstat (limited to 'spec/models')
| -rw-r--r-- | spec/models/group_spec.rb | 6 | ||||
| -rw-r--r-- | spec/models/project_spec.rb | 1 | ||||
| -rw-r--r-- | spec/models/user_spec.rb | 10 |
3 files changed, 9 insertions, 8 deletions
diff --git a/spec/models/group_spec.rb b/spec/models/group_spec.rb index 78c39548eb4..ce1aa05bcd7 100644 --- a/spec/models/group_spec.rb +++ b/spec/models/group_spec.rb @@ -26,10 +26,10 @@ describe Group do it { should validate_uniqueness_of(:name) } it { should validate_presence_of :path } it { should validate_uniqueness_of(:path) } - it { should validate_presence_of :owner } + it { should_not validate_presence_of :owner } describe :users do - it { group.users.should == [group.owner] } + it { group.users.should == group.owners } end describe :human_name do @@ -38,7 +38,7 @@ describe Group do describe :add_users do let(:user) { create(:user) } - before { group.add_users([user.id], UsersGroup::MASTER) } + before { group.add_user(user, UsersGroup::MASTER) } it { group.users_groups.masters.map(&:user).should include(user) } end diff --git a/spec/models/project_spec.rb b/spec/models/project_spec.rb index c3d2bf5d4a6..47ae760a7ed 100644 --- a/spec/models/project_spec.rb +++ b/spec/models/project_spec.rb @@ -85,7 +85,6 @@ describe Project do it { should respond_to(:execute_hooks) } it { should respond_to(:transfer) } it { should respond_to(:name_with_namespace) } - it { should respond_to(:namespace_owner) } it { should respond_to(:owner) } it { should respond_to(:path_with_namespace) } end diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index c879900f8fd..f6c9f82c4ee 100644 --- a/spec/models/user_spec.rb +++ b/spec/models/user_spec.rb @@ -130,11 +130,12 @@ describe User do before do ActiveRecord::Base.observers.enable(:user_observer) @user = create :user - @group = create :group, owner: @user + @group = create :group + @group.add_owner(@user) end it { @user.several_namespaces?.should be_true } - it { @user.namespaces.should include(@user.namespace, @group) } + it { @user.namespaces.should include(@user.namespace) } it { @user.authorized_groups.should == [@group] } it { @user.owned_groups.should == [@group] } end @@ -144,9 +145,10 @@ describe User do ActiveRecord::Base.observers.enable(:user_observer) @user = create :user @user2 = create :user - @group = create :group, owner: @user + @group = create :group + @group.add_owner(@user) - @group.add_users([@user2.id], UsersGroup::OWNER) + @group.add_user(@user2, UsersGroup::OWNER) end it { @user2.several_namespaces?.should be_true } |
