rspec-buildkite-analytics 0.3.4 → 0.3.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 23cd65346e813fe92bc69f52607c1dd580666879ca6ccc28b89fa277cff496e2
-  data.tar.gz: 3f1bc2b37f5e926d144b38b28a4406f0604e6458d8d288c32a6500869eea1258
+  metadata.gz: d55751f8e88a6ac7ad250c0c1e2b382dcaeb2d23504aa6d110a22a4a61af8d06
+  data.tar.gz: b21d3d7fe03e1d504870b2d1fba0f188ab94e1f5e1c5c3fa3f11b1536a9f99b1
 SHA512:
-  metadata.gz: c08b2ae468dda9fe8aabb7e52088eb2447243d4022f33402086e80968c1ef49382e2aa0aad4a3c596b4fb4337ce6b89ecc059862786393e08de8aa039de6230d
-  data.tar.gz: f7ed799c14eeaa6720b62e9ac25b9ea1c550019f776cf6a9c4437abdc9100e0ccc030ee97c3339b02f2e19fc3fa6518e7bf1977afdaf59bc97bfd495a9a1d94a
+  metadata.gz: 8bb6efbdc7f9b4f06ab3b0793c9ef5a1c2b41429a755040d0c6b558fbb24b308e59cd862a646b86717bb5bebf4464eec45e3551db043b8332ed37a94b642f789
+  data.tar.gz: 97089293de5331c1329979fd853ff15297e4460aaec7631867e2031398a31603296e53174da6a398134fe7a54bfa9b1e7082817acb989e6f74978a574afae9fc

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    rspec-buildkite-analytics (0.3.4)
+    rspec-buildkite-analytics (0.3.5)
       activesupport (>= 5.2, <= 7.0)
       rspec-core (~> 3.10)
       rspec-expectations (~> 3.10)
@@ -38,7 +38,7 @@ GEM
     tzinfo (2.0.4)
       concurrent-ruby (~> 1.0)
     websocket (1.2.9)
-    zeitwerk (2.4.2)
+    zeitwerk (2.5.1)
 
 PLATFORMS
   ruby

data/lib/rspec/buildkite/analytics/session.rb

@@ -10,6 +10,7 @@ module RSpec::Buildkite::Analytics
     WAIT_BETWEEN_RECONNECTIONS = 5
 
     class RejectedSubscription < StandardError; end
+    class InitialConnectionFailure < StandardError; end
 
     def initialize(url, authorization_header, channel)
       @queue = Queue.new
@@ -25,8 +26,8 @@ module RSpec::Buildkite::Analytics
       @authorization_header = authorization_header
 
       connect
-    rescue TimeoutError => e
-      $stderr.puts "rspec-buildkite-analytics could not establish an initial connection with Buildkite. Please contact support."
+    rescue TimeoutError, InitialConnectionFailure => e
+      $stderr.puts "rspec-buildkite-analytics could not establish an initial connection with Buildkite due to #{e.message}. You may be missing some data for this test suite, please contact support."
     end
 
     def disconnected(connection)
@@ -138,25 +139,25 @@ module RSpec::Buildkite::Analytics
       wait_for_confirm
     end
 
-    def pop_with_timeout
-      Timeout.timeout(30, RSpec::Buildkite::Analytics::TimeoutError, "Waited 30 seconds") do
+    def pop_with_timeout(message_type)
+      Timeout.timeout(30, RSpec::Buildkite::Analytics::TimeoutError, "Timeout: Waited 30 seconds for #{message_type}") do
         @queue.pop
       end
     end
 
     def wait_for_welcome
-      welcome = pop_with_timeout
+      welcome = pop_with_timeout("welcome")
 
       if welcome && welcome != { "type" => "welcome" }
-        raise "Not a welcome: #{welcome.inspect}"
+        raise InitialConnectionFailure.new("Wrong message received, expected a welcome, but received: #{welcome.inspect}")
       end
     end
 
     def wait_for_confirm
-      confirm = pop_with_timeout
+      confirm = pop_with_timeout("confirm")
 
       if confirm && confirm != { "type" => "confirm_subscription", "identifier" => @channel }
-        raise "Not a confirm: #{confirm.inspect}"
+        raise InitialConnectionFailure.new("Wrong message received, expected a confirm, but received: #{confirm.inspect}")
       end
     end
 

data/lib/rspec/buildkite/analytics/socket_connection.rb

@@ -82,6 +82,16 @@ module RSpec::Buildkite::Analytics
         end
       rescue IOError
         # This is fine to ignore
+      rescue IndexError
+        # I don't like that we're doing this but I think it's the best of the options
+        #
+        # This relates to this issue https://github.com/ruby/openssl/issues/452
+        # A fix for it has been released but the repercussions of overriding
+        # the OpenSSL version in the stdlib seem worse than catching this error here.
+        if @socket
+          @session.disconnected(self)
+          disconnect
+        end
       end
     end
 
@@ -96,6 +106,16 @@ module RSpec::Buildkite::Analytics
       return unless @socket
       @session.disconnected(self)
       disconnect
+    rescue IndexError
+      # I don't like that we're doing this but I think it's the best of the options
+      #
+      # This relates to this issue https://github.com/ruby/openssl/issues/452
+      # A fix for it has been released but the repercussions of overriding
+      # the OpenSSL version in the stdlib seem worse than catching this error here.
+      if @socket
+        @session.disconnected(self)
+        disconnect
+      end
     end
 
     def close

data/lib/rspec/buildkite/analytics/version.rb

@@ -3,7 +3,7 @@
 module RSpec
   module Buildkite
     module Analytics
-      VERSION = "0.3.4"
+      VERSION = "0.3.5"
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rspec-buildkite-analytics
 version: !ruby/object:Gem::Version
-  version: 0.3.4
+  version: 0.3.5
 platform: ruby
 authors:
 - Buildkite
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-07 00:00:00.000000000 Z
+date: 2021-10-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -121,7 +121,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.22
+rubygems_version: 3.1.4
 signing_key:
 specification_version: 4
 summary: Track execution of specs and report to Buildkite Analytics