buildkite-test_collector 2.3.2 → 2.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0b48c795c760f5a77634bc30024c344925036c3b4f935848caf6dcaf861b1692
-  data.tar.gz: 263e4faa13017fa5f8e1465dc76f245e05b3f26f703e933c87cb547b17e5dfbd
+  metadata.gz: '096d9a186b77909ac11460179d43bb4e57fd76ebea603f0d35d105264ae65793'
+  data.tar.gz: cc16f83917301ec0e20640545a985a800064777e549f434dfb56f9152efd2482
 SHA512:
-  metadata.gz: feaa2ec084069d17991bf5fda126b6b6d01fee4a807e4295bd644a02ccf30a02e9f1c0e912e2441347aee48f36bdae7eaa46ea4fe86075ac60e0b88424199aee
-  data.tar.gz: 7c31eafe74aa8cba48e74537940968c0b322b8939b33fa1cee2fdfb05600aa5ef9fc6192fb3b3c972bf4286edc3180cd375240eac2b6be8be08a4466bbed5567
+  metadata.gz: 58375f6b3c26113dc18d5af87b785a891b70045c657621450baaa959120ae36923732cb8992942148c0cfe2eaf199ae9e5e675e7199000f8e39aa6ed4717ab3e
+  data.tar.gz: b4bd0aa37da33506ae7bfe297497230a9d87e36aea074ad3e456d32aa2f13eaf76000dc4070fd6ab6ae6dfae00faeb39680c0945b1ffc1c1d9344d93cd82de1a

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # CHANGELOG
 
+## v2.5.0
+
+- Remove ActiveSupport as a primary dependancy #212 - @pat
+
+## v2.4.0
+
+- Feature release: Output links to Test Analytic tests within the rspec log #209 - @meghan-kradolfer
+
 ## v2.3.2
 
 - Add support for Codeship #198 - @swebb

data/Gemfile.lock

@@ -1,8 +1,7 @@
 PATH
   remote: .
   specs:
-    buildkite-test_collector (2.3.2)
-      activesupport (>= 4.2)
+    buildkite-test_collector (2.5.0)
 
 GEM
   remote: https://rubygems.org/
@@ -38,6 +37,7 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
+  activesupport (>= 4.2)
   buildkite-test_collector!
   rake (~> 13.0)
   rspec (~> 3.0)

data/buildkite-test_collector.gemspec

@@ -24,8 +24,7 @@ Gem::Specification.new do |spec|
 
   spec.required_ruby_version = Gem::Requirement.new(">= 2.3.0")
 
-  spec.add_dependency "activesupport", ">= 4.2"
-
+  spec.add_development_dependency "activesupport", ">= 4.2"
   spec.add_development_dependency "rspec-core", '~> 3.10'
   spec.add_development_dependency "rspec-expectations", '~> 3.10'
 end

data/lib/buildkite/test_collector/http_client.rb

@@ -41,6 +41,20 @@ module Buildkite::TestCollector
       http.request(contact)
     end
 
+    def metadata
+      contact_uri = URI.parse("#{url}/metadata")
+
+      http = Net::HTTP.new(contact_uri.host, contact_uri.port)
+      http.use_ssl = contact_uri.scheme == "https"
+
+      contact = Net::HTTP::Get.new(contact_uri.path, {
+        "Authorization" => authorization_header,
+        "Content-Type" => "application/json"
+      })
+
+      http.request(contact)
+    end
+
     private
 
     attr :url

data/lib/buildkite/test_collector/library_hooks/rspec.rb

@@ -5,12 +5,14 @@ require "rspec/expectations"
 
 require_relative "../rspec_plugin/reporter"
 require_relative "../rspec_plugin/trace"
+require_relative "../test_links_plugin/formatter"
 
 Buildkite::TestCollector.uploader = Buildkite::TestCollector::Uploader
 
 RSpec.configure do |config|
   config.before(:suite) do
     config.add_formatter Buildkite::TestCollector::RSpecPlugin::Reporter
+    config.add_formatter Buildkite::TestCollector::TestLinksPlugin::Formatter
   end
 
   config.around(:each) do |example|

data/lib/buildkite/test_collector/minitest_plugin/trace.rb

@@ -38,7 +38,7 @@ module Buildkite::TestCollector::MinitestPlugin
         failure_reason: failure_reason,
         failure_expanded: failure_expanded,
         history: history,
-      ).with_indifferent_access.select { |_, value| !value.nil? }
+      ).select { |_, value| !value.nil? }
     end
 
     private

data/lib/buildkite/test_collector/rspec_plugin/reporter.rb

@@ -40,6 +40,10 @@ module Buildkite::TestCollector::RSpecPlugin
       RSpec::Core::MultipleExceptionError
     ]
 
+    def blank?(string)
+      string.nil? || string.strip.empty?
+    end
+
     def failure_info(notification)
       failure_expanded = []
 
@@ -73,9 +77,9 @@ module Buildkite::TestCollector::RSpecPlugin
     def format_message_lines(message_lines)
       message_lines.map! { |l| strip_diff_colors(l) }
       # the first line is sometimes blank, depending on the error reported
-      message_lines.shift if message_lines.first.blank?
+      message_lines.shift if blank?(message_lines.first)
       # the last line is sometimes blank, depending on the error reported
-      message_lines.pop if message_lines.last.blank?
+      message_lines.pop if blank?(message_lines.last)
       message_lines
     end
 

data/lib/buildkite/test_collector/rspec_plugin/trace.rb

@@ -32,7 +32,7 @@ module Buildkite::TestCollector::RSpecPlugin
         failure_reason: failure_reason,
         failure_expanded: failure_expanded,
         history: history,
-      ).with_indifferent_access.select { |_, value| !value.nil? }
+      ).select { |_, value| !value.nil? }
     end
 
     private
@@ -60,7 +60,7 @@ module Buildkite::TestCollector::RSpecPlugin
     end
 
     def shared_example?
-      example.metadata[:shared_group_inclusion_backtrace].any?
+      !example.metadata[:shared_group_inclusion_backtrace].empty?
     end
 
     def shared_example_call_location

data/lib/buildkite/test_collector/test_links_plugin/formatter.rb

@@ -0,0 +1,62 @@
+# frozen_string_literal: true
+
+module Buildkite::TestCollector::TestLinksPlugin
+  class Formatter
+    RSpec::Core::Formatters.register self, :dump_failures
+
+    def initialize(output)
+      @output = output
+    end
+
+    def dump_failures(notification)
+      # Do not display summary if no failed examples
+      return if notification.failed_examples.empty?
+
+      # Check if a Test Analytics token is set
+      return unless Buildkite::TestCollector.api_token
+
+      metadata = fetch_metadata
+
+      # return if metadata was not fetched successfully
+      return if metadata.nil?
+
+      # return if suite url is nil
+      return if metadata['suite_url'].nil?
+
+      @output << "\n\n🔥 \x1b[31mTest Analytics failures 🔥\n"
+      @output << '_____________________________'
+      @output << "\n\n"
+
+      @output << notification.failed_examples.map do |example|
+        failed_example_output(example, metadata['suite_url'])
+      end.join("\n")
+
+      @output << "\n\n"
+    end
+
+    private
+
+    def generate_scope_name_digest(scope, name)
+      Digest::SHA256.hexdigest(scope + name)
+    end
+
+    def failed_example_output(example, url)
+      scope = example.example_group.metadata[:full_description]
+      name = example.description
+      scope_name_digest = generate_scope_name_digest(scope, name)
+      test_url = "#{url}/tests/#{scope_name_digest}"
+      "🔗 \x1b[4m\x1b[37m#{%(\x1b]1339;url=#{test_url};content="#{scope} #{name}"\x07)}\x1b[m"
+    end
+
+    def fetch_metadata
+      return unless Buildkite::TestCollector.api_token
+
+      http = Buildkite::TestCollector::HTTPClient.new(Buildkite::TestCollector.url)
+      response = http.metadata
+
+      JSON.parse(response.body) if response.code == '200'
+    rescue StandardError => e
+      # We don't need to output anything here
+    end
+  end
+end

data/lib/buildkite/test_collector/tracer.rb

@@ -43,7 +43,7 @@ module Buildkite::TestCollector
           duration: end_at - start_at,
           detail: detail,
           children: children.map(&:as_hash),
-        }.with_indifferent_access
+        }
       end
     end
 

data/lib/buildkite/test_collector/version.rb

@@ -2,7 +2,7 @@
 
 module Buildkite
   module TestCollector
-    VERSION = "2.3.2"
+    VERSION = "2.5.0"
     NAME = "buildkite-test_collector"
   end
 end

data/lib/buildkite/test_collector.rb

@@ -8,14 +8,11 @@ end
 require "json"
 require "logger"
 require "net/http"
+require "openssl"
 require "time"
 require "timeout"
 require "tmpdir"
 
-require "active_support/core_ext/object/blank"
-require "active_support/core_ext/hash/indifferent_access"
-require "active_support/notifications"
-
 require_relative "test_collector/version"
 require_relative "test_collector/error"
 require_relative "test_collector/ci"
@@ -72,6 +69,10 @@ module Buildkite
       Buildkite::TestCollector::Network.configure
       Buildkite::TestCollector::Object.configure
 
+      return unless defined?(ActiveSupport)
+
+      require "active_support/notifications"
+
       ActiveSupport::Notifications.subscribe("sql.active_record") do |name, start, finish, id, payload|
         Buildkite::TestCollector::Uploader.tracer&.backfill(:sql, finish - start, **{ query: payload[:sql] })
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: buildkite-test_collector
 version: !ruby/object:Gem::Version
-  version: 2.3.2
+  version: 2.5.0
 platform: ruby
 authors:
 - Buildkite
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-09-04 00:00:00.000000000 Z
+date: 2024-02-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -17,7 +17,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '4.2'
-  type: :runtime
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
@@ -52,7 +52,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.10'
-description:
+description: 
 email:
 - support+analytics@buildkite.com
 executables: []
@@ -88,6 +88,7 @@ files:
 - lib/buildkite/test_collector/rspec_plugin/reporter.rb
 - lib/buildkite/test_collector/rspec_plugin/trace.rb
 - lib/buildkite/test_collector/session.rb
+- lib/buildkite/test_collector/test_links_plugin/formatter.rb
 - lib/buildkite/test_collector/tracer.rb
 - lib/buildkite/test_collector/uploader.rb
 - lib/buildkite/test_collector/uuid.rb
@@ -99,7 +100,7 @@ licenses:
 metadata:
   homepage_uri: https://github.com/buildkite/test-collector-ruby
   source_code_uri: https://github.com/buildkite/test-collector-ruby
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -114,8 +115,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.26
-signing_key:
+rubygems_version: 3.1.6
+signing_key: 
 specification_version: 4
 summary: Track test executions and report to Buildkite Test Analytics
 test_files: []