Fix dependencies

Author: comandeo

gemfiles/standard.rb

@@ -41,6 +41,8 @@ def standard_dependencies
         gem 'ruby-debug-ide'
       end
     end
+
+    gem 'opentelemetry-api'
   end
 
   group :testing do

lib/mongo.rb

@@ -33,7 +33,6 @@
 autoload :CGI, 'cgi'
 
 require 'bson'
-require 'opentelemetry-api'
 
 require 'mongo/id'
 require 'mongo/bson'

lib/mongo/tracing/open_telemetry/tracer.rb

@@ -45,6 +45,7 @@ def initialize(enabled: nil, query_text_max_length: nil, otel_tracer: nil)
                      else
                        enabled
                      end
+          check_opentelemetry_loaded
           @query_text_max_length = if query_text_max_length.nil?
                                      ENV['OTEL_RUBY_INSTRUMENTATION_MONGODB_QUERY_TEXT_MAX_LENGTH'].to_i
                                    else
@@ -211,17 +212,23 @@ def transaction_map_key(session)
 
         private
 
+        def check_opentelemetry_loaded
+          return unless @enabled
+          return if defined?(::OpenTelemetry)
+
+          Logger.logger.warn('OpenTelemetry tracing for MongoDB is enabled, ' \
+                             'but the OpenTelemetry library is not loaded. ' \
+                             'Disabling tracing.')
+          @enabled = false
+        end
+
         def initialize_tracer
-          if enabled?
-            # Obtain the proper tracer from OpenTelemetry's tracer provider.
-            ::OpenTelemetry.tracer_provider.tracer(
-              'mongo-ruby-driver',
-              Mongo::VERSION
-            )
-          else
-            # No-op tracer when OpenTelemetry is not enabled.
-            ::OpenTelemetry::Trace::Tracer.new
-          end
+          return unless enabled?
+
+          ::OpenTelemetry.tracer_provider.tracer(
+            'mongo-ruby-driver',
+            Mongo::VERSION
+          )
         end
       end
     end

spec/mongo/tracing/open_telemetry/command_tracer_spec.rb

@@ -2,13 +2,16 @@
 
 require 'spec_helper'
 
+require 'opentelemetry'
+
 describe Mongo::Tracing::OpenTelemetry::CommandTracer do
   let(:otel_tracer) { double('OpenTelemetry::Trace::Tracer') }
   let(:parent_tracer) { double('Mongo::Tracing::OpenTelemetry::Tracer') }
   let(:query_text_max_length) { 0 }
   let(:command_tracer) do
     described_class.new(otel_tracer, parent_tracer, query_text_max_length: query_text_max_length)
   end
+  let(:lsid_value) { "55dcab94-2c82-445a-a7f2-5ce50213b753" }
 
   let(:connection) do
     double('Mongo::Server::Connection',
@@ -29,7 +32,7 @@
     {
       'find' => 'users',
       '$db' => 'test_db',
-      'lsid' => { 'id' => 'session-123' },
+      'lsid' => { 'id' => BSON::Binary.from_uuid(lsid_value) },
       'filter' => { 'name' => 'Alice' }
     }
   end
@@ -250,7 +253,7 @@
     end
 
     it 'includes db.mongodb.lsid' do
-      expect(subject['db.mongodb.lsid']).to eq('session-123')
+      expect(subject['db.mongodb.lsid']).to eq(lsid_value)
     end
 
     it 'does not include nil values' do
@@ -421,10 +424,10 @@
     subject { command_tracer.send(:lsid, message) }
 
     context 'with lsid present' do
-      let(:document) { { 'find' => 'users', 'lsid' => { 'id' => 'session-123' } } }
+      let(:document) { { 'find' => 'users', 'lsid' => { 'id' => BSON::Binary.from_uuid(lsid_value) } } }
 
       it 'returns the session ID' do
-        expect(subject).to eq('session-123')
+        expect(subject).to eq(lsid_value)
       end
     end
 
@@ -474,7 +477,7 @@
         {
           'find' => 'users',
           '$db' => 'test_db',
-          'lsid' => { 'id' => 'session-123' },
+          'lsid' => { 'id' => BSON::Binary.from_uuid(lsid_value) },
           'filter' => { 'name' => 'Alice' }
         }
       end

spec/mongo/tracing/open_telemetry/operation_tracer_spec.rb

@@ -2,9 +2,10 @@
 
 require 'spec_helper'
 
+require 'opentelemetry'
+
 describe Mongo::Tracing::OpenTelemetry::OperationTracer do
   # rubocop:disable RSpec/VerifiedDoubles
-  # OpenTelemetry classes may not be loaded, so we use regular doubles
   let(:otel_tracer) { double('OpenTelemetry::Trace::Tracer') }
   let(:parent_tracer) do
     instance_double(