Commit be34638c authored by Miguel Rincon's avatar Miguel Rincon Committed by Stan Hu

Remove runner list in group settings

This change expands on the implementation of the group runners list by
removing the old list implementation.

When `runner_list_group_view_vue_ui` is disabled, the old runner list
is still available, but it pending for full removal.

Changelog: removed
parent fe754f3e
...@@ -5,52 +5,73 @@ require 'spec_helper' ...@@ -5,52 +5,73 @@ require 'spec_helper'
RSpec.describe 'Group CI/CD settings' do RSpec.describe 'Group CI/CD settings' do
include WaitForRequests include WaitForRequests
let(:user) { create(:user) } let_it_be(:user) { create(:user) }
let(:group) { create(:group) } let_it_be(:group, reload: true) { create(:group) }
before do before_all do
group.add_owner(user) group.add_owner(user)
end
before do
sign_in(user) sign_in(user)
end end
describe 'new group runners view banner' do describe 'Runners section' do
it 'displays banner' do let(:shared_runners_toggle) { page.find('[data-testid="enable-runners-toggle"]') }
visit group_settings_ci_cd_path(group)
context 'with runner_list_group_view_vue_ui enabled' do
before do
visit group_settings_ci_cd_path(group)
end
it 'displays the new group runners view banner' do
expect(page).to have_content(s_('Runners|New group runners view'))
expect(page).to have_link(href: group_runners_path(group))
end
expect(page).to have_content(s_('Runners|New group runners view')) it 'has "Enable shared runners for this group" toggle', :js do
expect(page).to have_link(href: group_runners_path(group)) expect(shared_runners_toggle).to have_content(_('Enable shared runners for this group'))
end
end end
it 'does not display banner' do context 'with runner_list_group_view_vue_ui disabled' do
stub_feature_flags(runner_list_group_view_vue_ui: false) before do
stub_feature_flags(runner_list_group_view_vue_ui: false)
visit group_settings_ci_cd_path(group) visit group_settings_ci_cd_path(group)
end
expect(page).not_to have_content(s_('Runners|New group runners view')) it 'does not display the new group runners view banner' do
expect(page).not_to have_link(href: group_runners_path(group)) expect(page).not_to have_content(s_('Runners|New group runners view'))
end expect(page).not_to have_link(href: group_runners_path(group))
end end
describe 'runners registration token' do it 'has "Enable shared runners for this group" toggle', :js do
let!(:token) { group.runners_token } expect(shared_runners_toggle).to have_content(_('Enable shared runners for this group'))
end
before do context 'with runners registration token' do
visit group_settings_ci_cd_path(group) let!(:token) { group.runners_token }
end
it 'has a registration token' do before do
expect(page.find('#registration_token')).to have_content(token) visit group_settings_ci_cd_path(group)
end end
describe 'reload registration token' do it 'displays the registration token' do
let(:page_token) { find('#registration_token').text } expect(page.find('#registration_token')).to have_content(token)
end
before do describe 'reload registration token' do
click_button 'Reset registration token' let(:page_token) { find('#registration_token').text }
end
before do
click_button 'Reset registration token'
end
it 'changes registration token' do it 'changes the registration token' do
expect(page_token).not_to eq token expect(page_token).not_to eq token
end
end
end end
end end
end end
......
...@@ -352,6 +352,7 @@ RSpec.describe 'Runners' do ...@@ -352,6 +352,7 @@ RSpec.describe 'Runners' do
before do before do
group.add_owner(user) group.add_owner(user)
stub_feature_flags(runner_list_group_view_vue_ui: false)
end end
context 'group with no runners' do context 'group with no runners' do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment