Merge pull request #465 from wordpress-mobile/update/buildkite-test-collector

Update `buildkite-test_collector` to `2.2.0`

Author: spencertransier

CHANGELOG.md

@@ -23,6 +23,7 @@ _None_
 - Add the branch information to the 'This is not a release branch' error that's thrown from complete code freeze lane. [#461]
 - Update `octokit` to `5.6.1` This is a major version bump from version `4.18`, but is not a breaking change for the Release Toolkit because it doesn't change any public APIs for clients. [#464]
 - Update `danger` to `9.3.0`. This is an internal-only change and is not a breaking change for clients. [#464]
+- Replace `rspec-buildkite-analytics` with `buildkite-test_collector` (Buildkite renamed the gem) and update it to `2.2.0`. This is another internal-only change and is not a breaking change for clients. [#465]
 - Adds `ignore_pipeline_branch_filters=true` parameter to the API call triggering a Buildkite build [#468]
 
 ## 7.0.0

Gemfile

@@ -9,4 +9,4 @@ gem 'codecov', require: false
 gem 'webmock', require: false
 gem 'yard'
 
-gem 'rspec-buildkite-analytics', '~> 0.8.1'
+gem 'buildkite-test_collector', '~> 2.2'

Gemfile.lock

@@ -55,6 +55,8 @@ GEM
     bigdecimal (1.4.4)
     buildkit (1.5.0)
       sawyer (>= 0.6)
+    buildkite-test_collector (2.2.0)
+      activesupport (>= 4.2)
     chroma (0.2.0)
     claide (1.1.0)
     claide-plugins (0.9.2)
@@ -323,11 +325,6 @@ GEM
       rspec-core (~> 3.12.0)
       rspec-expectations (~> 3.12.0)
       rspec-mocks (~> 3.12.0)
-    rspec-buildkite-analytics (0.8.2)
-      activesupport (>= 5.2, < 8)
-      rspec-core (~> 3.10)
-      rspec-expectations (~> 3.10)
-      websocket (~> 1.2)
     rspec-core (3.12.2)
       rspec-support (~> 3.12.0)
     rspec-expectations (3.12.3)
@@ -398,7 +395,6 @@ GEM
       crack (>= 0.3.2)
       hashdiff (>= 0.4.0, < 2.0.0)
     webrick (1.8.1)
-    websocket (1.2.9)
     word_wrap (1.0.0)
     xcodeproj (1.22.0)
       CFPropertyList (>= 2.3.3, < 4.0)
@@ -417,6 +413,7 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
+  buildkite-test_collector (~> 2.1)
   bundler (~> 2.0)
   cocoapods (~> 1.10)
   codecov
@@ -427,7 +424,6 @@ DEPENDENCIES
   pry (~> 0.12.2)
   rmagick (~> 4.1)
   rspec (~> 3.8)
-  rspec-buildkite-analytics (~> 0.8.1)
   rspec_junit_formatter (~> 0.4.1)
   rubocop (~> 1.0)
   rubocop-require_tools (~> 0.1.2)

spec/spec_helper.rb

@@ -3,7 +3,7 @@
 require 'simplecov'
 require 'codecov'
 require 'webmock/rspec'
-require 'rspec/buildkite/analytics'
+require 'buildkite/test_collector'
 
 # SimpleCov.minimum_coverage 95
 SimpleCov.start
@@ -15,7 +15,7 @@
 
 # Buildkite Test Analytics
 WebMock.disable_net_connect!(allow: 'analytics-api.buildkite.com')
-RSpec::Buildkite::Analytics.configure(token: ENV['BUILDKITE_ANALYTICS_TOKEN'])
+Buildkite::TestCollector.configure(hook: :rspec)
 
 # This module is only used to check the environment is currently a testing env
 module SpecHelper