fitbit_api 0.14.2 → 0.15.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 417f3489c6b33f4acc4d8eebf14048c5a50b4f86905679f04a436472aea67bd7
-  data.tar.gz: 01ebe9bcc0db08753267015b0ac50a4950c8394c1f7b8563298d2b7657544007
+  metadata.gz: d911585848ded1bbf5c5abfd9090b96971592ce119e6525e7b82808468717880
+  data.tar.gz: 1d4984610d1bba6468662b5f22d986a324642685811c9eede3bf35127b6e9bfe
 SHA512:
-  metadata.gz: 66c83ce386d49728e5ec75f4e343c18a015f776c6cc8b8721ff1c4e2700098ece8f1c6b2c3788a7cc1521d3f1302319f25552ea7a1a70b0b9a93ffd0d8e47d52
-  data.tar.gz: 4e97184f094af8aed5a1fd8ea427a9be8c128ebbe439990a8c933a3463c7b8ddb0b31b64ce512d5998118a8f1988aa0ed56ad6009f6045e282d06fc131723bc3
+  metadata.gz: fbfd618baee4f3c58b957d7f4997befa6bf25ee9778cf45d340ed13ed64188eb663781aa6a6f326d63263e0d1de21328fbbdb5e2e04c399b18aa9f617c29b852
+  data.tar.gz: e4acd060d4d06f1a39ca1c951140db0413b1e6967f252c7abe0dc471d32295c401e67a34e20ea86531b629afcbc319d70265e0c4cbea7cd6afa646b23fe6e1ae

data/.github/workflows/{test.yml → ci.yml}

@@ -1,4 +1,4 @@
-name: Test
+name: CI
 
 on:
   - push
@@ -8,7 +8,7 @@ permissions:
   contents: read
 
 jobs:
-  test:
+  build:
     runs-on: ubuntu-latest
     strategy:
       matrix:
@@ -25,5 +25,7 @@ jobs:
         with:
           ruby-version: ${{ matrix.ruby-version }}
           bundler-cache: true
+      - name: Run Rubocop
+        run: bundle exec rubocop
       - name: Run tests
         run: bundle exec rake

data/.rubocop.yml

@@ -0,0 +1,28 @@
+AllCops:
+  TargetRubyVersion: 2.6
+  NewCops: enable
+
+Gemspec/RequiredRubyVersion:
+  Enabled: false
+
+Style/Documentation:
+  Enabled: false
+
+Metrics/AbcSize:
+  Enabled: false
+
+Metrics/CyclomaticComplexity:
+  Enabled: false
+
+Metrics/PerceivedComplexity:
+  Enabled: false
+
+Metrics/ClassLength:
+  Max: 200
+
+Metrics/MethodLength:
+  Max: 40
+
+Metrics/BlockLength:
+  Exclude:
+    - '**/*_spec.rb'

data/CHANGELOG.md

@@ -1,3 +1,17 @@
+0.15.1
+------
+- Improve gemspec metadata
+
+0.15.0
+------
+- The following methods now take a subscription_id as the first argument:
+  - create_subscription
+  - delete_subscription
+- Fix scope param handling for `auth_url`
+- Refactor configuration logic
+- Improve endpoint request specs
+- Format code with Rubocop
+
 0.14.2
 ------
 - Add documentation for more methods

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in fitbit_api.gemspec

data/README.md

@@ -1,7 +1,7 @@
 # FitbitAPI
 
 [![Gem Version](https://img.shields.io/gem/v/fitbit_api)](https://rubygems.org/gems/fitbit_api)
-[![Test Workflow](https://img.shields.io/github/workflow/status/zokioki/fitbit_api/Test)](https://github.com/zokioki/fitbit_api/actions/workflows/test.yml)
+[![CI Workflow](https://img.shields.io/github/workflow/status/zokioki/fitbit_api/CI)](https://github.com/zokioki/fitbit_api/actions/workflows/ci.yml)
 
 FitbitAPI provides a Ruby interface to the [Fitbit Web API](https://dev.fitbit.com/reference/web-api).
 
@@ -95,16 +95,16 @@ client.log_activity activity_id: 12345, durationMillis: '683300'
 
 When initializing a `FitbitAPI::Client` instance, you're given access to a handful of options:
 
-| option             | description                  |
-| ------------------ | -----------------------------|
-| api_version        | API version to be used when making requests (default: "1") |
-| unit_system        | The measurement unit system to use for response values (default: "en_US") |
-| locale             | The locale to use for response values (default: "en_US") |
-| scope              | A space-delimited list of permissions being requested (default: "activity nutrition profile settings sleep social weight heartrate respiratory_rate oxygen_saturation cardio_fitness temperature") |
-| snake_case_keys    | Transform response payload's keys to snake case format (default: false) |
-| symbolize_keys     | Transform response payload's keys to symbols (default: false) |
-| auto_refresh_token | Automatically refreshes the access token once expired (default: true) |
-| on_token_refresh   | A callback to be invoked whenever the access token is refreshed (default: nil) |
+| Option               | Description                  | Default        |
+| -------------------- | ---------------------------- | -------------- |
+| `api_version`        | API version to be used when making requests | `"1"` |
+| `unit_system`        | The measurement unit system to use for response values | `"en_US"` |
+| `locale`             | The locale to use for response values | `"en_US"` |
+| `scope`              | A list of permissions being requested (array or space-delimited string) | `%w[activity nutrition profile settings sleep social weight heartrate respiratory_rate oxygen_saturation cardio_fitness temperature]` |
+| `snake_case_keys`    | Transform response payload's keys to snake case format | `false` |
+| `symbolize_keys`     | Transform response payload's keys to symbols | `false` |
+| `auto_refresh_token` | Automatically refreshes the access token once expired | `true` |
+| `on_token_refresh`   | A callback to be invoked whenever the access token is refreshed | `nil` |
 
 If using this library in Rails, you can configure these options globally in an initializer:
 

data/Rakefile

@@ -1,6 +1,8 @@
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
+# frozen_string_literal: true
+
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
 
 RSpec::Core::RakeTask.new(:spec)
 
-task :default => :spec
+task default: :spec

data/fitbit_api.gemspec

@@ -1,6 +1,8 @@
-# coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
+# frozen_string_literal: true
+
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+
 require 'fitbit_api/version'
 
 Gem::Specification.new do |spec|
@@ -8,10 +10,17 @@ Gem::Specification.new do |spec|
   spec.version       = FitbitAPI::VERSION
   spec.authors       = ['Zoran']
 
-  spec.summary       = %q{A Ruby interface to the Fitbit Web API.}
+  spec.summary       = 'A Ruby interface to the Fitbit Web API.'
   spec.homepage      = 'https://github.com/zokioki/fitbit_api'
   spec.license       = 'MIT'
 
+  spec.metadata      = {
+    'source_code_uri' => spec.homepage,
+    'changelog_uri' => "#{spec.homepage}/blob/v#{FitbitAPI::VERSION}/CHANGELOG.md",
+    'documentation_uri' => "https://www.rubydoc.info/gems/fitbit_api/#{FitbitAPI::VERSION}",
+    'rubygems_mfa_required' => 'true'
+  }
+
   spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(bin|test|spec|features)/}) }
   spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
@@ -21,8 +30,10 @@ Gem::Specification.new do |spec|
 
   spec.add_runtime_dependency     'oauth2', '~> 1.0'
 
-  spec.add_development_dependency 'byebug', '~> 11.0'
   spec.add_development_dependency 'bundler', '~> 2.3'
+  spec.add_development_dependency 'byebug', '~> 11.0'
   spec.add_development_dependency 'rake', '~> 13.0'
   spec.add_development_dependency 'rspec', '~> 3.11'
+  spec.add_development_dependency 'rubocop', '~> 1.39'
+  spec.add_development_dependency 'webmock', '~> 3.18'
 end

data/lib/fitbit_api/activities.rb

@@ -1,21 +1,22 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class Client
-
-    ACTIVITY_RESOURCES = %w(calories caloriesBMR steps distance floors elevation
+    ACTIVITY_RESOURCES = %w[calories caloriesBMR steps distance floors elevation
                             minutesSedentary minutesLightlyActive minutesFairlyActive
                             minutesVeryActive activityCalories tracker/calories
                             tracker/steps tracker/distance tracker/floors
                             tracker/elevation tracker/minutesSedentary
                             tracker/minutesLightlyActive tracker/minutesFairlyActive
-                            tracker/minutesVeryActive tracker/activityCalories)
+                            tracker/minutesVeryActive tracker/activityCalories].freeze
 
-    ACTIVITY_INTRADAY_RESOURCES = %w(calories steps distance floors elevation)
+    ACTIVITY_INTRADAY_RESOURCES = %w[calories steps distance floors elevation].freeze
 
     # Retrieves a summary and list of a user's activities and activity log entries for a given day.
     #
     # @param date [Date] The date for which to retrieve the activity data.
 
-    def daily_activity_summary(date=Date.today)
+    def daily_activity_summary(date = Date.today)
       get("user/#{user_id}/activities/date/#{format_date(date)}.json")
     end
 
@@ -59,7 +60,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: 20)
 
-    def activity_logs_list(params={})
+    def activity_logs_list(params = {})
       default_params = { before_date: Date.today, after_date: nil, sort: 'desc', limit: 20, offset: 0 }
       get("user/#{user_id}/activities/list.json", default_params.merge(params))
     end
@@ -82,41 +83,42 @@ module FitbitAPI
       get("user/#{user_id}/activities.json")
     end
 
-    def activity_time_series(resource, opts={})
+    def activity_time_series(resource, opts = {})
       start_date = opts[:start_date]
       end_date   = opts[:end_date] || Date.today
       period     = opts[:period]
 
       unless ACTIVITY_RESOURCES.include?(resource)
-        raise FitbitAPI::InvalidArgumentError, "Invalid resource: \"#{resource}\". Please provide one of the following: #{ACTIVITY_RESOURCES}."
+        raise FitbitAPI::InvalidArgumentError,
+              "Invalid resource: \"#{resource}\". Please provide one of the following: #{ACTIVITY_RESOURCES}."
       end
 
-      if [start_date, period].none?
-        raise FitbitAPI::InvalidArgumentError, 'A start_date or period is required.'
-      end
+      raise FitbitAPI::InvalidArgumentError, 'A start_date or period is required.' if [start_date, period].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/#{resource}/date/#{format_date(end_date)}/#{period}.json")
-      else
-        result = get("user/#{user_id}/activities/#{resource}/date/#{format_date(start_date)}/#{format_date(end_date)}.json")
-      end
+      path = if period
+               "user/#{user_id}/activities/#{resource}/date/#{format_date(end_date)}/#{period}.json"
+             else
+               "user/#{user_id}/activities/#{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
 
-    def activity_intraday_time_series(resource, opts={})
+    def activity_intraday_time_series(resource, opts = {})
       date         = opts[:date] || Date.today
       detail_level = opts[:detail_level]
       start_time   = opts[:start_time]
       end_time     = opts[:end_time]
 
       unless ACTIVITY_INTRADAY_RESOURCES.include?(resource)
-        raise FitbitAPI::InvalidArgumentError, "Invalid resource: \"#{resource}\". Please provide one of the following: #{ACTIVITY_RESOURCES}."
+        raise FitbitAPI::InvalidArgumentError,
+              "Invalid resource: \"#{resource}\". Please provide one of the following: #{ACTIVITY_RESOURCES}."
       end
 
       if [date, detail_level].any?(&:nil?)
@@ -124,18 +126,18 @@ module FitbitAPI
       end
 
       unless %(1min 15min).include? detail_level
-        raise FitbitAPI::InvalidArgumentError, "Invalid detail_level: \"#{detail_level}\". Please provide one of the following: \"1min\" or \"15min\"."
+        raise FitbitAPI::InvalidArgumentError,
+              "Invalid detail_level: \"#{detail_level}\". Please provide one of the following: \"1min\" or \"15min\"."
       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/#{resource}/date/#{format_date(date)}/1d/#{detail_level}/time/#{format_time(start_time)}/#{format_time(end_time)}.json")
-      else
-        get("user/#{user_id}/activities/#{resource}/date/#{format_date(date)}/1d/#{detail_level}.json")
-      end
+      path = "user/#{user_id}/activities/#{resource}/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
 
     # Creates log entry for an activity or user's private custom activity using units
@@ -147,7 +149,7 @@ module FitbitAPI
     #
     # @option body :activity_id [Integer, String] The activity ID
     # @option body :activity_name [String] Custom activity name. Either activity ID or activity_name must be provided
-    # @option body :manual_calories [Integer] Calories burned, specified manually. Required with activity_name, otherwise optional
+    # @option body :manual_calories [Integer] Calories burned, specified manually. Required if activity_name is given
     # @option body :start_time [String] Activity start time; formatted in HH:mm:ss
     # @option body :duration_millis [Integer] Duration in milliseconds
     # @option body :date [String] Log entry date; formatted in yyyy-MM-dd

data/lib/fitbit_api/alarms.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class Client
     # Returns a list of the set alarms connected to a user's account.
@@ -18,9 +20,9 @@ module FitbitAPI
     # @option body :time [String] Time of day that the alarm vibrates with a UTC timezone offset, e.g. 07:15-08:00
     # @option body :enabled [Boolean] If false, alarm does not vibrate until enabled is set to true
     # @option body :recurring [Boolean] If false, the alarm is a single event
-    # @option body :week_days [String] Comma separated list of days of the week on which the alarm vibrates (MONDAY,TUESDAY)
+    # @option body :week_days [String] Comma separated list of days on which the alarm vibrates (e.g. MONDAY,TUESDAY)
 
-    def add_alarm(tracker_id, body={})
+    def add_alarm(tracker_id, body = {})
       post("user/#{user_id}/devices/tracker/#{tracker_id}/alarms.json", body)
     end
 
@@ -35,13 +37,13 @@ module FitbitAPI
     # @option body :time [String] Time of day that the alarm vibrates with a UTC timezone offset, e.g. 07:15-08:00
     # @option body :enabled [Boolean] If false, alarm does not vibrate until enabled is set to true
     # @option body :recurring [Boolean] If false, the alarm is a single event
-    # @option body :week_days [String] Comma separated list of days of the week on which the alarm vibrates (MONDAY,TUESDAY)
+    # @option body :week_days [String] Comma separated list of days on which the alarm vibrates (e.g. MONDAY,TUESDAY)
     # @option body :snooze_length [Integer] Minutes between alarms
     # @option body :snooze_count [Integer] Maximum snooze count
     # @option body :label [String] Label for alarm
     # @option body :vibe [String] Vibe pattern; only one value for now (DEFAULT)
 
-    def update_alarm(tracker_id, alarm_id, body={})
+    def update_alarm(tracker_id, alarm_id, body = {})
       post("user/#{user_id}/devices/tracker/#{tracker_id}/alarms/#{alarm_id}.json", body)
     end
 

data/lib/fitbit_api/base.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'fitbit_api/helpers/configuration'
 require 'fitbit_api/helpers/utils'
 require 'fitbit_api/helpers/exceptions'
@@ -15,7 +17,9 @@ module FitbitAPI
 
   define_setting :unit_system,     'en_US'
   define_setting :locale,          'en_US'
-  define_setting :scope,           'activity nutrition profile settings sleep social weight heartrate respiratory_rate oxygen_saturation cardio_fitness temperature'
+  define_setting :scope,           %w[activity nutrition profile settings sleep social weight
+                                      heartrate respiratory_rate oxygen_saturation cardio_fitness
+                                      temperature]
 
   define_setting :api_version,     '1'
 

data/lib/fitbit_api/body.rb

@@ -1,40 +1,41 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class Client
-    BODY_RESOURCES = %w(bmi fat weight)
+    BODY_RESOURCES = %w[bmi fat weight].freeze
 
-    def weight_logs(date=Date.today)
+    def weight_logs(date = Date.today)
       get("user/#{user_id}/body/log/weight/date/#{format_date(date)}.json")
     end
 
-    def body_fat_logs(date=Date.today)
+    def body_fat_logs(date = Date.today)
       get("user/#{user_id}/body/log/fat/date/#{format_date(date)}.json")
     end
 
-    def body_time_series(resource, opts={})
+    def body_time_series(resource, opts = {})
       start_date = opts[:start_date]
       end_date   = opts[:end_date] || Date.today
       period     = opts[:period]
 
       unless BODY_RESOURCES.include?(resource)
-        raise FitbitAPI::InvalidArgumentError, "Invalid resource: \"#{resource}\". Please provide one of the following: #{BODY_RESOURCES}."
+        raise FitbitAPI::InvalidArgumentError,
+              "Invalid resource: \"#{resource}\". Please provide one of the following: #{BODY_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}/body/#{resource}/date/#{format_date(end_date)}/#{period}.json")
-      else
-        result = get("user/#{user_id}/body/#{resource}/date/#{format_date(start_date)}/#{format_date(end_date)}.json")
-      end
+      result = if period
+                 get("user/#{user_id}/body/#{resource}/date/#{format_date(end_date)}/#{period}.json")
+               else
+                 get("user/#{user_id}/body/#{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
 
     def log_weight(body)

data/lib/fitbit_api/breathing_rate.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class Client
     # Returns the average breathing rate 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 breathing_rate_summary(opts={})
+    def breathing_rate_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}/br/date/#{format_date(start_date)}/#{format_date(end_date)}.json")
-      else
-        result = get("user/#{user_id}/br/date/#{format_date(date)}.json")
-      end
+      result = if start_date
+                 get("user/#{user_id}/br/date/#{format_date(start_date)}/#{format_date(end_date)}.json")
+               else
+                 get("user/#{user_id}/br/date/#{format_date(date)}.json")
+               end
 
-      # remove root key from response
-      result.values[0]
+      strip_root_key(result)
     end
 
     # Returns the intraday breathing rate 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 breathing_rate_intraday(opts={})
+    def breathing_rate_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}/br/date/#{format_date(start_date)}/#{format_date(end_date)}/all.json")
-      else
-        result = get("user/#{user_id}/br/date/#{format_date(date)}/all.json")
-      end
+      result = if start_date
+                 get("user/#{user_id}/br/date/#{format_date(start_date)}/#{format_date(end_date)}/all.json")
+               else
+                 get("user/#{user_id}/br/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/cardio_score.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module FitbitAPI
   class Client
     # Returns the cardio fitness score 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 cardio_score_summary(opts={})
+    def cardio_score_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}/cardioscore/date/#{format_date(start_date)}/#{format_date(end_date)}.json")
-      else
-        result = get("user/#{user_id}/cardioscore/date/#{format_date(date)}.json")
-      end
+      result = if start_date
+                 get("user/#{user_id}/cardioscore/date/#{format_date(start_date)}/#{format_date(end_date)}.json")
+               else
+                 get("user/#{user_id}/cardioscore/date/#{format_date(date)}.json")
+               end
 
-      # remove root key from response
-      result.values[0]
+      strip_root_key(result)
     end
   end
 end

data/lib/fitbit_api/client.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'fitbit_api/base'
 require 'fitbit_api/activities'
 require 'fitbit_api/breathing_rate'
@@ -24,7 +26,7 @@ module FitbitAPI
                   :snake_case_keys, :symbolize_keys, :auto_refresh_token, :on_token_refresh
     attr_reader   :token, :user_id
 
-    def initialize(opts={})
+    def initialize(opts = {})
       validate_args(opts)
       assign_attrs(opts)
       set_client
@@ -34,7 +36,7 @@ module FitbitAPI
     # Returns the authorize endpoint URL of the OAuth2 provider.
 
     def auth_url
-      @client.auth_code.authorize_url(redirect_uri: @redirect_uri, scope: @scope)
+      @client.auth_code.authorize_url(redirect_uri: @redirect_uri, scope: format_scope(@scope))
     end
 
     # Returns an OAuth2::AccessToken instance obtained from the given authorization code.
@@ -77,7 +79,7 @@ module FitbitAPI
     # @param params [Hash] The query parameters
     # @param opts [Hash] Additional request options (e.g. headers)
 
-    def get(path, params={}, opts={}, &block)
+    def get(path, params = {}, opts = {}, &block)
       request(:get, path, opts.merge(params: params), &block)
     end
 
@@ -87,7 +89,7 @@ module FitbitAPI
     # @param body [Hash] The request body
     # @param opts [Hash] Additional request options (e.g. headers)
 
-    def post(path, body={}, opts={}, &block)
+    def post(path, body = {}, opts = {}, &block)
       request(:post, path, opts.merge(body: body), &block)
     end
 
@@ -97,7 +99,7 @@ module FitbitAPI
     # @param params [Hash] The query parameters
     # @param opts [Hash] Additional request options (e.g. headers)
 
-    def delete(path, params={}, opts={}, &block)
+    def delete(path, params = {}, opts = {}, &block)
       request(:delete, path, opts.merge(params: params), &block)
     end
 
@@ -112,6 +114,7 @@ module FitbitAPI
       end
 
       return if missing_args.empty?
+
       raise FitbitAPI::InvalidArgumentError,
             "Required arguments: #{missing_args.join(', ')}"
     end
@@ -157,7 +160,7 @@ module FitbitAPI
       refresh_token! if @token.token.empty?
     end
 
-    def request(verb, path, opts={}, &block)
+    def request(verb, path, opts = {}, &block)
       request_path = "#{@api_version}/#{path}"
       request_headers = default_request_headers.merge(opts[:headers] || {})
       request_options = opts.merge(headers: request_headers)
@@ -174,7 +177,7 @@ module FitbitAPI
     end
 
     def auth_headers
-      { 'Authorization' => ('Basic ' + Base64.encode64(@client_id + ':' + @client_secret)) }
+      { 'Authorization' => "Basic #{Base64.encode64("#{@client_id}:#{@client_secret}")}" }
     end
 
     def default_request_headers

data/lib/fitbit_api/devices.rb

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