chore: replace rubocop with standardrb

Author: masterkain

.github/workflows/test.yml

@@ -41,7 +41,7 @@ jobs:
           bundle install --jobs 4 --retry 2
 
       - name: Lint
-        run: bundle exec rake lint
+        run: bundle exec standardrb
 
       - name: Strategy tests
         run: bundle exec rake test_unit

.rubocop.yml

@@ -0,0 +1,4 @@
+ruby_version: 3.2
+
+plugins:
+  - rubocop-minitest

.standard.yml

@@ -1,20 +1,20 @@
 # frozen_string_literal: true
 
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
 gemspec
 
-if ENV['OMNIAUTH_OAUTH2'] == 'head'
-  gem 'omniauth-oauth2', git: 'https://github.com/omniauth/omniauth-oauth2.git'
-elsif ENV['OMNIAUTH_OAUTH2']
-  gem 'omniauth-oauth2', ENV['OMNIAUTH_OAUTH2']
+if ENV["OMNIAUTH_OAUTH2"] == "head"
+  gem "omniauth-oauth2", git: "https://github.com/omniauth/omniauth-oauth2.git"
+elsif ENV["OMNIAUTH_OAUTH2"]
+  gem "omniauth-oauth2", ENV["OMNIAUTH_OAUTH2"]
 end
 
-gem 'rails', ENV['RAILS_VERSION'] if ENV['RAILS_VERSION']
+gem "rails", ENV["RAILS_VERSION"] if ENV["RAILS_VERSION"]
 
-gem 'minitest', '>= 5.20'
-gem 'rack-test', '>= 2.1'
-gem 'rake', '>= 13.1'
-gem 'rubocop', '>= 1.70'
-gem 'rubocop-minitest', '>= 0.36'
-gem 'webmock', '>= 3.24'
+gem "minitest", ">= 5.20"
+gem "rack-test", ">= 2.1"
+gem "rake", ">= 13.1"
+gem "rubocop-minitest", ">= 0.39", require: false
+gem "standard", ">= 1.54.0"
+gem "webmock", ">= 3.24"

Gemfile

@@ -1,27 +1,27 @@
 # frozen_string_literal: true
 
-require 'bundler/gem_tasks'
-require 'rake/testtask'
-require 'rubocop/rake_task'
+require "bundler/gem_tasks"
+require "rake/testtask"
+require "standard/rake"
 
 Rake::TestTask.new(:test_unit) do |task|
-  task.libs << 'lib'
-  task.libs << 'test'
-  task.test_files = FileList['test/omniauth_spotify_test.rb']
+  task.libs << "lib"
+  task.libs << "test"
+  task.test_files = FileList["test/omniauth_spotify_test.rb"]
 end
 
 Rake::TestTask.new(:test_rails_integration) do |task|
-  task.libs << 'lib'
-  task.libs << 'test'
-  task.test_files = FileList['test/rails_integration_test.rb']
+  task.libs << "lib"
+  task.libs << "test"
+  task.test_files = FileList["test/rails_integration_test.rb"]
 end
 
 Rake::TestTask.new(:test) do |task|
-  task.libs << 'lib'
-  task.libs << 'test'
-  task.test_files = FileList['test/**/*_test.rb']
+  task.libs << "lib"
+  task.libs << "test"
+  task.test_files = FileList["test/**/*_test.rb"]
 end
 
-RuboCop::RakeTask.new(:lint)
+task lint: :standard
 
-task default: %i[lint test_unit]
+task default: %i[standard test_unit]

Rakefile

@@ -1,3 +1,3 @@
 # frozen_string_literal: true
 
-require 'omniauth/spotify'
+require "omniauth/spotify"

lib/omniauth-spotify.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'omniauth/spotify/version'
+require "omniauth/spotify/version"
 
 # Backward compatibility for historical constant usage.
 module Omniauth

lib/omniauth-spotify/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 
-require 'omniauth/spotify/version'
-require 'omniauth/strategies/spotify'
+require "omniauth/spotify/version"
+require "omniauth/strategies/spotify"

lib/omniauth/spotify.rb

@@ -2,6 +2,6 @@
 
 module OmniAuth
   module Spotify
-    VERSION = '1.0.2'
+    VERSION = "1.0.2"
   end
 end

lib/omniauth/spotify/version.rb

@@ -1,67 +1,67 @@
 # frozen_string_literal: true
 
-require 'date'
-require 'omniauth-oauth2'
+require "date"
+require "omniauth-oauth2"
 
 module OmniAuth
   module Strategies
     # OmniAuth strategy for Spotify OAuth2.
     class Spotify < OmniAuth::Strategies::OAuth2
-      option :name, 'spotify'
+      option :name, "spotify"
 
-      FORCE_APPROVAL_KEY = 'omniauth_spotify_force_approval?'
-      LEGACY_FORCE_APPROVAL_KEY = 'ommiauth_spotify_force_approval?'
+      FORCE_APPROVAL_KEY = "omniauth_spotify_force_approval?"
+      LEGACY_FORCE_APPROVAL_KEY = "ommiauth_spotify_force_approval?"
 
       option :client_options,
-             site: 'https://api.spotify.com',
-             authorize_url: 'https://accounts.spotify.com/authorize',
-             token_url: 'https://accounts.spotify.com/api/token',
-             connection_opts: {
-               headers: {
-                 user_agent: 'icoretech-omniauth-spotify gem',
-                 accept: 'application/json',
-                 content_type: 'application/json'
-               }
-             }
-
-      uid { raw_info['id'] }
+        site: "https://api.spotify.com",
+        authorize_url: "https://accounts.spotify.com/authorize",
+        token_url: "https://accounts.spotify.com/api/token",
+        connection_opts: {
+          headers: {
+            user_agent: "icoretech-omniauth-spotify gem",
+            accept: "application/json",
+            content_type: "application/json"
+          }
+        }
+
+      uid { raw_info["id"] }
 
       info do
         {
-          name: raw_info['display_name'] || raw_info['id'],
-          nickname: raw_info['id'],
-          email: raw_info['email'],
-          urls: raw_info['external_urls'],
+          name: raw_info["display_name"] || raw_info["id"],
+          nickname: raw_info["id"],
+          email: raw_info["email"],
+          urls: raw_info["external_urls"],
           image: image_url,
-          birthdate: parse_birthdate(raw_info['birthdate']),
-          country_code: raw_info['country'],
-          product: raw_info['product'],
-          follower_count: raw_info.dig('followers', 'total')
+          birthdate: parse_birthdate(raw_info["birthdate"]),
+          country_code: raw_info["country"],
+          product: raw_info["product"],
+          follower_count: raw_info.dig("followers", "total")
         }.compact
       end
 
       credentials do
         {
-          'token' => access_token.token,
-          'refresh_token' => access_token.refresh_token,
-          'expires_at' => access_token.expires_at,
-          'expires' => access_token.expires?,
-          'scope' => token_scope
+          "token" => access_token.token,
+          "refresh_token" => access_token.refresh_token,
+          "expires_at" => access_token.expires_at,
+          "expires" => access_token.expires?,
+          "scope" => token_scope
         }.compact
       end
 
       extra do
         {
-          'raw_info' => raw_info
+          "raw_info" => raw_info
         }
       end
 
       def raw_info
-        @raw_info ||= access_token.get('v1/me').parsed
+        @raw_info ||= access_token.get("v1/me").parsed
       end
 
       def image_url
-        raw_info.fetch('images', []).first&.fetch('url', nil)
+        raw_info.fetch("images", []).first&.fetch("url", nil)
       end
 
       def authorize_params
@@ -71,18 +71,18 @@ def authorize_params
       end
 
       def request_phase
-        options[:authorize_params][:show_dialog] = request.params['show_dialog'] if request.params.key?('show_dialog')
+        options[:authorize_params][:show_dialog] = request.params["show_dialog"] if request.params.key?("show_dialog")
         super
       end
 
       def callback_url
-        return '' if @authorization_code_from_signed_request
+        return "" if @authorization_code_from_signed_request
 
         options[:callback_url] || super
       end
 
       def query_string
-        return '' if request.params['code']
+        return "" if request.params["code"]
 
         super
       end
@@ -91,7 +91,7 @@ def query_string
 
       def token_scope
         token_params = access_token.respond_to?(:params) ? access_token.params : {}
-        token_params['scope'] || (access_token['scope'] if access_token.respond_to?(:[]))
+        token_params["scope"] || (access_token["scope"] if access_token.respond_to?(:[]))
       end
 
       def force_approval_requested?
@@ -102,7 +102,7 @@ def force_approval_requested?
       end
 
       def flash_force_approval?(key)
-        flashes = session.dig(:flash, 'flashes') || session.dig('flash', 'flashes')
+        flashes = session.dig(:flash, "flashes") || session.dig("flash", "flashes")
         !!flashes&.[](key)
       end