buildkite-test_collector 1.1.0 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0f136ef927e80dbc0b442d01addc8f66041a79824508ea19158071594e78f7b2
-  data.tar.gz: 92136cc0d244357f96e5372af85b8923678cde70eece3582b6c02437e3a06146
+  metadata.gz: 96fb0d0c08ff10d420388b2438ae199c7fbf72878d1fa5631c68b27e70f11cac
+  data.tar.gz: 78846d669900e535d1e03bd7494611dd124f2c1bae59471ad41b5b0d9f1efb1b
 SHA512:
-  metadata.gz: 3f53e8887efbb7f8ee62df7693c8061950dc77b1168e8abe784352a6e168938a8ee530c739746722b26955f22547c7bddb809e1345633afe06a9605bd2cfad2c
-  data.tar.gz: 42e3568a83bb4d37dbb815fb27320f121af273c60d968a8fa4f75688708b3994f91dce02203e1a45ebf67630e27288665029e38b591f50e349e792b0e881b112
+  metadata.gz: c7151f06cf10d7aeb296b9d8c9294f9cdab0b07fc01112176cdc28ad55f7b65d4aa51fcc21dfd179eb58bc6c69592583f58335dece7f3b46ac1e2ba7718da369
+  data.tar.gz: cb1157c4a4cfaa719d852f2ac271bd09bbd25a461b4f6a9c002063d7461e3bb06e8107342cceb77dada1cda8090c3643aa3e2827aac7355fea90b51bbae4f7f0

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 # CHANGELOG
 
+## v1.1.1
+
+- Strip CR/LF from token input #127 - @gchan
+- Allow specify run name prefix and suffix #130 - @juanitofatas
+- Improve Minitest support by using after_teardown #131 - @davidstosik
+- Avoid breaking test suite stubs `Process.clock_gettime` #132 - @juanitofatas (thanks @ChrisBR)
+
 ## v1.1.0
 
 - Remove an internal debugging change #115 — @juanitofatas

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    buildkite-test_collector (1.1.0)
+    buildkite-test_collector (1.1.1)
       activesupport (>= 5.2, < 8)
       websocket (~> 1.2)
 
@@ -17,7 +17,7 @@ GEM
     diff-lcs (1.4.4)
     i18n (1.10.0)
       concurrent-ruby (~> 1.0)
-    minitest (5.15.0)
+    minitest (5.16.1)
     rake (13.0.6)
     rspec (3.10.0)
       rspec-core (~> 3.10.0)

data/lib/buildkite/test_collector/ci.rb

@@ -39,7 +39,9 @@ class Buildkite::TestCollector::CI
       "debug" => ENV["BUILDKITE_ANALYTICS_DEBUG_ENABLED"],
       "version" => Buildkite::TestCollector::VERSION,
       "collector" => Buildkite::TestCollector::NAME,
-  }.compact
+      "run_name_prefix" => ENV["BUILDKITE_ANALYTICS_RUN_NAME_PREFIX"],
+      "run_name_suffix" => ENV["BUILDKITE_ANALYTICS_RUN_NAME_SUFFIX"],
+    }.compact
   end
 
   def generic

data/lib/buildkite/test_collector/minitest_plugin.rb

@@ -12,9 +12,7 @@ module Buildkite::TestCollector::MinitestPlugin
     Thread.current[:_buildkite_tracer] = tracer
   end
 
-  def before_teardown
-    super
-
+  def after_teardown
     tracer = Thread.current[:_buildkite_tracer]
     if !tracer.nil?
       Thread.current[:_buildkite_tracer] = nil
@@ -23,5 +21,7 @@ module Buildkite::TestCollector::MinitestPlugin
       trace = Buildkite::TestCollector::MinitestPlugin::Trace.new(self, history: tracer.history)
       Buildkite::TestCollector.uploader.traces[trace.source_location] = trace
     end
+
+    super
   end
 end

data/lib/buildkite/test_collector/tracer.rb

@@ -4,6 +4,16 @@ require "active_support/core_ext/hash/indifferent_access"
 
 module Buildkite::TestCollector
   class Tracer
+    # https://github.com/buildkite/test-collector-ruby/issues/131
+    class MonotonicTime
+      GET_TIME = Process.method(:clock_gettime)
+      private_constant :GET_TIME
+
+      def self.call
+        GET_TIME.call Process::CLOCK_MONOTONIC
+      end
+    end
+
     class Span
       attr_accessor :section, :start_at, :end_at, :detail, :children
 
@@ -28,23 +38,23 @@ module Buildkite::TestCollector
     end
 
     def initialize
-      @top = Span.new(:top, Concurrent.monotonic_time, nil, {})
+      @top = Span.new(:top, MonotonicTime.call, nil, {})
       @stack = [@top]
     end
 
     def enter(section, **detail)
-      new_entry = Span.new(section, Concurrent.monotonic_time, nil, detail)
+      new_entry = Span.new(section, MonotonicTime.call, nil, detail)
       current_span.children << new_entry
       @stack << new_entry
     end
 
     def leave
-      current_span.end_at = Concurrent.monotonic_time
+      current_span.end_at = MonotonicTime.call
       @stack.pop
     end
 
     def backfill(section, duration, **detail)
-      new_entry = Span.new(section, Concurrent.monotonic_time - duration, Concurrent.monotonic_time, detail)
+      new_entry = Span.new(section, MonotonicTime.call - duration, MonotonicTime.call, detail)
       current_span.children << new_entry
     end
 
@@ -54,7 +64,7 @@ module Buildkite::TestCollector
 
     def finalize
       raise "Stack not empty" unless @stack.size == 1
-      @top.end_at = Concurrent.monotonic_time
+      @top.end_at = MonotonicTime.call
       self
     end
 

data/lib/buildkite/test_collector/version.rb

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

data/lib/buildkite/test_collector.rb

@@ -23,7 +23,7 @@ module Buildkite
     end
 
     def self.configure(hook:, token: nil, url: nil, debug_enabled: false, tracing_enabled: true)
-      self.api_token = token || ENV["BUILDKITE_ANALYTICS_TOKEN"]
+      self.api_token = (token || ENV["BUILDKITE_ANALYTICS_TOKEN"])&.strip
       self.url = url || DEFAULT_URL
       self.debug_enabled = debug_enabled || !!(ENV["BUILDKITE_ANALYTICS_DEBUG_ENABLED"])
       self.tracing_enabled = tracing_enabled

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: buildkite-test_collector
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.1
 platform: ruby
 authors:
 - Buildkite
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-06-21 00:00:00.000000000 Z
+date: 2022-06-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport