test/unit/cleaner_test.rb in foreman-tasks-0.10.8 vs test/unit/cleaner_test.rb in foreman-tasks-0.10.9
- old
+ new
@@ -8,17 +8,17 @@
describe ForemanTasks::Cleaner do
it 'is able to delete tasks (including the dynflow plans) based on filter' do
cleaner = ForemanTasks::Cleaner.new(:filter => 'label = "Actions::User::Create"', :after => '10d')
- tasks_to_delete = [FactoryBot.create(:dynflow_task, :user_create_task),
- FactoryBot.create(:dynflow_task, :user_create_task)]
- tasks_to_keep = [FactoryBot.create(:dynflow_task, :user_create_task) do |task|
+ 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.zone.now
task.save
end,
- FactoryBot.create(:dynflow_task, :product_create_task)]
+ FactoryGirl.create(:dynflow_task, :product_create_task)]
cleaner.expects(:tasks_to_csv)
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
@@ -29,29 +29,29 @@
.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 = [FactoryBot.create(:dynflow_task, :user_create_task),
- FactoryBot.create(:dynflow_task, :user_create_task) do |task|
+ 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.zone.now
task.save
end]
- tasks_to_keep = [FactoryBot.create(:dynflow_task, :product_create_task)]
+ tasks_to_keep = [FactoryGirl.create(:dynflow_task, :product_create_task)]
cleaner.expects(:tasks_to_csv)
cleaner.delete
ForemanTasks::Task.where(id: tasks_to_delete).must_be_empty
ForemanTasks::Task.where(id: tasks_to_keep).must_equal tasks_to_keep
end
it 'supports passing empty filter (just delete all)' do
cleaner = ForemanTasks::Cleaner.new(:filter => '', :after => '10d')
- tasks_to_delete = [FactoryBot.create(:dynflow_task, :user_create_task),
- FactoryBot.create(:dynflow_task, :product_create_task)]
+ tasks_to_delete = [FactoryGirl.create(:dynflow_task, :user_create_task),
+ FactoryGirl.create(:dynflow_task, :product_create_task)]
- tasks_to_keep = [FactoryBot.create(:dynflow_task, :user_create_task) do |task|
+ tasks_to_keep = [FactoryGirl.create(:dynflow_task, :user_create_task) do |task|
task.started_at = task.ended_at = Time.zone.now
task.save
end]
cleaner.expects(:tasks_to_csv)
cleaner.delete
@@ -60,11 +60,11 @@
end
it 'backs tasks up before deleting' do
dir = '/tmp'
cleaner = ForemanTasks::Cleaner.new(:filter => '', :after => '10d', :backup_dir => dir)
- tasks_to_delete = [FactoryBot.create(:dynflow_task, :user_create_task),
- FactoryBot.create(:dynflow_task, :product_create_task)]
+ tasks_to_delete = [FactoryGirl.create(:dynflow_task, :user_create_task),
+ FactoryGirl.create(:dynflow_task, :product_create_task)]
r, w = IO.pipe
cleaner.expects(:with_backup_file)
.with(dir, 'foreman_tasks.csv')
.yields(w, false)