Skip to content

Commit

Permalink
squash: use compatible? predicate
Browse files Browse the repository at this point in the history
  • Loading branch information
arielvalentin committed Jan 7, 2025
1 parent f5408aa commit 8b57d6a
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@

describe '#install' do
it 'accepts argument' do
skip "#{Rdkafka::VERSION} is not supported" unless instrumentation.compatible?
_(instrumentation.install({})).must_equal(true)
instrumentation.instance_variable_set(:@installed, false)
end
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@

describe '#each' do
it 'traces each call' do
skip "#{Rdkafka::VERSION} is not supported" unless instrumentation.installed?
skip "#{Rdkafka::VERSION} is not supported" unless instrumentation.compatible?

rand_hash = SecureRandom.hex(10)
topic_name = "consumer-patch-trace-#{rand_hash}"
Expand Down Expand Up @@ -114,7 +114,7 @@
end

it 'encodes messages keys depending on input format' do
skip "#{Rdkafka::VERSION} is not supported" unless instrumentation.installed?
skip "#{Rdkafka::VERSION} is not supported" unless instrumentation.compatible?

rand_hash = SecureRandom.hex(10)
topic_name = "consumer-patch-trace-#{rand_hash}"
Expand Down Expand Up @@ -168,7 +168,7 @@

describe '#each_batch' do
it 'traces each_batch call' do
skip "#{Rdkafka::VERSION} is not supported" unless instrumentation.installed?
skip "#{Rdkafka::VERSION} is not supported" unless instrumentation.compatible?

rand_hash = SecureRandom.hex(10)
topic_name = "consumer-patch-batch-trace-#{rand_hash}"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@

describe 'tracing' do
it 'traces sync produce calls' do
skip "#{Rdkafka::VERSION} is not supported" unless instrumentation.installed?
skip "#{Rdkafka::VERSION} is not supported" unless instrumentation.compatible?
topic_name = 'producer-patch-trace'
config = { 'bootstrap.servers': "#{host}:#{port}" }

Expand Down

0 comments on commit 8b57d6a

Please sign in to comment.