2017-05-05 08:26:04 -07:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
|
|
|
require 'rails_helper'
|
|
|
|
|
2018-05-02 09:58:48 -07:00
|
|
|
describe SearchService, type: :service do
|
2019-04-01 11:06:13 -07:00
|
|
|
let(:current_account) { Fabricate(:user).account }
|
|
|
|
|
2017-05-05 08:26:04 -07:00
|
|
|
subject { described_class.new }
|
|
|
|
|
|
|
|
describe '#call' do
|
|
|
|
describe 'with a blank query' do
|
|
|
|
it 'returns empty results without searching' do
|
|
|
|
allow(AccountSearchService).to receive(:new)
|
|
|
|
allow(Tag).to receive(:search_for)
|
2019-04-01 11:06:13 -07:00
|
|
|
results = subject.call('', current_account, 10)
|
2017-05-05 08:26:04 -07:00
|
|
|
|
|
|
|
expect(results).to eq(empty_results)
|
|
|
|
expect(AccountSearchService).not_to have_received(:new)
|
|
|
|
expect(Tag).not_to have_received(:search_for)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'with an url query' do
|
|
|
|
before do
|
|
|
|
@query = 'http://test.host/query'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'that does not find anything' do
|
|
|
|
it 'returns the empty results' do
|
|
|
|
service = double(call: nil)
|
2018-01-22 05:24:22 -08:00
|
|
|
allow(ResolveURLService).to receive(:new).and_return(service)
|
2019-04-01 11:06:13 -07:00
|
|
|
results = subject.call(@query, current_account, 10)
|
2017-05-05 08:26:04 -07:00
|
|
|
|
2019-04-01 11:06:13 -07:00
|
|
|
expect(service).to have_received(:call).with(@query, on_behalf_of: current_account)
|
2017-05-05 08:26:04 -07:00
|
|
|
expect(results).to eq empty_results
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'that finds an account' do
|
|
|
|
it 'includes the account in the results' do
|
2019-04-01 11:06:13 -07:00
|
|
|
account = Fabricate(:account)
|
2017-05-05 08:26:04 -07:00
|
|
|
service = double(call: account)
|
2018-01-22 05:24:22 -08:00
|
|
|
allow(ResolveURLService).to receive(:new).and_return(service)
|
2017-05-05 08:26:04 -07:00
|
|
|
|
2019-04-01 11:06:13 -07:00
|
|
|
results = subject.call(@query, current_account, 10)
|
|
|
|
expect(service).to have_received(:call).with(@query, on_behalf_of: current_account)
|
2017-05-05 08:26:04 -07:00
|
|
|
expect(results).to eq empty_results.merge(accounts: [account])
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'that finds a status' do
|
|
|
|
it 'includes the status in the results' do
|
2019-04-01 11:06:13 -07:00
|
|
|
status = Fabricate(:status)
|
2017-05-05 08:26:04 -07:00
|
|
|
service = double(call: status)
|
2018-01-22 05:24:22 -08:00
|
|
|
allow(ResolveURLService).to receive(:new).and_return(service)
|
2017-05-05 08:26:04 -07:00
|
|
|
|
2019-04-01 11:06:13 -07:00
|
|
|
results = subject.call(@query, current_account, 10)
|
|
|
|
expect(service).to have_received(:call).with(@query, on_behalf_of: current_account)
|
2017-05-05 08:26:04 -07:00
|
|
|
expect(results).to eq empty_results.merge(statuses: [status])
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'with a non-url query' do
|
|
|
|
context 'that matches an account' do
|
|
|
|
it 'includes the account in the results' do
|
|
|
|
query = 'username'
|
2019-04-01 11:06:13 -07:00
|
|
|
account = Fabricate(:account)
|
2017-05-05 08:26:04 -07:00
|
|
|
service = double(call: [account])
|
|
|
|
allow(AccountSearchService).to receive(:new).and_return(service)
|
|
|
|
|
2019-04-01 11:06:13 -07:00
|
|
|
results = subject.call(query, current_account, 10)
|
|
|
|
expect(service).to have_received(:call).with(query, current_account, limit: 10, offset: 0, resolve: false)
|
2017-05-05 08:26:04 -07:00
|
|
|
expect(results).to eq empty_results.merge(accounts: [account])
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'that matches a tag' do
|
|
|
|
it 'includes the tag in the results' do
|
|
|
|
query = '#tag'
|
|
|
|
tag = Tag.new
|
Add type, limit, offset, min_id, max_id, account_id to search API (#10091)
* Add type, limit, offset, min_id, max_id, account_id to search API
Fix #8939
* Make the offset work on accounts and hashtags search as well
* Assure brakeman we are not doing mass assignment here
* Do not allow paginating unless a type is chosen
* Fix search query and index id field on statuses instead of created_at
2019-02-26 06:21:36 -08:00
|
|
|
allow(Tag).to receive(:search_for).with('tag', 10, 0).and_return([tag])
|
2017-05-05 08:26:04 -07:00
|
|
|
|
2019-04-01 11:06:13 -07:00
|
|
|
results = subject.call(query, current_account, 10)
|
Add type, limit, offset, min_id, max_id, account_id to search API (#10091)
* Add type, limit, offset, min_id, max_id, account_id to search API
Fix #8939
* Make the offset work on accounts and hashtags search as well
* Assure brakeman we are not doing mass assignment here
* Do not allow paginating unless a type is chosen
* Fix search query and index id field on statuses instead of created_at
2019-02-26 06:21:36 -08:00
|
|
|
expect(Tag).to have_received(:search_for).with('tag', 10, 0)
|
2017-05-05 08:26:04 -07:00
|
|
|
expect(results).to eq empty_results.merge(hashtags: [tag])
|
|
|
|
end
|
|
|
|
it 'does not include tag when starts with @ character' do
|
|
|
|
query = '@username'
|
|
|
|
allow(Tag).to receive(:search_for)
|
|
|
|
|
2019-04-01 11:06:13 -07:00
|
|
|
results = subject.call(query, current_account, 10)
|
2017-05-05 08:26:04 -07:00
|
|
|
expect(Tag).not_to have_received(:search_for)
|
|
|
|
expect(results).to eq empty_results
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
def empty_results
|
|
|
|
{ accounts: [], hashtags: [], statuses: [] }
|
|
|
|
end
|
|
|
|
end
|