diff options
Diffstat (limited to 'spec')
-rw-r--r-- | spec/controllers/api/v1/accounts/relationships_controller_spec.rb | 6 | ||||
-rw-r--r-- | spec/controllers/api/v1/accounts_controller_spec.rb | 8 | ||||
-rw-r--r-- | spec/services/account_search_service_spec.rb | 10 | ||||
-rw-r--r-- | spec/services/post_status_service_spec.rb | 2 | ||||
-rw-r--r-- | spec/services/search_service_spec.rb | 2 |
5 files changed, 15 insertions, 13 deletions
diff --git a/spec/controllers/api/v1/accounts/relationships_controller_spec.rb b/spec/controllers/api/v1/accounts/relationships_controller_spec.rb index f25b86ac1..508415fc8 100644 --- a/spec/controllers/api/v1/accounts/relationships_controller_spec.rb +++ b/spec/controllers/api/v1/accounts/relationships_controller_spec.rb @@ -32,7 +32,7 @@ describe Api::V1::Accounts::RelationshipsController do json = body_as_json expect(json).to be_a Enumerable - expect(json.first[:following]).to be_truthy + expect(json.first[:following]).to be true expect(json.first[:followed_by]).to be false end end @@ -51,7 +51,8 @@ describe Api::V1::Accounts::RelationshipsController do expect(json).to be_a Enumerable expect(json.first[:id]).to eq simon.id.to_s - expect(json.first[:following]).to be_truthy + expect(json.first[:following]).to be true + expect(json.first[:showing_reblogs]).to be true expect(json.first[:followed_by]).to be false expect(json.first[:muting]).to be false expect(json.first[:requested]).to be false @@ -59,6 +60,7 @@ describe Api::V1::Accounts::RelationshipsController do expect(json.second[:id]).to eq lewis.id.to_s expect(json.second[:following]).to be false + expect(json.second[:showing_reblogs]).to be false expect(json.second[:followed_by]).to be true expect(json.second[:muting]).to be false expect(json.second[:requested]).to be false diff --git a/spec/controllers/api/v1/accounts_controller_spec.rb b/spec/controllers/api/v1/accounts_controller_spec.rb index f3b879421..053c53e5a 100644 --- a/spec/controllers/api/v1/accounts_controller_spec.rb +++ b/spec/controllers/api/v1/accounts_controller_spec.rb @@ -31,10 +31,10 @@ RSpec.describe Api::V1::AccountsController, type: :controller do expect(response).to have_http_status(:success) end - it 'returns JSON with following=truthy and requested=false' do + it 'returns JSON with following=true and requested=false' do json = body_as_json - expect(json[:following]).to be_truthy + expect(json[:following]).to be true expect(json[:requested]).to be false end @@ -50,11 +50,11 @@ RSpec.describe Api::V1::AccountsController, type: :controller do expect(response).to have_http_status(:success) end - it 'returns JSON with following=false and requested=truthy' do + it 'returns JSON with following=false and requested=true' do json = body_as_json expect(json[:following]).to be false - expect(json[:requested]).to be_truthy + expect(json[:requested]).to be true end it 'creates a follow request relation between user and target user' do diff --git a/spec/services/account_search_service_spec.rb b/spec/services/account_search_service_spec.rb index 25bb56bca..4685e619f 100644 --- a/spec/services/account_search_service_spec.rb +++ b/spec/services/account_search_service_spec.rb @@ -72,7 +72,7 @@ describe AccountSearchService do describe 'and there is no account provided' do it 'uses search_for to find matches' do allow(Account).to receive(:search_for) - subject.call('two@example.com', 10, false, nil) + subject.call('two@example.com', 10, nil, resolve: false) expect(Account).to have_received(:search_for).with('two example.com', 10) end @@ -82,9 +82,9 @@ describe AccountSearchService do it 'uses advanced_search_for to find matches' do account = Fabricate(:account) allow(Account).to receive(:advanced_search_for) - subject.call('two@example.com', 10, false, account) + subject.call('two@example.com', 10, account, resolve: false) - expect(Account).to have_received(:advanced_search_for).with('two example.com', account, 10) + expect(Account).to have_received(:advanced_search_for).with('two example.com', account, 10, nil) end end end @@ -125,7 +125,7 @@ describe AccountSearchService do service = double(call: nil) allow(ResolveRemoteAccountService).to receive(:new).and_return(service) - results = subject.call('newuser@remote.com', 10, true) + results = subject.call('newuser@remote.com', 10, nil, resolve: true) expect(service).to have_received(:call).with('newuser@remote.com') end @@ -133,7 +133,7 @@ describe AccountSearchService do service = double(call: nil) allow(ResolveRemoteAccountService).to receive(:new).and_return(service) - results = subject.call('newuser@remote.com', 10, false) + results = subject.call('newuser@remote.com', 10, nil, resolve: false) expect(service).not_to have_received(:call) end end diff --git a/spec/services/post_status_service_spec.rb b/spec/services/post_status_service_spec.rb index 91902ff69..92fbc73cd 100644 --- a/spec/services/post_status_service_spec.rb +++ b/spec/services/post_status_service_spec.rb @@ -182,7 +182,7 @@ RSpec.describe PostStatusService do expect(status2.id).to eq status1.id end - def create_status_with_options(options = {}) + def create_status_with_options(**options) subject.call(Fabricate(:account), 'test', nil, options) end end diff --git a/spec/services/search_service_spec.rb b/spec/services/search_service_spec.rb index 00475c699..3ffcc389b 100644 --- a/spec/services/search_service_spec.rb +++ b/spec/services/search_service_spec.rb @@ -68,7 +68,7 @@ describe SearchService do allow(AccountSearchService).to receive(:new).and_return(service) results = subject.call(query, 10) - expect(service).to have_received(:call).with(query, 10, false, nil) + expect(service).to have_received(:call).with(query, 10, nil, resolve: false) expect(results).to eq empty_results.merge(accounts: [account]) end end |