diff options
author | Eugen Rochko <eugen@zeonfederated.com> | 2020-12-18 09:18:31 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-12-18 09:18:31 +0100 |
commit | eb35be0431b2cdd2bbf3339beb9c5a0839e1088b (patch) | |
tree | 6e95bf9c68f7ce9b18c4a5ae404b484ef670655c /spec/models | |
parent | 941ff04b03a8a3e3f03e95c108f0cfa621226fb1 (diff) |
Fix follow limit preventing re-following of a moved account (#14207)
Diffstat (limited to 'spec/models')
-rw-r--r-- | spec/models/follow_spec.rb | 2 | ||||
-rw-r--r-- | spec/models/import_spec.rb | 10 |
2 files changed, 11 insertions, 1 deletions
diff --git a/spec/models/follow_spec.rb b/spec/models/follow_spec.rb index 0c84e5e7b..e723a1ef2 100644 --- a/spec/models/follow_spec.rb +++ b/spec/models/follow_spec.rb @@ -5,7 +5,7 @@ RSpec.describe Follow, type: :model do let(:bob) { Fabricate(:account, username: 'bob') } describe 'validations' do - subject { Follow.new(account: alice, target_account: bob) } + subject { Follow.new(account: alice, target_account: bob, rate_limit: true) } it 'has a valid fabricator' do follow = Fabricate.build(:follow) diff --git a/spec/models/import_spec.rb b/spec/models/import_spec.rb index 321761166..a5eec1722 100644 --- a/spec/models/import_spec.rb +++ b/spec/models/import_spec.rb @@ -20,5 +20,15 @@ RSpec.describe Import, type: :model do import = Import.create(account: account, type: type) expect(import).to model_have_error_on_field(:data) end + + it 'is invalid with too many rows in data' do + import = Import.create(account: account, type: type, data: StringIO.new("foo@bar.com\n" * (ImportService::ROWS_PROCESSING_LIMIT + 10))) + expect(import).to model_have_error_on_field(:data) + end + + it 'is invalid when there are more rows when following limit' do + import = Import.create(account: account, type: type, data: StringIO.new("foo@bar.com\n" * (FollowLimitValidator.limit_for_account(account) + 10))) + expect(import).to model_have_error_on_field(:data) + end end end |