Do not try and reindex link checkable object that is not indexable

Author: fbacall

app/models/link_monitor.rb

@@ -50,6 +50,6 @@ def status_changed?
   def reindex_resource
     return unless TeSS::Config.solr_enabled
     return unless status_changed?
-    link_checkable.solr_index
+    link_checkable.solr_index if link_checkable&.respond_to?(:solr_index)
   end
 end

test/fixtures/external_resources.yml

@@ -15,7 +15,6 @@ biotools:
   url: https://bio.tools/tool/SR-Tesseler
   title: SR-Tesseler
 
-
 biotwools:
   source: event_with_external_resource (Event)
   url: https://bio.tools/tool/FuNeL

test/models/link_monitor_test.rb

@@ -120,4 +120,27 @@ class LinkMonitorTest < ActiveSupport::TestCase
     refute @link_monitor.failing?
     refute @link_monitor.status_changed?
   end
+
+  test 'reindex after status changing' do
+    with_settings(solr_enabled: true) do
+      e = events(:one)
+      lm = e.create_link_monitor
+      index = []
+      e.stub(:solr_index, -> { index << e }) do
+        assert_empty index
+        lm.fail!(404)
+        assert_includes index, e
+      end
+    end
+  end
+
+  test 'do not try and reindex non-solr resource' do
+    with_settings(solr_enabled: true) do
+      er = external_resources(:biotools)
+      lm = er.create_link_monitor
+      assert_nothing_raised do
+        lm.fail!(404)
+      end
+    end
+  end
 end