wavefront-sdk 3.3.3 → 3.3.4

data/lib/wavefront-sdk/event.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'core/api'
 require_relative 'api_mixins/tag'
 
@@ -28,19 +30,11 @@ module Wavefront
     #
     def list(from = nil, to = Time.now, limit = 100, cursor = nil)
       raise ArgumentError unless from && to
-      wf_event_id?(cursor) if cursor
-
-      from = parse_time(from, true)
-      to   = parse_time(to, true)
-
-      wf_ms_ts?(from)
-      wf_ms_ts?(to)
-
-      body = { earliestStartTimeEpochMillis: from,
-               latestStartTimeEpochMillis:   to,
-               cursor:                       cursor,
-               limit:                        limit }
 
+      body = list_body(from, to, limit, cursor)
+      wf_event_id?(cursor) if cursor
+      wf_ms_ts?(body[:earliestStartTimeEpochMillis])
+      wf_ms_ts?(body[:latestStartTimeEpochMillis])
       api.get('', body.cleanse)
     end
 
@@ -56,6 +50,7 @@ module Wavefront
     #
     def create(body)
       raise ArgumentError unless body.is_a?(Hash)
+
       api.post('', body, 'application/json')
     end
 
@@ -121,5 +116,14 @@ module Wavefront
     def valid_id?(id)
       wf_event_id?(id)
     end
+
+    private
+
+    def list_body(t_start, t_end, limit, cursor)
+      { earliestStartTimeEpochMillis: parse_time(t_start, true),
+        latestStartTimeEpochMillis: parse_time(t_end, true),
+        cursor: cursor,
+        limit: limit }
+    end
   end
 end

data/lib/wavefront-sdk/externallink.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'core/api'
 
 module Wavefront
@@ -32,6 +34,7 @@ module Wavefront
     #
     def create(body)
       raise ArgumentError unless body.is_a?(Hash)
+
       api.post('', body, 'application/json')
     end
 

data/lib/wavefront-sdk/integration.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'core/api'
 
 module Wavefront

data/lib/wavefront-sdk/maintenancewindow.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'core/api'
 
 module Wavefront
@@ -32,6 +34,7 @@ module Wavefront
     #
     def create(body)
       raise ArgumentError unless body.is_a?(Hash)
+
       api.post('', body, 'application/json')
     end
 
@@ -110,7 +113,7 @@ module Wavefront
       ret = pending
 
       items = { UPCOMING: ret.response.items,
-                CURRENT:  ongoing.response.items }
+                CURRENT: ongoing.response.items }
 
       ret.response.items = items
       ret

data/lib/wavefront-sdk/message.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'core/api'
 
 module Wavefront

data/lib/wavefront-sdk/metric.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'core/api'
 
 module Wavefront
@@ -24,6 +26,7 @@ module Wavefront
 
       if cursor
         raise ArgumentError unless cursor.is_a?(String)
+
         query.<< [:c, cursor]
       end
 

data/lib/wavefront-sdk/metric_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'write'
 
 module Wavefront
@@ -18,7 +20,7 @@ module Wavefront
     #
     def initialize(creds, opts = {})
       @opts        = opts
-      @buf         = { gauges:   empty_gauges,
+      @buf         = { gauges: empty_gauges,
                        counters: empty_counters }
       @writer      = setup_writer(creds, opts)
       @dist_writer = setup_dist_writer(creds, opts) if opts[:dist_port]
@@ -139,9 +141,9 @@ module Wavefront
     def dists_to_wf(dists)
       dists.map do |k, v|
         path, interval, tags = k
-        dist = { path:     path,
-                 value:    dist_writer.mk_distribution(v),
-                 ts:       Time.now.utc.to_i,
+        dist = { path: path,
+                 value: dist_writer.mk_distribution(v),
+                 ts: Time.now.utc.to_i,
                  interval: interval }
         dist[:tags] = tags unless tags.nil?
         dist

data/lib/wavefront-sdk/notificant.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'core/api'
 
 module Wavefront
@@ -23,6 +25,7 @@ module Wavefront
     #
     def create(body)
       raise ArgumentError unless body.is_a?(Hash)
+
       api.post('', body, 'application/json')
     end
 

data/lib/wavefront-sdk/paginator/base.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../defs/constants'
 
 module Wavefront
@@ -71,6 +73,7 @@ module Wavefront
       def user_page_size(args)
         arg_val = limit_and_offset(args)[:offset].to_i
         return arg_val if arg_val&.positive?
+
         PAGE_SIZE
       end
 
@@ -114,6 +117,7 @@ module Wavefront
           api_caller.verbosity(conn, method, *p_args)
           resp = api_caller.respond(conn.public_send(method, *p_args))
           raise StopIteration unless resp.ok?
+
           ret.response.items += resp.response.items
           return finalize_response(ret) unless resp.more_items?
         end
@@ -154,6 +158,7 @@ module Wavefront
             unless resp.ok?
               raise(Wavefront::Exception::EnumerableError, resp.status)
             end
+
             p_args = set_pagination(offset, page_size, p_args)
             resp.response.items.map { |i| y.<< i }
             raise StopIteration unless resp.more_items?

data/lib/wavefront-sdk/paginator/delete.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'base'
 
 module Wavefront

data/lib/wavefront-sdk/paginator/get.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'base'
 
 module Wavefront

data/lib/wavefront-sdk/paginator/post.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'json'
 require_relative 'base'
 

data/lib/wavefront-sdk/paginator/put.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'base'
 
 module Wavefront

data/lib/wavefront-sdk/proxy.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'core/api'
 
 module Wavefront

data/lib/wavefront-sdk/query.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'core/api'
 
 module Wavefront
@@ -32,6 +34,7 @@ module Wavefront
               options = {})
 
       raise ArgumentError unless query.is_a?(String)
+
       wf_granularity?(granularity)
       raise Wavefront::Exception::InvalidTimestamp if t_start.nil?
 
@@ -87,9 +90,9 @@ module Wavefront
       status = 400 if status == 200 && !err_msg.empty?
 
       { response: resp,
-        status:   { result:  status == 200 ? 'OK' : 'ERROR',
-                    message: err_msg,
-                    code:    status } }.to_json
+        status: { result: status == 200 ? 'OK' : 'ERROR',
+                  message: err_msg,
+                  code: status } }.to_json
     end
 
     # A bad query doesn't send back a JSON object. It sends back a

data/lib/wavefront-sdk/report.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'write'
 
 module Wavefront

data/lib/wavefront-sdk/savedsearch.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'core/api'
 
 module Wavefront
@@ -26,6 +28,7 @@ module Wavefront
     #
     def create(body)
       raise ArgumentError unless body.is_a?(Hash)
+
       api.post('', body, 'application/json')
     end
 
@@ -60,6 +63,7 @@ module Wavefront
     def update(id, body)
       wf_savedsearch_id?(id)
       raise ArgumentError unless body.is_a?(Hash)
+
       api.put(id, body)
     end
 

data/lib/wavefront-sdk/search.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'core/api'
 
 module Wavefront
@@ -39,6 +41,7 @@ module Wavefront
     #
     def search(entity, query, options = {})
       raise ArgumentError unless options.is_a?(Hash)
+
       raw_search(entity, body(query, options), options[:deleted] || false)
     end
 
@@ -71,7 +74,7 @@ module Wavefront
     def sort_field(options, query)
       field = options[:sort_field] || [query].flatten.first[:key]
 
-      { field:     field,
+      { field: field,
         ascending: !options[:desc] || true }
     end
 

data/lib/wavefront-sdk/settings.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'core/api'
 
 module Wavefront

data/lib/wavefront-sdk/source.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'core/api'
 require_relative 'api_mixins/tag'
 
@@ -37,6 +39,7 @@ module Wavefront
     #
     def create(body)
       raise ArgumentError unless body.is_a?(Hash)
+
       api.post('', body, 'application/json')
     end
 

data/lib/wavefront-sdk/stdlib/array.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Extensions to stdlib Array
 #
 class Array

data/lib/wavefront-sdk/stdlib/hash.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'string'
 
 # Extensions to stdlib Hash
@@ -7,7 +9,9 @@ class Hash
   # the WF wire-format guide. No validation is performed here.
   #
   def to_wf_tag
-    map { |k, v| format('%s="%s"', k, v.tagescape) }.join(' ')
+    map do |k, v|
+      format('%<key>s="%<value>s"', key: k, value: v.tagescape)
+    end.join(' ')
   end
 
   # Drop any key-value pairs where the value is not truthy

data/lib/wavefront-sdk/stdlib/string.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Extensions to stdlib String
 #
 class String

data/lib/wavefront-sdk/support/mixins.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'date'
 require_relative 'parse_time'
 require_relative '../core/exception'
@@ -22,6 +24,7 @@ module Wavefront
     #
     def parse_time(time, in_ms = false)
       return relative_time(time, in_ms) if time =~ /^[\-+]/
+
       ParseTime.new(time, in_ms).parse!
     end
 
@@ -57,7 +60,7 @@ module Wavefront
       end
 
       m = in_ms ? 1000 : 1
-      time.delete!('+')
+      time = time.delete('+')
       match = time.match(/^(-?\d*\.?\d*)([smhdwy])$/)
       (match[1].to_f * time_multiplier(match[2]) * m).to_i
     end
@@ -85,6 +88,7 @@ module Wavefront
       u = { s: 1, m: 60, h: 3600, d: 86_400, w: 604_800, y: 31_536_000 }
 
       return u[suffix.to_sym] if u.key?(suffix.to_sym)
+
       raise Wavefront::Exception::InvalidTimeUnit
     end
 

data/lib/wavefront-sdk/support/parse_time.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'time'
 
 module Wavefront
@@ -33,6 +35,7 @@ module Wavefront
     #
     def parse_time_string
       return t.to_i if t =~ /^\d+$/
+
       @t = Time.parse("#{t} #{Time.now.getlocal.zone}")
       parse_time_time
     end

data/lib/wavefront-sdk/types/status.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Wavefront
   #
   # Status types are used by the Wavefront::Response class. They
@@ -37,6 +39,7 @@ module Wavefront
       def message
         return obj[:message] if obj[:message]
         return obj[:error] if obj[:error]
+
         nil
       end
 
@@ -47,6 +50,7 @@ module Wavefront
       def result
         return obj[:result] if obj[:result]
         return 'OK' if status.between?(200, 299)
+
         'ERROR'
       end
     end

data/lib/wavefront-sdk/user.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'core/api'
 require_relative 'api_mixins/user'
 
@@ -25,6 +27,7 @@ module Wavefront
     #
     def create(body, send_email = false)
       raise ArgumentError unless body.is_a?(Hash)
+
       api.post("?sendEmail=#{send_email}", body, 'application/json')
     end
 
@@ -115,6 +118,7 @@ module Wavefront
     def grant(id, pgroup)
       wf_user_id?(id)
       raise ArgumentError unless pgroup.is_a?(String)
+
       api.post([id, 'grant'].uri_concat, "group=#{pgroup}",
                'application/x-www-form-urlencoded')
     end
@@ -131,6 +135,7 @@ module Wavefront
     def revoke(id, pgroup)
       wf_user_id?(id)
       raise ArgumentError unless pgroup.is_a?(String)
+
       api.post([id, 'revoke'].uri_concat, "group=#{pgroup}",
                'application/x-www-form-urlencoded')
     end
@@ -145,6 +150,7 @@ module Wavefront
     #
     def delete_users(user_list)
       raise ArgumentError unless user_list.is_a?(Array)
+
       validate_user_list(user_list)
       api.post('deleteUsers', user_list, 'application/json')
     end
@@ -160,6 +166,7 @@ module Wavefront
     #
     def grant_permission(permission, user_list)
       raise ArgumentError unless user_list.is_a?(Array)
+
       validate_user_list(user_list)
       api.post(['grant', permission].uri_concat, user_list,
                'application/json')
@@ -176,6 +183,7 @@ module Wavefront
     #
     def revoke_permission(permission, user_list)
       raise ArgumentError unless user_list.is_a?(Array)
+
       validate_user_list(user_list)
       api.post(['revoke', permission].uri_concat, user_list,
                'application/json')
@@ -190,6 +198,7 @@ module Wavefront
     def invite(body)
       raise ArgumentError unless body.is_a?(Array)
       raise ArgumentError unless body.first.is_a?(Hash)
+
       api.post('invite', body, 'application/json')
     end
 
@@ -213,21 +222,21 @@ module Wavefront
     end
 
     def itemize_response(body_obj)
-      { status:   body_obj[:status],
+      { status: body_obj[:status],
         response: { items: [body_obj[:response]].flatten } }.to_json
     end
 
     # Construct a response almost from scratch. Used for 'list', among others.
     #
     def construct_response(body_obj, status)
-      { status:   { result:     status.to_s.start_with?('2') ? 'OK' : 'ERROR',
-                    message:    extract_api_message(status, body_obj),
-                    code:       status },
-        response: { items:      [body_obj].flatten,
-                    offset:     0,
-                    limit:      body_obj.size,
+      { status: { result: status.to_s.start_with?('2') ? 'OK' : 'ERROR',
+                  message: extract_api_message(status, body_obj),
+                  code: status },
+        response: { items: [body_obj].flatten,
+                    offset: 0,
+                    limit: body_obj.size,
                     totalItems: body_obj.size,
-                    moreItems:  false } }.to_json
+                    moreItems: false } }.to_json
     end
 
     # the user API class does not support pagination. Be up-front
@@ -248,6 +257,7 @@ module Wavefront
 
     def extract_api_message(status, items)
       return '' if status < 300
+
       items.fetch(:message, 'no message from API')
     end