fitbit_api 0.14.2 → 0.15.1

data/lib/fitbit_api/food.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class Client
-    FOOD_RESOURCES = %w(caloriesIn water)
+    FOOD_RESOURCES = %w[caloriesIn water].freeze
 
-    def food_logs(date=Date.today)
+    def food_logs(date = Date.today)
       get("user/#{user_id}/foods/log/date/#{format_date(date)}.json")
     end
 
@@ -11,7 +13,7 @@ module FitbitAPI
     # @param query [String] The search query
 
     def search_foods(params)
-      get("foods/search.json", params)
+      get('foods/search.json', params)
     end
 
     def recent_foods
@@ -81,43 +83,43 @@ module FitbitAPI
       delete("user/#{user_id}/foods/log/favorite/#{food_id}.json")
     end
 
-    def food_time_series(resource, opts={})
+    def food_time_series(resource, opts = {})
       start_date = opts[:start_date]
       end_date   = opts[:end_date] || Date.today
       period     = opts[:period]
 
       unless FOOD_RESOURCES.include?(resource)
-        raise FitbitAPI::InvalidArgumentError, "Invalid resource: \"#{resource}\". Please provide one of the following: #{FOOD_RESOURCES}."
+        raise FitbitAPI::InvalidArgumentError,
+              "Invalid resource: \"#{resource}\". Please provide one of the following: #{FOOD_RESOURCES}."
       end
 
-      if [period, start_date].none?
-        raise FitbitAPI::InvalidArgumentError, 'A start_date or period is required.'
-      end
+      raise FitbitAPI::InvalidArgumentError, 'A start_date or period is required.' if [period, start_date].none?
 
       if period && !PERIODS.include?(period)
-        raise FitbitAPI::InvalidArgumentError, "Invalid period: \"#{period}\". Please provide one of the following: #{PERIODS}."
+        raise FitbitAPI::InvalidArgumentError,
+              "Invalid period: \"#{period}\". Please provide one of the following: #{PERIODS}."
       end
 
-      if period
-        result = get("user/#{user_id}/foods/log/#{resource}/date/#{format_date(end_date)}/#{period}.json")
-      else
-        result = get("user/#{user_id}/foods/log/#{resource}/date/#{format_date(start_date)}/#{format_date(end_date)}.json")
-      end
+      path = if period
+               "user/#{user_id}/foods/log/#{resource}/date/#{format_date(end_date)}/#{period}.json"
+             else
+               "user/#{user_id}/foods/log/#{resource}/date/#{format_date(start_date)}/#{format_date(end_date)}.json"
+             end
+      result = get(path)
 
-      # remove root key from response
-      result.values[0]
+      strip_root_key(result)
     end
 
     # Retrieves the food locales used to search, log or create food
 
     def food_locales
-      get("foods/locales.json")
+      get('foods/locales.json')
     end
 
     # Retrieves a list of all valid Fitbit food units
 
     def food_units
-      get("foods/units.json")
+      get('foods/units.json')
     end
   end
 end

data/lib/fitbit_api/friends.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class Client
     # Retrieves a list of the Fitbit user's friends.

data/lib/fitbit_api/goals.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class Client
     # Retrieves a user's current weight goal.
@@ -55,7 +57,7 @@ module FitbitAPI
     # @option body :distance [Integer, Float] Distance goal value
     # @option body :steps [Integer] Steps goal value
 
-    def update_daily_activity_goals(body={})
+    def update_daily_activity_goals(body = {})
       post("user/#{user_id}/activities/goals/daily.json", body)
     end
 
@@ -72,7 +74,7 @@ module FitbitAPI
     # @option body :distance [Integer, Float] Distance goal value
     # @option body :steps [Integer] Steps goal value
 
-    def update_weekly_activity_goals(body={})
+    def update_weekly_activity_goals(body = {})
       post("user/#{user_id}/activities/goals/weekly.json", body)
     end
 

data/lib/fitbit_api/heart_rate.rb

@@ -1,29 +1,29 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class Client
-    def heart_rate_time_series(opts={})
+    def heart_rate_time_series(opts = {})
       start_date = opts[:start_date]
       end_date   = opts[:end_date] || Date.today
       period     = opts[:period]
 
-      if [period, start_date].none?
-        raise FitbitAPI::InvalidArgumentError, 'A start_date or period is required.'
-      end
+      raise FitbitAPI::InvalidArgumentError, 'A start_date or period is required.' if [period, start_date].none?
 
       if period && !PERIODS.include?(period)
-        raise FitbitAPI::InvalidArgumentError, "Invalid period: \"#{period}\". Please provide one of the following: #{PERIODS}."
+        raise FitbitAPI::InvalidArgumentError,
+              "Invalid period: \"#{period}\". Please provide one of the following: #{PERIODS}."
       end
 
-      if period
-        result = get("user/#{user_id}/activities/heart/date/#{format_date(end_date)}/#{period}.json")
-      else
-        result = get("user/#{user_id}/activities/heart/date/#{format_date(start_date)}/#{format_date(end_date)}.json")
-      end
+      result = if period
+                 get("user/#{user_id}/activities/heart/date/#{format_date(end_date)}/#{period}.json")
+               else
+                 get("user/#{user_id}/activities/heart/date/#{format_date(start_date)}/#{format_date(end_date)}.json")
+               end
 
-      # remove root key from response
-      result.values[0]
+      strip_root_key(result)
     end
 
-    def heart_rate_intraday_time_series(opts={})
+    def heart_rate_intraday_time_series(opts = {})
       date         = opts[:date] || Date.today
       detail_level = opts[:detail_level]
       start_time   = opts[:start_time]
@@ -34,18 +34,18 @@ module FitbitAPI
       end
 
       unless %(1sec 1min).include? detail_level
-        raise FitbitAPI::InvalidArgumentError, "Invalid detail_level: \"#{detail_level}\". Please provide one of the following: \"1sec\" or \"1min\"."
+        raise FitbitAPI::InvalidArgumentError,
+              "Invalid detail_level: \"#{detail_level}\". Please provide one of the following: \"1sec\" or \"1min\"."
       end
 
       if (start_time || end_time) && !(start_time && end_time)
         raise FitbitAPI::InvalidArgumentError, 'Both start_time and end_time are required if time is being specified.'
       end
 
-      if (start_time && end_time)
-        get("user/#{user_id}/activities/heart/date/#{format_date(date)}/1d/#{detail_level}/time/#{format_time(start_time)}/#{format_time(end_time)}.json")
-      else
-        get("user/#{user_id}/activities/heart/date/#{format_date(date)}/1d/#{detail_level}.json")
-      end
+      path = "user/#{user_id}/activities/heart/date/#{format_date(date)}/1d/#{detail_level}"
+      path += "/time/#{format_time(start_time)}/#{format_time(end_time)}" if start_time && end_time
+
+      get("#{path}.json")
     end
   end
 end

data/lib/fitbit_api/heart_rate_variability.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class Client
     # Returns the heart rate variability data for a given date or date range.
@@ -12,27 +14,22 @@ module FitbitAPI
     # @option params :start_date [Date] The start of the date range
     # @option params :end_date [Date] The end of the date range
 
-    def heart_rate_variability_summary(opts={})
+    def heart_rate_variability_summary(opts = {})
       date       = opts[:date] || Date.today
       start_date = opts[:start_date]
       end_date   = opts[:end_date]
 
-      if start_date && !end_date
-        end_date = Date.today
-      end
+      end_date = Date.today if start_date && !end_date
 
-      unless date || start_date
-        raise FitbitAPI::InvalidArgumentError, 'A date or start_date and end_date are required.'
-      end
+      raise FitbitAPI::InvalidArgumentError, 'A date or start_date and end_date are required.' unless date || start_date
 
-      if start_date
-        result = get("user/#{user_id}/hrv/date/#{format_date(start_date)}/#{format_date(end_date)}.json")
-      else
-        result = get("user/#{user_id}/hrv/date/#{format_date(date)}.json")
-      end
+      result = if start_date
+                 get("user/#{user_id}/hrv/date/#{format_date(start_date)}/#{format_date(end_date)}.json")
+               else
+                 get("user/#{user_id}/hrv/date/#{format_date(date)}.json")
+               end
 
-      # remove root key from response
-      result.values[0]
+      strip_root_key(result)
     end
 
     # Returns the heart rate variability intraday data for a given date or date range.
@@ -47,27 +44,22 @@ module FitbitAPI
     # @option params :start_date [Date] The start of the date range
     # @option params :end_date [Date] The end of the date range
 
-    def heart_rate_variability_intraday(opts={})
+    def heart_rate_variability_intraday(opts = {})
       date       = opts[:date] || Date.today
       start_date = opts[:start_date]
       end_date   = opts[:end_date]
 
-      if start_date && !end_date
-        end_date = Date.today
-      end
+      end_date = Date.today if start_date && !end_date
 
-      unless date || start_date
-        raise FitbitAPI::InvalidArgumentError, 'A date or start_date and end_date are required.'
-      end
+      raise FitbitAPI::InvalidArgumentError, 'A date or start_date and end_date are required.' unless date || start_date
 
-      if start_date
-        result = get("user/#{user_id}/hrv/date/#{format_date(start_date)}/#{format_date(end_date)}/all.json")
-      else
-        result = get("user/#{user_id}/hrv/date/#{format_date(date)}/all.json")
-      end
+      result = if start_date
+                 get("user/#{user_id}/hrv/date/#{format_date(start_date)}/#{format_date(end_date)}/all.json")
+               else
+                 get("user/#{user_id}/hrv/date/#{format_date(date)}/all.json")
+               end
 
-      # remove root key from response
-      result.values[0]
+      strip_root_key(result)
     end
   end
 end

data/lib/fitbit_api/helpers/configuration.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   module Configuration
     def configure
@@ -5,22 +7,14 @@ module FitbitAPI
     end
 
     def define_setting(name, default = nil)
-      class_variable_set("@@#{name}", default)
-
-      define_class_method "#{name}=" do |value|
-        class_variable_set("@@#{name}", value)
-      end
+      instance_variable_set("@#{name}", default)
 
-      define_class_method name do
-        class_variable_get("@@#{name}")
+      define_singleton_method("#{name}=") do |value|
+        instance_variable_set("@#{name}", value)
       end
-    end
-
-    private
 
-    def define_class_method(name, &block)
-      (class << self; self; end).instance_eval do
-        define_method name, &block
+      define_singleton_method(name) do
+        instance_variable_get("@#{name}")
       end
     end
   end

data/lib/fitbit_api/helpers/exceptions.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class InvalidArgumentError < ArgumentError
   end

data/lib/fitbit_api/helpers/utils.rb

@@ -1,18 +1,22 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class Client
-    PERIODS = %w(1d 7d 30d 1w 1m 3m 6m 1y max).freeze
+    PERIODS = %w[1d 7d 30d 1w 1m 3m 6m 1y max].freeze
 
     def format_date(date)
       if [Date, Time, DateTime].include?(date.class)
         date.strftime('%Y-%m-%d')
       elsif date.is_a? String
-        if date =~ /\d{4}\-\d{2}\-\d{2}/
+        if date =~ /\d{4}-\d{2}-\d{2}/
           date
         else
-          raise FitbitAPI::InvalidArgumentError, "Invalid argument [\"#{date}\"] - string must follow yyyy-MM-dd format."
+          raise FitbitAPI::InvalidArgumentError,
+                "Invalid argument [\"#{date}\"] - string must follow yyyy-MM-dd format."
         end
       else
-        raise FitbitAPI::InvalidArgumentError, "Invalid type [#{date.class}] - provide a Date/Time/DateTime or a String(yyyy-MM-dd format)."
+        raise FitbitAPI::InvalidArgumentError,
+              "Invalid type [#{date.class}] - provide a Date/Time/DateTime or a String(yyyy-MM-dd format)."
       end
     end
 
@@ -20,13 +24,15 @@ module FitbitAPI
       if [Time, DateTime].include?(time.class)
         time.strftime('%H:%M')
       elsif time.is_a? String
-        if time =~ /\d{2}\:\d{2}/
-          time
-        else
+        unless time =~ /\d{2}:\d{2}/
           raise FitbitAPI::InvalidArgumentError, "Invalid argument [\"#{time}\"] - string must follow HH:mm format."
         end
+
+        time
+
       else
-        raise FitbitAPI::InvalidArgumentError, "Invalid type [#{time.class}] - provide a Time/DateTime or a String(HH:mm format)."
+        raise FitbitAPI::InvalidArgumentError,
+              "Invalid type [#{time.class}] - provide a Time/DateTime or a String(HH:mm format)."
       end
     end
 
@@ -34,6 +40,12 @@ module FitbitAPI
       scope.is_a?(Array) ? scope.join(' ') : scope
     end
 
+    def strip_root_key(object)
+      return object unless object.is_a?(Hash) && object.keys.length == 1
+
+      object.values[0]
+    end
+
     def deep_keys_to_snake_case!(object)
       deep_transform_keys!(object) { |key| to_snake_case(key, replace_dashes: true) }
     end
@@ -43,40 +55,49 @@ module FitbitAPI
     end
 
     def deep_symbolize_keys!(object)
-      deep_transform_keys!(object) { |key| key.to_sym rescue key }
+      deep_transform_keys!(object) do |key|
+        key.to_sym
+      rescue StandardError
+        key
+      end
     end
 
     # Inspired by ActiveSupport's implementation
     def deep_transform_keys!(object, &block)
       case object
       when Hash
-        object.keys.each do |key|
+        keys = object.keys
+        keys.each do |key|
           value = object.delete(key)
-          object[yield(key)] = deep_transform_keys!(value) { |key| yield(key) }
+          object[yield(key)] = deep_transform_keys!(value, &block)
         end
         object
       when Array
-        object.map! { |e| deep_transform_keys!(e) { |key| yield(key) } }
+        object.map! { |e| deep_transform_keys!(e, &block) }
       else
         object
       end
     end
 
-    def to_snake_case(word, opts={})
+    def to_snake_case(word, opts = {})
       string = word.to_s.dup
       return string.downcase if string.match(/\A[A-Z]+\z/)
+
       string.gsub!(/([A-Z]+)([A-Z][a-z])/, '\1_\2')
       string.gsub!(/([a-z])([A-Z])/, '\1_\2')
       string.gsub!('-', '_') if opts[:replace_dashes]
       string.downcase
     end
 
-    def to_camel_case(word, opts={})
+    def to_camel_case(word, opts = {})
       string = word.to_s
-      return string if string.match(/[A-Z]|[a-z]([A-Z0-9]*[a-z][a-z0-9]*[A-Z]|[a-z0-9]*[A-Z][A-Z0-9]*[a-z])[A-Za-z0-9]*/)
+      if string.match(/[A-Z]|[a-z]([A-Z0-9]*[a-z][a-z0-9]*[A-Z]|[a-z0-9]*[A-Z][A-Z0-9]*[a-z])[A-Za-z0-9]*/)
+        return string
+      end
+
       string = word.to_s.split('_').collect(&:capitalize).join
-      string.gsub!(/^\w{1}/) { |word| word.downcase } if opts[:lower]
-      return string
+      string.gsub!(/^\w{1}/, &:downcase) if opts[:lower]
+      string
     end
   end
 end

data/lib/fitbit_api/meals.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class Client
     # Retrieves a list of meals created by the user from their food log

data/lib/fitbit_api/oxygen_saturation.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class Client
     # Returns the oxygen saturation summary data for a given date or date range.
@@ -12,18 +14,14 @@ module FitbitAPI
     # @option params :start_date [Date] The start of the date range
     # @option params :end_date [Date] The end of the date range
 
-    def oxygen_saturation_summary(opts={})
+    def oxygen_saturation_summary(opts = {})
       date       = opts[:date] || Date.today
       start_date = opts[:start_date]
       end_date   = opts[:end_date]
 
-      if start_date && !end_date
-        end_date = Date.today
-      end
+      end_date = Date.today if start_date && !end_date
 
-      unless date || start_date
-        raise FitbitAPI::InvalidArgumentError, 'A date or start_date and end_date are required.'
-      end
+      raise FitbitAPI::InvalidArgumentError, 'A date or start_date and end_date are required.' unless date || start_date
 
       if start_date
         get("user/#{user_id}/spo2/date/#{format_date(start_date)}/#{format_date(end_date)}.json")
@@ -44,18 +42,14 @@ module FitbitAPI
     # @option params :start_date [Date] The start of the date range
     # @option params :end_date [Date] The end of the date range
 
-    def oxygen_saturation_intraday(opts={})
+    def oxygen_saturation_intraday(opts = {})
       date       = opts[:date] || Date.today
       start_date = opts[:start_date]
       end_date   = opts[:end_date]
 
-      if start_date && !end_date
-        end_date = Date.today
-      end
+      end_date = Date.today if start_date && !end_date
 
-      unless date || start_date
-        raise FitbitAPI::InvalidArgumentError, 'A date or start_date and end_date are required.'
-      end
+      raise FitbitAPI::InvalidArgumentError, 'A date or start_date and end_date are required.' unless date || start_date
 
       if start_date
         get("user/#{user_id}/spo2/date/#{format_date(start_date)}/#{format_date(end_date)}/all.json")

data/lib/fitbit_api/sleep.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class Client
-    SLEEP_RESOURCES = %w(startTime timeInBed minutesAsleep awakeningsCount
-                         minutesAwake minutesToFallAsleep minutesAfterWakeup efficiency)
+    SLEEP_RESOURCES = %w[startTime timeInBed minutesAsleep awakeningsCount
+                         minutesAwake minutesToFallAsleep minutesAfterWakeup efficiency].freeze
 
     # Returns a list of a user's sleep log entries for a given date. The data returned can include sleep
     # periods that began on the previous date. For example, if you request a Sleep Log for 2021-12-22,
@@ -9,7 +11,7 @@ module FitbitAPI
     #
     # @param date [Date, String] The date for the sleep log to be returned in the format yyyy-MM-dd
 
-    def sleep_logs(date=Date.today)
+    def sleep_logs(date = Date.today)
       get("user/#{user_id}/sleep/date/#{format_date(date)}.json")
     end
 
@@ -28,7 +30,7 @@ module FitbitAPI
     # @option params :offset [Integer] The offset number of entries. Must always be 0
     # @option params :limit [Integer] The max of the number of entries returned (max: 100)
 
-    def sleep_logs_list(params={})
+    def sleep_logs_list(params = {})
       default_params = { before_date: Date.today, after_date: nil, sort: 'desc', limit: 20, offset: 0 }
       get("user/#{user_id}/sleep/list.json", default_params.merge(params))
     end
@@ -44,31 +46,30 @@ module FitbitAPI
       delete("user/#{user_id}/sleep/#{sleep_log_id}.json")
     end
 
-    def sleep_time_series(resource, opts={})
+    def sleep_time_series(resource, opts = {})
       start_date = opts[:start_date]
       end_date   = opts[:end_date] || Date.today
       period     = opts[:period]
 
       unless SLEEP_RESOURCES.include?(resource)
-        raise FitbitAPI::InvalidArgumentError, "Invalid resource: \"#{resource}\". Please provide one of the following: #{SLEEP_RESOURCES}."
+        raise FitbitAPI::InvalidArgumentError,
+              "Invalid resource: \"#{resource}\". Please provide one of the following: #{SLEEP_RESOURCES}."
       end
 
-      if [period, start_date].none?
-        raise FitbitAPI::InvalidArgumentError, 'A start_date or period is required.'
-      end
+      raise FitbitAPI::InvalidArgumentError, 'A start_date or period is required.' if [period, start_date].none?
 
       if period && !PERIODS.include?(period)
-        raise FitbitAPI::InvalidArgumentError, "Invalid period: \"#{period}\". Please provide one of the following: #{PERIODS}."
+        raise FitbitAPI::InvalidArgumentError,
+              "Invalid period: \"#{period}\". Please provide one of the following: #{PERIODS}."
       end
 
-      if period
-        result = get("user/#{user_id}/sleep/#{resource}/date/#{format_date(end_date)}/#{period}.json")
-      else
-        result = get("user/#{user_id}/sleep/#{resource}/date/#{format_date(start_date)}/#{format_date(end_date)}.json")
-      end
+      result = if period
+                 get("user/#{user_id}/sleep/#{resource}/date/#{format_date(end_date)}/#{period}.json")
+               else
+                 get("user/#{user_id}/sleep/#{resource}/date/#{format_date(start_date)}/#{format_date(end_date)}.json")
+               end
 
-      # remove root key from response
-      result.values[0]
+      strip_root_key(result)
     end
   end
 end

data/lib/fitbit_api/subscriptions.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class Client
     # Retrieves a list of subscriptions created by your application for a specific user.
@@ -6,31 +8,31 @@ module FitbitAPI
     #
     # @param collection_path [String] Collection of data to retrieve notifications
 
-    def subscriptions(collection_path=nil)
+    def subscriptions(collection_path = nil)
       get("#{subscriptions_path(collection_path)}.json")
     end
 
     # Creates a subscription to notify the application when a user has new data available.
     #
-    # @param collection_path [String] Collection of data to retrieve notifications
     # @param subscription_id [Integer] The unique ID of the subscription created by the API client application
+    # @param collection_path [String] Collection of data to retrieve notifications
 
-    def create_subscription(collection_path=nil, subscription_id)
+    def create_subscription(subscription_id, collection_path = nil)
       post("#{subscriptions_path(collection_path)}/#{subscription_id}.json")
     end
 
     # Deletes a subscription for a specific user.
     #
-    # @param collection_path [String] Collection of data to retrieve notifications
     # @param subscription_id [Integer] The unique ID of the subscription created by the API client application
+    # @param collection_path [String] Collection of data to retrieve notifications
 
-    def delete_subscription(collection_path=nil, subscription_id)
+    def delete_subscription(subscription_id, collection_path = nil)
       delete("#{subscriptions_path(collection_path)}/#{subscription_id}.json")
     end
 
     private
 
-    def subscriptions_path(collection_path=nil)
+    def subscriptions_path(collection_path = nil)
       collection_path = "#{collection_path}/" if collection_path
       "user/#{user_id}/#{collection_path}apiSubscriptions"
     end