percy-common 2.1.1 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 8c72b6ac18f0d0481edf3be4436cd9d509f01494
-  data.tar.gz: a117e47adcb706883d4565ddb5c47100adebf926
+SHA256:
+  metadata.gz: c0fe4132dddef08590d485a30d755fa43e95bc0cc4569b0f716e25c756c5dca5
+  data.tar.gz: 9de9dbc2f26404e03f22175bfc742080df122824d9510197a23a0f55c8ca0c65
 SHA512:
-  metadata.gz: 38fa4e30554c706d6d8b77764bab5e0851a9bc6ee126bb64df7a1d2063f58fdfc73142739f7b7476a2eca1e8cdbe4078b00f9791d576626c60661dbb05e5e697
-  data.tar.gz: 729ee85dd9b7177de77cd5dab68cdfc519b2e7ff8a0628ce58811a0f7b67f3986390153acf4647879d4a69f18aa86411965f6474231e23155b1ab6d835d1bb14
+  metadata.gz: 4063a9ab97a6a5e30774733dbe8922dc690b6556f80f6a3c28651043e050043ea5a76b4de1220a2e3f3fa0688968a259fe9eaba7a77f11699fb9a9e5348f68ab
+  data.tar.gz: 89cb3605660c98e521e3f08c72be55b3517236ee3702dda10c65d09c35eaa855b5b6715d2e349359c34e8fdb41020139dce834e8a3acb2ff7457e0bfe4e6b641

data/.circleci/config.yml

@@ -0,0 +1,18 @@
+version: 2.1
+
+jobs:
+  ruby_25:
+    docker:
+      - image: circleci/ruby:2.5-node-browsers
+    steps:
+      - checkout
+      - run: gem update --system
+      - run: gem install bundler
+      - run: bundle install
+      - run: bundle exec rubocop -D
+
+workflows:
+  version: 2
+  test:
+    jobs:
+      - ruby_25

data/.rubocop.yml

@@ -2,6 +2,10 @@ inherit_gem:
   percy-style:
     - default.yml
 
+inherit_mode:
+  merge:
+    - Include
+
 RSpec/DescribeClass:
   Exclude:
     - spec/percy/logger_spec.rb

data/lib/percy/common/version.rb

@@ -1,5 +1,5 @@
 module Percy
   module Common
-    VERSION = '2.1.1'.freeze
+    VERSION = '3.0.0'.freeze
   end
 end

data/lib/percy/stats.rb

@@ -3,41 +3,58 @@ require 'time'
 
 module Percy
   class Stats < ::Datadog::Statsd
-    def initialize(host = nil, port = nil, opts = {}, max_buffer_size = 50)
-      host ||= ENV.fetch('DATADOG_AGENT_HOST', ::Datadog::Statsd::DEFAULT_HOST)
-      port ||= Integer(ENV.fetch('DATADOG_AGENT_PORT', ::Datadog::Statsd::DEFAULT_PORT))
-      opts[:tags] ||= []
-      opts[:tags] << "env:#{ENV['PERCY_ENV'] || 'development'}"
-      retry_delay = opts[:retry_delay] || 1
-      retry_count = opts[:retry_count] || 3
-      retries = 0
-
-      begin
-        super(host, port, opts, max_buffer_size)
-      rescue SocketError
-        host = 'localhost' if retries >= retry_count
-        sleep retry_delay
-        retries += 1
-        retry
-      end
+    DEFAULT_HOST = ENV.fetch(
+      'DATADOG_AGENT_HOST',
+      ::Datadog::Statsd::Connection::DEFAULT_HOST,
+    )
+
+    DEFAULT_PORT = Integer(
+      ENV.fetch(
+        'DATADOG_AGENT_PORT',
+        ::Datadog::Statsd::Connection::DEFAULT_PORT,
+      ),
+    )
+
+    DEFAULT_TAGS = %W[
+      env:#{ENV.fetch('PERCY_ENV', 'development')}
+    ].freeze
+
+    def initialize(
+      host = DEFAULT_HOST,
+      port = DEFAULT_PORT,
+      tags: DEFAULT_TAGS,
+      **kwargs
+    )
+      super(host, port, tags: tags, **kwargs)
     end
 
-    # Equivalent to stats.time, but without wrapping in blocks and dealing with var scoping issues.
+    # Equivalent to stats.time, but without wrapping in blocks and dealing with
+    # var scoping issues.
     #
     # @example Report the time taken to activate an account.
     #   stats.start_timing
     #   account.activate!
     #   stats.stop_timing('account.activate')
     def start_timing
-      @_timing_start = Time.now
+      @_timing_start = now
       true
     end
 
     def stop_timing(stat, options = {})
-      raise 'no timing started' unless @_timing_start # Programmer mistake, so raise an error.
+      # Programmer mistake, so raise an error.
+      raise 'no timing started' unless @_timing_start
+
       time_since(stat, @_timing_start, options)
       @_timing_start = nil
       true
     end
+
+    def time_since(stat, start, opts = {})
+      timing(stat, ((now.to_f - start.to_f) * 1000).round, opts)
+    end
+
+    private def now
+      Process.clock_gettime(Process::CLOCK_MONOTONIC)
+    end
   end
 end

data/percy-common.gemspec

@@ -16,14 +16,12 @@ Gem::Specification.new do |spec|
   spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   spec.require_paths = ['lib']
 
-  # Pinned as we rely on `time_since` which was taken out after this version.
-  spec.add_dependency 'dogstatsd-ruby', '3.2.0'
-
+  spec.add_dependency 'dogstatsd-ruby', '~> 4.4.0'
   spec.add_dependency 'excon', '~> 0.57'
 
-  spec.add_development_dependency 'bundler', '~> 1.15'
+  spec.add_development_dependency 'bundler', '~> 2.0.2'
   spec.add_development_dependency 'rake', '~> 10.0'
   spec.add_development_dependency 'rspec', '~> 3.2'
   spec.add_development_dependency 'guard-rspec', '~> 4.7'
-  spec.add_development_dependency 'percy-style'
+  spec.add_development_dependency 'percy-style', '~> 0.6.0'
 end

metadata

@@ -1,29 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: percy-common
 version: !ruby/object:Gem::Version
-  version: 2.1.1
+  version: 3.0.0
 platform: ruby
 authors:
 - Perceptual Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-02-26 00:00:00.000000000 Z
+date: 2019-07-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dogstatsd-ruby
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.2.0
+        version: 4.4.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.2.0
+        version: 4.4.0
 - !ruby/object:Gem::Dependency
   name: excon
   requirement: !ruby/object:Gem::Requirement
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.15'
+        version: 2.0.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.15'
+        version: 2.0.2
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -98,16 +98,16 @@ dependencies:
   name: percy-style
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.6.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.6.0
 description: ''
 email:
 - team@percy.io
@@ -115,10 +115,10 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".circleci/config.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
-- ".travis.yml"
 - Gemfile
 - Guardfile
 - LICENSE
@@ -153,7 +153,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.13
+rubygems_version: 2.7.8
 signing_key: 
 specification_version: 4
 summary: Server-side common library for Percy.

data/.travis.yml

@@ -1,12 +0,0 @@
-language: ruby
-sudo: false
-cache: bundler
-rvm:
-  - 2.4
-  - 2.5
-before_install:
-  - gem update --system
-  - gem update bundler
-script:
-  - bundle exec rubocop -D
-  - bundle exec rspec