fix(tests): skip vernier on Ruby head due to compilation issues#2856
Closed
fix(tests): skip vernier on Ruby head due to compilation issues#2856
Conversation
The vernier gem (v1.9.0) fails to load its native extension on Ruby head (4.1.0+1). Skip loading vernier on Ruby >= 4.1 until the gem is updated to support the development Ruby version. This fixes the test failures for Ruby head with Rack 2, 3, and 3.1. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
mjq
force-pushed
the
fix-ruby-head-vernier-issue
branch
from
299e9c1 to
0b1a0a2
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
It seemed better to me to temporarily skip vernier on Ruby head so that we still get the actual test runs, rather than have them fail upfront when vernier fails to load.
Summary
verniergem on Ruby head (>= 4.1) to fix test failuresRoot Cause
The vernier gem (v1.9.0) has a native C extension that fails to load on Ruby head:
This was causing all "Ruby head" test matrix runs to fail on master.
Solution
Updated
sentry-ruby/spec/spec_helper.rbto skip requiring vernier whenRUBY_VERSION >= "4.1".This is a temporary workaround until vernier is updated to support Ruby 4.1 development versions.
Test Plan
✅ All existing tests on Ruby 2.7 - 4.0 should continue to pass (vernier still loads)
✅ Ruby head tests should now pass (vernier is skipped)
Fixes the Ruby head test failures seen in #2855 and on master branch.
🤖 Generated with Claude Code