diff options
Diffstat (limited to 'spec/features/security/project/internal_access_spec.rb')
-rw-r--r-- | spec/features/security/project/internal_access_spec.rb | 27 |
1 files changed, 15 insertions, 12 deletions
diff --git a/spec/features/security/project/internal_access_spec.rb b/spec/features/security/project/internal_access_spec.rb index 9c1c81918fa..5c74b566ef0 100644 --- a/spec/features/security/project/internal_access_spec.rb +++ b/spec/features/security/project/internal_access_spec.rb @@ -7,10 +7,6 @@ describe "Internal Project Access" do set(:project) { create(:project, :internal, :repository) } - before do - stub_feature_flags(job_log_json: false) - end - describe "Project should be internal" do describe '#internal?' do subject { project.internal? } @@ -89,7 +85,7 @@ describe "Internal Project Access" do it { is_expected.to be_denied_for(:visitor) } end - describe "GET /:project_path/settings/members" do + describe "GET /:project_path/-/settings/members" do subject { project_settings_members_path(project) } it { is_expected.to be_allowed_for(:admin) } @@ -103,7 +99,7 @@ describe "Internal Project Access" do it { is_expected.to be_denied_for(:external) } end - describe "GET /:project_path/settings/ci_cd" do + describe "GET /:project_path/-/settings/ci_cd" do subject { project_settings_ci_cd_path(project) } it { is_expected.to be_allowed_for(:admin) } @@ -117,7 +113,7 @@ describe "Internal Project Access" do it { is_expected.to be_denied_for(:external) } end - describe "GET /:project_path/settings/repository" do + describe "GET /:project_path/-/settings/repository" do subject { project_settings_repository_path(project) } it { is_expected.to be_allowed_for(:admin) } @@ -133,6 +129,7 @@ describe "Internal Project Access" do describe "GET /:project_path/blob" do let(:commit) { project.repository.commit } + subject { project_blob_path(project, File.join(commit.id, '.gitignore')) } it { is_expected.to be_allowed_for(:admin) } @@ -190,6 +187,7 @@ describe "Internal Project Access" do describe "GET /:project_path/issues/:id/edit" do let(:issue) { create(:issue, project: project) } + subject { edit_project_issue_path(project, issue) } it { is_expected.to be_allowed_for(:admin) } @@ -301,7 +299,7 @@ describe "Internal Project Access" do it { is_expected.to be_denied_for(:visitor) } end - describe "GET /:project_path/settings/integrations" do + describe "GET /:project_path/-/settings/integrations" do subject { project_settings_integrations_path(project) } it { is_expected.to be_allowed_for(:admin) } @@ -331,6 +329,7 @@ describe "Internal Project Access" do describe "GET /:project_path/pipelines/:id" do let(:pipeline) { create(:ci_pipeline, project: project) } + subject { project_pipeline_path(project, pipeline) } it { is_expected.to be_allowed_for(:admin) } @@ -383,6 +382,7 @@ describe "Internal Project Access" do describe "GET /:project_path/builds/:id" do let(:pipeline) { create(:ci_pipeline, project: project) } let(:build) { create(:ci_build, pipeline: pipeline) } + subject { project_job_path(project, build.id) } context "when allowed for public and internal" do @@ -421,6 +421,7 @@ describe "Internal Project Access" do describe 'GET /:project_path/builds/:id/trace' do let(:pipeline) { create(:ci_pipeline, project: project) } let(:build) { create(:ci_build, pipeline: pipeline) } + subject { trace_project_job_path(project, build.id) } context 'when allowed for public and internal' do @@ -470,7 +471,7 @@ describe "Internal Project Access" do it { is_expected.to be_denied_for(:visitor) } end - describe "GET /:project_path/environments" do + describe "GET /:project_path/-/environments" do subject { project_environments_path(project) } it { is_expected.to be_allowed_for(:admin) } @@ -484,8 +485,9 @@ describe "Internal Project Access" do it { is_expected.to be_denied_for(:visitor) } end - describe "GET /:project_path/environments/:id" do + describe "GET /:project_path/-/environments/:id" do let(:environment) { create(:environment, project: project) } + subject { project_environment_path(project, environment) } it { is_expected.to be_allowed_for(:admin) } @@ -499,8 +501,9 @@ describe "Internal Project Access" do it { is_expected.to be_denied_for(:visitor) } end - describe "GET /:project_path/environments/:id/deployments" do + describe "GET /:project_path/-/environments/:id/deployments" do let(:environment) { create(:environment, project: project) } + subject { project_environment_deployments_path(project, environment) } it { is_expected.to be_allowed_for(:admin) } @@ -514,7 +517,7 @@ describe "Internal Project Access" do it { is_expected.to be_denied_for(:visitor) } end - describe "GET /:project_path/environments/new" do + describe "GET /:project_path/-/environments/new" do subject { new_project_environment_path(project) } it { is_expected.to be_allowed_for(:admin) } |