rspec-mergify 0.1.0 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9719211e5ed0a1fa3ac6f1f642ae5076bcd3aa0d3ad1dd432d7bc5740178f7c2
-  data.tar.gz: 6a2da476d401beac81bafcc30473e1a5daf792ec77faf9d8b8871846ff4994f9
+  metadata.gz: 379b00755c80dfcba26213f7747fd42f2739c39f2cc450e36123e1703b9312f4
+  data.tar.gz: 979aaf645e9f2ad882dd00f900a5168be6f5c3ff041360e79698860e7be3790d
 SHA512:
-  metadata.gz: 778948de1e1f38ac758cfada2e563bbebf9623dead6ec0fbbd89f26a52ec100cbd7ad7c1e678c04025510101146e71df1424f2be447e6efc3829dfa8f2aca8b1
-  data.tar.gz: 747f74f4ede1ede9e3186b53c98497eb9e12280e2e38b29f3ee56a1655e8e9c98e77a6e7f51dc169706c3793b830c20f9a51c390f38c07e389c0e28e883da919
+  metadata.gz: dc0c1dae123ed41a0ca6290ed79d4bcd82ec79989f8d8cf3e232e3e505592bd791d3387435d99721f5c42218839990e609d4d595ba4317e544e3b65255ed14c8
+  data.tar.gz: 1827d2eefe6a13d8efb9b6e10cc99da36e6e64b842cfee830cc872d687caabb72881b61cdffb496d74e6e8cf91ccfaf44f7bee866f31858fafc8998e23488016

data/README.md

@@ -35,6 +35,8 @@ The plugin activates automatically when running in CI (detected via the `CI` env
 | `MERGIFY_TRACEPARENT` | W3C distributed trace context | — |
 | `MERGIFY_TEST_JOB_NAME` | Mergify test job name | — |
 
+For detailed documentation, see the [official guide](https://docs.mergify.com/ci-insights/test-frameworks/rspec/).
+
 ## Development
 
 ### Prerequisites

data/lib/mergify/rspec/ci_insights.rb

@@ -8,6 +8,7 @@ require_relative 'resources/ci'
 require_relative 'resources/git'
 require_relative 'resources/github_actions'
 require_relative 'resources/jenkins'
+require_relative 'resources/buildkite'
 require_relative 'resources/mergify'
 require_relative 'resources/rspec'
 
@@ -102,6 +103,7 @@ module Mergify
           Resources::Git.detect,
           Resources::GitHubActions.detect,
           Resources::Jenkins.detect,
+          Resources::Buildkite.detect,
           Resources::Mergify.detect,
           Resources::RSpec.detect
         ]

data/lib/mergify/rspec/quarantine.rb

@@ -54,11 +54,52 @@ module Mergify
 
       private
 
-      # rubocop:disable Metrics/MethodLength,Metrics/AbcSize
       def fetch_quarantined_tests(api_url, token, owner, repo, branch_name)
         uri = URI("#{api_url}/v1/ci/#{owner}/repositories/#{repo}/quarantines")
-        uri.query = URI.encode_www_form(branch: branch_name)
+        uri.query = URI.encode_www_form(branch: branch_name, per_page: 100)
+        collected = walk_paginated_quarantines(uri, token)
+        @quarantined_tests = collected if collected
+      rescue Net::OpenTimeout, Net::ReadTimeout, Errno::ECONNREFUSED, SocketError => e
+        @init_error_msg = "Failed to connect to Mergify API: #{e.message}"
+      rescue JSON::ParserError => e
+        @init_error_msg = "Mergify API returned a malformed quarantine list: #{e.message}"
+      end
 
+      # Follows the RFC 5988 `next` link until exhausted. Returns the full list
+      # on success, or `nil` when the run was aborted (subscription missing or
+      # an error already recorded in @init_error_msg).
+      # rubocop:disable Metrics/MethodLength,Metrics/AbcSize
+      def walk_paginated_quarantines(uri, token)
+        collected = []
+        # Guard against a server returning a `next` link that loops back to a
+        # URL we have already fetched.
+        seen = Set.new
+        while uri
+          if seen.include?(uri.to_s)
+            @init_error_msg = 'Mergify API returned a cyclic `next` link, aborting.'
+            return nil
+          end
+          seen.add(uri.to_s)
+
+          response = perform_request(uri, token)
+          case response.code.to_i
+          when 200
+            data = JSON.parse(response.body)
+            collected.concat(data.fetch('quarantined_tests', []).map { |t| t['test_name'] })
+            next_url = parse_next_link(response['Link'])
+            uri = next_url ? URI(next_url) : nil
+          when 402
+            return nil
+          else
+            @init_error_msg = "Mergify API returned HTTP #{response.code}"
+            return nil
+          end
+        end
+        collected
+      end
+      # rubocop:enable Metrics/MethodLength,Metrics/AbcSize
+
+      def perform_request(uri, token)
         http = Net::HTTP.new(uri.host, uri.port)
         http.use_ssl = uri.scheme == 'https'
         http.open_timeout = 10
@@ -66,23 +107,27 @@ module Mergify
 
         request = Net::HTTP::Get.new(uri)
         request['Authorization'] = "Bearer #{token}"
+        http.request(request)
+      end
 
-        response = http.request(request)
-        handle_response(response)
-      rescue Net::OpenTimeout, Net::ReadTimeout, Errno::ECONNREFUSED, SocketError => e
-        @init_error_msg = "Failed to connect to Mergify API: #{e.message}"
+      # Parses RFC 8288 Link headers tolerantly: accepts both quoted
+      # (`rel="next"`) and token (`rel=next`) forms, and matches when `next`
+      # is one of several space-separated rel-types (`rel="next prev"`).
+      def parse_next_link(link_header)
+        return nil if link_header.nil? || link_header.empty?
+
+        link_header.split(',').each do |part|
+          match = part.strip.match(/\A<([^>]+)>\s*;\s*(.+)\z/)
+          next unless match && next_rel?(match[2])
+
+          return match[1]
+        end
+        nil
       end
-      # rubocop:enable Metrics/MethodLength,Metrics/AbcSize
 
-      def handle_response(response)
-        case response.code.to_i
-        when 200
-          data = JSON.parse(response.body)
-          @quarantined_tests = data.fetch('quarantined_tests', []).map { |t| t['test_name'] }
-        when 402
-          # No subscription — silently skip
-        else
-          @init_error_msg = "Mergify API returned HTTP #{response.code}"
+      def next_rel?(params)
+        params.scan(/rel\s*=\s*(?:"([^"]+)"|([^\s;,]+))/i).any? do |quoted, token|
+          (quoted || token).split.include?('next')
         end
       end
     end

data/lib/mergify/rspec/resources/buildkite.rb

@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+
+require 'opentelemetry-sdk'
+require_relative '../utils'
+require_relative 'git'
+
+module Mergify
+  module RSpec
+    module Resources
+      # Detects OpenTelemetry Resource attributes for Buildkite.
+      module Buildkite
+        module_function
+
+        BUILDKITE_MAPPING = {
+          'cicd.pipeline.name' => [:to_s, 'BUILDKITE_PIPELINE_SLUG'],
+          'cicd.pipeline.task.name' => [
+            :to_s,
+            lambda {
+              label = ENV.fetch('BUILDKITE_LABEL', nil)
+              label && !label.empty? ? label : ENV.fetch('BUILDKITE_STEP_KEY', nil)
+            }
+          ],
+          'cicd.pipeline.run.id' => [:to_s, 'BUILDKITE_BUILD_ID'],
+          'cicd.pipeline.run.url' => [:to_s, 'BUILDKITE_BUILD_URL'],
+          'cicd.pipeline.run.attempt' => [
+            :to_i,
+            -> { ENV.fetch('BUILDKITE_RETRY_COUNT', '0').to_i + 1 }
+          ],
+          'cicd.pipeline.runner.name' => [:to_s, 'BUILDKITE_AGENT_NAME'],
+          'vcs.ref.head.name' => [:to_s, 'BUILDKITE_BRANCH'],
+          'vcs.ref.base.name' => [:to_s, 'BUILDKITE_PULL_REQUEST_BASE_BRANCH'],
+          'vcs.ref.head.revision' => [:to_s, 'BUILDKITE_COMMIT'],
+          'vcs.repository.url.full' => [:to_s, 'BUILDKITE_REPO'],
+          'vcs.repository.name' => [
+            :to_s,
+            lambda {
+              url = ENV.fetch('BUILDKITE_REPO', nil)
+              Utils.repository_name_from_url(url) if url
+            }
+          ]
+        }.freeze
+
+        def detect
+          return OpenTelemetry::SDK::Resources::Resource.create({}) if Utils.ci_provider != :buildkite
+
+          git_attrs = Utils.get_attributes(Git::GIT_MAPPING)
+          buildkite_attrs = Utils.get_attributes(BUILDKITE_MAPPING)
+          merged = git_attrs.merge(buildkite_attrs)
+          OpenTelemetry::SDK::Resources::Resource.create(merged)
+        end
+      end
+    end
+  end
+end

data/lib/mergify/rspec/utils.rb

@@ -15,6 +15,7 @@ module Mergify
         'GITHUB_ACTIONS' => :github_actions,
         'CIRCLECI' => :circleci,
         'JENKINS_URL' => :jenkins,
+        'BUILDKITE' => :buildkite,
         '_RSPEC_MERGIFY_TEST' => :rspec_mergify_suite
       }.freeze
 
@@ -101,12 +102,7 @@ module Mergify
       # Attributes whose env var is unset or whose callable returns nil are omitted.
       def get_attributes(mapping)
         mapping.each_with_object({}) do |(attr, (cast, env_or_callable)), result|
-          value =
-            if env_or_callable.respond_to?(:call)
-              env_or_callable.call
-            else
-              ENV.fetch(env_or_callable, nil)
-            end
+          value = env_or_callable.respond_to?(:call) ? env_or_callable.call : ENV.fetch(env_or_callable, nil)
 
           next if value.nil?
           next if value.respond_to?(:empty?) && value.empty?
@@ -130,6 +126,9 @@ module Mergify
         when :circleci
           url = ENV.fetch('CIRCLE_REPOSITORY_URL', nil)
           return repository_name_from_url(url) if url
+        when :buildkite
+          url = ENV.fetch('BUILDKITE_REPO', nil)
+          return repository_name_from_url(url) if url
         when :rspec_mergify_suite
           return 'Mergifyio/rspec-mergify'
         end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rspec-mergify
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.2
 platform: ruby
 authors:
 - Mergify
@@ -67,6 +67,7 @@ files:
 - lib/mergify/rspec/flaky_detection.rb
 - lib/mergify/rspec/formatter.rb
 - lib/mergify/rspec/quarantine.rb
+- lib/mergify/rspec/resources/buildkite.rb
 - lib/mergify/rspec/resources/ci.rb
 - lib/mergify/rspec/resources/git.rb
 - lib/mergify/rspec/resources/github_actions.rb