test/unit/triggering_test.rb in foreman-tasks-9.1.0 vs test/unit/triggering_test.rb in foreman-tasks-9.1.1

@@ -1,52 +1,52 @@
 require 'foreman_tasks_test_helper'
 
 class TriggeringTest < ActiveSupport::TestCase
   describe 'validation' do
     it 'is valid when immediate' do
-      _(FactoryBot.build(:triggering)).must_be :valid?
+      assert_predicate(FactoryBot.build(:triggering), :valid?)
     end
 
     it 'is validates future execution' do
       triggering = FactoryBot.build(:triggering, :future)
-      _(triggering).must_be :valid?
+      assert_predicate(triggering, :valid?)
       triggering.start_before = triggering.start_at - 120
-      _(triggering).wont_be :valid?
+      assert_not_predicate(triggering, :valid?)
     end
 
     it 'is invalid when recurring logic is invalid' do
       triggering = FactoryBot.build(:triggering, :recurring)
-      _(triggering).must_be :valid?
+      assert_predicate(triggering, :valid?)
       triggering.recurring_logic.stubs(:valid?).returns(false)
-      _(triggering).wont_be :valid?
+      assert_not_predicate(triggering, :valid?)
     end
 
     it 'is valid when recurring logic has purpose' do
       logic = FactoryBot.build(:recurring_logic, :purpose => 'test', :state => 'active')
       triggering = FactoryBot.build(:triggering, :recurring_logic => logic, :mode => :recurring, :input_type => :cronline, :cronline => '* * * * *')
-      _(triggering).must_be :valid?
+      assert_predicate(triggering, :valid?)
     end
 
     it 'is invalid when recurring logic with given purpose exists' do
       FactoryBot.create(:recurring_logic, :purpose => 'test', :state => 'active')
       logic = FactoryBot.build(:recurring_logic, :purpose => 'test', :state => 'active')
       triggering = FactoryBot.build(:triggering, :recurring_logic => logic, :mode => :recurring, :input_type => :cronline, :cronline => '* * * * *')
-      _(triggering).wont_be :valid?
+      assert_not_predicate(triggering, :valid?)
     end
 
     it 'is valid when recurring logic with given purpose exists and is not active or disabled' do
       ['finished', 'cancelled', 'failed'].each do |item|
         FactoryBot.create(:recurring_logic, :purpose => 'test', :state => item)
       end
       logic = FactoryBot.build(:recurring_logic, :purpose => 'test')
       triggering = FactoryBot.build(:triggering, :recurring_logic => logic, :mode => :recurring, :input_type => :cronline, :cronline => '* * * * *')
-      _(triggering).must_be :valid?
+      assert_predicate(triggering, :valid?)
     end
   end
 
   it 'cannot have mode set to arbitrary value' do
     triggering = FactoryBot.build(:triggering)
-    _(triggering).must_be :valid?
-    _(proc { triggering.mode = 'bogus' }).must_raise ArgumentError
-    _(proc { triggering.mode = 27 }).must_raise ArgumentError
+    assert_predicate(triggering, :valid?)
+    assert_raises(ArgumentError) { triggering.mode = 'bogus' }
+    assert_raises(ArgumentError) { triggering.mode = 27 }
   end
 end