Tidy up user project specs

1 files changed, 8 insertions, 13 deletions

diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb
index 06d1ca3b7da..10e7e693571 100644
--- a/spec/models/user_spec.rb
+++ b/spec/models/user_spec.rb
@@ -233,8 +233,6 @@ describe User, models: true do
       @project = create :project, namespace: @user.namespace
       @project_2 = create :project, group: create(:group) # Grant MASTER access to the user
       @project_3 = create :project, group: create(:group) # Grant DEVELOPER access to the user
-      @project_4 = create :project, group: create(:group)
-      @project_5 = create :project, group: create(:group)
 
       @project_2.team << [@user, :master]
       @project_3.team << [@user, :developer]
@@ -787,23 +785,20 @@ describe User, models: true do
 
   describe '#viewable_starred_projects' do
     let(:user) { create(:user) }
-    let(:public_project) { create(:project, :public) }
-    let(:private_project) { create(:project, :private) }
-    let(:private_viewable_project) { create(:project, :private) }
-    let(:viewable?) { -> (project) { user.can?(:read_project, project) } }
-    let(:projects) { [public_project, private_project, private_viewable_project] }
+    let(:public_project) { create(:empty_project, :public) }
+    let(:private_project) { create(:empty_project, :private) }
+    let(:private_viewable_project) { create(:empty_project, :private) }
 
     before do
       private_viewable_project.team << [user, Gitlab::Access::MASTER]
-      projects.each { |project| user.toggle_star(project) }
-    end
 
-    it 'returns only starred projects the user can view' do
-      expect(user.viewable_starred_projects).to all(satisfy(&viewable?))
+      [public_project, private_project, private_viewable_project].each do |project|
+        user.toggle_star(project)
+      end
     end
 
-    it 'rejects only starred projects the user can not view' do
-      expect(projects - user.viewable_starred_projects).not_to include(satisfy(&viewable?))
+    it 'returns only starred projects the user can view' do
+      expect(user.viewable_starred_projects).not_to include(private_project)
     end
   end
 end