From 3d227e27222d93291fbb3a10621ed474a9f0e151 Mon Sep 17 00:00:00 2001 From: "Luke \"Jared\" Bennett" Date: Wed, 12 Apr 2017 11:53:25 +0100 Subject: [PATCH] Use empty_project for resources that are independent of the repo --- spec/features/discussion_comments_spec.rb | 25 ++++++++++++++++------- 1 file changed, 18 insertions(+), 7 deletions(-) diff --git a/spec/features/discussion_comments_spec.rb b/spec/features/discussion_comments_spec.rb index ae778118c5c..cc813873cf0 100644 --- a/spec/features/discussion_comments_spec.rb +++ b/spec/features/discussion_comments_spec.rb @@ -247,18 +247,14 @@ describe 'Discussion Comments', :feature, :js do include RepoHelpers let(:user) { create(:user) } - let(:project) { create(:project) } - - before do - project.team << [user, :developer] - - login_as(user) - end describe 'on a merge request' do + let(:project) { create(:project) } let(:merge_request) { create(:merge_request, source_project: project) } before do + login + visit namespace_project_merge_request_path(project.namespace, project, merge_request) end @@ -266,9 +262,12 @@ describe 'Discussion Comments', :feature, :js do end describe 'on an issue' do + let(:project) { create(:empty_project) } let(:issue) { create(:issue, project: project) } before do + login + visit namespace_project_issue_path(project.namespace, project, issue) end @@ -276,9 +275,12 @@ describe 'Discussion Comments', :feature, :js do end describe 'on an snippet' do + let(:project) { create(:empty_project) } let(:snippet) { create(:project_snippet, :private, project: project, author: user) } before do + login + visit namespace_project_snippet_path(project.namespace, project, snippet) end @@ -286,10 +288,19 @@ describe 'Discussion Comments', :feature, :js do end describe 'on a commit' do + let(:project) { create(:project) } + before do + login + visit namespace_project_commit_path(project.namespace, project, sample_commit.id) end it_behaves_like 'discussion comments', 'commit' end + + def login + project.add_master(user) + login_as(user) + end end