fix: mock the connect method when using it in net-http test case

Author: xuan-cao-swi

instrumentation/net_http/test/opentelemetry/instrumentation/net/http/instrumentation_test.rb

@@ -151,6 +151,7 @@
       end
 
       it 'does not create a span on connect when request ignored using a regexp' do
+        # this works because http://bazqux.com is reachable site; try http://asdfasdfsef.com will fail
         uri = URI.parse('http://bazqux.com')
         http = Net::HTTP.new(uri.host, uri.port)
         http.send(:connect)
@@ -169,7 +170,16 @@
       it 'creates a span on connect for a non-ignored request' do
         uri = URI.parse('http://example.com')
         http = Net::HTTP.new(uri.host, uri.port)
-        http.send(:connect)
+
+        fake_socket = Object.new
+        def fake_socket.setsockopt(*args); end
+        def fake_socket.close; end
+
+        # Replace the TCP socket creation with our fake socket
+        TCPSocket.stub(:open, fake_socket) do
+          http.send(:connect)
+        end
+
         http.send(:do_finish)
         _(exporter.finished_spans.size).must_equal 1
         _(span.name).must_equal('connect')
@@ -199,6 +209,10 @@
         OpenTelemetry::Common::Utilities.untraced do
           uri = URI.parse('http://example.com/body')
           http = Net::HTTP.new(uri.host, uri.port)
+
+          # Mock the connect
+          http.define_singleton_method(:connect) { true }
+
           http.send(:connect)
           http.send(:do_finish)
         end