From e1e41a962eb6fb20f4e9efac18f5c7a41edf326c Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 23 Nov 2016 21:26:49 +0200 Subject: [PATCH] Remove unneeded log #186 --- app/models/domain_cron.rb | 3 +-- spec/models/domain_cron_spec.rb | 11 ----------- 2 files changed, 1 insertion(+), 13 deletions(-) diff --git a/app/models/domain_cron.rb b/app/models/domain_cron.rb index c3a698fbc..b4da056a2 100644 --- a/app/models/domain_cron.rb +++ b/app/models/domain_cron.rb @@ -32,12 +32,11 @@ class DomainCron end def self.start_expire_period - Rails.logger.info('Expiring domains') - ::PaperTrail.whodunnit = "cron - #{__method__}" domains = Domain.expired marked = 0 real = 0 + domains.each do |domain| next unless domain.expirable? real += 1 diff --git a/spec/models/domain_cron_spec.rb b/spec/models/domain_cron_spec.rb index a9c3e2d2f..50c4ca80a 100644 --- a/spec/models/domain_cron_spec.rb +++ b/spec/models/domain_cron_spec.rb @@ -38,15 +38,4 @@ RSpec.describe DomainCron do @domain.reload @domain.statuses.include?(DomainStatus::EXPIRED).should == true end - - describe '::start_expire_period', db: false do - before :example do - travel_to Time.zone.parse('05.07.2010') - end - - it 'logs start time' do - expect(Rails.logger).to receive(:info).with('Expiring domains') - described_class.start_expire_period - end - end end