diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2024-01-03 21:12:53 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2024-01-03 21:12:53 +0300 |
commit | b87af16bf2b9b09309deb902889edc1bff05256a (patch) | |
tree | c1088a650a2c26fb9636ad2d01744697caf80091 /spec | |
parent | 5429e3d4e0d5f1601ff8989bcb26bd822b3c7c5a (diff) |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'spec')
4 files changed, 9 insertions, 8 deletions
diff --git a/spec/features/projects/members/manage_members_spec.rb b/spec/features/projects/members/manage_members_spec.rb index 8bd97566acf..e6b6241c2fe 100644 --- a/spec/features/projects/members/manage_members_spec.rb +++ b/spec/features/projects/members/manage_members_spec.rb @@ -93,7 +93,8 @@ RSpec.describe 'Projects > Members > Manage members', :js, feature_category: :on end end - context 'uses ProjectMember valid_access_level_roles for the invite members modal options', :aggregate_failures do + context 'uses ProjectMember valid_access_level_roles for the invite members modal options', :aggregate_failures, + quarantine: 'https://gitlab.com/gitlab-org/gitlab/-/issues/436958' do before do sign_in(current_user) @@ -118,7 +119,7 @@ RSpec.describe 'Projects > Members > Manage members', :js, feature_category: :on context 'when maintainer' do let(:current_user) { project_maintainer } - it 'does not show the Owner option', quarantine: 'https://gitlab.com/gitlab-org/gitlab/-/issues/436958' do + it 'does not show the Owner option' do within_modal do toggle_listbox expect_listbox_items(%w[Guest Reporter Developer Maintainer]) diff --git a/spec/helpers/webpack_helper_spec.rb b/spec/helpers/webpack_helper_spec.rb index 8cbc4db9108..675fd0c0f04 100644 --- a/spec/helpers/webpack_helper_spec.rb +++ b/spec/helpers/webpack_helper_spec.rb @@ -6,6 +6,10 @@ RSpec.describe WebpackHelper do let(:source) { 'foo.js' } let(:asset_path) { "/assets/webpack/#{source}" } + before do + allow(helper).to receive(:vite_enabled?).and_return(false) + end + describe '#prefetch_link_tag' do it 'returns prefetch link tag' do expect(helper.prefetch_link_tag(source)).to eq("<link rel=\"prefetch\" href=\"/#{source}\">") @@ -40,7 +44,6 @@ RSpec.describe WebpackHelper do before do stub_rails_env('development') - stub_feature_flags(vite: true) allow(helper).to receive(:vite_javascript_tag).and_return('vite') allow(helper).to receive(:vite_enabled?).and_return(true) diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb index 816dec08b53..d5e301cb986 100644 --- a/spec/spec_helper.rb +++ b/spec/spec_helper.rb @@ -320,9 +320,6 @@ RSpec.configure do |config| # Postgres is the primary data source, and ClickHouse only when enabled in certain cases. stub_feature_flags(clickhouse_data_collection: false) - - # This is going to be removed with https://gitlab.com/gitlab-org/gitlab/-/issues/431041 - stub_feature_flags(vite: false) else unstub_all_feature_flags end diff --git a/spec/support/shared_examples/controllers/base_action_controller_shared_examples.rb b/spec/support/shared_examples/controllers/base_action_controller_shared_examples.rb index 2eab533ef7f..97748fe9e89 100644 --- a/spec/support/shared_examples/controllers/base_action_controller_shared_examples.rb +++ b/spec/support/shared_examples/controllers/base_action_controller_shared_examples.rb @@ -53,7 +53,7 @@ RSpec.shared_examples 'Base action controller' do skip: 'https://gitlab.com/gitlab-org/gitlab/-/issues/424334' do before do stub_rails_env('development') - stub_feature_flags(vite: true) + allow(ViteHelper).to receive(:vite_enabled?).and_return(true) end it 'adds vite csp' do @@ -65,7 +65,7 @@ RSpec.shared_examples 'Base action controller' do context 'when vite disabled' do before do - stub_feature_flags(vite: false) + allow(ViteHelper).to receive(:vite_enabled?).and_return(false) end it "doesn't add vite csp" do |