Never required class is being used

[sergio-bobillier]

This issue refers to the file lib/elastic/transport/transport/errors.rb On line 27 the file declares the following class:

# Reloading connections timeout (1 sec by default)
#
class SnifferTimeoutError < Timeout::Error; end

However, the file never requires timeout which means that if nothing else is requiring timeout the following error happens:

NameError: uninitialized constant Elasticsearch::Transport::Transport::Timeout
from /usr/share/.rbenv/versions/2.7.7/lib/ruby/gems/2.7.0/gems/elasticsearch-transport-7.9.0/lib/elasticsearch/transport/transport/errors.rb:28:in `<module:Transport>'

In our case, this happens because we have a small class to handle Elasticsearch errors, so we are directly requiring elasticsearch/transport/transport/errors and hence we are getting this error. Mostly when executing a single spec file instead of the whole test suite.

This is not a particularly nasty issue, but classes should, in general, require other classes that they are using.

NOTE: Note that here the error shows elasticsearch-transport instead of elastic-transport. The issue was there before the repository was split, and it is still there in the new repository.