fastly_nsq 1.15.0 → 1.17.1

data/lib/fastly_nsq/priority_thread_pool.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 class FastlyNsq::PriorityThreadPool < Concurrent::ThreadPoolExecutor
-  alias max_threads max_length
+  alias_method :max_threads, :max_length
 
   def initialize(*)
     super
@@ -15,7 +15,7 @@ class FastlyNsq::PriorityThreadPool < Concurrent::ThreadPoolExecutor
   # @!visibility private
   def ns_enqueue(*args, &task)
     if !ns_limited_queue? || @queue.size < @max_queue
-      @queue.push([task, args[1..-1]], args[0])
+      @queue.push([task, args[1..]], args[0])
       true
     else
       false
@@ -27,6 +27,6 @@ class FastlyNsq::PriorityThreadPool < Concurrent::ThreadPoolExecutor
   #
   # @!visibility private
   def ns_assign_worker(*args, &task)
-    super(args[1..-1], &task)
+    super(args[1..], &task)
   end
 end

data/lib/fastly_nsq/producer.rb

@@ -31,10 +31,10 @@ class FastlyNsq::Producer
   # @param logger [Logger] defaults to FastlyNsq.logger
   # @param connect_timeout [Integer] NSQ connection timeout in seconds
   def initialize(topic:, tls_options: nil, logger: FastlyNsq.logger, connect_timeout: DEFAULT_CONNECTION_TIMEOUT)
-    @topic           = topic
-    @tls_options     = FastlyNsq::TlsOptions.as_hash(tls_options)
+    @topic = topic
+    @tls_options = FastlyNsq::TlsOptions.as_hash(tls_options)
     @connect_timeout = connect_timeout
-    @logger          = logger
+    @logger = logger
 
     connect
   end
@@ -75,14 +75,14 @@ class FastlyNsq::Producer
 
     @connection ||= Nsq::Producer.new(
       tls_options.merge(
-        nsqlookupd:  lookupd,
-        topic:       topic,
-      ),
+        nsqlookupd: lookupd,
+        topic: topic
+      )
     )
 
     timeout_args = [connect_timeout, FastlyNsq::ConnectionFailed]
 
-    if RUBY_VERSION > '2.4.0'
+    if RUBY_VERSION > "2.4.0"
       timeout_args << "Failed connection to #{lookupd} within #{connect_timeout} seconds"
     end
 

data/lib/fastly_nsq/safe_thread.rb

@@ -3,7 +3,7 @@
 module FastlyNsq::SafeThread
   def safe_thread(name, &block)
     Thread.new do
-      Thread.current['fastly_nsq_label'] = name
+      Thread.current["fastly_nsq_label"] = name
       watchdog(name, &block)
     end
   end

data/lib/fastly_nsq/testing.rb

@@ -98,7 +98,7 @@ module FastlyNsq
       #   processor_klass.call(test_message)
       #   expect(Post.find(post_data['id']).not_to be nil
       def message(data:, meta: nil)
-        test_message = FastlyNsq::TestMessage.new(JSON.dump('data' => data, 'meta' => meta))
+        test_message = FastlyNsq::TestMessage.new(JSON.dump("data" => data, "meta" => meta))
         FastlyNsq::Message.new(test_message)
       end
     end
@@ -121,7 +121,7 @@ module FastlyNsq
 
     def initialize(raw_body)
       @raw_body = raw_body
-      @id       = Digest::SHA1.hexdigest(raw_body.to_s + Time.now.to_s)
+      @id = Digest::SHA1.hexdigest(raw_body.to_s + Time.now.to_s)
       @attempts = 0
     end
 
@@ -249,6 +249,11 @@ module FastlyNsq
 
   FastlyNsq::Listener.prepend(ListenerTesting)
 
+  class FakeConnection
+    def connected?
+    end
+  end
+
   module ConsumerTesting
     module ClassMethods
       def messages(topic = nil)
@@ -278,6 +283,11 @@ module FastlyNsq
       @connected
     end
 
+    def connect(*args)
+      return super(*args) unless FastlyNsq::Testing.enabled?
+      @connected = FakeConnection.new
+    end
+
     def empty?
       FastlyNsq::Testing.enabled? ? messages.empty? : super
     end

data/lib/fastly_nsq/tls_options.rb

@@ -20,7 +20,7 @@ module FastlyNsq
       else
         {
           tls_v1: true,
-          tls_options: @context,
+          tls_options: @context
         }
       end
     end
@@ -28,19 +28,19 @@ module FastlyNsq
     private
 
     def env_key
-      ENV.fetch('NSQ_SSL_KEY', nil)
+      ENV.fetch("NSQ_SSL_KEY", nil)
     end
 
     def env_certificate
-      ENV.fetch('NSQ_SSL_CERTIFICATE', nil)
+      ENV.fetch("NSQ_SSL_CERTIFICATE", nil)
     end
 
     def env_ca_certificate
-      ENV.fetch('NSQ_SSL_CA_CERTIFICATE', nil)
+      ENV.fetch("NSQ_SSL_CA_CERTIFICATE", nil)
     end
 
     def verify_mode
-      ENV.fetch('NSQ_SSL_VERIFY_MODE', nil)
+      ENV.fetch("NSQ_SSL_VERIFY_MODE", nil)
     end
 
     def env_default_hash
@@ -48,7 +48,7 @@ module FastlyNsq
         key: env_key,
         certificate: env_certificate,
         ca_certificate: env_ca_certificate,
-        verify_mode: verify_mode,
+        verify_mode: verify_mode
       }
     end
 

data/lib/fastly_nsq/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module FastlyNsq
-  VERSION = '1.15.0'
+  VERSION = "1.17.1"
 end

data/lib/fastly_nsq.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
-require 'nsq'
-require 'concurrent'
-require 'fc'
-require 'set'
-require 'logger'
-require 'forwardable'
-require 'digest/md5'
+require "nsq"
+require "concurrent"
+require "fc"
+require "set"
+require "logger"
+require "forwardable"
+require "digest/md5"
 
 module FastlyNsq
   NotConnectedError = Class.new(StandardError)
@@ -58,7 +58,7 @@ module FastlyNsq
     def logger
       return @logger if @logger
 
-      self.logger = Logger.new(STDERR)
+      self.logger = Logger.new($stderr)
     end
 
     ##
@@ -112,21 +112,21 @@ module FastlyNsq
     # @return [Integer]
     # @see https://nsq.io/components/nsqd.html#command-line-options
     def max_req_timeout
-      @max_req_timeout ||= ENV.fetch('MAX_REQ_TIMEOUT', 60 * 60 * 1_000).to_i
+      @max_req_timeout ||= ENV.fetch("MAX_REQ_TIMEOUT", 60 * 60 * 1_000).to_i
     end
 
     # Maximum number of threads for FastlyNsq::PriorityThreadPool
     # Default setting is 5 and can be set via ENV['MAX_PROCESSING_POOL_THREADS']
     # @return [Integer]
     def max_processing_pool_threads
-      @max_processing_pool_threads ||= ENV.fetch('MAX_PROCESSING_POOL_THREADS', 5).to_i
+      @max_processing_pool_threads ||= ENV.fetch("MAX_PROCESSING_POOL_THREADS", 5).to_i
     end
 
     ##
     # Return an array of NSQ lookupd http addresses sourced from ENV['NSQLOOKUPD_HTTP_ADDRESS']
     # @return [Array<String>] list of nsqlookupd http addresses
     def lookupd_http_addresses
-      ENV.fetch('NSQLOOKUPD_HTTP_ADDRESS').split(',').map(&:strip)
+      ENV.fetch("NSQLOOKUPD_HTTP_ADDRESS").split(",").map(&:strip)
     end
 
     # Register a block to run at a point in the lifecycle.
@@ -151,11 +151,9 @@ module FastlyNsq
     def fire_event(event)
       blocks = FastlyNsq.events.fetch(event)
       blocks.each do |block|
-        begin
-          block.call
-        rescue => e
-          logger.error "[#{event}] #{e.inspect}"
-        end
+        block.call
+      rescue => e
+        logger.error "[#{event}] #{e.inspect}"
       end
     end
 
@@ -168,16 +166,16 @@ module FastlyNsq
   end
 end
 
-require 'fastly_nsq/consumer'
-require 'fastly_nsq/feeder'
-require 'fastly_nsq/launcher'
-require 'fastly_nsq/listener'
-require 'fastly_nsq/manager'
-require 'fastly_nsq/message'
-require 'fastly_nsq/messenger'
-require 'fastly_nsq/new_relic'
-require 'fastly_nsq/priority_queue'
-require 'fastly_nsq/priority_thread_pool'
-require 'fastly_nsq/producer'
-require 'fastly_nsq/tls_options'
-require 'fastly_nsq/version'
+require "fastly_nsq/consumer"
+require "fastly_nsq/feeder"
+require "fastly_nsq/launcher"
+require "fastly_nsq/listener"
+require "fastly_nsq/manager"
+require "fastly_nsq/message"
+require "fastly_nsq/messenger"
+require "fastly_nsq/new_relic"
+require "fastly_nsq/priority_queue"
+require "fastly_nsq/priority_thread_pool"
+require "fastly_nsq/producer"
+require "fastly_nsq/tls_options"
+require "fastly_nsq/version"

data/spec/cli_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
-require 'fastly_nsq/cli'
+require "spec_helper"
+require "fastly_nsq/cli"
 
 RSpec.describe FastlyNsq::CLI do
 end

data/spec/consumer_spec.rb

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 RSpec.describe FastlyNsq::Consumer do
-  let!(:topic)   { 'fnsq' }
-  let!(:channel) { 'fnsq' }
-  let!(:queue)   { nil }
+  let!(:topic) { "fnsq" }
+  let!(:channel) { "fnsq" }
+  let!(:queue) { nil }
 
   subject { described_class.new(topic: topic, channel: channel, queue: queue) }
 
@@ -16,15 +16,15 @@ RSpec.describe FastlyNsq::Consumer do
 
   it { should be_connected }
 
-  it 'should terminate' do
+  it "should terminate" do
     expect { subject.terminate }.to change(subject, :connected?).to(false)
   end
 
-  describe 'with a specified queue' do
+  describe "with a specified queue" do
     let!(:queue) { Queue.new }
 
-    it 'passes #queue to Nsq::Consumer' do
-      message = 'foo'
+    it "passes #queue to Nsq::Consumer" do
+      message = "foo"
 
       FastlyNsq::Messenger.deliver(message: message, topic: topic)
 
@@ -41,16 +41,16 @@ RSpec.describe FastlyNsq::Consumer do
   it { should delegate(:pop).to(:connection) }
   it { should delegate(:pop_without_blocking).to(:connection) }
 
-  describe 'with a message' do
-    let(:message) { 'foo' }
+  describe "with a message" do
+    let(:message) { "foo" }
 
     before { FastlyNsq::Messenger.deliver(message: message, topic: topic) }
 
-    it 'should not be empty' do
+    it "should not be empty" do
       expect { subject }.to eventually(be_empty).within(15)
     end
 
-    describe 'that has finished' do
+    describe "that has finished" do
       before { subject.pop.finish }
 
       it { should be_empty }

data/spec/fastly_nsq_spec.rb

@@ -1,89 +1,89 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 RSpec.describe FastlyNsq do
-  describe '#configure' do
+  describe "#configure" do
     specify { expect { |b| described_class.configure(&b) }.to yield_with_args(described_class) }
   end
 
-  describe '#listen' do
+  describe "#listen" do
     let!(:default_channel) { subject.channel }
-    let!(:topic) { 'fnsq' }
+    let!(:topic) { "fnsq" }
 
-    before { subject.channel = 'fnsq' }
+    before { subject.channel = "fnsq" }
     after { subject.channel = default_channel }
 
-    it 'creates a listener' do
+    it "creates a listener" do
       expect { subject.listen topic, ->(*) {} }.to change { subject.manager.topics }.to([topic])
     end
 
-    it 'creates a listener with a specific priority' do
+    it "creates a listener with a specific priority" do
       listener = subject.listen topic, ->(*) {}, priority: 10
       expect(listener.priority).to eq(10)
     end
   end
 
-  describe '#channel=' do
+  describe "#channel=" do
     let!(:default_channel) { subject.channel }
     after { subject.channel = default_channel }
 
-    it 'allows the channel to be set and retrieved' do
+    it "allows the channel to be set and retrieved" do
       expect(subject.channel).to be_nil
-      subject.channel = 'foo'
-      expect(subject.channel).to eq('foo')
+      subject.channel = "foo"
+      expect(subject.channel).to eq("foo")
     end
   end
 
-  describe '#logger' do
+  describe "#logger" do
     let!(:default_logger) { subject.logger }
     after { subject.logger = default_logger }
 
-    it 'returns the set logger' do
+    it "returns the set logger" do
       logger = Logger.new(nil)
       subject.logger = logger
 
       expect(subject.logger).to eq logger
     end
 
-    it 'sets the default logger if none is set' do
+    it "sets the default logger if none is set" do
       subject.instance_variable_set(:@logger, nil)
       expect(subject.instance_variable_get(:@logger)).to be nil
       logger = subject.logger
 
       expect(logger).to be_instance_of(Logger)
-      expect(logger.instance_variable_get(:@logdev).dev).to eq(STDERR)
+      expect(logger.instance_variable_get(:@logdev).dev).to eq($stderr)
       expect(logger).to eq(Nsq.logger)
     end
   end
 
-  describe '#logger=' do
+  describe "#logger=" do
     let!(:default_logger) { subject.logger }
     after { subject.logger = default_logger }
 
-    it 'allows the logger to be set and retrieved' do
-      logger = Logger.new(STDOUT)
+    it "allows the logger to be set and retrieved" do
+      logger = Logger.new($stdout)
       subject.logger = logger
 
       expect(subject.logger).to eq logger
     end
 
-    it 'sets Nsq.logger' do
-      logger = Logger.new(STDOUT)
+    it "sets Nsq.logger" do
+      logger = Logger.new($stdout)
       subject.logger = logger
 
       expect(Nsq.logger).to eq logger
     end
   end
 
-  describe '#manager' do
-    it 'represents the active default manager' do
+  describe "#manager" do
+    it "represents the active default manager" do
       expect(subject.manager).not_to be_stopped
     end
   end
 
-  describe '#manager=' do
-    it 'transfers to specified manager' do
+  describe "#manager=" do
+    it "transfers to specified manager" do
       old_manager = subject.manager
       new_manager = FastlyNsq::Manager.new
 
@@ -93,17 +93,17 @@ RSpec.describe FastlyNsq do
     end
   end
 
-  describe '#lookupd_http_addresses' do
-    it 'retreives NSQLOOKUPD_HTTP_ADDRESS' do
-      expect(subject.lookupd_http_addresses).to eq(ENV['NSQLOOKUPD_HTTP_ADDRESS'].split(','))
+  describe "#lookupd_http_addresses" do
+    it "retreives NSQLOOKUPD_HTTP_ADDRESS" do
+      expect(subject.lookupd_http_addresses).to eq(ENV["NSQLOOKUPD_HTTP_ADDRESS"].split(","))
     end
   end
 
-  describe '#on' do
+  describe "#on" do
     before { FastlyNsq.events.each { |(_, v)| v.clear } }
-    after  { FastlyNsq.events.each { |(_, v)| v.clear } }
+    after { FastlyNsq.events.each { |(_, v)| v.clear } }
 
-    it 'registers callbacks for events' do
+    it "registers callbacks for events" do
       %i[startup shutdown heartbeat].each do |event|
         block = -> {}
         FastlyNsq.on(event, &block)
@@ -111,7 +111,7 @@ RSpec.describe FastlyNsq do
       end
     end
 
-    it 'limits callback registration to valid events' do
+    it "limits callback registration to valid events" do
       expect { FastlyNsq.on(:foo, &-> {}) }.to raise_error(ArgumentError, /Invalid event name/)
     end
   end

data/spec/feeder_spec.rb

@@ -1,14 +1,14 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
+require "spec_helper"
 
 RSpec.describe FastlyNsq::Feeder do
-  describe '#push' do
-    it 'sends message to processor with the specified priority' do
+  describe "#push" do
+    it "sends message to processor with the specified priority" do
       messages = []
       processor = ->(m) { messages << m }
       priority = 5
-      message = 'foo'
+      message = "foo"
 
       feeder = described_class.new(processor, priority)
 

data/spec/http/nsqd_spec.rb

@@ -1,77 +1,77 @@
 # frozen_string_literal: true
 
-require 'spec_helper'
-require 'fastly_nsq/http/nsqd'
+require "spec_helper"
+require "fastly_nsq/http/nsqd"
 
 RSpec.describe FastlyNsq::Http::Nsqd, :webmock do
-  let(:base_uri) { 'http://example.com' }
+  let(:base_uri) { "http://example.com" }
 
-  it 'makes simple get requests' do
+  it "makes simple get requests" do
     %w[ping info config/nsqlookupd_tcp_addresses].each do |api|
       url = "#{base_uri}/#{api}"
       stub_request(:get, url)
-      FastlyNsq::Http::Nsqd.send(api.tr('/', '_').to_sym, base_uri: base_uri)
+      FastlyNsq::Http::Nsqd.send(api.tr("/", "_").to_sym, base_uri: base_uri)
 
       expect(a_request(:get, url)).to have_been_requested
     end
   end
 
-  describe 'stats' do
-    it 'can fetch stats' do
+  describe "stats" do
+    it "can fetch stats" do
       url = "#{base_uri}/stats?topic=lol&channel=foo&format=json"
       stub_request(:get, url)
-      data = { topic: 'lol', channel: 'foo', format: 'json' }
+      data = {topic: "lol", channel: "foo", format: "json"}
 
-      FastlyNsq::Http::Nsqd.stats(topic: 'lol', channel: 'foo', base_uri: base_uri)
+      FastlyNsq::Http::Nsqd.stats(topic: "lol", channel: "foo", base_uri: base_uri)
 
       expect(a_request(:get, url).with(query: data)).to have_been_requested
     end
 
-    it 'raises InvaildFormatError if provided format is not in list' do
+    it "raises InvaildFormatError if provided format is not in list" do
       expect do
-        FastlyNsq::Http::Nsqd.stats(format: 'foo')
+        FastlyNsq::Http::Nsqd.stats(format: "foo")
       end.to raise_error(FastlyNsq::Http::Nsqd::InvalidFormatError)
     end
   end
 
-  it 'can publish messages' do
+  it "can publish messages" do
     url = "#{base_uri}/pub?topic=lol&defer=999"
     stub_request(:post, url)
-    data = { topic: 'lol', defer: 999 }
+    data = {topic: "lol", defer: 999}
 
-    FastlyNsq::Http::Nsqd.pub(topic: 'lol', defer: 999, message: 'SOMETHING', base_uri: base_uri)
+    FastlyNsq::Http::Nsqd.pub(topic: "lol", defer: 999, message: "SOMETHING", base_uri: base_uri)
 
-    expect(a_request(:post, url).with(query: data, body: 'SOMETHING')).to have_been_requested
+    expect(a_request(:post, url).with(query: data, body: "SOMETHING")).to have_been_requested
   end
 
-  it 'can publish multiple messages' do
+  it "can publish multiple messages" do
     url = "#{base_uri}/mpub?topic=lol&binary=false"
     stub_request(:post, url)
-    data = { topic: 'lol' }
+    data = {topic: "lol"}
     body = "ONE MESSAGE\nTWO MESSAGE\nRED MESSAGE\nBLUE MESSAGE"
 
-    FastlyNsq::Http::Nsqd.mpub(topic: 'lol', message: body, base_uri: base_uri)
+    FastlyNsq::Http::Nsqd.mpub(topic: "lol", message: body, base_uri: base_uri)
 
     expect(a_request(:post, url).with(query: data, body: body)).to have_been_requested
   end
 
-  it 'can create, delete, empty, pause and unpause topics and channels' do
+  it "can create, delete, empty, pause and unpause topics and channels" do
     verbs = %w[create delete empty pause unpause]
 
     verbs.each do |verb|
       url = "#{base_uri}/topic/#{verb}?topic=lol"
       stub_request(:post, url)
-      data = { topic: 'lol' }
+      data = {topic: "lol"}
 
-      FastlyNsq::Http::Nsqd.send("topic_#{verb}".to_sym, topic: 'lol', base_uri: base_uri)
+      FastlyNsq::Http::Nsqd.send("topic_#{verb}".to_sym, topic: "lol", base_uri: base_uri)
 
       expect(a_request(:post, url).with(query: data)).to have_been_requested
 
       url = "#{base_uri}/channel/#{verb}?topic=lol&channel=foo"
       stub_request(:post, url)
-      data = { topic: 'lol', channel: 'foo' }
+      data = {topic: "lol", channel: "foo"}
 
-      FastlyNsq::Http::Nsqd.send("channel_#{verb}".to_sym, topic: 'lol', channel: 'foo', base_uri: base_uri)
+      FastlyNsq::Http::Nsqd.send("channel_#{verb}".to_sym, topic: "lol", channel: "foo", base_uri: base_uri)
 
       expect(a_request(:post, url).with(query: data)).to have_been_requested
     end