Replace the 'search.feature' spinach test with an rspec analog

10 files changed, 554 insertions, 0 deletions

diff --git a/spec/features/search/user_searches_for_code_spec.rb b/spec/features/search/user_searches_for_code_spec.rb
new file mode 100644
index 00000000000..0ed797a62ea
--- /dev/null
+++ b/spec/features/search/user_searches_for_code_spec.rb
@@ -0,0 +1,67 @@
+require 'spec_helper'
+
+describe 'User searches for code' do
+  let(:user) { create(:user) }
+  let(:project) { create(:project, :repository, namespace: user.namespace) }
+
+  context 'when signed in' do
+    before do
+      project.add_master(user)
+      sign_in(user)
+    end
+
+    it 'finds a file' do
+      visit(project_path(project))
+
+      page.within('.search') do
+        fill_in('search', with: 'application.js')
+        click_button('Go')
+      end
+
+      click_link('Code')
+
+      expect(page).to have_selector('.file-content .code')
+      expect(page).to have_selector("span.line[lang='javascript']")
+    end
+
+    context 'when on a project page', :js do
+      before do
+        visit(search_path)
+      end
+
+      include_examples 'top right search form'
+
+      it 'finds code' do
+        find('.js-search-project-dropdown').trigger('click')
+
+        page.within('.project-filter') do
+          click_link(project.name_with_namespace)
+        end
+
+        fill_in('dashboard_search', with: 'rspec')
+        find('.btn-search').trigger('click')
+
+        page.within('.results') do
+          expect(find(:css, '.search-results')).to have_content('Update capybara, rspec-rails, poltergeist to recent versions')
+        end
+      end
+    end
+  end
+
+  context 'when signed out' do
+    let(:project) { create(:project, :public, :repository) }
+
+    before do
+      visit(project_path(project))
+    end
+
+    it 'finds code' do
+      fill_in('search', with: 'rspec')
+      click_button('Go')
+
+      page.within('.results') do
+        expect(find(:css, '.search-results')).to have_content('Update capybara, rspec-rails, poltergeist to recent versions')
+      end
+    end
+  end
+end
diff --git a/spec/features/search/user_searches_for_comments_spec.rb b/spec/features/search/user_searches_for_comments_spec.rb
new file mode 100644
index 00000000000..c7c469a262c
--- /dev/null
+++ b/spec/features/search/user_searches_for_comments_spec.rb
@@ -0,0 +1,47 @@
+require 'spec_helper'
+
+describe 'User searches for comments' do
+  let(:project) { create(:project, :repository) }
+  let(:user) { create(:user) }
+
+  before do
+    project.add_reporter(user)
+    sign_in(user)
+
+    visit(project_path(project))
+  end
+
+  context 'when a comment is in commits' do
+    context 'when comment belongs to an invalid commit' do
+      let(:comment) { create(:note_on_commit, author: user, project: project, commit_id: 12345678, note: 'Bug here') }
+
+      it 'finds a commit' do
+        page.within('.search') do
+          fill_in('search', with: comment.note)
+          click_button('Go')
+        end
+
+        click_link('Comments')
+
+        expect(page).to have_text('Commit deleted')
+        expect(page).to have_text('12345678')
+      end
+    end
+  end
+
+  context 'when a comment is in a snippet' do
+    let(:snippet) { create(:project_snippet, :private, project: project, author: user, title: 'Some title') }
+    let(:comment) { create(:note, noteable: snippet, author: user, note: 'Supercalifragilisticexpialidocious', project: project) }
+
+    it 'finds a snippet' do
+      page.within('.search') do
+        fill_in('search', with: comment.note)
+        click_button('Go')
+      end
+
+      click_link('Comments')
+
+      expect(page).to have_link(snippet.title)
+    end
+  end
+end
diff --git a/spec/features/search/user_searches_for_commits_spec.rb b/spec/features/search/user_searches_for_commits_spec.rb
new file mode 100644
index 00000000000..28cae444588
--- /dev/null
+++ b/spec/features/search/user_searches_for_commits_spec.rb
@@ -0,0 +1,49 @@
+require 'spec_helper'
+
+describe 'User searches for commits' do
+  let(:project) { create(:project, :repository) }
+  let(:sha) { '6d394385cf567f80a8fd85055db1ab4c5295806f' }
+  let(:user) { create(:user) }
+
+  before do
+    project.add_reporter(user)
+    sign_in(user)
+
+    visit(search_path(project_id: project.id))
+  end
+
+  context 'when searching by SHA' do
+    it 'finds a commit and redirects to its page' do
+      fill_in('search', with: sha)
+      click_button('Search')
+
+      expect(page).to have_current_path(project_commit_path(project, sha))
+    end
+
+    it 'finds a commit in uppercase and redirects to its page' do
+      fill_in('search', with: sha.upcase)
+      click_button('Search')
+
+      expect(page).to have_current_path(project_commit_path(project, sha))
+    end
+  end
+
+  context 'when searching by message' do
+    it 'finds a commit and holds on /search page' do
+      create_commit('Message referencing another sha: "deadbeef"', project, user, 'master')
+
+      fill_in('search', with: 'deadbeef')
+      click_button('Search')
+
+      expect(page).to have_current_path('/search', only_path: true)
+    end
+
+    it 'finds multiple commits' do
+      fill_in('search', with: 'See merge request')
+      click_button('Search')
+      click_link('Commits')
+
+      expect(page).to have_selector('.commit-row-description', count: 9)
+    end
+  end
+end
diff --git a/spec/features/search/user_searches_for_issues_spec.rb b/spec/features/search/user_searches_for_issues_spec.rb
new file mode 100644
index 00000000000..630a81b1c5e
--- /dev/null
+++ b/spec/features/search/user_searches_for_issues_spec.rb
@@ -0,0 +1,76 @@
+require 'spec_helper'
+
+describe 'User searches for issues', :js do
+  let(:user) { create(:user) }
+  let(:project) { create(:project, namespace: user.namespace) }
+  let!(:issue1) { create(:issue, title: 'Foo', project: project) }
+  let!(:issue2) { create(:issue, title: 'Bar', project: project) }
+
+  context 'when signed in' do
+    before do
+      project.add_master(user)
+      sign_in(user)
+
+      visit(search_path)
+    end
+
+    include_examples 'top right search form'
+
+    it 'finds an issue' do
+      fill_in('dashboard_search', with: issue1.title)
+      find('.btn-search').trigger('click')
+
+      page.within('.search-filter') do
+        click_link('Issues')
+      end
+
+      page.within('.results') do
+        expect(find(:css, '.search-results')).to have_link(issue1.title).and have_no_link(issue2.title)
+      end
+    end
+
+    context 'when on a project page' do
+      it 'finds an issue' do
+        find('.js-search-project-dropdown').trigger('click')
+
+        page.within('.project-filter') do
+          click_link(project.name_with_namespace)
+        end
+
+        fill_in('dashboard_search', with: issue1.title)
+        find('.btn-search').trigger('click')
+
+        page.within('.search-filter') do
+          click_link('Issues')
+        end
+
+        page.within('.results') do
+          expect(find(:css, '.search-results')).to have_link(issue1.title).and have_no_link(issue2.title)
+        end
+      end
+    end
+  end
+
+  context 'when signed out' do
+    let(:project) { create(:project, :public) }
+
+    before do
+      visit(search_path)
+    end
+
+    include_examples 'top right search form'
+
+    it 'finds an issue' do
+      fill_in('dashboard_search', with: issue1.title)
+      find('.btn-search').trigger('click')
+
+      page.within('.search-filter') do
+        click_link('Issues')
+      end
+
+      page.within('.results') do
+        expect(find(:css, '.search-results')).to have_link(issue1.title).and have_no_link(issue2.title)
+      end
+    end
+  end
+end
diff --git a/spec/features/search/user_searches_for_merge_requests_spec.rb b/spec/features/search/user_searches_for_merge_requests_spec.rb
new file mode 100644
index 00000000000..116256682f4
--- /dev/null
+++ b/spec/features/search/user_searches_for_merge_requests_spec.rb
@@ -0,0 +1,51 @@
+require 'spec_helper'
+
+describe 'User searches for merge requests', :js do
+  let(:user) { create(:user) }
+  let(:project) { create(:project, namespace: user.namespace) }
+  let!(:merge_request1) { create(:merge_request, title: 'Foo', source_project: project, target_project: project) }
+  let!(:merge_request2) { create(:merge_request, :simple, title: 'Bar', source_project: project, target_project: project) }
+
+  before do
+    project.add_master(user)
+    sign_in(user)
+
+    visit(search_path)
+  end
+
+  include_examples 'top right search form'
+
+  it 'finds a merge request' do
+    fill_in('dashboard_search', with: merge_request1.title)
+    find('.btn-search').trigger('click')
+
+    page.within('.search-filter') do
+      click_link('Merge requests')
+    end
+
+    page.within('.results') do
+      expect(find(:css, '.search-results')).to have_link(merge_request1.title).and have_no_link(merge_request2.title)
+    end
+  end
+
+  context 'when on a project page' do
+    it 'finds a merge request' do
+      find('.js-search-project-dropdown').trigger('click')
+
+      page.within('.project-filter') do
+        click_link(project.name_with_namespace)
+      end
+
+      fill_in('dashboard_search', with: merge_request1.title)
+      find('.btn-search').trigger('click')
+
+      page.within('.search-filter') do
+        click_link('Merge requests')
+      end
+
+      page.within('.results') do
+        expect(find(:css, '.search-results')).to have_link(merge_request1.title).and have_no_link(merge_request2.title)
+      end
+    end
+  end
+end
diff --git a/spec/features/search/user_searches_for_milestones_spec.rb b/spec/features/search/user_searches_for_milestones_spec.rb
new file mode 100644
index 00000000000..4fa9fe9ce8c
--- /dev/null
+++ b/spec/features/search/user_searches_for_milestones_spec.rb
@@ -0,0 +1,51 @@
+require 'spec_helper'
+
+describe 'User searches for milestones', :js do
+  let(:user) { create(:user) }
+  let(:project) { create(:project, namespace: user.namespace) }
+  let!(:milestone1) { create(:milestone, title: 'Foo', project: project) }
+  let!(:milestone2) { create(:milestone, title: 'Bar', project: project) }
+
+  before do
+    project.add_master(user)
+    sign_in(user)
+
+    visit(search_path)
+  end
+
+  include_examples 'top right search form'
+
+  it 'finds a milestone' do
+    fill_in('dashboard_search', with: milestone1.title)
+    find('.btn-search').trigger('click')
+
+    page.within('.search-filter') do
+      click_link('Milestones')
+    end
+
+    page.within('.results') do
+      expect(find(:css, '.search-results')).to have_link(milestone1.title).and have_no_link(milestone2.title)
+    end
+  end
+
+  context 'when on a project page' do
+    it 'finds a milestone' do
+      find('.js-search-project-dropdown').trigger('click')
+
+      page.within('.project-filter') do
+        click_link(project.name_with_namespace)
+      end
+
+      fill_in('dashboard_search', with: milestone1.title)
+      find('.btn-search').trigger('click')
+
+      page.within('.search-filter') do
+        click_link('Milestones')
+      end
+
+      page.within('.results') do
+        expect(find(:css, '.search-results')).to have_link(milestone1.title).and have_no_link(milestone2.title)
+      end
+    end
+  end
+end
diff --git a/spec/features/search/user_searches_for_projects_spec.rb b/spec/features/search/user_searches_for_projects_spec.rb
new file mode 100644
index 00000000000..242e437e41c
--- /dev/null
+++ b/spec/features/search/user_searches_for_projects_spec.rb
@@ -0,0 +1,36 @@
+require 'spec_helper'
+
+describe 'User searches for projects' do
+  let!(:project) { create(:project, :public, name: 'Shop') }
+
+  context 'when signed out' do
+    include_examples 'top right search form'
+
+    it 'finds a project' do
+      visit(search_path)
+
+      fill_in('dashboard_search', with: project.name[0..3])
+      click_button('Search')
+
+      expect(page).to have_link(project.name)
+    end
+
+    it 'preserves the group being searched in' do
+      visit(search_path(group_id: project.namespace.id))
+
+      fill_in('search', with: 'foo')
+      click_button('Search')
+
+      expect(find('#group_id', visible: false).value).to eq(project.namespace.id.to_s)
+    end
+
+    it 'preserves the project being searched in' do
+      visit(search_path(project_id: project.id))
+
+      fill_in('search', with: 'foo')
+      click_button('Search')
+
+      expect(find('#project_id', visible: false).value).to eq(project.id.to_s)
+    end
+  end
+end
diff --git a/spec/features/search/user_searches_for_wiki_pages_spec.rb b/spec/features/search/user_searches_for_wiki_pages_spec.rb
new file mode 100644
index 00000000000..1ea56479ecc
--- /dev/null
+++ b/spec/features/search/user_searches_for_wiki_pages_spec.rb
@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+describe 'User searches for wiki pages', :js do
+  let(:user) { create(:user) }
+  let(:project) { create(:project, namespace: user.namespace) }
+  let!(:wiki_page) { create(:wiki_page, wiki: project.wiki, attrs: { title: 'test_wiki', content: 'Some Wiki content' }) }
+
+  before do
+    project.add_master(user)
+    sign_in(user)
+
+    visit(search_path)
+  end
+
+  include_examples 'top right search form'
+
+  it 'finds a page' do
+    find('.js-search-project-dropdown').trigger('click')
+
+    page.within('.project-filter') do
+      click_link(project.name_with_namespace)
+    end
+
+    fill_in('dashboard_search', with: 'content')
+    find('.btn-search').trigger('click')
+
+    page.within('.search-filter') do
+      click_link('Wiki')
+    end
+
+    page.within('.results') do
+      expect(find(:css, '.search-results')).to have_link(wiki_page.title)
+    end
+  end
+end
diff --git a/spec/features/search/user_uses_header_search_field_spec.rb b/spec/features/search/user_uses_header_search_field_spec.rb
new file mode 100644
index 00000000000..5ddea36add5
--- /dev/null
+++ b/spec/features/search/user_uses_header_search_field_spec.rb
@@ -0,0 +1,90 @@
+require 'spec_helper'
+
+describe 'User uses header search field' do
+  include FilteredSearchHelpers
+
+  let(:project) { create(:project) }
+  let(:user) { create(:user) }
+
+  before do
+    project.add_reporter(user)
+    sign_in(user)
+
+    visit(project_path(project))
+  end
+
+  it 'starts searching by pressing the enter key', :js do
+    fill_in('search', with: 'gitlab')
+    find('#search').native.send_keys(:enter)
+
+    page.within('.breadcrumbs-sub-title') do
+      expect(page).to have_content('Search')
+    end
+  end
+
+  it 'contains location badge' do
+    expect(page).to have_selector('.has-location-badge')
+  end
+
+  context 'when clicking the search field', :js do
+    before do
+      page.find('#search').click
+    end
+
+    it 'shows category search dropdown' do
+      expect(page).to have_selector('.dropdown-header', text: /#{project.name}/i)
+    end
+
+    context 'when clicking issues' do
+      let!(:issue) { create(:issue, project: project, author: user, assignees: [user]) }
+
+      it 'shows assigned issues' do
+        find('.dropdown-menu').click_link('Issues assigned to me')
+
+        expect(page).to have_selector('.filtered-search')
+        expect_tokens([assignee_token(user.name)])
+        expect_filtered_search_input_empty
+      end
+
+      it 'shows created issues' do
+        find('.dropdown-menu').click_link("Issues I've created")
+
+        expect(page).to have_selector('.filtered-search')
+        expect_tokens([author_token(user.name)])
+        expect_filtered_search_input_empty
+      end
+    end
+
+    context 'when clicking merge requests' do
+      let!(:merge_request) { create(:merge_request, source_project: project, author: user, assignee: user) }
+
+      it 'shows assigned merge requests' do
+        find('.dropdown-menu').click_link('Merge requests assigned to me')
+
+        expect(page).to have_selector('.merge-requests-holder')
+        expect_tokens([assignee_token(user.name)])
+        expect_filtered_search_input_empty
+      end
+
+      it 'shows created merge requests' do
+        find('.dropdown-menu').click_link("Merge requests I've created")
+
+        expect(page).to have_selector('.merge-requests-holder')
+        expect_tokens([author_token(user.name)])
+        expect_filtered_search_input_empty
+      end
+    end
+  end
+
+  context 'when entering text into the search field', :js do
+    before do
+      page.within('.search-input-wrap') do
+        fill_in('search', with: project.name[0..3])
+      end
+    end
+
+    it 'does not display the category search dropdown' do
+      expect(page).not_to have_selector('.dropdown-header', text: /#{project.name}/i)
+    end
+  end
+end
diff --git a/spec/features/search/user_uses_search_filters_spec.rb b/spec/features/search/user_uses_search_filters_spec.rb
new file mode 100644
index 00000000000..95f3eb5e805
--- /dev/null
+++ b/spec/features/search/user_uses_search_filters_spec.rb
@@ -0,0 +1,52 @@
+require 'spec_helper'
+
+describe 'User uses search filters', :js do
+  let(:group) { create(:group) }
+  let!(:group_project) { create(:project, group: group) }
+  let(:project) { create(:project, namespace: user.namespace) }
+  let(:user) { create(:user) }
+
+  before do
+    project.add_reporter(user)
+    group.add_owner(user)
+    sign_in(user)
+
+    visit(search_path)
+  end
+
+  context' when filtering by group' do
+    it 'shows group projects' do
+      find('.js-search-group-dropdown').trigger('click')
+
+      wait_for_requests
+
+      page.within('.search-holder') do
+        click_link(group.name)
+      end
+
+      expect(find('.js-search-group-dropdown')).to have_content(group.name)
+
+      page.within('.project-filter') do
+        find('.js-search-project-dropdown').trigger('click')
+
+        wait_for_requests
+
+        expect(page).to have_link(group_project.name_with_namespace)
+      end
+    end
+  end
+
+  context' when filtering by project' do
+    it 'shows a project' do
+      page.within('.project-filter') do
+        find('.js-search-project-dropdown').trigger('click')
+
+        wait_for_requests
+
+        click_link(project.name_with_namespace)
+      end
+
+      expect(find('.js-search-project-dropdown')).to have_content(project.name_with_namespace)
+    end
+  end
+end