01456fe029
* Prepare factories and associations for Users::DeleteActivity specs * Altered user associations, delete the needed associations in Users::DeleteActivity * Moved and refined user deletion spec * Removed useless commented user associations * Refactored BackupData factory * Refactored ProfilePin factory * Refactored Reaction factory * Fixed specs: Keep default _type in factories or specify it in tests * Ok, notes shouldn't be deleted on user deletion * Fixed creating reactable for specs * Specify commentable/commentable_type explicitly when creating data for tests * Fixed typo in the comments spec * Removed unnneeded space * Added aggregate_failures for testing user associations deletion * Keep feedback_messages while deleting user activities
93 lines
3.5 KiB
Ruby
93 lines
3.5 KiB
Ruby
require "rails_helper"
|
|
|
|
RSpec.describe RateLimitChecker, type: :labor do
|
|
let(:user) { create(:user) }
|
|
let(:article) { create(:article, user_id: user.id) }
|
|
|
|
describe "#limit_by_action" do
|
|
let(:rate_limit_checker) { described_class.new(user) }
|
|
|
|
it "returns false for invalid action" do
|
|
expect(rate_limit_checker.limit_by_action("random-nothing")).to eq(false)
|
|
end
|
|
|
|
context "when creating comments" do
|
|
before do
|
|
allow(SiteConfig).to receive(:rate_limit_comment_creation).and_return(1)
|
|
end
|
|
|
|
it "returns true if too many comments at once" do
|
|
create_list(:comment, 2, user_id: user.id, commentable: article)
|
|
expect(rate_limit_checker.limit_by_action("comment_creation")).to eq(true)
|
|
end
|
|
|
|
it "triggers ping admin when too many comments" do
|
|
allow(RateLimitCheckerWorker).to receive(:perform_async)
|
|
create_list(:comment, 2, user_id: user.id, commentable: article)
|
|
rate_limit_checker.limit_by_action("comment_creation")
|
|
expect(RateLimitCheckerWorker).to have_received(:perform_async).with(user.id, "comment_creation")
|
|
end
|
|
|
|
it "returns false if allowed comment" do
|
|
expect(rate_limit_checker.limit_by_action("comment_creation")).to eq(false)
|
|
end
|
|
end
|
|
|
|
it "returns true if too many published articles at once" do
|
|
allow(SiteConfig).to receive(:rate_limit_published_article_creation).and_return(1)
|
|
create_list(:article, 2, user_id: user.id, published: true)
|
|
expect(rate_limit_checker.limit_by_action("published_article_creation")).to eq(true)
|
|
end
|
|
|
|
it "returns true if a user has followed more than <daily_limit> accounts today" do
|
|
allow(rate_limit_checker).
|
|
to receive(:user_today_follow_count).
|
|
and_return(SiteConfig.rate_limit_follow_count_daily + 1)
|
|
|
|
expect(rate_limit_checker.limit_by_action("follow_account")).to eq(true)
|
|
end
|
|
|
|
it "returns false if a user's following_users_count is less than <daily_limit>" do
|
|
allow(user).
|
|
to receive(:following_users_count).
|
|
and_return(SiteConfig.rate_limit_follow_count_daily - 1)
|
|
|
|
expect(rate_limit_checker.limit_by_action("follow_account")).to eq(false)
|
|
end
|
|
|
|
it "returns false if a user has followed less than <daily_limit> accounts today" do
|
|
allow(rate_limit_checker).
|
|
to receive(:user_today_follow_count).
|
|
and_return(SiteConfig.rate_limit_image_upload + 1)
|
|
|
|
expect(rate_limit_checker.limit_by_action("follow_account")).to eq(false)
|
|
end
|
|
|
|
it "returns false if published articles limit has not been reached" do
|
|
expect(described_class.new(user).limit_by_action("published_article_creation")).to eq(false)
|
|
end
|
|
|
|
it "returns false if a user uploads too many images" do
|
|
allow(rate_limit_checker).
|
|
to receive(:track_image_uploads).
|
|
and_return(SiteConfig.rate_limit_follow_count_daily - 1)
|
|
|
|
expect(rate_limit_checker.limit_by_action("image_upload")).to eq(false)
|
|
end
|
|
end
|
|
|
|
describe "#limit_by_email_recipient_address" do
|
|
before do
|
|
allow(SiteConfig).to receive(:rate_limit_email_recipient).and_return(1)
|
|
end
|
|
|
|
it "returns true if too many emails are sent to the same recipient" do
|
|
2.times { EmailMessage.create(to: user.email, sent_at: Time.current) }
|
|
expect(described_class.new.limit_by_email_recipient_address(user.email)).to eq(true)
|
|
end
|
|
|
|
it "returns false if we are below the message limit for this recipient" do
|
|
expect(described_class.new.limit_by_email_recipient_address(user.email)).to eq(false)
|
|
end
|
|
end
|
|
end
|