cleaner_test.rb in foreman-tasks-0.9.0

- old
+ new

@@ -6,30 +6,30 @@
       cleaner = ForemanTasks::Cleaner.new(:filter => 'label = "Actions::User::Create"', :after => '10d')
 
       tasks_to_delete = [FactoryGirl.create(:dynflow_task, :user_create_task),
                          FactoryGirl.create(:dynflow_task, :user_create_task)]
       tasks_to_keep   = [FactoryGirl.create(:dynflow_task, :user_create_task) do |task|
-                           task.started_at = task.ended_at = Time.now
+                           task.started_at = task.ended_at = Time.zone.now
                            task.save
                          end,
                          FactoryGirl.create(:dynflow_task, :product_create_task)]
       cleaner.delete
       ForemanTasks::Task.where(id: tasks_to_delete).must_be_empty
       ForemanTasks::Task.where(id: tasks_to_keep).order(:id).map(&:id).must_equal tasks_to_keep.map(&:id).sort
 
-      ForemanTasks.dynflow.world.persistence.
-          find_execution_plans(filters: {'uuid' => tasks_to_delete.map(&:external_id)}).size.must_equal 0
+      ForemanTasks.dynflow.world.persistence
+                  .find_execution_plans(filters: { 'uuid' => tasks_to_delete.map(&:external_id) }).size.must_equal 0
 
-      ForemanTasks.dynflow.world.persistence.
-          find_execution_plans(filters: {'uuid' => tasks_to_keep.map(&:external_id)}).size.must_equal tasks_to_keep.size
+      ForemanTasks.dynflow.world.persistence
+                  .find_execution_plans(filters: { 'uuid' => tasks_to_keep.map(&:external_id) }).size.must_equal tasks_to_keep.size
     end
 
     it 'deletes all tasks matching the filter when the time limit is not specified' do
       cleaner = ForemanTasks::Cleaner.new(:filter => 'label = "Actions::User::Create"')
       tasks_to_delete = [FactoryGirl.create(:dynflow_task, :user_create_task),
                          FactoryGirl.create(:dynflow_task, :user_create_task) do |task|
-                           task.started_at = task.ended_at = Time.now
+                           task.started_at = task.ended_at = Time.zone.now
                            task.save
                          end]
 
       tasks_to_keep   = [FactoryGirl.create(:dynflow_task, :product_create_task)]
       cleaner.delete
@@ -41,11 +41,11 @@
       cleaner = ForemanTasks::Cleaner.new(:filter => '', :after => '10d')
       tasks_to_delete = [FactoryGirl.create(:dynflow_task, :user_create_task),
                          FactoryGirl.create(:dynflow_task, :product_create_task)]
 
       tasks_to_keep   = [FactoryGirl.create(:dynflow_task, :user_create_task) do |task|
-                           task.started_at = task.ended_at = Time.now
+                           task.started_at = task.ended_at = Time.zone.now
                            task.save
                          end]
       cleaner.delete
       ForemanTasks::Task.where(id: tasks_to_delete).must_be_empty
       ForemanTasks::Task.where(id: tasks_to_keep).must_equal tasks_to_keep
@@ -55,20 +55,19 @@
       def self.cleanup_after
         '15d'
       end
     end
 
-    describe "default behaviour" do
-      it "searches for the actions that have the cleanup_after defined" do
+    describe 'default behaviour' do
+      it 'searches for the actions that have the cleanup_after defined' do
         ForemanTasks::Cleaner.stubs(:cleanup_settings => {})
         ForemanTasks::Cleaner.actions_with_default_cleanup[ActionWithCleanup].must_equal '15d'
       end
 
-      it "searches for the actions that have the cleanup_after defined" do
+      it 'searches for the actions that have the cleanup_after defined' do
         ForemanTasks::Cleaner.stubs(:cleanup_settings =>
-                                     { :actions => [{:name => ActionWithCleanup.name, :after => '5d'}]})
+                                     { :actions => [{ :name => ActionWithCleanup.name, :after => '5d' }] })
         ForemanTasks::Cleaner.actions_with_default_cleanup[ActionWithCleanup].must_equal '5d'
       end
-
     end
   end
 end