diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2020-08-17 09:10:12 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2020-08-17 09:10:12 +0300 |
commit | c64b892786865a3b87701450c6ca62fcab0fa044 (patch) | |
tree | 81193337273f6360b9d0814064454248cf0a7b6a /spec | |
parent | d17008f99ed08e7c231b7e8e65cba1be51e2e747 (diff) |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'spec')
5 files changed, 5 insertions, 28 deletions
diff --git a/spec/controllers/admin/application_settings_controller_spec.rb b/spec/controllers/admin/application_settings_controller_spec.rb index 6ae9dc38016..4f223811be8 100644 --- a/spec/controllers/admin/application_settings_controller_spec.rb +++ b/spec/controllers/admin/application_settings_controller_spec.rb @@ -142,23 +142,12 @@ RSpec.describe Admin::ApplicationSettingsController do end describe 'verify panel actions' do - before do - stub_feature_flags(instance_level_integrations: false) - end - Admin::ApplicationSettingsController::VALID_SETTING_PANELS.each do |valid_action| it_behaves_like 'renders correct panels' do let(:action) { valid_action } end end end - end - - describe 'PATCH #integrations' do - before do - stub_feature_flags(instance_level_integrations: false) - sign_in(admin) - end describe 'EKS integration' do let(:application_setting) { ApplicationSetting.current } @@ -172,7 +161,7 @@ RSpec.describe Admin::ApplicationSettingsController do end it 'updates EKS settings' do - patch :integrations, params: { application_setting: settings_params } + put :update, params: { application_setting: settings_params } expect(application_setting.eks_integration_enabled).to be_truthy expect(application_setting.eks_account_id).to eq '123456789012' @@ -186,7 +175,7 @@ RSpec.describe Admin::ApplicationSettingsController do it 'does not update the secret key' do application_setting.update!(eks_secret_access_key: 'dummy secret key') - patch :integrations, params: { application_setting: settings_params } + put :update, params: { application_setting: settings_params } expect(application_setting.reload.eks_secret_access_key).to eq 'dummy secret key' end diff --git a/spec/controllers/admin/integrations_controller_spec.rb b/spec/controllers/admin/integrations_controller_spec.rb index 7e7b60db2dc..4a5d5ede728 100644 --- a/spec/controllers/admin/integrations_controller_spec.rb +++ b/spec/controllers/admin/integrations_controller_spec.rb @@ -10,16 +10,6 @@ RSpec.describe Admin::IntegrationsController do end describe '#edit' do - context 'when instance_level_integrations not enabled' do - it 'returns not_found' do - stub_feature_flags(instance_level_integrations: false) - - get :edit, params: { id: Service.available_services_names.sample } - - expect(response).to have_gitlab_http_status(:not_found) - end - end - Service.available_services_names.each do |integration_name| context "#{integration_name}" do it 'successfully displays the template' do diff --git a/spec/features/admin/admin_settings_spec.rb b/spec/features/admin/admin_settings_spec.rb index 55f6a9930ff..f5b05c76e90 100644 --- a/spec/features/admin/admin_settings_spec.rb +++ b/spec/features/admin/admin_settings_spec.rb @@ -209,8 +209,7 @@ RSpec.describe 'Admin updates settings', :clean_gitlab_redis_shared_state, :do_n context 'Integrations page' do before do - stub_feature_flags(instance_level_integrations: false) - visit integrations_admin_application_settings_path + visit general_admin_application_settings_path end it 'Enable hiding third party offers' do diff --git a/spec/features/projects/clusters/gcp_spec.rb b/spec/features/projects/clusters/gcp_spec.rb index 2e6a366f77a..63e5546b43c 100644 --- a/spec/features/projects/clusters/gcp_spec.rb +++ b/spec/features/projects/clusters/gcp_spec.rb @@ -205,11 +205,10 @@ RSpec.describe 'Gcp Cluster', :js, :do_not_mock_admin_mode do let(:admin) { create(:admin) } before do - stub_feature_flags(instance_level_integrations: false) stub_env('IN_MEMORY_APPLICATION_SETTINGS', 'false') sign_in(admin) gitlab_enable_admin_mode_sign_in(admin) - visit integrations_admin_application_settings_path + visit general_admin_application_settings_path end it 'user does not see the offer' do diff --git a/spec/support/shared_examples/requests/api/composer_packages_shared_examples.rb b/spec/support/shared_examples/requests/api/composer_packages_shared_examples.rb index 6ef4e852428..09743c20fba 100644 --- a/spec/support/shared_examples/requests/api/composer_packages_shared_examples.rb +++ b/spec/support/shared_examples/requests/api/composer_packages_shared_examples.rb @@ -72,7 +72,7 @@ RSpec.shared_examples 'Composer package creation' do |user_type, status, add_mem expect(response).to have_gitlab_http_status(status) end - it_behaves_like 'a gitlab tracking event', described_class.name, 'register_package' + it_behaves_like 'a gitlab tracking event', described_class.name, 'push_package' end end |