wavefront-sdk 5.2.0 → 5.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fc04400fbf51a67e94d228b3145a9fdd5fdb484c24035f1c83ff583e755d705e
-  data.tar.gz: 3acd358086967628300c65bff916b541e28004a1a5ca55b44b764a7ddb277d1e
+  metadata.gz: 5e623522f872c4204c1e723ee5a399c513251933e64445f6f5b65ff66d2fbffc
+  data.tar.gz: 6f55f44177169c24d4c0aa89490464097a6fe2278e24a1e3b83e5f7520757dfd
 SHA512:
-  metadata.gz: 665e0f09c85149b0879b868ec51336b88ac9c9c1af542df52ce0e00b932249a06236268e6398e6b828b7a7deb4421669611cdcaaf838965dd2eb560a720f0181
-  data.tar.gz: 519dc5ca20b01d81454bf8255815bd85ba3e016a4639eda688f1e2544b6d6c6e26ce6ff097e125fdf1eb40ab3ac10530307e0df3a31e590fac24279c89965b64
+  metadata.gz: 58bb998cc92b36c369497631f766af8844a692fbd10508962ee7bd30f4fbe65d222bc512d06b63feb19ca94cdcc4e8429b8c61f6227e4ff1041cd6ffff141758
+  data.tar.gz: 198dfadd7561b5c3772211716d23f9653ace3bbf472e41d12fa8b64d744ed16567bb11796ef3aff252054bc0945200ea71e454b5989824656c178e74503b4675

data/.rubocop.yml

@@ -2,47 +2,17 @@
 
 AllCops:
   TargetRubyVersion: 2.4
+  NewCops: enable
 
 Metrics/ClassLength:
   Max: 150
 
-# New cops
-#
-Lint/RaiseException:
-  Enabled: true
-Lint/StructNewOverride:
-  Enabled: true
-Style/ExponentialNotation:
-  Enabled: true
-Style/HashEachMethods:
-  Enabled: true
-Style/HashTransformKeys:
-  Enabled: true
-Style/HashTransformValues:
-  Enabled: true
-Layout/EmptyLinesAroundAttributeAccessor:
-  Enabled: true
-Layout/SpaceAroundMethodCallOperator:
-  Enabled: true
-Style/SlicingWithRange:
-  Enabled: true
-Lint/DeprecatedOpenSSLConstant:
-  Enabled: true
-Lint/MixedRegexpCaptureTypes:
-  Enabled: true
-Style/RedundantRegexpCharacterClass:
-  Enabled: true
-Style/RedundantRegexpEscape:
-  Enabled: true
-Style/AccessorGrouping:
-  Enabled: true
-Style/BisectedAttrAccessor:
-  Enabled: true
-Style/RedundantAssignment:
-  Enabled: true
-Style/RedundantFetchBlock:
-  Enabled: true
-
 # Is nothing sacred?
 Layout/LineLength:
   Max: 80
+
+Style/StringConcatenation:
+  Enabled: false
+
+Style/OptionalBooleanParameter:
+  Enabled: false

data/HISTORY.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## 5.2.1 (2020-09-18)
+* Remove necessity for user to `require 'pathname'`
+
 ## 5.2.0 (2020-09-03)
 * Add `:raise_on_no_profile` option to `Wavefront::Credentials` constructor
   options. If this is true and a specific config stanza is requested but not

data/lib/wavefront-sdk/defs/version.rb

@@ -1,4 +1,6 @@
 # frozen_string_literal: true
 
-WF_SDK_VERSION = '5.2.0'
+require 'pathname'
+
+WF_SDK_VERSION = '5.2.1'
 WF_SDK_LOCATION = Pathname.new(__dir__).parent.parent.parent

data/lib/wavefront-sdk/distribution.rb

@@ -65,6 +65,7 @@ module Wavefront
       raise Wavefront::Exception::InvalidDistribution
     end
 
+    # rubocop:disable Metrics/AbcSize
     def dist_hash(dist)
       dist.dup.tap do |d|
         d[:interval] = distribution_interval(dist)
@@ -75,6 +76,7 @@ module Wavefront
         d[:opttags] = tags_or_nothing(opts[:tags])
       end
     end
+    # rubocop:enable Metrics/AbcSize
 
     def distribution_timestamp(dist)
       parse_time(dist.fetch(:ts, Time.now))

data/lib/wavefront-sdk/writers/core.rb

@@ -138,8 +138,8 @@ module Wavefront
       end
 
       def log_invalid_point(rawpoint, exception)
-        logger.log('Invalid point, skipping.', :info)
-        logger.log(exception.class, :info)
+        logger.log('Invalid point, skipping.', :warn)
+        logger.log(exception.class, :warn)
         logger.log(format('Invalid point: %<rawpoint>s (%<message>s)',
                           rawpoint: rawpoint,
                           message: exception.to_s), :debug)

data/spec/.rubocop.yml

@@ -1,54 +1,23 @@
-# Offense count: 64
+AllCops:
+  NewCops: enable
+
 Metrics/AbcSize:
   Max: 64
 
-# Offense count: 5
 # Configuration parameters: CountComments.
 Metrics/ClassLength:
   Max: 400
 
-# Offense count: 46
 # Configuration parameters: CountComments, ExcludedMethods.
 Metrics/MethodLength:
   Max: 39
 
-# New cops
-#
-Lint/RaiseException:
-  Enabled: true
-Lint/StructNewOverride:
-  Enabled: true
-Style/ExponentialNotation:
-  Enabled: true
-Style/HashEachMethods:
-  Enabled: true
-Style/HashTransformKeys:
-  Enabled: true
-Style/HashTransformValues:
-  Enabled: true
-Layout/EmptyLinesAroundAttributeAccessor:
-  Enabled: true
-Layout/SpaceAroundMethodCallOperator:
-  Enabled: true
-Style/SlicingWithRange:
-  Enabled: true
-Lint/DeprecatedOpenSSLConstant:
-  Enabled: true
-Lint/MixedRegexpCaptureTypes:
-  Enabled: true
-Style/RedundantRegexpCharacterClass:
-  Enabled: true
-Style/RedundantRegexpEscape:
-  Enabled: true
-Style/AccessorGrouping:
-  Enabled: true
-Style/BisectedAttrAccessor:
-  Enabled: true
-Style/RedundantAssignment:
-  Enabled: true
-Style/RedundantFetchBlock:
-  Enabled: true
-
 # Is nothing sacred?
 Layout/LineLength:
   Max: 80
+
+Style/StringConcatenation:
+  Enabled: false
+
+Style/OptionalBooleanParameter:
+  Enabled: false

data/spec/support/minitest_assertions.rb

@@ -95,18 +95,12 @@ module Minitest
     end
 
     def header_lookup(type)
-      ctype = case type
-              when :plain
-                'text/plain'
-              when :json
-                'application/json'
-              when :octet
-                'application/octet-stream'
-              when :form
-                'application/x-www-form-urlencoded'
-              end
+      ctypes = { plain: 'text/plain',
+                 json: 'application/json',
+                 octet: 'application/octet-stream',
+                 form: 'application/x-www-form-urlencoded' }
 
-      { 'Content-Type': ctype, Accept: 'application/json' }
+      { 'Content-Type': ctypes[type], Accept: 'application/json' }
     end
   end
 end

data/spec/wavefront-sdk/core/logger_spec.rb

@@ -41,21 +41,21 @@ class WavefrontLoggerTest < MiniTest::Test
   end
 
   def test_log_logger_debug
-    l = Wavefront::Logger.new(logger: Logger.new(STDOUT))
+    l = Wavefront::Logger.new(logger: Logger.new($stdout))
     out, err = capture_subprocess_io { l.log('my message', :debug) }
     assert_match(/DEBUG -- : my message$/, out)
     assert_empty(err)
   end
 
   def test_log_logger_info
-    l = Wavefront::Logger.new(logger: Logger.new(STDOUT))
+    l = Wavefront::Logger.new(logger: Logger.new($stdout))
     out, err = capture_subprocess_io { l.log('my message', :info) }
     assert_match(/INFO -- : my message$/, out)
     assert_empty(err)
   end
 
   def test_log_logger_error
-    l = Wavefront::Logger.new(logger: Logger.new(STDOUT))
+    l = Wavefront::Logger.new(logger: Logger.new($stdout))
     out, err = capture_subprocess_io { l.log('my message', :error) }
     assert_match(/ERROR -- : my message$/, out)
     assert_empty(err)

data/spec/wavefront-sdk/credentials_spec.rb

@@ -60,9 +60,11 @@ end
 # Test individual methods. We must override the constructor to do
 # this.
 #
+# rubocop:disable Lint/MissingSuper
 class Giblets < Wavefront::Credentials
   def initialize; end
 end
+# rubocop:enable Lint/MissingSuper
 
 # And here are the tests
 #

data/spec/wavefront-sdk/writers/core_spec.rb

@@ -22,7 +22,7 @@ class TestClassNoTags
   end
 
   def logger
-    Logger.new(STDOUT)
+    Logger.new($stdout)
   end
 
   def manage_conn

data/wavefront-sdk.gemspec

@@ -29,7 +29,7 @@ Gem::Specification.new do |gem|
 
   gem.add_development_dependency 'minitest', '~> 5.14'
   gem.add_development_dependency 'rake', '~> 13.0'
-  gem.add_development_dependency 'rubocop', '0.87.1'
+  gem.add_development_dependency 'rubocop', '0.91.0'
   gem.add_development_dependency 'simplecov', '~> 0.18'
   gem.add_development_dependency 'spy', '1.0.0'
   gem.add_development_dependency 'webmock', '~> 3.8'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: wavefront-sdk
 version: !ruby/object:Gem::Version
-  version: 5.2.0
+  version: 5.2.1
 platform: ruby
 authors:
 - Robert Fisher
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-09-03 00:00:00.000000000 Z
+date: 2020-09-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable
@@ -100,14 +100,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.87.1
+        version: 0.91.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.87.1
+        version: 0.91.0
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement