statsd-instrument 3.0.0 → 3.0.1

data/lib/statsd/instrument/helpers.rb

@@ -1,11 +1,15 @@
 # frozen_string_literal: true
 
-module StatsD::Instrument::Helpers
-  def capture_statsd_datagrams(client: nil, &block)
-    client ||= StatsD.singleton_client
-    client.capture(&block)
-  end
+module StatsD
+  module Instrument
+    module Helpers
+      def capture_statsd_datagrams(client: nil, &block)
+        client ||= StatsD.singleton_client
+        client.capture(&block)
+      end
 
-  # For backwards compatibility
-  alias_method :capture_statsd_calls, :capture_statsd_datagrams
+      # For backwards compatibility
+      alias_method :capture_statsd_calls, :capture_statsd_datagrams
+    end
+  end
 end

data/lib/statsd/instrument/log_sink.rb

@@ -1,24 +1,28 @@
 # frozen_string_literal: true
 
-# @note This class is part of the new Client implementation that is intended
-#   to become the new default in the next major release of this library.
-class StatsD::Instrument::LogSink
-  attr_reader :logger, :severity
+module StatsD
+  module Instrument
+    # @note This class is part of the new Client implementation that is intended
+    #   to become the new default in the next major release of this library.
+    class LogSink
+      attr_reader :logger, :severity
 
-  def initialize(logger, severity: Logger::DEBUG)
-    @logger = logger
-    @severity = severity
-  end
+      def initialize(logger, severity: Logger::DEBUG)
+        @logger = logger
+        @severity = severity
+      end
 
-  def sample?(_sample_rate)
-    true
-  end
+      def sample?(_sample_rate)
+        true
+      end
 
-  def <<(datagram)
-    # Some implementations require a newline at the end of datagrams.
-    # When logging, we make sure those newlines are removed using chomp.
+      def <<(datagram)
+        # Some implementations require a newline at the end of datagrams.
+        # When logging, we make sure those newlines are removed using chomp.
 
-    logger.add(severity, "[StatsD] #{datagram.chomp}")
-    self
+        logger.add(severity, "[StatsD] #{datagram.chomp}")
+        self
+      end
+    end
   end
 end

data/lib/statsd/instrument/matchers.rb

@@ -3,97 +3,113 @@
 require 'rspec/expectations'
 require 'rspec/core/version'
 
-module StatsD::Instrument::Matchers
-  CUSTOM_MATCHERS = {
-    increment: :c,
-    measure: :ms,
-    gauge: :g,
-    histogram: :h,
-    distribution: :d,
-    set: :s,
-  }
-
-  class Matcher
-    include RSpec::Matchers::Composable if RSpec::Core::Version::STRING.start_with?('3')
-    include StatsD::Instrument::Helpers
-
-    def initialize(metric_type, metric_name, options = {})
-      @metric_type = metric_type
-      @metric_name = metric_name
-      @options = options
-    end
+module StatsD
+  module Instrument
+    module Matchers
+      class Matcher
+        include(RSpec::Matchers::Composable) if RSpec::Core::Version::STRING.start_with?('3')
+        include StatsD::Instrument::Helpers
+
+        def initialize(metric_type, metric_name, options = {})
+          @metric_type = metric_type
+          @metric_name = metric_name
+          @options = options
+        end
 
-    def matches?(block)
-      expect_statsd_call(@metric_type, @metric_name, @options, &block)
-    rescue RSpec::Expectations::ExpectationNotMetError => e
-      @message = e.message
-      false
-    end
+        def matches?(block)
+          expect_statsd_call(@metric_type, @metric_name, @options, &block)
+        rescue RSpec::Expectations::ExpectationNotMetError => e
+          @message = e.message
+          false
+        end
 
-    def failure_message
-      @message
-    end
+        def failure_message
+          @message
+        end
 
-    def failure_message_when_negated
-      "No StatsD calls for metric #{@metric_name} expected."
-    end
+        def failure_message_when_negated
+          "No StatsD calls for metric #{@metric_name} expected."
+        end
 
-    def supports_block_expectations?
-      true
-    end
+        def supports_block_expectations?
+          true
+        end
 
-    private
+        private
 
-    def expect_statsd_call(metric_type, metric_name, options, &block)
-      metrics = capture_statsd_calls(&block)
-      metrics = metrics.select { |m| m.type == metric_type && m.name == metric_name }
+        def expect_statsd_call(metric_type, metric_name, options, &block)
+          metrics = capture_statsd_calls(&block)
+          metrics = metrics.select { |m| m.type == metric_type && m.name == metric_name }
 
-      if metrics.empty?
-        raise RSpec::Expectations::ExpectationNotMetError, "No StatsD calls for metric #{metric_name} were made."
-      elsif options[:times] && options[:times] != metrics.length
-        raise RSpec::Expectations::ExpectationNotMetError, "The numbers of StatsD calls for metric #{metric_name} " \
-         "was unexpected. Expected #{options[:times].inspect}, got #{metrics.length}"
-      end
+          if metrics.empty?
+            raise RSpec::Expectations::ExpectationNotMetError, "No StatsD calls for metric #{metric_name} were made."
+          elsif options[:times] && options[:times] != metrics.length
+            raise RSpec::Expectations::ExpectationNotMetError, "The numbers of StatsD calls for metric " \
+             "#{metric_name} was unexpected. Expected #{options[:times].inspect}, got #{metrics.length}"
+          end
+
+          [:sample_rate, :value, :tags].each do |expectation|
+            next unless options[expectation]
+
+            num_matches = metrics.count do |m|
+              matcher = RSpec::Matchers::BuiltIn::Match.new(options[expectation])
+              matcher.matches?(m.public_send(expectation))
+            end
 
-      [:sample_rate, :value, :tags].each do |expectation|
-        next unless options[expectation]
+            found = options[:times] ? num_matches == options[:times] : num_matches > 0
 
-        num_matches = metrics.count do |m|
-          matcher = RSpec::Matchers::BuiltIn::Match.new(options[expectation])
-          matcher.matches?(m.public_send(expectation))
+            unless found
+              message = metric_information(metric_name, options, metrics, expectation)
+              raise RSpec::Expectations::ExpectationNotMetError, message
+            end
+          end
+
+          true
         end
 
-        found = options[:times] ? num_matches == options[:times] : num_matches > 0
+        def metric_information(metric_name, options, metrics, expectation)
+          message = "expected StatsD #{expectation.inspect} for metric '#{metric_name}' to be called"
+
+          message += "\n  "
+          message += options[:times] ? "exactly #{options[:times]} times" : "at least once"
+          message += " with: #{options[expectation]}"
 
-        unless found
-          message = metric_information(metric_name, options, metrics, expectation)
-          raise RSpec::Expectations::ExpectationNotMetError, message
+          message += "\n  captured metric values: #{metrics.map(&expectation).join(', ')}"
+
+          message
         end
       end
 
-      true
-    end
+      Increment = Class.new(Matcher)
+      Measure = Class.new(Matcher)
+      Gauge = Class.new(Matcher)
+      Set = Class.new(Matcher)
+      Histogram = Class.new(Matcher)
+      Distribution = Class.new(Matcher)
 
-    def metric_information(metric_name, options, metrics, expectation)
-      message = "expected StatsD #{expectation.inspect} for metric '#{metric_name}' to be called"
+      def trigger_statsd_increment(metric_name, options = {})
+        Increment.new(:c, metric_name, options)
+      end
 
-      message += "\n  "
-      message += options[:times] ? "exactly #{options[:times]} times" : "at least once"
-      message += " with: #{options[expectation]}"
+      def trigger_statsd_measure(metric_name, options = {})
+        Measure.new(:ms, metric_name, options)
+      end
 
-      message += "\n  captured metric values: #{metrics.map(&expectation).join(', ')}"
+      def trigger_statsd_gauge(metric_name, options = {})
+        Gauge.new(:g, metric_name, options)
+      end
 
-      message
-    end
-  end
+      def trigger_statsd_set(metric_name, options = {})
+        Set.new(:s, metric_name, options)
+      end
 
-  CUSTOM_MATCHERS.each do |method_name, metric_type|
-    klass = Class.new(Matcher)
+      def trigger_statsd_histogram(metric_name, options = {})
+        Histogram.new(:h, metric_name, options)
+      end
 
-    define_method "trigger_statsd_#{method_name}" do |metric_name, options = {}|
-      klass.new(metric_type, metric_name, options)
+      def trigger_statsd_distribution(metric_name, options = {})
+        Distribution.new(:d, metric_name, options)
+      end
     end
-
-    StatsD::Instrument::Matchers.const_set(method_name.capitalize, klass)
   end
 end

data/lib/statsd/instrument/null_sink.rb

@@ -1,13 +1,17 @@
 # frozen_string_literal: true
 
-# @note This class is part of the new Client implementation that is intended
-#   to become the new default in the next major release of this library.
-class StatsD::Instrument::NullSink
-  def sample?(_sample_rate)
-    true
-  end
+module StatsD
+  module Instrument
+    # @note This class is part of the new Client implementation that is intended
+    #   to become the new default in the next major release of this library.
+    class NullSink
+      def sample?(_sample_rate)
+        true
+      end
 
-  def <<(_datagram)
-    self # noop
+      def <<(_datagram)
+        self # noop
+      end
+    end
   end
 end

data/lib/statsd/instrument/railtie.rb

@@ -1,11 +1,15 @@
 # frozen_string_literal: true
 
-# This Railtie runs some initializers that will set the logger to <tt>Rails#logger</tt>,
-# and will initialize the {StatsD#backend} based on the Rails environment.
-#
-# @see StatsD::Instrument::Environment
-class StatsD::Instrument::Railtie < Rails::Railtie
-  initializer 'statsd-instrument.use_rails_logger' do
-    ::StatsD.logger = Rails.logger
+module StatsD
+  module Instrument
+    # This Railtie runs some initializers that will set the logger to <tt>Rails#logger</tt>,
+    # and will initialize the {StatsD#backend} based on the Rails environment.
+    #
+    # @see StatsD::Instrument::Environment
+    class Railtie < Rails::Railtie
+      initializer 'statsd-instrument.use_rails_logger' do
+        ::StatsD.logger = Rails.logger
+      end
+    end
   end
 end

data/lib/statsd/instrument/statsd_datagram_builder.rb

@@ -1,14 +1,18 @@
 # frozen_string_literal: true
 
-# @note This class is part of the new Client implementation that is intended
-#   to become the new default in the next major release of this library.
-class StatsD::Instrument::StatsDDatagramBuilder < StatsD::Instrument::DatagramBuilder
-  unsupported_datagram_types :h, :d, :kv
+module StatsD
+  module Instrument
+    # @note This class is part of the new Client implementation that is intended
+    #   to become the new default in the next major release of this library.
+    class StatsDDatagramBuilder < StatsD::Instrument::DatagramBuilder
+      unsupported_datagram_types :h, :d, :kv
 
-  protected
+      protected
 
-  def normalize_tags(tags)
-    raise NotImplementedError, "#{self.class.name} does not support tags" if tags
-    super
+      def normalize_tags(tags)
+        raise NotImplementedError, "#{self.class.name} does not support tags" if tags
+        super
+      end
+    end
   end
 end

data/lib/statsd/instrument/udp_sink.rb

@@ -1,62 +1,66 @@
 # frozen_string_literal: true
 
-# @note This class is part of the new Client implementation that is intended
-#   to become the new default in the next major release of this library.
-class StatsD::Instrument::UDPSink
-  def self.for_addr(addr)
-    host, port_as_string = addr.split(':', 2)
-    new(host, Integer(port_as_string))
-  end
+module StatsD
+  module Instrument
+    # @note This class is part of the new Client implementation that is intended
+    #   to become the new default in the next major release of this library.
+    class UDPSink
+      def self.for_addr(addr)
+        host, port_as_string = addr.split(':', 2)
+        new(host, Integer(port_as_string))
+      end
 
-  attr_reader :host, :port
+      attr_reader :host, :port
 
-  def initialize(host, port)
-    @host = host
-    @port = port
-    @mutex = Mutex.new
-    @socket = nil
-  end
+      def initialize(host, port)
+        @host = host
+        @port = port
+        @mutex = Mutex.new
+        @socket = nil
+      end
 
-  def sample?(sample_rate)
-    sample_rate == 1 || rand < sample_rate
-  end
+      def sample?(sample_rate)
+        sample_rate == 1 || rand < sample_rate
+      end
 
-  def <<(datagram)
-    with_socket { |socket| socket.send(datagram, 0) > 0 }
-    self
+      def <<(datagram)
+        with_socket { |socket| socket.send(datagram, 0) > 0 }
+        self
 
-  rescue ThreadError
-    # In cases where a TERM or KILL signal has been sent, and we send stats as
-    # part of a signal handler, locks cannot be acquired, so we do our best
-    # to try and send the datagram without a lock.
-    socket.send(datagram, 0) > 0
+      rescue ThreadError
+        # In cases where a TERM or KILL signal has been sent, and we send stats as
+        # part of a signal handler, locks cannot be acquired, so we do our best
+        # to try and send the datagram without a lock.
+        socket.send(datagram, 0) > 0
 
-  rescue SocketError, IOError, SystemCallError
-    # TODO: log?
-    invalidate_socket
-  end
+      rescue SocketError, IOError, SystemCallError
+        # TODO: log?
+        invalidate_socket
+      end
 
-  def addr
-    "#{host}:#{port}"
-  end
+      def addr
+        "#{host}:#{port}"
+      end
 
-  private
+      private
 
-  def with_socket
-    @mutex.synchronize { yield(socket) }
-  end
+      def with_socket
+        @mutex.synchronize { yield(socket) }
+      end
 
-  def socket
-    if @socket.nil?
-      @socket = UDPSocket.new
-      @socket.connect(@host, @port)
-    end
-    @socket
-  end
+      def socket
+        if @socket.nil?
+          @socket = UDPSocket.new
+          @socket.connect(@host, @port)
+        end
+        @socket
+      end
 
-  def invalidate_socket
-    @mutex.synchronize do
-      @socket = nil
+      def invalidate_socket
+        @mutex.synchronize do
+          @socket = nil
+        end
+      end
     end
   end
 end

data/lib/statsd/instrument/version.rb

@@ -2,6 +2,6 @@
 
 module StatsD
   module Instrument
-    VERSION = "3.0.0"
+    VERSION = "3.0.1"
   end
 end