Remove :new_import_ui feature flag

Cleanup feature flag from controllers, templates and specs

app/controllers/import/bitbucket_controller.rb

@@ -22,20 +22,9 @@ class Import::BitbucketController < Import::BaseController
     redirect_to status_import_bitbucket_url
   end
 
-  # rubocop: disable CodeReuse/ActiveRecord
   def status
-    return super if Feature.enabled?(:new_import_ui)
-
-    bitbucket_client = Bitbucket::Client.new(credentials)
-    repos = bitbucket_client.repos(filter: sanitized_filter_param)
-    @repos, @incompatible_repos = repos.partition { |repo| repo.valid? }
-
-    @already_added_projects = find_already_added_projects('bitbucket')
-    already_added_projects_names = @already_added_projects.pluck(:import_source)
-
-    @repos.to_a.reject! { |repo| already_added_projects_names.include?(repo.full_name) }
+    super
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 
   def jobs
     render json: find_jobs('bitbucket')

app/controllers/import/bitbucket_server_controller.rb

@@ -52,20 +52,9 @@ class Import::BitbucketServerController < Import::BaseController
     redirect_to status_import_bitbucket_server_path
   end
 
-  # rubocop: disable CodeReuse/ActiveRecord
   def status
-    return super if Feature.enabled?(:new_import_ui)
-
-    @collection = client.repos(page_offset: page_offset, limit: limit_per_page, filter: sanitized_filter_param)
-    @repos, @incompatible_repos = @collection.partition { |repo| repo.valid? }
-
-    # Use the import URL to filter beyond what BaseService#find_already_added_projects
-    @already_added_projects = filter_added_projects('bitbucket_server', @repos.map(&:browse_url))
-    already_added_projects_names = @already_added_projects.pluck(:import_source)
-
-    @repos.reject! { |repo| already_added_projects_names.include?(repo.browse_url) }
+    super
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 
   def jobs
     render json: find_jobs('bitbucket_server')

app/controllers/import/fogbugz_controller.rb

@@ -50,14 +50,7 @@ class Import::FogbugzController < Import::BaseController
       return redirect_to new_import_fogbugz_path
     end
 
-    return super if Feature.enabled?(:new_import_ui)
-
-    @repos = client.repos
-
-    @already_added_projects = find_already_added_projects('fogbugz')
-    already_added_projects_names = @already_added_projects.pluck(:import_source)
-
-    @repos.reject! { |repo| already_added_projects_names.include? repo.name }
+    super
   end
   # rubocop: enable CodeReuse/ActiveRecord
 

app/controllers/import/gitlab_controller.rb

@@ -16,18 +16,9 @@ class Import::GitlabController < Import::BaseController
     redirect_to status_import_gitlab_url
   end
 
-  # rubocop: disable CodeReuse/ActiveRecord
   def status
-    return super if Feature.enabled?(:new_import_ui)
-
-    @repos = client.projects(starting_page: 1, page_limit: MAX_PROJECT_PAGES, per_page: PER_PAGE_PROJECTS)
-
-    @already_added_projects = find_already_added_projects('gitlab')
-    already_added_projects_names = @already_added_projects.pluck(:import_source)
-
-    @repos = @repos.to_a.reject { |repo| already_added_projects_names.include? repo["path_with_namespace"] }
+    super
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 
   def jobs
     render json: find_jobs('gitlab')

app/views/import/bitbucket/status.html.haml

@@ -5,93 +5,4 @@
   %i.fa.fa-bitbucket
   = _('Import projects from Bitbucket')
 
-- if Feature.enabled?(:new_import_ui)
-  = render 'import/githubish_status', provider: 'bitbucket'
-- else
-  - if @repos.any?
-    %p.light
-      = _('Select projects you want to import.')
-    %p
-    - if @incompatible_repos.any?
-      = button_tag class: 'btn btn-import btn-success js-import-all' do
-        = _('Import all compatible projects')
-        = icon('spinner spin', class: 'loading-icon')
-    - else
-      = button_tag class: 'btn btn-import btn-success js-import-all' do
-        = _('Import all projects')
-        = icon('spinner spin', class: 'loading-icon')
-
-  .position-relative.ms-no-clear.d-flex.flex-fill.float-right.append-bottom-10
-    = form_tag status_import_bitbucket_path, method: 'get' do
-      = text_field_tag :filter, @filter, class: 'form-control pr-5', placeholder: _('Filter projects'), size: 40, autofocus: true, 'aria-label': _('Search')
-      .position-absolute.position-top-0.d-flex.align-items-center.text-muted.position-right-0.h-100
-        .border-left
-          %button{ class: 'btn btn-transparent btn-secondary', 'aria-label': _('Search Button'), type: 'submit' }
-            %i{ class: 'fa fa-search', 'aria-hidden': true }
-
-  .table-responsive
-    %table.table.import-jobs
-      %colgroup.import-jobs-from-col
-      %colgroup.import-jobs-to-col
-      %colgroup.import-jobs-status-col
-      %thead
-        %tr
-          %th= _('From Bitbucket')
-          %th= _('To GitLab')
-          %th= _('Status')
-      %tbody
-        - @already_added_projects.each do |project|
-          %tr{ id: "project_#{project.id}", class: "#{project_status_css_class(project.import_status)}" }
-            %td
-              = link_to project.import_source, "https://bitbucket.org/#{project.import_source}", target: '_blank', rel: 'noopener noreferrer'
-            %td
-              = link_to project.full_path, [project.namespace.becomes(Namespace), project]
-            %td.job-status
-              - case project.import_status
-              - when 'finished'
-                %span
-                  %i.fa.fa-check
-                  = _('done')
-              - when 'started'
-                %i.fa.fa-spinner.fa-spin
-                = _('started')
-              - else
-                = project.human_import_status_name
-
-        - @repos.each do |repo|
-          %tr{ id: "repo_#{repo.owner}___#{repo.slug}" }
-            %td
-              = link_to repo.full_name, "https://bitbucket.org/#{repo.full_name}", target: '_blank', rel: 'noopener noreferrer'
-            %td.import-target
-              %fieldset.row
-              .input-group
-                .project-path.input-group-prepend
-                  - if current_user.can_select_namespace?
-                    - selected = params[:namespace_id] || :current_user
-                    - opts = current_user.can_create_group? ? { extra_group: Group.new(name: repo.owner, path: repo.owner) } : {}
-                    = select_tag :namespace_id, namespaces_options(selected, opts.merge({ display_path: true })), { class: 'select2 js-select-namespace', tabindex: 1 }
-                  - else
-                    = text_field_tag :path, current_user.namespace_path, class: "input-group-text input-large form-control", tabindex: 1, disabled: true
-                %span.input-group-prepend
-                  .input-group-text /
-                = text_field_tag :path, sanitize_project_name(repo.slug), class: "input-mini form-control", tabindex: 2, autofocus: true, required: true
-            %td.import-actions.job-status
-              = button_tag class: 'btn btn-import js-add-to-import' do
-                = _('Import')
-                = icon('spinner spin', class: 'loading-icon')
-        - @incompatible_repos.each do |repo|
-          %tr{ id: "repo_#{repo.owner}___#{repo.slug}" }
-            %td
-              = link_to repo.full_name, "https://bitbucket.org/#{repo.full_name}", target: '_blank', rel: 'noopener noreferrer'
-            %td.import-target
-            %td.import-actions-job-status
-              = label_tag _('Incompatible Project'), nil, class: 'label badge-danger'
-
-  - if @incompatible_repos.any?
-    %p
-      = _("One or more of your Bitbucket projects cannot be imported into GitLab directly because they use Subversion or Mercurial for version control, rather than Git.")
-      - link_to_git = link_to(_('Git'), 'https://www.atlassian.com/git/tutorials/migrating-overview')
-      - link_to_import_flow = link_to(_('import flow'), status_import_bitbucket_path)
-      = _("Please convert them to %{link_to_git}, and go through the %{link_to_import_flow} again.").html_safe % { link_to_git: link_to_git, link_to_import_flow: link_to_import_flow }
-
-  .js-importer-status{ data: { jobs_import_path: "#{jobs_import_bitbucket_path}", import_path: "#{import_bitbucket_path}" } }
+= render 'import/githubish_status', provider: 'bitbucket'

app/views/import/bitbucket_server/status.html.haml

@@ -5,94 +5,4 @@
   %i.fa.fa-bitbucket-square
   = _('Import projects from Bitbucket Server')
 
-- if Feature.enabled?(:new_import_ui)
-  = render 'import/githubish_status', provider: 'bitbucket_server', extra_data: { reconfigure_path: configure_import_bitbucket_server_path }
-- else
-  - if @repos.any?
-    %p.light
-      = _('Select projects you want to import.')
-    .btn-group
-      - if @incompatible_repos.any?
-        = button_tag class: 'btn btn-import btn-success js-import-all' do
-          = _('Import all compatible projects')
-          = icon('spinner spin', class: 'loading-icon')
-      - else
-        = button_tag class: 'btn btn-import btn-success js-import-all' do
-          = _('Import all projects')
-          = icon('spinner spin', class: 'loading-icon')
-
-  .btn-group
-    = link_to('Reconfigure', configure_import_bitbucket_server_path, class: 'btn btn-primary', method: :post)
-
-  .input-btn-group.float-right
-    = form_tag status_import_bitbucket_server_path, :method => 'get' do
-      = text_field_tag :filter, sanitize(params[:filter]), class: 'form-control append-bottom-10', placeholder: _('Filter your projects by name'), size: 40, autoFocus: true
-
-  .table-responsive.prepend-top-10
-    %table.table.import-jobs
-      %colgroup.import-jobs-from-col
-      %colgroup.import-jobs-to-col
-      %colgroup.import-jobs-status-col
-      %thead
-        %tr
-          %th= _('From Bitbucket Server')
-          %th= _('To GitLab')
-          %th= _('Status')
-      %tbody
-        - @already_added_projects.each do |project|
-          %tr{ id: "project_#{project.id}", class: "#{project_status_css_class(project.import_status)}" }
-            %td
-              = link_to project.import_source, project.import_source, target: '_blank', rel: 'noopener noreferrer'
-            %td
-              = link_to project.full_path, [project.namespace.becomes(Namespace), project]
-            %td.job-status
-              - case project.import_status
-              - when 'finished'
-                = icon('check', text: 'Done')
-              - when 'started'
-                = icon('spin', text: 'started')
-              - else
-                = project.human_import_status_name
-
-        - @repos.each do |repo|
-          %tr{ data: { id: "#{repo.project_key}/#{repo.slug}" } }
-            %td
-              = sanitize(link_to(repo.browse_url, repo.browse_url, target: '_blank', rel: 'noopener noreferrer'), attributes: %w(href target rel))
-            %td.import-target
-              %fieldset.row
-              .input-group
-                .project-path.input-group-prepend
-                  - if current_user.can_select_namespace?
-                    - selected = params[:namespace_id] || :extra_group
-                    - opts = current_user.can_create_group? ? { extra_group: Group.new(name: sanitize_project_name(repo.project_key), path: sanitize_project_name(repo.project_key)) } : {}
-                    = select_tag :namespace_id, namespaces_options(selected, opts.merge({ display_path: true })), { class: 'select2 js-select-namespace', tabindex: 1 }
-                  - else
-                    = text_field_tag :path, current_user.namespace_path, class: "input-group-text input-large form-control", tabindex: 1, disabled: true
-                %span.input-group-prepend
-                  .input-group-text /
-                = text_field_tag :path, sanitize_project_name(repo.slug), class: "input-mini form-control", tabindex: 2, required: true
-            %td.import-actions.job-status
-              = button_tag class: 'btn btn-import js-add-to-import' do
-                Import
-                = icon('spinner spin', class: 'loading-icon')
-        - @incompatible_repos.each do |repo|
-          %tr{ id: "repo_#{repo.project_key}___#{repo.slug}" }
-            %td
-              = sanitize(link_to(repo.browse_url, repo.browse_url, target: '_blank', rel: 'noopener noreferrer'), attributes: %w(href target rel))
-            %td.import-target
-            %td.import-actions-job-status
-              = label_tag 'Incompatible Project', nil, class: 'label badge-danger'
-
-  - if @incompatible_repos.any?
-    %p
-      One or more of your Bitbucket Server projects cannot be imported into GitLab
-      directly because they use Subversion or Mercurial for version control,
-      rather than Git. Please convert
-      = link_to 'them to Git,', 'https://www.atlassian.com/git/tutorials/migrating-overview'
-      and go through the
-      = link_to 'import flow', status_import_bitbucket_server_path
-      again.
-
-  = paginate_without_count(@collection)
-
-  .js-importer-status{ data: { jobs_import_path: "#{jobs_import_bitbucket_server_path}", import_path: "#{import_bitbucket_server_path}" } }
+= render 'import/githubish_status', provider: 'bitbucket_server', extra_data: { reconfigure_path: configure_import_bitbucket_server_path }

app/views/import/fogbugz/status.html.haml

@@ -4,63 +4,8 @@
   %i.fa.fa-bug
   = _('Import projects from FogBugz')
 
-- if Feature.enabled?(:new_import_ui)
-  %p.light
-    - link_to_customize = link_to('customize', new_user_map_import_fogbugz_path)
-    = _('Optionally, you can %{link_to_customize} how FogBugz email addresses and usernames are imported into GitLab.').html_safe % { link_to_customize: link_to_customize }
-  %hr
-  = render 'import/githubish_status', provider: 'fogbugz', filterable: false
-- else
-  - if @repos.any?
-    %p.light
-      = _('Select projects you want to import.')
-    %p.light
-      - link_to_customize = link_to('customize', new_user_map_import_fogbugz_path)
-      = _('Optionally, you can %{link_to_customize} how FogBugz email addresses and usernames are imported into GitLab.').html_safe % { link_to_customize: link_to_customize }
-    %hr
-    %p
-    = button_tag class: 'btn btn-import btn-success js-import-all' do
-      = _('Import all projects')
-      = icon("spinner spin", class: "loading-icon")
-
-  .table-responsive
-    %table.table.import-jobs
-      %colgroup.import-jobs-from-col
-      %colgroup.import-jobs-to-col
-      %colgroup.import-jobs-status-col
-      %thead
-        %tr
-          %th= _("From FogBugz")
-          %th= _("To GitLab")
-          %th= _("Status")
-      %tbody
-        - @already_added_projects.each do |project|
-          %tr{ id: "project_#{project.id}", class: "#{project_status_css_class(project.import_status)}" }
-            %td
-              = project.import_source
-            %td
-              = link_to project.full_path, [project.namespace.becomes(Namespace), project]
-            %td.job-status
-              - case project.import_status
-              - when 'finished'
-                %span
-                  %i.fa.fa-check
-                  = _("done")
-              - when 'started'
-                %i.fa.fa-spinner.fa-spin
-                = _("started")
-              - else
-                = project.human_import_status_name
-
-        - @repos.each do |repo|
-          %tr{ id: "repo_#{repo.id}" }
-            %td
-              = repo.name
-            %td.import-target
-              #{current_user.username}/#{repo.name}
-            %td.import-actions.job-status
-              = button_tag class: "btn btn-import js-add-to-import" do
-                = _("Import")
-                = icon("spinner spin", class: "loading-icon")
-
-  .js-importer-status{ data: { jobs_import_path: "#{jobs_import_fogbugz_path}", import_path: "#{import_fogbugz_path}" } }
+%p.light
+  - link_to_customize = link_to('customize', new_user_map_import_fogbugz_path)
+  = _('Optionally, you can %{link_to_customize} how FogBugz email addresses and usernames are imported into GitLab.').html_safe % { link_to_customize: link_to_customize }
+%hr
+= render 'import/githubish_status', provider: 'fogbugz', filterable: false

app/views/import/gitlab/status.html.haml

@@ -4,55 +4,4 @@
   = sprite_icon('heart', size: 16, css_class: 'gl-vertical-align-middle')
   = _('Import projects from GitLab.com')
 
-- if Feature.enabled?(:new_import_ui)
-  = render 'import/githubish_status', provider: 'gitlab', filterable: false
-- else
-  %p.light
-    = _('Select projects you want to import.')
-  %hr
-  %p
-    = button_tag class: "btn btn-import btn-success js-import-all" do
-      = _('Import all projects')
-      = icon("spinner spin", class: "loading-icon")
-
-  .table-responsive
-    %table.table.import-jobs
-      %colgroup.import-jobs-from-col
-      %colgroup.import-jobs-to-col
-      %colgroup.import-jobs-status-col
-      %thead
-        %tr
-          %th= _('From GitLab.com')
-          %th= _('To this GitLab instance')
-          %th= _('Status')
-      %tbody
-        - @already_added_projects.each do |project|
-          %tr{ id: "project_#{project.id}", class: "#{project_status_css_class(project.import_status)}" }
-            %td
-              = link_to project.import_source, "https://gitlab.com/#{project.import_source}", target: "_blank"
-            %td
-              = link_to project.full_path, [project.namespace.becomes(Namespace), project]
-            %td.job-status
-              - case project.import_status
-              - when 'finished'
-                %span
-                  %i.fa.fa-check
-                  = _('done')
-              - when 'started'
-                %i.fa.fa-spinner.fa-spin
-                = _('started')
-              - else
-                = project.human_import_status_name
-
-        - @repos.each do |repo|
-          %tr{ id: "repo_#{repo["id"]}" }
-            %td
-              = link_to repo["path_with_namespace"], "https://gitlab.com/#{repo["path_with_namespace"]}", target: "_blank", rel: 'noopener noreferrer'
-            %td.import-target
-              = import_project_target(repo['namespace']['path'], repo['name'])
-            %td.import-actions.job-status
-              = button_tag class: "btn btn-import js-add-to-import" do
-                = _('Import')
-                = icon("spinner spin", class: "loading-icon")
-
-  .js-importer-status{ data: { jobs_import_path: "#{jobs_import_gitlab_path}", import_path: "#{import_gitlab_path}" } }
+= render 'import/githubish_status', provider: 'gitlab', filterable: false

spec/controllers/import/bitbucket_controller_spec.rb

@@ -58,12 +58,12 @@ RSpec.describe Import::BitbucketController do
     before do
       @repo = double(name: 'vim', slug: 'vim', owner: 'asd', full_name: 'asd/vim', clone_url: 'http://test.host/demo/url.git', 'valid?' => true)
       @invalid_repo = double(name: 'mercurialrepo', slug: 'mercurialrepo', owner: 'asd', full_name: 'asd/mercurialrepo', clone_url: 'http://test.host/demo/mercurialrepo.git', 'valid?' => false)
+      allow(controller).to receive(:provider_url).and_return('http://demobitbucket.org')
 
       assign_session_tokens
-      stub_feature_flags(new_import_ui: false)
     end
 
-    it_behaves_like 'import controller with new_import_ui feature flag' do
+    it_behaves_like 'import controller status' do
       before do
         allow(controller).to receive(:provider_url).and_return('http://demobitbucket.org')
       end
@@ -75,44 +75,16 @@ RSpec.describe Import::BitbucketController do
       let(:client_repos_field) { :repos }
     end
 
-    context 'with new_import_ui feature flag enabled' do
-      before do
-        stub_feature_flags(new_import_ui: true)
-        allow(controller).to receive(:provider_url).and_return('http://demobitbucket.org')
-      end
-
-      it 'returns invalid repos' do
-        allow_any_instance_of(Bitbucket::Client).to receive(:repos).and_return([@repo, @invalid_repo])
-
-        get :status, format: :json
-
-        expect(response).to have_gitlab_http_status(:ok)
-        expect(json_response['incompatible_repos'].length).to eq(1)
-        expect(json_response.dig("incompatible_repos", 0, "id")).to eq(@invalid_repo.full_name)
-        expect(json_response['provider_repos'].length).to eq(1)
-        expect(json_response.dig("provider_repos", 0, "id")).to eq(@repo.full_name)
-      end
-    end
-
-    it "assigns variables" do
-      @project = create(:project, import_type: 'bitbucket', creator_id: user.id)
-      allow_any_instance_of(Bitbucket::Client).to receive(:repos).and_return([@repo])
+    it 'returns invalid repos' do
+      allow_any_instance_of(Bitbucket::Client).to receive(:repos).and_return([@repo, @invalid_repo])
 
-      get :status
-
-      expect(assigns(:already_added_projects)).to eq([@project])
-      expect(assigns(:repos)).to eq([@repo])
-      expect(assigns(:incompatible_repos)).to eq([])
-    end
+      get :status, format: :json
 
-    it "does not show already added project" do
-      @project = create(:project, import_type: 'bitbucket', creator_id: user.id, import_source: 'asd/vim')
-      allow_any_instance_of(Bitbucket::Client).to receive(:repos).and_return([@repo])
-
-      get :status
-
-      expect(assigns(:already_added_projects)).to eq([@project])
-      expect(assigns(:repos)).to eq([])
+      expect(response).to have_gitlab_http_status(:ok)
+      expect(json_response['incompatible_repos'].length).to eq(1)
+      expect(json_response.dig("incompatible_repos", 0, "id")).to eq(@invalid_repo.full_name)
+      expect(json_response['provider_repos'].length).to eq(1)
+      expect(json_response.dig("provider_repos", 0, "id")).to eq(@repo.full_name)
     end
 
     context 'when filtering' do

spec/controllers/import/bitbucket_server_controller_spec.rb

@@ -148,28 +148,21 @@ RSpec.describe Import::BitbucketServerController do
       @invalid_repo = double(slug: 'invalid', project_key: 'foobar', full_name: 'asd/foobar', "valid?" => false, browse_url: 'http://bad-repo', name: 'invalid')
       @created_repo = double(slug: 'created', project_key: 'existing', full_name: 'group/created', "valid?" => true, browse_url: 'http://existing')
       assign_session_tokens
-      stub_feature_flags(new_import_ui: false)
     end
 
-    context 'with new_import_ui feature flag enabled' do
-      before do
-        stub_feature_flags(new_import_ui: true)
-      end
-
-      it 'returns invalid repos' do
-        allow(client).to receive(:repos).with(filter: nil, limit: 25, page_offset: 0).and_return([@repo, @invalid_repo])
+    it 'returns invalid repos' do
+      allow(client).to receive(:repos).with(filter: nil, limit: 25, page_offset: 0).and_return([@repo, @invalid_repo])
 
-        get :status, format: :json
+      get :status, format: :json
 
-        expect(response).to have_gitlab_http_status(:ok)
-        expect(json_response['incompatible_repos'].length).to eq(1)
-        expect(json_response.dig("incompatible_repos", 0, "id")).to eq(@invalid_repo.full_name)
-        expect(json_response['provider_repos'].length).to eq(1)
-        expect(json_response.dig("provider_repos", 0, "id")).to eq(@repo.full_name)
-      end
+      expect(response).to have_gitlab_http_status(:ok)
+      expect(json_response['incompatible_repos'].length).to eq(1)
+      expect(json_response.dig("incompatible_repos", 0, "id")).to eq(@invalid_repo.full_name)
+      expect(json_response['provider_repos'].length).to eq(1)
+      expect(json_response.dig("provider_repos", 0, "id")).to eq(@repo.full_name)
     end
 
-    it_behaves_like 'import controller with new_import_ui feature flag' do
+    it_behaves_like 'import controller status' do
       let(:repo) { @repo }
       let(:repo_id) { @repo.full_name }
       let(:import_source) { @repo.browse_url }
@@ -177,28 +170,11 @@ RSpec.describe Import::BitbucketServerController do
       let(:client_repos_field) { :repos }
     end
 
-    it 'assigns repository categories' do
-      created_project = create(:project, :import_finished, import_type: 'bitbucket_server', creator_id: user.id, import_source: @created_repo.browse_url)
-
-      expect(repos).to receive(:partition).and_return([[@repo, @created_repo], [@invalid_repo]])
-      expect(repos).to receive(:current_page).and_return(1)
-      expect(repos).to receive(:next_page).and_return(2)
-      expect(repos).to receive(:prev_page).and_return(nil)
-      expect(client).to receive(:repos).and_return(repos)
-
-      get :status
-
-      expect(assigns(:already_added_projects)).to eq([created_project])
-      expect(assigns(:repos)).to eq([@repo])
-      expect(assigns(:incompatible_repos)).to eq([@invalid_repo])
-    end
-
     context 'when filtering' do
       let(:filter) { 'test' }
 
       it 'passes filter param to bitbucket client' do
-        expect(repos).to receive(:partition).and_return([[@repo, @created_repo], [@invalid_repo]])
-        expect(client).to receive(:repos).with(filter: filter, limit: 25, page_offset: 0).and_return(repos)
+        expect(client).to receive(:repos).with(filter: filter, limit: 25, page_offset: 0).and_return([@repo])
 
         get :status, params: { filter: filter }, as: :json
       end

spec/controllers/import/fogbugz_controller_spec.rb

@@ -82,36 +82,15 @@ RSpec.describe Import::FogbugzController do
     before do
       @repo = OpenStruct.new(id: 'demo', name: 'vim')
       stub_client(valid?: true)
-      stub_feature_flags(new_import_ui: false)
     end
 
-    it_behaves_like 'import controller with new_import_ui feature flag' do
+    it_behaves_like 'import controller status' do
       let(:repo) { @repo }
       let(:repo_id) { @repo.id }
       let(:import_source) { @repo.name }
       let(:provider_name) { 'fogbugz' }
       let(:client_repos_field) { :repos }
     end
-
-    it 'assigns variables' do
-      @project = create(:project, import_type: 'fogbugz', creator_id: user.id)
-      stub_client(repos: [@repo])
-
-      get :status
-
-      expect(assigns(:already_added_projects)).to eq([@project])
-      expect(assigns(:repos)).to eq([@repo])
-    end
-
-    it 'does not show already added project' do
-      @project = create(:project, import_type: 'fogbugz', creator_id: user.id, import_source: 'vim')
-      stub_client(repos: [@repo])
-
-      get :status
-
-      expect(assigns(:already_added_projects)).to eq([@project])
-      expect(assigns(:repos)).to eq([])
-    end
   end
 
   describe 'POST create' do

spec/controllers/import/gitlab_controller_spec.rb

@@ -36,36 +36,15 @@ RSpec.describe Import::GitlabController do
     before do
       @repo = OpenStruct.new(id: 1, path: 'vim', path_with_namespace: 'asd/vim', web_url: 'https://gitlab.com/asd/vim')
       assign_session_token
-      stub_feature_flags(new_import_ui: false)
     end
 
-    it_behaves_like 'import controller with new_import_ui feature flag' do
+    it_behaves_like 'import controller status' do
       let(:repo) { @repo }
       let(:repo_id) { @repo.id }
       let(:import_source) { @repo.path_with_namespace }
       let(:provider_name) { 'gitlab' }
       let(:client_repos_field) { :projects }
     end
-
-    it "assigns variables" do
-      @project = create(:project, import_type: 'gitlab', creator_id: user.id)
-      stub_client(projects: [@repo])
-
-      get :status
-
-      expect(assigns(:already_added_projects)).to eq([@project])
-      expect(assigns(:repos)).to eq([@repo])
-    end
-
-    it "does not show already added project" do
-      @project = create(:project, import_type: 'gitlab', creator_id: user.id, import_source: 'asd/vim')
-      stub_client(projects: [@repo])
-
-      get :status
-
-      expect(assigns(:already_added_projects)).to eq([@project])
-      expect(assigns(:repos)).to eq([])
-    end
   end
 
   describe "POST create" do

spec/support/shared_examples/controllers/import_controller_new_import_ui_shared_examples.rb

-# frozen_string_literal: true
-
-RSpec.shared_examples 'import controller with new_import_ui feature flag' do
-  include ImportSpecHelper
-
-  context 'with new_import_ui feature flag enabled' do
-    let(:group) { create(:group) }
-
-    before do
-      stub_feature_flags(new_import_ui: true)
-      group.add_owner(user)
-    end
-
-    it "returns variables for json request" do
-      project = create(:project, import_type: provider_name, creator_id: user.id)
-      stub_client(client_repos_field => [repo])
-
-      get :status, format: :json
-
-      expect(response).to have_gitlab_http_status(:ok)
-      expect(json_response.dig("imported_projects", 0, "id")).to eq(project.id)
-      expect(json_response.dig("provider_repos", 0, "id")).to eq(repo_id)
-      expect(json_response.dig("namespaces", 0, "id")).to eq(group.id)
-    end
-
-    it "does not show already added project" do
-      project = create(:project, import_type: provider_name, namespace: user.namespace, import_status: :finished, import_source: import_source)
-      stub_client(client_repos_field => [repo])
-
-      get :status, format: :json
-
-      expect(json_response.dig("imported_projects", 0, "id")).to eq(project.id)
-      expect(json_response.dig("provider_repos")).to eq([])
-    end
-  end
-end

spec/support/shared_examples/controllers/import_controller_status_shared_examples.rb

+# frozen_string_literal: true
+
+RSpec.shared_examples 'import controller status' do
+  include ImportSpecHelper
+
+  let(:group) { create(:group) }
+
+  before do
+    group.add_owner(user)
+  end
+
+  it "returns variables for json request" do
+    project = create(:project, import_type: provider_name, creator_id: user.id)
+    stub_client(client_repos_field => [repo])
+
+    get :status, format: :json
+
+    expect(response).to have_gitlab_http_status(:ok)
+    expect(json_response.dig("imported_projects", 0, "id")).to eq(project.id)
+    expect(json_response.dig("provider_repos", 0, "id")).to eq(repo_id)
+    expect(json_response.dig("namespaces", 0, "id")).to eq(group.id)
+  end
+
+  it "does not show already added project" do
+    project = create(:project, import_type: provider_name, namespace: user.namespace, import_status: :finished, import_source: import_source)
+    stub_client(client_repos_field => [repo])
+
+    get :status, format: :json
+
+    expect(json_response.dig("imported_projects", 0, "id")).to eq(project.id)
+    expect(json_response.dig("provider_repos")).to eq([])
+  end
+end