scraper_event_check test

lib/scraper.rb

@@ -168,15 +168,17 @@ def scrape(source, user = get_user, index: 0)
     end
   end
 
-  def scraper_event_check(_data_sources)
-    return unless TeSS::Config&.scraper_event_check&.[]('enabled')
+  def scraper_event_check(data_sources)
+    result = false
+    return result unless TeSS::Config&.scraper_event_check&.[]('enabled')
 
     data_sources.each do |_key, sources|
       sources.each do |source|
-        event_check_rejected(source)
-        event_check_stale(source)
+        result ||= event_check_rejected(source)
+        result ||= event_check_stale(source)
       end
     end
+    result
   end
 
   def event_check_stale(source)

test/unit/ingestors/scraper_test.rb

@@ -310,27 +310,27 @@ def set_up_event_check
     freeze_time(2019) do
       set_up_event_check
       with_settings({ scraper_event_check: { enabled: true, stale_threshold: 0.3 } }) do
-        assert_not @scraper.event_check_stale(@source)
+        assert_not @scraper.scraper_event_check({config: [@source]})
         @source.content_provider.events.each do |event|
           event.last_scraped = 10.days.freeze.ago
           event.timezone = 'Amsterdam'
           event.save!
           event.reload
         end
-        assert @scraper.event_check_stale(@source)
+        assert @scraper.scraper_event_check({config: [@source]})
       end
     end
   end
 
   test 'event_check_rejected' do
     set_up_event_check
     with_settings({ scraper_event_check: { enabled: true, rejected_threshold: 0.3 } }) do
-      assert_not @scraper.event_check_rejected(@source)
+      assert_not @scraper.scraper_event_check({config: [@source]})
       @source.records_written = 10
       @source.resources_rejected = 90
       @source.save!
       @source.reload
-      assert @scraper.event_check_rejected(@source)
+      assert @scraper.scraper_event_check({config: [@source]})
     end
   end