mirror of
https://github.com/discourse/discourse.git
synced 2024-11-24 21:35:48 +08:00
4ea21fa2d0
This change both speeds up specs (less strings to allocate) and helps catch cases where methods in Discourse are mutating inputs. Overall we will be migrating everything to use #frozen_string_literal: true it will take a while, but this is the first and safest move in this direction
46 lines
1.2 KiB
Ruby
46 lines
1.2 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
require 'rails_helper'
|
|
|
|
RSpec.describe Jobs::CleanUpInactiveUsers do
|
|
it "should clean up new users that have been inactive" do
|
|
SiteSetting.clean_up_inactive_users_after_days = 0
|
|
|
|
user = Fabricate(:user,
|
|
last_seen_at: 5.days.ago,
|
|
trust_level: TrustLevel.levels[:newuser]
|
|
)
|
|
|
|
Fabricate(:active_user)
|
|
|
|
Fabricate(:post, user: Fabricate(:user,
|
|
trust_level: TrustLevel.levels[:newuser],
|
|
last_seen_at: 5.days.ago
|
|
)).user
|
|
|
|
Fabricate(:user,
|
|
trust_level: TrustLevel.levels[:newuser],
|
|
last_seen_at: 2.days.ago
|
|
)
|
|
|
|
Fabricate(:user, trust_level: TrustLevel.levels[:basic])
|
|
|
|
expect { described_class.new.execute({}) }.to_not change { User.count }
|
|
|
|
SiteSetting.clean_up_inactive_users_after_days = 4
|
|
|
|
expect { described_class.new.execute({}) }
|
|
.to change { User.count }.by(-1)
|
|
|
|
expect(User.exists?(id: user.id)).to eq(false)
|
|
end
|
|
|
|
it "doesn't delete inactive admins" do
|
|
SiteSetting.clean_up_inactive_users_after_days = 4
|
|
admin = Fabricate(:admin, last_seen_at: 5.days.ago, trust_level: TrustLevel.levels[:newuser])
|
|
|
|
expect { described_class.new.execute({}) }.to_not change { User.count }
|
|
expect(User.exists?(admin.id)).to eq(true)
|
|
end
|
|
end
|