diff options
Diffstat (limited to 'spec/features/projects/ci/editor_spec.rb')
-rw-r--r-- | spec/features/projects/ci/editor_spec.rb | 34 |
1 files changed, 27 insertions, 7 deletions
diff --git a/spec/features/projects/ci/editor_spec.rb b/spec/features/projects/ci/editor_spec.rb index ad4381a526a..2f960c09936 100644 --- a/spec/features/projects/ci/editor_spec.rb +++ b/spec/features/projects/ci/editor_spec.rb @@ -12,6 +12,8 @@ RSpec.describe 'Pipeline Editor', :js do let(:other_branch) { 'test' } before do + stub_feature_flags(pipeline_editor_file_tree: false) + sign_in(user) project.add_developer(user) @@ -22,11 +24,7 @@ RSpec.describe 'Pipeline Editor', :js do wait_for_requests end - it 'user sees the Pipeline Editor page' do - expect(page).to have_content('Pipeline Editor') - end - - describe 'Branch switcher' do + shared_examples 'default branch switcher behavior' do def switch_to_branch(branch) find('[data-testid="branch-selector"]').click @@ -53,7 +51,7 @@ RSpec.describe 'Pipeline Editor', :js do end it 'displays new branch as selected after commiting on a new branch' do - find('#target-branch-field').set('new_branch', clear: :backspace) + find('#source-branch-field').set('new_branch', clear: :backspace) page.within('#source-editor-') do find('textarea').send_keys '123' @@ -68,6 +66,28 @@ RSpec.describe 'Pipeline Editor', :js do end end + it 'user sees the Pipeline Editor page' do + expect(page).to have_content('Pipeline Editor') + end + + describe 'Branch Switcher (pipeline_editor_file_tree disabled)' do + it_behaves_like 'default branch switcher behavior' + end + + describe 'Branch Switcher (pipeline_editor_file_tree enabled)' do + before do + stub_feature_flags(pipeline_editor_file_tree: true) + + visit project_ci_pipeline_editor_path(project) + wait_for_requests + + # close button for the popover + find('[data-testid="close-button"]').click + end + + it_behaves_like 'default branch switcher behavior' + end + describe 'Editor navigation' do context 'when no change is made' do it 'user can navigate away without a browser alert' do @@ -112,7 +132,7 @@ RSpec.describe 'Pipeline Editor', :js do it 'user who creates a MR is taken to the merge request page without warnings' do expect(page).not_to have_content('New merge request') - find_field('Target Branch').set 'new_branch' + find_field('Branch').set 'new_branch' find_field('Start a new merge request with these changes').click click_button 'Commit changes' |