<script>alert("Hello")</script>
' + end + end + + context 'contains html (xss attack)' do + let(:local_text) { %q{<img src="javascript:alert('XSS');">
' + end + end end describe '#reformat' do diff --git a/spec/lib/language_detector_spec.rb b/spec/lib/language_detector_spec.rb new file mode 100644 index 000000000..2e68ec63b --- /dev/null +++ b/spec/lib/language_detector_spec.rb @@ -0,0 +1,70 @@ +# frozen_string_literal: true +require 'rails_helper' + +describe LanguageDetector do + describe 'to_iso_s' do + it 'detects english language' do + string = 'Hello and welcome to mastodon' + result = described_class.new(string).to_iso_s + + expect(result).to eq :en + end + + it 'detects spanish language' do + string = 'Obtener un Hola y bienvenidos a Mastodon' + result = described_class.new(string).to_iso_s + + expect(result).to eq :es + end + + describe 'when language can\'t be detected' do + it 'confirm language engine cant detect' do + result = WhatLanguage.new(:all).language_iso('') + expect(result).to be_nil + end + + describe 'with an account' do + it 'uses the account locale when present' do + user = double(:user, locale: 'fr') + account = double(:account, user: user) + result = described_class.new('', account).to_iso_s + + expect(result).to eq :fr + end + + it 'uses default locale when account is present but has no locale' do + user = double(:user, locale: nil) + account = double(:accunt, user: user) + result = described_class.new('', account).to_iso_s + + expect(result).to eq :en + end + end + + describe 'with an `en` default locale' do + it 'uses the default locale' do + string = '' + result = described_class.new(string).to_iso_s + + expect(result).to eq :en + end + end + + describe 'with a non-`en` default locale' do + around(:each) do |example| + before = I18n.default_locale + I18n.default_locale = :ja + example.run + I18n.default_locale = before + end + + it 'uses the default locale' do + string = '' + result = described_class.new(string).to_iso_s + + expect(result).to eq :ja + end + end + end + end +end diff --git a/spec/models/account_spec.rb b/spec/models/account_spec.rb index c39e7986e..3edbc3253 100644 --- a/spec/models/account_spec.rb +++ b/spec/models/account_spec.rb @@ -410,6 +410,20 @@ RSpec.describe Account, type: :model do end end + describe 'by_domain_accounts' do + it 'returns accounts grouped by domain sorted by accounts' do + 2.times { Fabricate(:account, domain: 'example.com') } + Fabricate(:account, domain: 'example2.com') + + results = Account.by_domain_accounts + expect(results.length).to eq 2 + expect(results.first.domain).to eq 'example.com' + expect(results.first.accounts_count).to eq 2 + expect(results.last.domain).to eq 'example2.com' + expect(results.last.accounts_count).to eq 1 + end + end + describe 'local' do it 'returns an array of accounts who do not have a domain' do account_1 = Fabricate(:account, domain: nil) diff --git a/spec/models/report_filter_spec.rb b/spec/models/report_filter_spec.rb new file mode 100644 index 000000000..fc0b7d7b5 --- /dev/null +++ b/spec/models/report_filter_spec.rb @@ -0,0 +1,31 @@ +require 'rails_helper' + +describe ReportFilter do + describe 'with empty params' do + it 'defaults to unresolved reports list' do + filter = ReportFilter.new({}) + + expect(filter.results).to eq Report.unresolved + end + end + + describe 'with invalid params' do + it 'raises with key error' do + filter = ReportFilter.new(wrong: true) + + expect { filter.results }.to raise_error(/wrong/) + end + end + + describe 'with valid params' do + it 'combines filters on Report' do + filter = ReportFilter.new(account_id: '123', resolved: true) + + allow(Report).to receive(:where).and_return(Report.none) + allow(Report).to receive(:resolved).and_return(Report.none) + filter.results + expect(Report).to have_received(:where).with(account_id: '123') + expect(Report).to have_received(:resolved) + end + end +end diff --git a/spec/requests/link_headers_spec.rb b/spec/requests/link_headers_spec.rb new file mode 100644 index 000000000..3947806cc --- /dev/null +++ b/spec/requests/link_headers_spec.rb @@ -0,0 +1,33 @@ +# frozen_string_literal: true + +require 'rails_helper' + +describe 'Link headers' do + describe 'on the account show page' do + let(:account) { Fabricate(:account, username: 'test') } + + before do + get short_account_path(username: account) + end + + it 'contains webfinger url in link header' do + link_header = link_header_with_type('application/xrd+xml') + + expect(link_header.href).to match 'http://www.example.com/.well-known/webfinger?resource=acct%3Atest%40cb6e6126.ngrok.io' + expect(link_header.attr_pairs.first).to eq %w[rel lrdd] + end + + it 'contains atom url in link header' do + link_header = link_header_with_type('application/atom+xml') + + expect(link_header.href).to eq 'http://www.example.com/users/test.atom' + expect(link_header.attr_pairs.first).to eq %w[rel alternate] + end + + def link_header_with_type(type) + response.headers['Link'].links.find do |link| + link.attr_pairs.any? { |pair| pair == ['type', type] } + end + end + end +end diff --git a/spec/routing/accounts_routing_spec.rb b/spec/routing/accounts_routing_spec.rb new file mode 100644 index 000000000..d04cb27f0 --- /dev/null +++ b/spec/routing/accounts_routing_spec.rb @@ -0,0 +1,31 @@ +require 'rails_helper' + +describe 'Routes under accounts/' do + describe 'the route for accounts who are followers of an account' do + it 'routes to the followers action with the right username' do + expect(get('/users/name/followers')). + to route_to('follower_accounts#index', account_username: 'name') + end + end + + describe 'the route for accounts who are followed by an account' do + it 'routes to the following action with the right username' do + expect(get('/users/name/following')). + to route_to('following_accounts#index', account_username: 'name') + end + end + + describe 'the route for following an account' do + it 'routes to the follow create action with the right username' do + expect(post('/users/name/follow')). + to route_to('account_follow#create', account_username: 'name') + end + end + + describe 'the route for unfollowing an account' do + it 'routes to the unfollow create action with the right username' do + expect(post('/users/name/unfollow')). + to route_to('account_unfollow#create', account_username: 'name') + end + end +end diff --git a/spec/services/post_status_service_spec.rb b/spec/services/post_status_service_spec.rb index 0e39cd969..c9d80257f 100644 --- a/spec/services/post_status_service_spec.rb +++ b/spec/services/post_status_service_spec.rb @@ -64,6 +64,18 @@ RSpec.describe PostStatusService do expect(status.application).to eq application end + it 'creates a status with a language set' do + detector = double(to_iso_s: :en) + allow(LanguageDetector).to receive(:new).and_return(detector) + + account = Fabricate(:account) + text = 'test status text' + + subject.call(account, text) + + expect(LanguageDetector).to have_received(:new).with(text, account) + end + it 'processes mentions' do mention_service = double(:process_mentions_service) allow(mention_service).to receive(:call) diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb index aa180a387..2bc462121 100644 --- a/spec/spec_helper.rb +++ b/spec/spec_helper.rb @@ -21,10 +21,6 @@ RSpec.configure do |config| end end - config.before :each do - stub_request(:post, 'https://fcm.googleapis.com/fcm/send').to_return(status: 200, body: '') - end - config.after :suite do FileUtils.rm_rf(Dir["#{Rails.root}/spec/test_files/"]) end