Prevent lazy comment fetching from failing if file gets deleted (#2661)

lib/ruby_indexer/lib/ruby_indexer/entry.rb

@@ -110,6 +110,10 @@ def comments
         else
           ""
         end
+      rescue Errno::ENOENT
+        # If the file was deleted, but the entry hasn't been removed yet (could happen due to concurrency), then we do
+        # not want to fail. Just set the comments to an empty string
+        ""
       end
     end
 

lib/ruby_indexer/test/classes_and_modules_test.rb

@@ -630,5 +630,17 @@ def test_lazy_comment_fetching_uses_correct_line_breaks_for_rendering
         its namespace nesting, and the surrounding CallNode (e.g. a method call).
       COMMENTS
     end
+
+    def test_lazy_comment_fetching_does_not_fail_if_file_gets_deleted
+      indexable = IndexablePath.new("#{Dir.pwd}/lib", "lib/ruby_lsp/does_not_exist.rb")
+
+      @index.index_single(indexable, <<~RUBY, collect_comments: false)
+        class Foo
+        end
+      RUBY
+
+      entry = @index["Foo"].first
+      assert_empty(entry.comments)
+    end
   end
 end