foreman_expire_hosts 6.0.0 → 6.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8d016e02f798e790c84d0ff29badd0fbfbaf122b941787842dce237f19fcb0d9
-  data.tar.gz: 361195ee1b65ac38de1bad8e9ff7ecd6172202f52d524c3729bfbff022fe0a85
+  metadata.gz: 66c086d3dc04916321804be89e8d5f027691b5aa5dc10b0cef317b7ac5d4ee9e
+  data.tar.gz: 84e16983231ab5651c03c5d56484c8645bd1c6910bd677aa7bcacefb139b06d2
 SHA512:
-  metadata.gz: 55d8ac2d1ddb0b96effb841ce5b897ef3c583b3b5bd619b337015c35a0f4744f7db57e70919ec8c946ad0a2d3a424263d1fad0e01ded37e540ca220d258158c7
-  data.tar.gz: 94c8f531186c1c55b70297e8c3905f17fb823326e3cf06c94c221aa7daa675deefc9f017161d8b0070d48164b58c8a6f8031eebf951d839990da24beeb656676
+  metadata.gz: b2990701bc9efa238f25f6a6a7def12d87e9403e4b6cf0f41b477c1d5cc16a467f2edcb211fabd724034b3c149d00d654087a42e4a8e323fd403a2030ac81492
+  data.tar.gz: 60a5e736f9e54223e5c62bcf426514e10aec9569708d3f58b672da60e41667b3537848c8fa6f00c8904c3c36aa87038304cdf44bc7d51e6a087d5c906ff3302d

data/app/helpers/concerns/foreman_expire_hosts/audits_helper_extensions.rb

@@ -4,11 +4,13 @@ module ForemanExpireHosts
 
     def destroyed_expired_host_audit_comment_in_list(audit)
       return unless audit.auditable_type.to_s == 'Host' && audit.action == 'destroy' && audit.comment.present?
+
       "<div style='color: #737373;font-size: 14px'>Comment: #{audit.comment}</div>".html_safe
     end
 
     def destroyed_expired_host_audit_comment_in_show(audit)
       return unless audit.auditable_type.to_s == 'Host' && audit.action == 'destroy' && audit.comment.present?
+
       "<tr><td>Comment</td><td>#{audit.comment}</td></tr>".html_safe
     end
   end

data/app/helpers/concerns/foreman_expire_hosts/hosts_helper_extensions.rb

@@ -16,6 +16,7 @@ module ForemanExpireHosts
 
     def host_expiry_warning_message(host)
       return nil unless host.expires?
+
       if host.expired_past_grace_period?
         message = _('This host has expired %s ago and needs to be deleted manually.') % time_ago_in_words(host.expired_on)
       elsif host.expired?

data/app/helpers/expire_hosts_mailer_helper.rb

@@ -1,6 +1,7 @@
 module ExpireHostsMailerHelper
   def relative_date(date, opts = {})
     return _('N/A') if date.blank?
+
     opts[:tense] ||= :future if Date.today < date
     opts[:tense] ||= :past if Date.today > date
 

data/app/mailers/expire_hosts_mailer.rb

@@ -80,6 +80,7 @@ class ExpireHostsMailer < ApplicationMailer
   def recipient_mail(recipient)
     return recipient if recipient.is_a?(Array)
     return recipient.mail if recipient.mail.present?
+
     admin_email
   end
 

data/app/models/concerns/foreman_expire_hosts/host_ext.rb

@@ -40,27 +40,32 @@ module ForemanExpireHosts
 
     def expires_today?
       return false unless expires?
+
       expired_on.to_date == Date.today
     end
 
     def expired?
       return false unless expires?
+
       expired_on.to_date < Date.today
     end
 
     def expiration_grace_period_end_date
       return nil unless expires?
+
       expired_on + Setting[:days_to_delete_after_host_expiration].to_i
     end
 
     def expired_past_grace_period?
       return false unless expires?
+
       expiration_grace_period_end_date <= Date.today
     end
 
     def pending_expiration?
       return false unless expires?
       return false if expired?
+
       expired_on - Setting['notify1_days_before_host_expiry'].to_i <= Date.today
     end
 
@@ -70,6 +75,7 @@ module ForemanExpireHosts
       return true unless User.current
       return true if Authorizer.new(User.current).can?(:edit_host_expiry, self)
       return true if self.owner_type.nil? || self.owner.nil?
+
       Setting[:can_owner_modify_host_expiry_date] &&
         ((self.owner_type == 'User' && self.owner == User.current) ||
          (self.owner_type == 'Usergroup' && self.owner.all_users.include?(User.current)))

data/app/models/host_status/expiration_status.rb

@@ -15,6 +15,7 @@ module HostStatus
       return EXPIRED if host.expired_past_grace_period?
       return IN_GRACE_PERIOD if host.expired?
       return PENDING if host.pending_expiration?
+
       OK
     end
 

data/app/overrides/add_expired_on_field_to_host_form.rb

@@ -1,6 +1,6 @@
 Deface::Override.new(
   :virtual_path => 'hosts/_form',
-  :name         => 'host_form_expired_on_field',
+  :name => 'host_form_expired_on_field',
   :insert_after => 'div#model_name',
-  :partial      => 'hosts/expired_on_field'
+  :partial => 'hosts/expired_on_field'
 )

data/app/overrides/add_expired_on_field_to_host_show.rb

@@ -1,6 +1,6 @@
 Deface::Override.new(
-  :virtual_path  => 'hosts/show',
-  :name          => 'host_expiry_waring_in_show',
+  :virtual_path => 'hosts/show',
+  :name => 'host_expiry_waring_in_show',
   :insert_before => '#host-show',
-  :partial       => 'hosts/expired_message.html.erb'
+  :partial => 'hosts/expired_message.html.erb'
 )

data/app/overrides/add_js_to_host_index.rb

@@ -1,6 +1,6 @@
 Deface::Override.new(
-  :virtual_path  => 'hosts/_list',
-  :name          => 'host_list_expiration_js',
+  :virtual_path => 'hosts/_list',
+  :name => 'host_list_expiration_js',
   :insert_before => '#confirmation-modal',
-  :text          => "<%= stylesheet 'foreman_expire_hosts/application' %><%= javascript 'foreman_expire_hosts/application' %>"
+  :text => "<%= stylesheet 'foreman_expire_hosts/application' %><%= javascript 'foreman_expire_hosts/application' %>"
 )

data/app/overrides/deleted_expired_host_comment_in_audits.rb

@@ -1,13 +1,13 @@
 Deface::Override.new(
-  :virtual_path  => 'audits/_list',
-  :name          => 'deleted_expired_host_audit_comment_in_list',
+  :virtual_path => 'audits/_list',
+  :name => 'deleted_expired_host_audit_comment_in_list',
   :insert_bottom => 'div.row div.audit-content',
-  :text          => "\n <%= destroyed_expired_host_audit_comment_in_list(audit) %>"
+  :text => "\n <%= destroyed_expired_host_audit_comment_in_list(audit) %>"
 )
 
 Deface::Override.new(
-  :virtual_path  => 'audits/show',
-  :name          => 'deleted_expired_host_audit_comment_in_show',
+  :virtual_path => 'audits/show',
+  :name => 'deleted_expired_host_audit_comment_in_show',
   :insert_bottom => 'div#tab1 table',
-  :text          => "\n <%= destroyed_expired_host_audit_comment_in_show(@audit) %>"
+  :text => "\n <%= destroyed_expired_host_audit_comment_in_show(@audit) %>"
 )

data/app/services/foreman_expire_hosts/action/base.rb

@@ -17,7 +17,10 @@ module ForemanExpireHosts
 
       def process
         hosts.each do |host|
-          if action(host)
+          result = action(host)
+          next if result.nil?
+
+          if result
             self.successful_hosts << host
           else
             self.failed_hosts << host

data/app/services/foreman_expire_hosts/action/stop_expired_hosts.rb

@@ -8,7 +8,9 @@ module ForemanExpireHosts
       end
 
       def action(host)
-        return true unless host.supports_power_and_running?
+        return false unless host.supports_power?
+        return unless host.power.ready?
+
         logger.info "Powering down expired host in grace period #{host}"
         host.power.stop
       rescue StandardError

data/app/services/foreman_expire_hosts/notification/base.rb

@@ -24,6 +24,8 @@ module ForemanExpireHosts
       end
 
       def deliver_mail_notification(recipient, hosts)
+        return true if hosts.empty?
+
         build_mail_notification(recipient, hosts).deliver_now
       rescue SocketError, Net::SMTPError => error
         message = _('Failed to deliver %{notification_name} for Hosts %{hosts}') % {
@@ -59,11 +61,13 @@ module ForemanExpireHosts
         return global_recipients if global_recipients.present?
         return [User.anonymous_admin] if host.owner.blank?
         return [host.owner] if host.owner_type == 'User'
+
         host.owner.all_users
       end
 
       def additional_recipients
         return [] if Setting[:host_expiry_email_recipients].nil?
+
         Setting[:host_expiry_email_recipients].split(',').compact.map(&:strip)
       end
     end

data/app/services/foreman_expire_hosts/ui_notifications/hosts/base.rb

@@ -6,6 +6,7 @@ module ForemanExpireHosts
 
         def create
           return add_notification unless find_notification
+
           redeliver! if redeliver?
           find_notification
         end

data/foreman_expire_hosts.gemspec

@@ -22,5 +22,5 @@ Gem::Specification.new do |s|
   s.add_dependency 'deface'
 
   s.add_development_dependency 'rdoc'
-  s.add_development_dependency 'rubocop', '0.54.0'
+  s.add_development_dependency 'rubocop', '0.59.2'
 end

data/lib/expire_hosts_notifications.rb

@@ -10,6 +10,7 @@ module ExpireHostsNotifications
 
     def deliver_expiry_warning_notification(num = 1) # notify1_days_before_expiry
       return unless [1, 2].include?(num)
+
       days_before_expiry = Setting["notify#{num}_days_before_host_expiry"].to_i
       expiry_date        = (Date.today + days_before_expiry)
       notifiable_hosts   = Host.with_expire_date(expiry_date).preload(:owner)

data/lib/foreman_expire_hosts/version.rb

@@ -1,3 +1,3 @@
 module ForemanExpireHosts
-  VERSION = '6.0.0'.freeze
+  VERSION = '6.0.1'.freeze
 end

data/test/factories/foreman_expire_hosts_factories.rb

@@ -6,26 +6,26 @@ FactoryBot.modify do
     end
 
     trait :expires_in_a_year do
-      expired_on Date.today + 365
+      expired_on { Date.today + 365 }
     end
 
     trait :expires_in_a_week do
-      expired_on Date.today + 7
+      expired_on { Date.today + 7 }
     end
 
     trait :expires_today do
       without_validation
-      expired_on Date.today
+      expired_on { Date.today }
     end
 
     trait :expired_grace do
       without_validation
-      expired_on Date.today - 1
+      expired_on { Date.today - 1 }
     end
 
     trait :expired do
       without_validation
-      expired_on Date.today - 356
+      expired_on { Date.today - 356 }
     end
   end
 end

data/test/functional/api/v2/hosts_controller_test.rb

@@ -11,7 +11,7 @@ class Api::V2::HostsControllerTest < ActionController::TestCase
     get :show, params: { :id => host.to_param }
     assert_response :success
     show_response = ActiveSupport::JSON.decode(@response.body)
-    assert !show_response.empty?
+    assert_not_empty show_response
     assert_equal host.expired_on, show_response['expired_on'].to_date
   end
 end

data/test/lib/expire_hosts_notifications_test.rb

@@ -61,17 +61,22 @@ class ExpireHostsNotificationsTest < ActiveSupport::TestCase
     context 'with additional recipients' do
       setup do
         Setting[:host_expiry_email_recipients] = 'test@example.com, test2.example.com'
-        FactoryBot.create_list(:host, 2, :expired, :owner => user)
       end
 
       test 'should deliver notification to additional recipients' do
+        FactoryBot.create_list(:host, 2, :expired, :owner => user)
         ExpireHostsNotifications.delete_expired_hosts
-        assert_equal 3, ActionMailer::Base.deliveries.count
+        assert_equal 2, ActionMailer::Base.deliveries.count
         assert_includes ActionMailer::Base.deliveries.first.subject, 'Deleted expired hosts'
         assert_includes ActionMailer::Base.deliveries.flat_map(&:to), user.mail
         assert_includes ActionMailer::Base.deliveries.flat_map(&:to), 'test@example.com'
         assert_includes ActionMailer::Base.deliveries.flat_map(&:to), 'test2.example.com'
       end
+
+      test 'should not deliver a mail notification for empty host list' do
+        ExpireHostsNotifications.delete_expired_hosts
+        assert_equal 0, ActionMailer::Base.deliveries.count
+      end
     end
   end
 
@@ -100,6 +105,12 @@ class ExpireHostsNotificationsTest < ActiveSupport::TestCase
       end
     end
 
+    test 'should send no message if host is already be stopped' do
+      power_mock.stubs(:ready?).returns(false)
+      ExpireHostsNotifications.stop_expired_hosts
+      assert_equal 0, ActionMailer::Base.deliveries.count
+    end
+
     test 'should send failure message if host cannot be stopped' do
       power_mock.expects(:stop).returns(false)
       ExpireHostsNotifications.stop_expired_hosts
@@ -114,59 +125,75 @@ class ExpireHostsNotificationsTest < ActiveSupport::TestCase
 
     setup do
       Setting['notify1_days_before_host_expiry'] = 7
-      hosts
     end
 
-    test 'should send a ui notification per host' do
-      assert_difference('blueprint.notifications.count', 2) do
+    context 'with expired hosts' do
+      setup do
+        hosts
+      end
+
+      test 'should send a ui notification per host' do
+        assert_difference('blueprint.notifications.count', 2) do
+          ExpireHostsNotifications.deliver_expiry_warning_notification
+        end
+        hosts.each do |host|
+          notification = Notification.find_by(
+            notification_blueprint_id: blueprint.id,
+            subject_id: host.id,
+            subject_type: 'Host::Base'
+          )
+          assert_equal 1, notification.notification_recipients.where(user_id: user.id).count
+        end
+      end
+
+      test 'should redisplay read ui notification' do
         ExpireHostsNotifications.deliver_expiry_warning_notification
+        notification = Notification.find_by(notification_blueprint_id: blueprint.id, subject_id: hosts.first.id)
+        assert_not_nil notification
+        assert_equal 1, NotificationRecipient.where(notification_id: notification.id).update_all(seen: true) # rubocop:disable Rails/SkipsModelValidations
+        ExpireHostsNotifications.deliver_expiry_warning_notification
+        assert_equal 1, NotificationRecipient.where(notification_id: notification.id, seen: false).count
       end
-      hosts.each do |host|
-        notification = Notification.find_by(
-          notification_blueprint_id: blueprint.id,
-          subject_id: host.id,
-          subject_type: 'Host::Base'
-        )
-        assert_equal 1, notification.notification_recipients.where(user_id: user.id).count
+
+      test 'should send a single notification' do
+        ExpireHostsNotifications.deliver_expiry_warning_notification
+        assert_equal 1, ActionMailer::Base.deliveries.count
+        assert_includes ActionMailer::Base.deliveries.first.subject, 'Expiring hosts in foreman'
       end
-    end
 
-    test 'should redisplay read ui notification' do
-      ExpireHostsNotifications.deliver_expiry_warning_notification
-      notification = Notification.find_by(notification_blueprint_id: blueprint.id, subject_id: hosts.first.id)
-      assert_not_nil notification
-      assert_equal 1, NotificationRecipient.where(notification_id: notification.id).update_all(seen: true) # rubocop:disable Rails/SkipsModelValidations
-      ExpireHostsNotifications.deliver_expiry_warning_notification
-      assert_equal 1, NotificationRecipient.where(notification_id: notification.id, seen: false).count
-    end
+      test 'should send two notifications for two users' do
+        owner2 = FactoryBot.create(:user, :with_mail)
+        FactoryBot.create(:host, :expires_in_a_week, :owner => owner2)
+        ExpireHostsNotifications.deliver_expiry_warning_notification
+        assert_equal 2, ActionMailer::Base.deliveries.count
+        assert_includes ActionMailer::Base.deliveries.first.subject, 'Expiring hosts in foreman'
+        assert_includes ActionMailer::Base.deliveries.last.subject, 'Expiring hosts in foreman'
+      end
 
-    test 'should send a single notification' do
-      ExpireHostsNotifications.deliver_expiry_warning_notification
-      assert_equal 1, ActionMailer::Base.deliveries.count
-      assert_includes ActionMailer::Base.deliveries.first.subject, 'Expiring hosts in foreman'
+      test 'should send three notifications for three users' do
+        user2 = FactoryBot.create(:user, :with_mail)
+        user3 = FactoryBot.create(:user, :with_mail, :usergroups => [usergroup])
+        FactoryBot.create(:host, :expires_in_a_week, :owner => user2)
+        FactoryBot.create(:host, :expires_in_a_week, :owner => usergroup)
+        ExpireHostsNotifications.deliver_expiry_warning_notification
+        assert_equal 3, ActionMailer::Base.deliveries.count
+        assert_includes ActionMailer::Base.deliveries.first.subject, 'Expiring hosts in foreman'
+        assert_includes ActionMailer::Base.deliveries.flat_map(&:to), user.mail
+        assert_includes ActionMailer::Base.deliveries.flat_map(&:to), user2.mail
+        assert_includes ActionMailer::Base.deliveries.flat_map(&:to), user3.mail
+        assert_equal 1, ActionMailer::Base.deliveries.flat_map(&:subject).uniq.count
+      end
     end
 
-    test 'should send two notifications for two users' do
-      owner2 = FactoryBot.create(:user, :with_mail)
-      FactoryBot.create(:host, :expires_in_a_week, :owner => owner2)
-      ExpireHostsNotifications.deliver_expiry_warning_notification
-      assert_equal 2, ActionMailer::Base.deliveries.count
-      assert_includes ActionMailer::Base.deliveries.first.subject, 'Expiring hosts in foreman'
-      assert_includes ActionMailer::Base.deliveries.last.subject, 'Expiring hosts in foreman'
-    end
+    context 'with additional recipients' do
+      setup do
+        Setting[:host_expiry_email_recipients] = 'test@example.com, test2.example.com'
+      end
 
-    test 'should send three notifications for three users' do
-      user2 = FactoryBot.create(:user, :with_mail)
-      user3 = FactoryBot.create(:user, :with_mail, :usergroups => [usergroup])
-      FactoryBot.create(:host, :expires_in_a_week, :owner => user2)
-      FactoryBot.create(:host, :expires_in_a_week, :owner => usergroup)
-      ExpireHostsNotifications.deliver_expiry_warning_notification
-      assert_equal 3, ActionMailer::Base.deliveries.count
-      assert_includes ActionMailer::Base.deliveries.first.subject, 'Expiring hosts in foreman'
-      assert_includes ActionMailer::Base.deliveries.flat_map(&:to), user.mail
-      assert_includes ActionMailer::Base.deliveries.flat_map(&:to), user2.mail
-      assert_includes ActionMailer::Base.deliveries.flat_map(&:to), user3.mail
-      assert_equal 1, ActionMailer::Base.deliveries.flat_map(&:subject).uniq.count
+      test 'should not send empty reminders' do
+        ExpireHostsNotifications.deliver_expiry_warning_notification
+        assert_equal 0, ActionMailer::Base.deliveries.count
+      end
     end
   end
 end

data/test/unit/concerns/host_extensions_test.rb

@@ -26,7 +26,7 @@ class ForemanExpireHostsHostExtTest < ActiveSupport::TestCase
 
     test 'should require expired on' do
       host = FactoryBot.build(:host)
-      refute host.valid?, "Can not be valid without expiration date: #{host.errors.messages}"
+      assert_not host.valid?, "Can not be valid without expiration date: #{host.errors.messages}"
       assert_includes host.errors.messages.keys, :expired_on
     end
   end
@@ -99,23 +99,23 @@ class ForemanExpireHostsHostExtTest < ActiveSupport::TestCase
     end
 
     test 'should not expire' do
-      refute @host.expires?
+      assert_not @host.expires?
     end
 
     test 'should not expire today' do
-      refute @host.expires_today?
+      assert_not @host.expires_today?
     end
 
     test 'should not be expired' do
-      refute @host.expired?
+      assert_not @host.expired?
     end
 
     test 'should not be expired past grace period' do
-      refute @host.expired_past_grace_period?
+      assert_not @host.expired_past_grace_period?
     end
 
     test 'should not be pending expiration' do
-      refute @host.pending_expiration?
+      assert_not @host.pending_expiration?
     end
 
     test 'should not be in any expiration scopes' do
@@ -133,7 +133,7 @@ class ForemanExpireHostsHostExtTest < ActiveSupport::TestCase
     end
 
     test 'should not expire today' do
-      refute @host.expires_today?
+      assert_not @host.expires_today?
     end
 
     test 'should be expired' do
@@ -145,7 +145,7 @@ class ForemanExpireHostsHostExtTest < ActiveSupport::TestCase
     end
 
     test 'should not be pending expiration' do
-      refute @host.pending_expiration?
+      assert_not @host.pending_expiration?
     end
 
     test 'should only exist in correct scopes' do
@@ -168,11 +168,11 @@ class ForemanExpireHostsHostExtTest < ActiveSupport::TestCase
     end
 
     test 'should not be expired' do
-      refute @host.expired?
+      assert_not @host.expired?
     end
 
     test 'should not be expired past grace period' do
-      refute @host.expired_past_grace_period?
+      assert_not @host.expired_past_grace_period?
     end
 
     test 'should be pending expiration' do
@@ -195,19 +195,19 @@ class ForemanExpireHostsHostExtTest < ActiveSupport::TestCase
     end
 
     test 'should not expire today' do
-      refute @host.expires_today?
+      assert_not @host.expires_today?
     end
 
     test 'should not be expired' do
-      refute @host.expired?
+      assert_not @host.expired?
     end
 
     test 'should not be expired past grace period' do
-      refute @host.expired_past_grace_period?
+      assert_not @host.expired_past_grace_period?
     end
 
     test 'should not be pending expiration' do
-      refute @host.pending_expiration?
+      assert_not @host.pending_expiration?
     end
 
     test 'should only exist in correct scopes' do
@@ -225,7 +225,7 @@ class ForemanExpireHostsHostExtTest < ActiveSupport::TestCase
     end
 
     test 'should not expire today' do
-      refute @host.expires_today?
+      assert_not @host.expires_today?
     end
 
     test 'should be expired' do
@@ -233,11 +233,11 @@ class ForemanExpireHostsHostExtTest < ActiveSupport::TestCase
     end
 
     test 'should not be expired past grace period' do
-      refute @host.expired_past_grace_period?
+      assert_not @host.expired_past_grace_period?
     end
 
     test 'should not be pending expiration' do
-      refute @host.pending_expiration?
+      assert_not @host.pending_expiration?
     end
 
     test 'should only exist in correct scopes' do
@@ -250,7 +250,7 @@ class ForemanExpireHostsHostExtTest < ActiveSupport::TestCase
   def exists_only_in_scopes(host, valid_scopes)
     host.save(validate: false)
     (EXPIRATION_SCOPES - valid_scopes).each do |scope|
-      refute Host::Managed.send(scope).exists?(host.id), "Host should not exist in #{scope} scope"
+      assert_not Host::Managed.send(scope).exists?(host.id), "Host should not exist in #{scope} scope"
     end
     valid_scopes.each do |scope|
       assert Host::Managed.send(scope).exists?(host.id), "Host should exist in #{scope} scope"

data/test/unit/host_status/expiration_status_test.rb

@@ -23,6 +23,6 @@ class ExpirationStatusTest < ActiveSupport::TestCase
     assert @status.relevant?
 
     @host.expired_on = nil
-    refute @status.relevant?
+    assert_not @status.relevant?
   end
 end

data/test/unit/safe_destroy_test.rb

@@ -29,7 +29,7 @@ module ForemanExpireHosts
       test 'deletes a host' do
         assert Host::Managed.find_by(id: host.id)
         assert SafeDestroy.new(host).destroy!
-        refute Host::Managed.find_by(id: host.id)
+        assert_not Host::Managed.find_by(id: host.id)
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_expire_hosts
 version: !ruby/object:Gem::Version
-  version: 6.0.0
+  version: 6.0.1
 platform: ruby
 authors:
 - Nagarjuna Rachaneni
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-03-08 00:00:00.000000000 Z
+date: 2019-07-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bootstrap-datepicker-rails
@@ -59,14 +59,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.54.0
+        version: 0.59.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.54.0
+        version: 0.59.2
 description: This Plugin will add new column expired_on to hosts to limit the lifetime
   of a host.
 email:
@@ -164,7 +164,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.7.3
+rubygems_version: 2.7.6.2
 signing_key: 
 specification_version: 4
 summary: Foreman plugin for limiting host lifetime