rspec-mergify 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a69562ff8faf56e93122d4a11c50cf94e4d328a06a57b84cbc7eff1a80dc3dc9
-  data.tar.gz: d5cdd6a938db90e73a0771a2646b0aa9f303c120ffd72e26df8c02df3a19cbbe
+  metadata.gz: 379b00755c80dfcba26213f7747fd42f2739c39f2cc450e36123e1703b9312f4
+  data.tar.gz: 979aaf645e9f2ad882dd00f900a5168be6f5c3ff041360e79698860e7be3790d
 SHA512:
-  metadata.gz: 4d6f8fdc77e89ecb4a69c3c44804b2b75ee6749cd4f0f664e903629a6216e4c1be7860a0a774df2b1c90e6eee233f18a5d1e9d07b94949b5c106972f7849cb30
-  data.tar.gz: 2228d6143a76bfba2f0e955ee6ca988d89786e0aeedb9ed2a00d2dde40678dff529f4dd276a30fc7b31f77290389da1b09e40f467021203f578fa760b84c5ee8
+  metadata.gz: dc0c1dae123ed41a0ca6290ed79d4bcd82ec79989f8d8cf3e232e3e505592bd791d3387435d99721f5c42218839990e609d4d595ba4317e544e3b65255ed14c8
+  data.tar.gz: 1827d2eefe6a13d8efb9b6e10cc99da36e6e64b842cfee830cc872d687caabb72881b61cdffb496d74e6e8cf91ccfaf44f7bee866f31858fafc8998e23488016

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
 
@@ -125,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.1
+  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